[ARCHETYPE-360] Wrong assignment in loop results in wrong debug log output

• Type: Bug
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 2.0
• Fix Version/s: 2.1
• Component/s: Generator
• Assignee: Herve Boutemy
• Reporter: Marc Wirth
• Created: 12/Jan/11 10:10
• Updated: 24/Jan/11 17:16
• Resolved: 24/Jan/11 17:16

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.