This is an automated email from the ASF dual-hosted git repository.

khmarbaise pushed a commit to branch MENFORCER-320
in repository https://gitbox.apache.org/repos/asf/maven-enforcer.git

commit d94f3b5f61ff482939f3aae2823f61f6d1aa6761
Author: Alex Shesterov <alex.sheste...@gmail.com>
AuthorDate: Fri Oct 5 15:29:18 2018 +0200

    [MENFORCER-320] - RequireProfileIdsExist check fails if no profiles have 
been specified
     - RequireProfileIdsExist check must not fail if there are no profiles
       in the pom.xml and no profiles have been specified in the command line
     - fix site: breadcrumbs on the "requireProfileIdsExist" page.
---
 .../maven/plugins/enforcer/RequireProfileIdsExist.java       | 12 ++++++------
 enforcer-rules/src/site/apt/requireProfileIdsExist.apt.vm    |  2 +-
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git 
a/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireProfileIdsExist.java
 
b/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireProfileIdsExist.java
index 9a16546..770f644 100644
--- 
a/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireProfileIdsExist.java
+++ 
b/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireProfileIdsExist.java
@@ -64,13 +64,13 @@ public class RequireProfileIdsExist extends 
AbstractNonCacheableEnforcerRule
             for ( org.apache.maven.settings.Profile profile : 
session.getSettings().getProfiles() )
             {
                 profileIds.remove( profile.getId() );
-                
-                if ( profileIds.isEmpty() )
-                {
-                    return;
-                }
             }
-            
+
+            if ( profileIds.isEmpty() )
+            {
+                return;
+            }
+
             StringBuilder sb = new StringBuilder();
             if ( profileIds.size() > 1 )
             {
diff --git a/enforcer-rules/src/site/apt/requireProfileIdsExist.apt.vm 
b/enforcer-rules/src/site/apt/requireProfileIdsExist.apt.vm
index 8da72a4..49e84ac 100644
--- a/enforcer-rules/src/site/apt/requireProfileIdsExist.apt.vm
+++ b/enforcer-rules/src/site/apt/requireProfileIdsExist.apt.vm
@@ -16,7 +16,7 @@
  ~~ under the License.
 
  -----
- Require Upper Bound Dependencies
+ Require Existence of Profiles Specified on the Commandline
  -----
  -----
  2017-09-25

Reply via email to