February 2013 - Cantharella-devel - mailman.cloud.codelutin.com

Méthode equals() et hashCode() non redéfinie sur les entités
by Eric Chatellier 26 Feb '13

26 Feb '13

Bonjour, Je rencontre actuellement un petit problème visible sur la modification d'un document. Lorsque l'on tente de modifier un document existant, le créateur n'est sélectionné avec la bonne valeur. Le code étant le suivant: final DropDownChoice<Personne> createurInput = new DropDownChoice<Personne>("Document.createur", new PropertyModel<Personne>(documentModel, "createur"), personnes); Je pense que le problème se situe au niveau des méthodes equals() des entités qui ne sont pas redéfini, l’implémentation par defaut étant 'this == obj', ca peut passer dans la majorité des cas car il y a le cache hibernate, mais dans ce cas la cela ne fonctionne pas. A code lutin, nous redéfinissons systématiquement le equals() et le hashCode() en fonction des id car il existe toujours. Mais dans le cas de cantharella ils sont générés par la base de données. Je ne vois pas comment corriger simplement le problème. J'ai testé en ajoutant un equals() rapidement et cela corrige le problème. -- Éric Chatellier - Code Lutin Tel: 02.40.50.29.28 - http://www.codelutin.com

1 0

Re: [Cantharella-devel] CR Réunion Cantharella du 19/02/2013
by Adrien Cheype 26 Feb '13

26 Feb '13

