[MASSEMBLY-599] Mistake in javadoc for dependencySet/includes

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.2.2, 2.3
• Fix Version/s: 2.4
• Assignee: Dennis Lundberg
• Reporter: Alexander Kormushin
• Created: 08/Feb/12 05:26
• Updated: 08/Apr/15 08:02
• Resolved: 27/Oct/12 14:19

Description

Here it's stated that fully qualified artifact coordinates in dependencySet/includes/include* should be in form
groupId:artifactId:type:version[:classifier]

But actually accepted following form
groupId:artifactId:type[:classifier]:version

Fragment of source calculating artifact.id for filtering:

.m2/repository/org/apache/maven/maven-artifact/2.2.1/maven-artifact-2.2.1-sources.jar!/org/apache/maven/artifact/DefaultArtifact.java

    public String getId()
    {
        return getDependencyConflictId() + ":" + getBaseVersion();
    }

    public String getDependencyConflictId()
    {
        StringBuffer sb = new StringBuffer();
        sb.append( getGroupId() );
        sb.append( ":" );
        appendArtifactTypeClassifierString( sb );
        return sb.toString();
    }

    private void appendArtifactTypeClassifierString( StringBuffer sb )
    {
        sb.append( getArtifactId() );
        sb.append( ":" );
        sb.append( getType() );
        if ( hasClassifier() )
        {
            sb.append( ":" );
            sb.append( getClassifier() );
        }
    }

Issue Links

• relates to: MASSEMBLY-567 Version wildcard in dependencySet include does not work