[jira] [Created] (MNG-6094) Maven Profile activation (via property) bug (or misleading doc): profile active even if property doesn't exist

["A. Di Matteo (JIRA)" <ji...@apache.org>]

A. Di Matteo created MNG-6094:
---------------------------------

             Summary: Maven Profile activation (via property) bug (or misleading doc): profile active even if property doesn't exist
                 Key: MNG-6094
                 URL: https://issues.apache.org/jira/browse/MNG-6094
             Project: Maven
          Issue Type: Bug
          Components: core
    Affects Versions: 3.3.9
         Environment: Windows 7
            Reporter: A. Di Matteo


Based on the following profile:

{code}
<profile>
    <id>test-profile</id>
    <activation>
        <property>
            <name>something</name>
            <value>!a</value>
        </property>
    </activation>
    ...
</profile>
{code}

And not setting the something property, the profile is always active (by default). This is in contradiction with the official Maven profile documentation on the !true value example:

> The following profile will be activated when the system property "debug" is defined with a value which is not "true".

A bug or misleading documentation?

If the mechanism is still implemented by the SystemPropertyProfileActivator, deprecated but not documented by which componenet it was replaced, if already replaced, then the (potential) bug is confirmed by the following shortened code:

ActivationProperty property = activation.getProperty();

if ( property != null ) {
    String name = property.getName();
    boolean reverseName = false;

    String sysValue = properties.getProperty( name );

    String propValue = property.getValue();
    if ( StringUtils.isNotEmpty( propValue ) ) {
        boolean reverseValue = false;
        if ( propValue.startsWith( "!" ) ) {
            reverseValue = true;
            propValue = propValue.substring( 1 );
        }

        // we have a value, so it has to match the system value...
        boolean result = propValue.equals( sysValue );

        if ( reverseValue ) {
            return !result;
        }
        else {
            return result;
        }
    }
}

The issue is: the sysValue real property value by the defined name is not taken into account, it will be null in this specific case. However, its check on equals will return false, but then the further check on reverseValue would turn it to true, regardless of the existence of the name property and effectively activating the profile in this case.

Full description at:
http://stackoverflow.com/q/39632024/5606016



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