Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WebApplication.java] Soucis de faute de frappe dans un commentaire #9

Open
dokor opened this issue Nov 14, 2024 · 5 comments
Open
Labels
Milestone

Comments

@dokor
Copy link
Member

dokor commented Nov 14, 2024

Dans l'archétype "classique" de plume, on retrouve un fichier WebApplication.java.
Ce dernier comporte en son centre un commentaire qui contient une erreur de frappe qui est visible lorsqu'on décommente le code lors de l'utilisation de plume-scheduler.

			// Add a shutdown hook to execute some code when the JVM receive a kill signal before it stops
			addShutDownListener(httpServer);
			// If Plume Scheduler / Wisp is used, uncomment next line
			// addShutDownListerner(httpServer, injector.getInstance(Scheduler.class));

Il y a un R en trop dans // addShutDownListeRner(XXXXXX);

(Antoine LE LOUËT)

@amanteaux
Copy link
Member

Normalement c'est déjà corrigé dans la release 4.2.0.

Le commit de correction est celui là : 7623f1c

Tu reproduits le problème quand tu crées un projet avec la commande décrite dans https://github.com/Coreoz/Plume-archetypes/tree/master/plume-archetype-querydsl-jersey-guice-grizzly ?

@dokor
Copy link
Member Author

dokor commented Nov 15, 2024

Yes j'ai bien utilisé la commande normal qui est décrite dans la doc

@dokor
Copy link
Member Author

dokor commented Nov 15, 2024

HA !

Le commit en question corrige bien le nom de la méthode et de son utilisation, mais la modif n'a pas été impacté sur le commentaire de la ligne 49 justement.
Ce qui fait que lorsqu'on décommente, la fonction n'est pas trouvé.

		// addShutDownListerner(httpServer, injector.getInstance(Scheduler.class));

@amanteaux
Copy link
Member

Ah oui effectivement, j'avais pas compris, c'est la ligne commentée qui pose problème !

Je note ça pour la release qui arrive normalement très bientôt !

@amanteaux amanteaux added the bug label Nov 18, 2024
@amanteaux amanteaux added this to the V5 milestone Nov 18, 2024
amanteaux added a commit that referenced this issue Nov 18, 2024
@amanteaux
Copy link
Member

Merci pour l'ouverture de l'issue @dokor !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants