sling-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From k...@apache.org
Subject [sling-org-apache-sling-validation-core] branch master updated: SLING-7289 really remove the map entry
Date Thu, 07 Dec 2017 16:40:37 GMT
This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/master by this push:
     new 10c2515  SLING-7289 really remove the map entry
10c2515 is described below

commit 10c2515830fe0b05e9ce01a517725665ca674703
Author: Konrad Windszus <kwin@apache.org>
AuthorDate: Thu Dec 7 17:40:33 2017 +0100

    SLING-7289 really remove the map entry
---
 src/main/java/org/apache/sling/validation/impl/ValidatorMap.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/main/java/org/apache/sling/validation/impl/ValidatorMap.java b/src/main/java/org/apache/sling/validation/impl/ValidatorMap.java
index 71d80d5..846edfb 100644
--- a/src/main/java/org/apache/sling/validation/impl/ValidatorMap.java
+++ b/src/main/java/org/apache/sling/validation/impl/ValidatorMap.java
@@ -222,7 +222,7 @@ public class ValidatorMap {
         } else {
             // only actually remove if the service reference is equal
             if (entry.serviceReference.equals(serviceReference)) {
-                //validatorMap.remove(id);
+                validatorMap.remove(id);
                 return true;
             } else {
                 LOG.warn("Could not remove validator with id '{}' from map because it is
only contained with a different service reference!", id);

-- 
To stop receiving notification emails like this one, please contact
['"commits@sling.apache.org" <commits@sling.apache.org>'].

Mime
View raw message