[ http://jira.codehaus.org/browse/ARCHETYPE-360?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Herve Boutemy updated ARCHETYPE-360: ------------------------------------ Description: The "artifactId" parameter in the processFilesetModule() method is reassigned in the while loop iterating over submodules, but the parameter is not used in the loop. The only consequence of that is that the artifact ID logged by the {{getLogger().debug( "Processed " + artifactId );}} line at the end of the method does not match the artifact ID logged at the start of the method {{getLogger().debug( "Processing module " + artifactId );}} if there are submodules. Proposal: Just delete the line, see attached patch file. was: The "artifactId" parameter in the processFilesetModule() method is reassigned in the while loop iterating over submodules, but the parameter is not used in the loop. The only consequence of that is that the artifact ID logged by the {{ getLogger().debug( "Processed " + artifactId );}} line at the end of the method does not match the artifact ID logged at the start of the method {{getLogger().debug( "Processing module " + artifactId );}} if there are submodules. Proposal: Just delete the line, see attached patch file. > Wrong assignment in loop results in wrong debug log output > ---------------------------------------------------------- > > Key: ARCHETYPE-360 > URL: http://jira.codehaus.org/browse/ARCHETYPE-360 > Project: Maven Archetype > Issue Type: Bug > Components: Generator > Affects Versions: 2.0 > Reporter: Marc Wirth > Priority: Trivial > Attachments: wrong_assignment.patch > > > The "artifactId" parameter in the processFilesetModule() method is reassigned > in the while loop iterating over submodules, but the parameter is not used in > the loop. > The only consequence of that is that the artifact ID logged by the > {{getLogger().debug( "Processed " + artifactId );}} line at the end of the > method does not match the artifact ID logged at the start of the method > {{getLogger().debug( "Processing module " + artifactId );}} if there are > submodules. > Proposal: Just delete the line, see attached patch file. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira