extra-enforce-rules ignorableDependency code broken
---------------------------------------------------

                 Key: MOJO-1769
                 URL: https://jira.codehaus.org/browse/MOJO-1769
             Project: Mojo
          Issue Type: Bug
          Components: extra-enforcer-rules
    Affects Versions: extra-enforcer-rules-1.0-alpha-2
            Reporter: Ian Robertson


In BanDuplicateClasses.java, there is the following code:

                if ( ignorableDependency.groupId != null )
                {
                    ignorableDependency.groupId = Pattern.compile( asRegex( 
dependency.getGroupId() ) );
                }
                if ( ignorableDependency.artifactId != null )
                {
                    ignorableDependency.artifactId = Pattern.compile( asRegex( 
dependency.getArtifactId() ) );
                }

the if clauses should be looking at dependency.getXxx() instead of 
ignorableDependency.xxx. Indeed, the following lines do that for type and 
classifier.

I'm working on a unit test and patch locally, and will be submitting those 
shortly. The integration test will probably rely on work I'm doing for 
MOJO-1767.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email


Reply via email to