[jira] [Closed] (MSHARED-495) Add convenience method to move code into Maven Filtering

["Karl Heinz Marbaise (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/MSHARED-495?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Karl Heinz Marbaise closed MSHARED-495.
---------------------------------------
    Resolution: Fixed
      Assignee: Karl Heinz Marbaise

Fixed in [r1729092|http://svn.apache.org/r1729092]
Follow up in [r1729093|http://svn.apache.org/r1729093]

> Add convenience method to move code into Maven Filtering
> --------------------------------------------------------
>
>                 Key: MSHARED-495
>                 URL: https://issues.apache.org/jira/browse/MSHARED-495
>             Project: Maven Shared Components
>          Issue Type: Improvement
>    Affects Versions: maven-filtering-3.0.0
>            Reporter: Karl Heinz Marbaise
>            Assignee: Karl Heinz Marbaise
>            Priority: Minor
>             Fix For: maven-filtering-3.0.1
>
>
> A few plugins contain the following code which belong into maven-filtering instead:
> {code:java}
>         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 );
> {code}
> Moving this code into maven-fitlering where it belongs. New method introduced:
> {code:java}
> setDelimiters( LinkedHashSet<String> delimiters, boolean useDefaultDelimiters )
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)