April 2011 - Chorem-devel - mailman.cloud.codelutin.com

Re: [Chorem-devel] [Chorem-commits] r89 - in trunk/chorem-entities: . src src/test src/test/java src/test/java/org src/test/java/org/chorem src/test/java/org/chorem/entities
by Tony Chemit 15 Apr '11

15 Apr '11

On Fri, 15 Apr 2011 11:22:37 +0200 (CEST) bpoussin(a)users.chorem.org wrote: > - > + <dependency> > + <groupId>junit</groupId> > + <artifactId>junit</artifactId> > + <version>4.8.2</version> > + <scope>test</scope> > + </dependency> > </dependencies> rohhh pas de dépendances en dure dans des modules :) en plus celle-là est géré par mavenpom donc un simple <dependency> <groupId>junit</groupId> <artifactId>junit</artifactId> </dependency> suffira ;) -- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 email: chemit(a)codelutin.com http://www.codelutin.com

1 0

Re: [Chorem-devel] [Chorem-commits] r71 - in trunk/chorem-web/src/main: java/org/chorem/billy/action java/org/chorem/gepeto/action resources resources/i18n webapp/WEB-INF/jsp/billy webapp/WEB-INF/jsp/gepeto
by Tony Chemit 08 Apr '11

08 Apr '11

On Fri, 8 Apr 2011 16:03:55 +0200 (CEST) vbriand(a)users.chorem.org wrote: > > + /** > + * Retrieves the information of the projectOrder referenced by > the > + * projectOrderId > + * > + * @return SUCCESS if the information have been retrieved > + * @return ERROR either if the UUID is invalid or the > projectOrderId doesn't > + * exist > + */ > + public String projectOrderDetails() { Dans la javadoc il ne faut pas mettre deux @return c'est pas valide :) -- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 email: chemit(a)codelutin.com http://www.codelutin.com

1 0

Re: [Chorem-devel] [Chorem-commits] r68 - trunk/chorem-web/src/main/java/org/chorem/gepeto/action
by Tony Chemit 08 Apr '11

08 Apr '11

On Fri, 8 Apr 2011 11:11:46 +0200 (CEST) vbriand(a)users.chorem.org wrote: > + > + /** > + * Tests if a quotation already has a project order > + * > + * @param quotation > + * @return true if the quotation already has a project order, > false otherwise > + */ > + protected boolean quotationHasProjectOrder(Quotation quotation) { Deux petites remarques valentin : 1) la première ligne d'une javadoc doit être une phrase (donc se terminer par un point) 2) le nom de méthode est pas top, j'opterais plutôt pour un *isQuotationHasProjectOrder* Du détail..., mais qui permets de faire avancer un peu le chmilblick :) -- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 email: chemit(a)codelutin.com http://www.codelutin.com

3 6

Re: [Chorem-devel] [Chorem-commits] r60 - in trunk/chorem-web/src/main: java/org/chorem/action webapp/WEB-INF/jsp
by Tony Chemit 07 Apr '11

07 Apr '11

On Wed, 6 Apr 2011 18:43:43 +0200 (CEST) bpoussin(a)users.chorem.org wrote: > <div class="block"> > <ul> > - <li>Project open: > <%=action.getNbProjectOrderOpen()%></li> > + <li>Project status: > <%=action.getNbProjectOrderStatus()%></li> <li>Task: > <%=action.getNbTaskOpenForMe()%>/<%=action.getNbTaskOpen()%> > </li> On pourrait quand même un minimum utiliser les tag-libs plutôt que de faire de la jsp pure et dure avec du <%=action.XXX%> dans tous les sens, a la place plutôt utiliser un %{nbTaskForMe} par exemple. Je comprends toujours pas cette volonté de coder sans tag-lib à la mode JSP 0.1 :( Je veux bien quand c'est compliqué mais bon là ça me parait pas bien sorcier sauf si vous trouvez ça esthétique d'avoir du <%=action.getXXX()%> partout dans vos jsp, pour ma part je trouve ça complètement illisible, redondant et ultra-lourdingue et cela utilise pas la puissance du framework :( qui de base est quand même là pour ça :) -- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 email: chemit(a)codelutin.com http://www.codelutin.com

2 3