You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@rave.apache.org by ja...@apache.org on 2011/07/14 15:19:06 UTC

svn commit: r1146689 - /incubator/rave/trunk/rave-shindig/src/main/java/org/apache/rave/commoncontainer/ConfigurablePropertiesModule.java

Author: jasha
Date: Thu Jul 14 13:19:04 2011
New Revision: 1146689

URL: http://svn.apache.org/viewvc?rev=1146689&view=rev
Log:
RAVE-83 apply Sonar recommendation for iterating over entrySet

Modified:
    incubator/rave/trunk/rave-shindig/src/main/java/org/apache/rave/commoncontainer/ConfigurablePropertiesModule.java

Modified: incubator/rave/trunk/rave-shindig/src/main/java/org/apache/rave/commoncontainer/ConfigurablePropertiesModule.java
URL: http://svn.apache.org/viewvc/incubator/rave/trunk/rave-shindig/src/main/java/org/apache/rave/commoncontainer/ConfigurablePropertiesModule.java?rev=1146689&r1=1146688&r2=1146689&view=diff
==============================================================================
--- incubator/rave/trunk/rave-shindig/src/main/java/org/apache/rave/commoncontainer/ConfigurablePropertiesModule.java (original)
+++ incubator/rave/trunk/rave-shindig/src/main/java/org/apache/rave/commoncontainer/ConfigurablePropertiesModule.java Thu Jul 14 13:19:04 2011
@@ -29,6 +29,7 @@ import org.springframework.core.io.Resou
 
 import java.io.IOException;
 import java.util.Arrays;
+import java.util.Map;
 import java.util.Properties;
 
 /**
@@ -84,10 +85,10 @@ public class ConfigurablePropertiesModul
 
         try {
             initProperties.load(propertyResource.getInputStream());
-            for (Object key : initProperties.keySet()) {
-                String value = (String) initProperties.get(key);
+            for (Map.Entry entry : initProperties.entrySet()) {
+                String value = (String) entry.getValue();
                 if (value != null && value.contains(CONTEXT_ROOT_PLACEHOLDER)) {
-                    initProperties.put(key,
+                    initProperties.put(entry.getKey(),
                             value.replace((CONTEXT_ROOT_PLACEHOLDER), getContextRoot()));
                 }
             }