[sling-org-apache-sling-resourceresolver] 01/02: SLING-8006: Remove empty list of handlers if we removed the last handler from a handler list for a given path because the handler didn't activate.

[pauls]

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

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

commit bbccfed4e12e8ee909a8d3eb6b86ef962768ae5e
Author: Karl Pauls <karlpa...@gmail.com>
AuthorDate: Wed Oct 10 23:53:36 2018 +0200

    SLING-8006: Remove empty list of handlers if we removed the last handler 
from a handler list for a given path because the handler didn't activate.
---
 .../sling/resourceresolver/impl/providers/ResourceProviderTracker.java | 3 +++
 1 file changed, 3 insertions(+)

diff --git 
a/src/main/java/org/apache/sling/resourceresolver/impl/providers/ResourceProviderTracker.java
 
b/src/main/java/org/apache/sling/resourceresolver/impl/providers/ResourceProviderTracker.java
index 85cc1c3..97bb488 100644
--- 
a/src/main/java/org/apache/sling/resourceresolver/impl/providers/ResourceProviderTracker.java
+++ 
b/src/main/java/org/apache/sling/resourceresolver/impl/providers/ResourceProviderTracker.java
@@ -202,6 +202,9 @@ public class ResourceProviderTracker implements 
ResourceProviderStorageProvider
                        final List<ResourceProviderHandler> matchingHandlers = 
this.handlers.get(info.getPath());
                        if ( matchingHandlers != null && 
!matchingHandlers.isEmpty() && matchingHandlers.remove(activate) ) {
                            storage = null;
+                           if ( matchingHandlers.isEmpty() ) {
+                               this.handlers.remove(info.getPath());
+                           }
                        }
                    }
                }