[MSHARED-495] Add convenience method to move code into Maven Filtering

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: maven-filtering-3.0.0
• Fix Version/s: maven-filtering-3.1.0
• Assignee: Karl Heinz Marbaise
• Reporter: Karl Heinz Marbaise
• Created: 08/Feb/16 07:14
• Updated: 08/Feb/16 08:33
• Resolved: 08/Feb/16 08:21

Description

A few plugins contain the following code which belong into maven-filtering instead:

        if ( delimiters != null && !delimiters.isEmpty() )
        {
            LinkedHashSet<String> delims = new LinkedHashSet<String>();
            if ( useDefaultDelimiters )
            {
                delims.addAll( this.getDelimiters() );
            }

            for ( String delim : delimiters )
            {
                if ( delim == null )
                {
                    // FIXME: ${filter:*} could also trigger this condition. Need a better long-term solution.
                    delims.add( "${*}" );
                }
                else
                {
                    delims.add( delim );
                }
            }

            this.setDelimiters( delims );

Moving this code into maven-fitlering where it belongs. New method introduced:

setDelimiters( LinkedHashSet<String> delimiters, boolean useDefaultDelimiters )