Le 26/02/2013 03:33, Eric Chatellier a écrit : > Le 25/02/2013 16:23, Eric Chatellier a écrit : >> Je ne comprend pas ce problème. >> >> Sur la page des molécules, il y a deux tableaux : provenances et >> documents. >> Le premier est bien mis à jour mais pas le deuxième et il n'y >> a aucun code particulier qui force le rechargement des tableaux. > J'ai fini par trouver finalement, je ne maitrise pas encore toutes > les subtilités de wicket ;-). > > Dans ce cas là, j'utilisais directement les instances des > entités 'Molecule' au lieu de Model<Molecule>. > Les pages sont instanciées une seule fois, les modèles > aussi, mais les données des modèles sont détacher/rattacher > lorsque l'on change de page. > Il faut donc utiliser des modèles pour que les entités soient > rechargées et les bonnes listes affichées. > Bonjour Eric, En effet, les modèles sont au début un peu déroutants. On en voit partout dans wicket et on se demande pourquoi faut-il systématiquement passé par eux (une simplification ne serait-elle pas meilleure ?). Mais petit à petit on rentre dans la logique du framework et on comprend leur force. Dans ton cas, tu as bien identifié le problème, tu ne passais pas par un modèle pour construire ton listview. Cependant tu peux encore clarifier ton code. Actuellement tu charges la liste des documents dans un modèle de list de document après chaque cycle de sérialisation/désérialisation (load()). Cependant cette liste existe déjà, de la même manière que les autres propriétés d'une molécule. La simple utilisation d'un PropertyModel pour constituer ton modèle de liste suffit donc dans ce cas. Tu peux d'ailleurs le passer en paramètre de ton ReadListDocumentsPanel au lieu du modèle de molécule. Ce serait plus cohérent : le panel sert à afficher le modèle de ce qu'il doit afficher. Tu peux trouver un exemple (sans panel) dans ReadPurificationPage, notamment : > ListView<ParamMethoPuriEffectif> paramMethodes = new > ListView<ParamMethoPuriEffectif>( > "Purification.paramsMethode.List", new > PropertyModel<List<ParamMethoPuriEffectif>>(purificationModel, > "sortedParamsMetho")) { > ... Bonne journée, Adrien -- Adrien Cheype Ingénieur en Systèmes d'Information Service « Informatique Scientifique et Appui aux Partenaires du Sud » Direction du Système d'Information (DSI) http://www.ird.fr/dsi/ http://www.ird.fr/informatique-scientifique/ INSTITUT DE RECHERCHE POUR LE DEVELOPPEMENT BP A5 - 98848 Nouméa - Nouvelle Calédonie Tél. +687 260 789

2 1

Re: [Cantharella-devel] Mots clés svn et site web (javadoc)
by Adrien Cheype 20 Feb '13

20 Feb '13

Merci pour ces infos, Eric. Mes réponses ci-dessous. Le 20/02/2013 04:55, Eric Chatellier a écrit : > Bonjour, > > Les mots clés svn étaient bien manquants, je viens de les ajouter > sur tous les fichiers via la commande: > svn propset svn:keywords "Author Date Id Revision HeadURL" -R . > > Pour que les mots clé soient automatiquement ajouté sur > un nouveau fichiers créé par un développeur, il est possible > de le specifier dans le fichier $HOME/.subversion/config > en ajoutant, dans la section [auto-props] la ligne: > *.java = svn:keywords="Author Date Id Revision HeadURL" Ok, si je comprends bien le module qui gère les licences utilisent ces propriétés pour compléter le message d'header. Les fichiers n'ayant pas ces propriétés se retrouve alors juste avec "$Id:$" et "$HeadURL:$" sans texte associé, c'est bien cela ? A quels fichiers le module ajoute les headers de licence ? A des fichiers qui comportent certaines extensions ? Ou à tous les fichiers texte ? > Concernant, la génération du site, celle-ci s'effectue > normalement via la commande "mvn site". > Sur notre forge, cette commande a été allégée et pour > généré le site entierrement (avec les rapports, javadocs...) > il faut activer un profile supplémentaire: > mvn site -Preporting > > > Le site une fois buildé peut être déployé comme par exemple > à cette adresse : > http://maven-site.forge.codelutin.com/cantharella/ > Pour consulter la javadoc, il faut aller dans les rapports > de chaque module (le module principal n'ayant pas de code java > donc pas de javadoc). > > > Plus globalement, lors d'une release, beaucoup de choses sont > effectuées tardivement et seulement lors de la release. > la commande "mvn install -DperformRelease" active tous les > plugins maven devant s’exécuter à la release mais sans > réellement l'effectuer. > > Si la commande "mvn install -DperformRelease", la release > fonctionnera ensuite. > > Cependant, cette commande nécessite une configuration > un peu lourde de l'envirronement de developpement pour > avoir accès notamment aux services web redmine de la forge. > > Je ne pense pas que cela vaille le coup de configurer tout > ca, mais si vous y tener (pour pouvoir faire des déploiements) > c'est tout à faire possible. Je suis d'accord, pas la peine que je configure mon environnement pour effectuer des releases. J'imagine que la commande "mvn site" nécessite également ce type de configuration. Si j'ai besoin de ces fonctionnalités (par ex. mise à jour des sources pour le site de présentation), mieux vaut que je vous demande de lancer la commande adéquate. Bonne journée -- Adrien Cheype Ingénieur en Systèmes d'Information Service « Informatique Scientifique et Appui aux Partenaires du Sud » Direction du Système d'Information (DSI) http://www.ird.fr/dsi/ http://www.ird.fr/informatique-scientifique/ INSTITUT DE RECHERCHE POUR LE DEVELOPPEMENT BP A5 - 98848 Nouméa - Nouvelle Calédonie Tél. +687 260 789

2 1

Re: [Cantharella-devel] Conventions de codage - Cantharella
by Adrien Cheype 20 Feb '13

20 Feb '13

Pour terminer sur le sujet du passage de la JDK à la v.7, le compilateur rajoute davantage de warnings du style : > Some input files use unchecked or unsafe operations. J'avais par exemple deux warnings qui n'était pas détectés avec le JDK 6 dans la classe AssertTools de cantharella.utils En recompilant avec l'option -Xlint:uncheked pour avoir davantage de détails, encore d'autres avertissements sont apparus sur la classe BeanTools, et ils comportaient tous le message suivant : > |[unchecked] Possible heap pollution from parameterized vararg type T| L'explication est que depuis Java 7, le compilateur avertit lorsqu'il détecte des arguments paramétrés de méthode qui sont des "varargs", comme on peut le voir dans l'exemple suivant : > public static <T> void addToList (List<T> listArg, T... elements) { > ... Il semblerait en effet que des erreurs à l'exécution peuvent se produire pour certains cas extrêmes où il y a affectation d'un élément de la "varargs" à une liste d'un autre type. La solution pour éviter ces warnings est que le développeur certifie la bonne exécution du code avec l'annotation @SafeVarargs. On peut également utilisé l'annotation @SuppressWarnings({"unchecked", "varargs"}) mais cette dernière laissera tout de même un warning à l'exécution. Pour plus de détails, cf. http://docs.oracle.com/javase/7/docs/technotes/guides/language/non-reifiabl… Le 20/02/2013 09:47, Adrien Cheype a écrit : > Ok, à notre niveau cela ne semble pas poser de problème. > > Et juste pour info, Cantharella est hébergé actuellement en production > sur un serveur virtuel où il y a Tomcat 7 et Java 6 d'installé. > Nous passerons donc à Tomcat 7 et Java 7, tout en continuant > d'utiliser le standard Servlet 2.5. > > Le 19/02/2013 20:08, Mickaël Tricot a écrit : >> Tant mieux ! :) >> Je crois avoir lu que certains ont eu des problèmes à faire tourner >> Tomcat 6 + Java 7, mais ce sont peut-être des problèmes spécifiques à >> leur infrastructure ou applications. >> >> 2013/2/18 Eric Chatellier <chatellier(a)codelutin.com >> <mailto:chatellier@codelutin.com>> >> >> Il n'y a pas de rapport entre Java 7 et Tomcat 7. >> >> Tomcat 7, impose une JRE 6 ou supérieur et peut faire tourner des >> servlet 2.5 (voire avant). >> >> Les servlet 3.0 sont d'ailleurs inclut dans la norme JEE 6 et non >> JSE 7. >> >> >> >> _______________________________________________ >> Cantharella-devel mailing list >> Cantharella-devel(a)list.forge.codelutin.com >> http://list.forge.codelutin.com/cgi-bin/mailman/listinfo/cantharella-devel > > > -- > Adrien Cheype > Ingénieur en Systèmes d'Information > Service « Informatique Scientifique et Appui aux Partenaires du Sud » > Direction du Système d'Information (DSI) > http://www.ird.fr/dsi/ > http://www.ird.fr/informatique-scientifique/ > > INSTITUT DE RECHERCHE POUR LE DEVELOPPEMENT > BP A5 - 98848 Nouméa - Nouvelle Calédonie > Tél. +687 260 789 -- Adrien Cheype Ingénieur en Systèmes d'Information Service « Informatique Scientifique et Appui aux Partenaires du Sud » Direction du Système d'Information (DSI) http://www.ird.fr/dsi/ http://www.ird.fr/informatique-scientifique/ INSTITUT DE RECHERCHE POUR LE DEVELOPPEMENT BP A5 - 98848 Nouméa - Nouvelle Calédonie Tél. +687 260 789

1 0

Re: [Cantharella-devel] Conventions de codage - Cantharella
by Adrien Cheype 19 Feb '13

19 Feb '13

Ok, à notre niveau cela ne semble pas poser de problème. Et juste pour info, Cantharella est hébergé actuellement en production sur un serveur virtuel où il y a Tomcat 7 et Java 6 d'installé. Nous passerons donc à Tomcat 7 et Java 7, tout en continuant d'utiliser le standard Servlet 2.5. Le 19/02/2013 20:08, Mickaël Tricot a écrit : > Tant mieux ! :) > Je crois avoir lu que certains ont eu des problèmes à faire tourner > Tomcat 6 + Java 7, mais ce sont peut-être des problèmes spécifiques à > leur infrastructure ou applications. > > 2013/2/18 Eric Chatellier <chatellier(a)codelutin.com > <mailto:chatellier@codelutin.com>> > > Il n'y a pas de rapport entre Java 7 et Tomcat 7. > > Tomcat 7, impose une JRE 6 ou supérieur et peut faire tourner des > servlet 2.5 (voire avant). > > Les servlet 3.0 sont d'ailleurs inclut dans la norme JEE 6 et non > JSE 7. > > > > _______________________________________________ > Cantharella-devel mailing list > Cantharella-devel(a)list.forge.codelutin.com > http://list.forge.codelutin.com/cgi-bin/mailman/listinfo/cantharella-devel -- Adrien Cheype Ingénieur en Systèmes d'Information Service « Informatique Scientifique et Appui aux Partenaires du Sud » Direction du Système d'Information (DSI) http://www.ird.fr/dsi/ http://www.ird.fr/informatique-scientifique/ INSTITUT DE RECHERCHE POUR LE DEVELOPPEMENT BP A5 - 98848 Nouméa - Nouvelle Calédonie Tél. +687 260 789

