commit b42db5ee252afe8747cc6d8d2902d9c5543e6001
author David Bosschaert <david.bosschaert@gmail.com> Wed Jun 03 08:32:43 2020 +0100
committer GitHub <noreply@github.com> Wed Jun 03 08:32:43 2020 +0100
tree 61ec9d840a2d26af0b75901b1e02da70f9eb9fcc
parent 67f065b96baa81b5e5efd8743a0e41fc588f875c
parent 134f9d941b6489cf1d8d9666d373acf273228b11

Merge pull request #10 from bosschaert/SLING-9496

SLING-9496 Dead code in the API Regions runtime

src/main/java/org/apache/sling/feature/apiregions/impl/ResolverHookImpl.java

diff --git a/src/main/java/org/apache/sling/feature/apiregions/impl/ResolverHookImpl.java b/src/main/java/org/apache/sling/feature/apiregions/impl/ResolverHookImpl.java
index 1476fc4..d6a1881 100644
--- a/src/main/java/org/apache/sling/feature/apiregions/impl/ResolverHookImpl.java
+++ b/src/main/java/org/apache/sling/feature/apiregions/impl/ResolverHookImpl.java
@@ -103,16 +103,7 @@
                 continue nextCapability;
             }
 
-            if (capFeatures.isEmpty())
-                capFeatures = Collections.singleton(null);
-
             for (String capFeat : capFeatures) {
-                if (capFeat == null) {
-                    // everyone can access capability not coming from a feature
-                    coveredCaps.put(bc, RegionConstants.GLOBAL_REGION);
-                    continue nextCapability;
-                }
-
                 if (reqFeatures.contains(capFeat)) {
                     // Within a single feature everything can wire to everything else
                     coveredCaps.put(bc, null); // null value means same bundle, same feature or system bundle