[sling-org-apache-sling-feature-analyser] branch master updated: SLING-8137 Create an analyser that checks the order of API Regions

[davidb]

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

davidb pushed a commit to branch master
in repository 
https://gitbox.apache.org/repos/asf/sling-org-apache-sling-feature-analyser.git


The following commit(s) were added to refs/heads/master by this push:
     new deb26b7  SLING-8137 Create an analyser that checks the order of API 
Regions
deb26b7 is described below

commit deb26b70a14dea5290341634579443d1c14129b7
Author: David Bosschaert <bossc...@adobe.com>
AuthorDate: Thu Nov 29 17:03:26 2018 +0000

    SLING-8137 Create an analyser that checks the order of API Regions
---
 .../sling/feature/analyser/task/impl/CheckApiRegionsOrder.java      | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git 
a/src/main/java/org/apache/sling/feature/analyser/task/impl/CheckApiRegionsOrder.java
 
b/src/main/java/org/apache/sling/feature/analyser/task/impl/CheckApiRegionsOrder.java
index cd7d6bc..cc9408b 100644
--- 
a/src/main/java/org/apache/sling/feature/analyser/task/impl/CheckApiRegionsOrder.java
+++ 
b/src/main/java/org/apache/sling/feature/analyser/task/impl/CheckApiRegionsOrder.java
@@ -39,6 +39,11 @@ public class CheckApiRegionsOrder implements AnalyserTask {
     }
 
     @Override
+    public String getName() {
+        return "Api Regions check order analyser task";
+    }
+
+    @Override
     public final void execute(AnalyserTaskContext ctx) throws Exception {
         String order = ctx.getConfiguration().get("order");
         Feature feature = ctx.getFeature();
@@ -97,6 +102,7 @@ public class CheckApiRegionsOrder implements AnalyserTask {
                     if (regionIdx < 0) {
                         reportError(ctx, "Region '" + name + "' appears in the 
wrong order. It appears after '"
                                 + prescribedOrder.get(prevIdx) + "'. Order of 
regions should be " + prescribedOrder);
+                        return;
                     }
                 }
             }