1 0

Mots clés svn et site web (javadoc)
by Eric Chatellier 19 Feb '13

19 Feb '13

Bonjour, Les mots clés svn étaient bien manquants, je viens de les ajouter sur tous les fichiers via la commande: svn propset svn:keywords "Author Date Id Revision HeadURL" -R . Pour que les mots clé soient automatiquement ajouté sur un nouveau fichiers créé par un développeur, il est possible de le specifier dans le fichier $HOME/.subversion/config en ajoutant, dans la section [auto-props] la ligne: *.java = svn:keywords="Author Date Id Revision HeadURL" Concernant, la génération du site, celle-ci s'effectue normalement via la commande "mvn site". Sur notre forge, cette commande a été allégée et pour généré le site entierrement (avec les rapports, javadocs...) il faut activer un profile supplémentaire: mvn site -Preporting Le site une fois buildé peut être déployé comme par exemple à cette adresse : http://maven-site.forge.codelutin.com/cantharella/ Pour consulter la javadoc, il faut aller dans les rapports de chaque module (le module principal n'ayant pas de code java donc pas de javadoc). Plus globalement, lors d'une release, beaucoup de choses sont effectuées tardivement et seulement lors de la release. la commande "mvn install -DperformRelease" active tous les plugins maven devant s’exécuter à la release mais sans réellement l'effectuer. Si la commande "mvn install -DperformRelease", la release fonctionnera ensuite. Cependant, cette commande nécessite une configuration un peu lourde de l'envirronement de developpement pour avoir accès notamment aux services web redmine de la forge. Je ne pense pas que cela vaille le coup de configurer tout ca, mais si vous y tener (pour pouvoir faire des déploiements) c'est tout à faire possible. -- Éric Chatellier - Code Lutin Tel: 02.40.50.29.28 - http://www.codelutin.com

