You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by ss...@apache.org on 2016/12/03 16:07:19 UTC
svn commit: r1772475 -
/sling/trunk/contrib/extensions/contextaware-config/impl/src/main/java/org/apache/sling/caconfig/impl/def/DefaultConfigurationPersistenceStrategy.java
Author: sseifert
Date: Sat Dec 3 16:07:19 2016
New Revision: 1772475
URL: http://svn.apache.org/viewvc?rev=1772475&view=rev
Log:
SLING-6347 don't forget to commit changes in persistence strategy
Modified:
sling/trunk/contrib/extensions/contextaware-config/impl/src/main/java/org/apache/sling/caconfig/impl/def/DefaultConfigurationPersistenceStrategy.java
Modified: sling/trunk/contrib/extensions/contextaware-config/impl/src/main/java/org/apache/sling/caconfig/impl/def/DefaultConfigurationPersistenceStrategy.java
URL: http://svn.apache.org/viewvc/sling/trunk/contrib/extensions/contextaware-config/impl/src/main/java/org/apache/sling/caconfig/impl/def/DefaultConfigurationPersistenceStrategy.java?rev=1772475&r1=1772474&r2=1772475&view=diff
==============================================================================
--- sling/trunk/contrib/extensions/contextaware-config/impl/src/main/java/org/apache/sling/caconfig/impl/def/DefaultConfigurationPersistenceStrategy.java (original)
+++ sling/trunk/contrib/extensions/contextaware-config/impl/src/main/java/org/apache/sling/caconfig/impl/def/DefaultConfigurationPersistenceStrategy.java Sat Dec 3 16:07:19 2016
@@ -88,6 +88,7 @@ public class DefaultConfigurationPersist
return false;
}
getOrCreateResource(resourceResolver, configResourcePath, data.getProperties());
+ commit(resourceResolver);
return true;
}
@@ -111,6 +112,7 @@ public class DefaultConfigurationPersist
replaceProperties(configResourceParent, data.getProperties());
}
+ commit(resourceResolver);
return true;
}
@@ -148,5 +150,14 @@ public class DefaultConfigurationPersist
}
modValueMap.putAll(properties);
}
+
+ private void commit(ResourceResolver resourceResolver) {
+ try {
+ resourceResolver.commit();
+ }
+ catch (PersistenceException ex) {
+ throw new ConfigurationPersistenceException("Unable to save configuration: " + ex.getMessage(), ex);
+ }
+ }
}