1 0

Problème internet explorer
by Eric Chatellier 19 Feb '13

19 Feb '13

Bonjour, Le problème IE venait du header de licence qui est avant la déclaration de doctype. IE bascule en "Quirks mode". Heureusement, on est habitué aux bug d'IE et j'avais déjà rencontré ce soucis: http://en.wikipedia.org/wiki/Quirks_mode#Triggering_different_rendering_mod… Il suffit de mettre la doctype avant. Sinon, suite à l'inclusion de chemdoodle, je pensais: * changer la doctype en html 5 * changer les dtd wicket 1.4 en xsd wicket 6 A noter, que cantharella nécessite au minimum ie9 rien que pour chemdoodle. Cordialement, Eric Chatellier. -- Éric Chatellier - Code Lutin Tel: 02.40.50.29.28 - http://www.codelutin.com

1 0

Conventions de codage - Cantharella
by Adrien Cheype 19 Feb '13

19 Feb '13

Bonjour, Pour rappel, voici les conventions de codages sur lesquelles nous nous sommes entendu lors de la dernière réunion : - Les commentaires doivent toujours être en anglais (bien qu'il y ait actuellement un mélange français/anglais) - Chaque classe, méthode et attribut doit être associé à un commentaire de javadoc - Les indentations doivent être constituées de 4 espaces (et non d'une tabulation) - La taille des lignes doit être de 120 caractères - Le formatage du texte doit respecter les conventions JAVA Pour uniformiser le code du projet et ce quelque soit l'IDE utilisé, j'ai mis en place le plugin maven-java-formatter-plugin. Chaque sous-projet maven partage le même fichier de configuration qui est servi par un assembly. Ce plugin prenant en entrée un fichier de configuration du formatter d'Eclipse, il a été facile pour moi d'y renseigner le paramétrage voulu (120 caractère de largeur, 4 espaces pour une indentation). Je suis parti d'une configuration de convention Java et j'ai seulement modifié une règle pour avoir un saut de ligne entre deux propriétés d'une classe (je trouve cela plus lisible, mais ça reste subjectif). A chaque build Maven, l'ensemble des classes est parsé et le plugin Maven formate les classes qui en ont besoin. Le plugin répond donc bien aux trois derniers points des conventions plus haut. Pour le premier point, nous avons convenu qu'on mettrait dorénavant les commentaires en anglais et que ce n'est pas la peine de revenir sur ceux qui existent déjà. Il reste donc à régler le deuxième point, c'est à dire de mettre à jour l'ensemble des commentaires de javadoc qui manquent sur le code source produit depuis le début de la prestation. Une dernière chose, j'ai spécifié dans le pom.xml la version du JDK utilisée pour à la fois la compilation et le formatage. J'ai mis 1.6 car le code a été initialement écrit pour du Java SE 6. Préférez-vous qu'on passe directement à la version 1.7 ? ou au contraire que cela pourrait engendrer des problèmes de compatibilité ? Adrien -- Adrien Cheype Ingénieur en Systèmes d'Information Service « Informatique Scientifique et Appui aux Partenaires du Sud » Direction du Système d'Information (DSI) http://www.ird.fr/dsi/ http://www.ird.fr/informatique-scientifique/ INSTITUT DE RECHERCHE POUR LE DEVELOPPEMENT BP A5 - 98848 Nouméa - Nouvelle Calédonie Tél. +687 260 789

3 4

Re: [Cantharella-devel] Conventions de codage - Cantharella
by Adrien Cheype 19 Feb '13

19 Feb '13

Le 18/02/2013 21:25, Eric Chatellier a écrit : > Le 18/02/2013 10:28, Mickaël Tricot a écrit : >> Bonjour, >> >> Il me semble que passer de Java 6 à 7 implique un certain nombre de >> changements, à commencer par passer de Tomcat 6 à 7 avec tout ce que cela >> implique (Servlet 3...). Voir : >> http://tomcat.apache.org/migration-7.html#Migrating_from_6.0.x_to_7.0.x >> A prévoir aussi, la migration vers Jetty 8. > Il n'y a pas de rapport entre Java 7 et Tomcat 7. > > Tomcat 7, impose une JRE 6 ou supérieur et peut faire tourner des > servlet 2.5 (voire avant). > > Les servlet 3.0 sont d'ailleurs inclut dans la norme JEE 6 et non JSE 7. Le pom.xml a été mis à jour aujourd'hui avec la version 7 du JDK. Aucun problème majeur, juste deux warnings "Some input files use unchecked or unsafe operations." facilement corrigeable. -- Adrien Cheype Ingénieur en Systèmes d'Information Service « Informatique Scientifique et Appui aux Partenaires du Sud » Direction du Système d'Information (DSI) http://www.ird.fr/dsi/ http://www.ird.fr/informatique-scientifique/ INSTITUT DE RECHERCHE POUR LE DEVELOPPEMENT BP A5 - 98848 Nouméa - Nouvelle Calédonie Tél. +687 260 789

1 0

Re: [Cantharella-devel] [Cantharella-commits] r125 - in trunk: cantharella.data/src/main/java/nc/ird/cantharella/data/config
by Adrien Cheype 18 Feb '13

18 Feb '13

Ok, merci Eric d'avoir vérifié. J'ai dû me tromper de fichier de configuration. Le 18/02/2013 21:14, Eric Chatellier a écrit : > Le 18/02/2013 11:11, echatellier(a)users.forge.codelutin.com a écrit : >> Author: echatellier >> Date: 2013-02-18 11:11:43 +0100 (Mon, 18 Feb 2013) >> New Revision: 125 >> >> Url: http://forge.codelutin.com/projects/cantharella/repository/revisions/125 >> >> Log: >> Use "space only" tabulation policy with size = 4 > Je viens de changer ces deux valeurs et fait un reformat global > également car le code étaient indenté avec des tabulations. >> Modified: trunk/src/code-formatter-conf/code-format.xml >> =================================================================== >> --- trunk/src/code-formatter-conf/code-format.xml 2013-02-18 09:56:08 UTC (rev 124) >> +++ trunk/src/code-formatter-conf/code-format.xml 2013-02-18 10:11:43 UTC (rev 125) >> @@ -30,7 +30,7 @@ >> <setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_method_invocation_arguments" value="do not insert"/> >> <setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_cast" value="do not insert"/> >> <setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_bracket_in_array_allocation_expression" value="do not insert"/> >> -<setting id="org.eclipse.jdt.core.formatter.tabulation.size" value="8"/> >> +<setting id="org.eclipse.jdt.core.formatter.tabulation.size" value="4"/> >> <setting id="org.eclipse.jdt.core.formatter.comment.indent_parameter_description" value="true"/> >> <setting id="org.eclipse.jdt.core.formatter.insert_new_line_before_else_in_if_statement" value="do not insert"/> >> <setting id="org.eclipse.jdt.core.formatter.brace_position_for_type_declaration" value="end_of_line"/> >> @@ -64,7 +64,7 @@ >> <setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_parenthesized_expression" value="do not insert"/> >> <setting id="org.eclipse.jdt.core.formatter.insert_new_line_after_annotation_on_local_variable" value="insert"/> >> <setting id="org.eclipse.jdt.core.formatter.alignment_for_binary_expression" value="16"/> >> -<setting id="org.eclipse.jdt.core.formatter.tabulation.char" value="mixed"/> >> +<setting id="org.eclipse.jdt.core.formatter.tabulation.char" value="space"/> > -- Adrien Cheype Ingénieur en Systèmes d'Information Service « Informatique Scientifique et Appui aux Partenaires du Sud » Direction du Système d'Information (DSI) http://www.ird.fr/dsi/ http://www.ird.fr/informatique-scientifique/ INSTITUT DE RECHERCHE POUR LE DEVELOPPEMENT BP A5 - 98848 Nouméa - Nouvelle Calédonie Tél. +687 260 789

1 0