Author: michiel
Date: 2010-03-18 11:23:01 +0100 (Thu, 18 Mar 2010)
New Revision: 41493

Modified:
   
mmbase/trunk/applications/vpro-wizards/src/main/java/org/mmbase/applications/vprowizards/spring/BasicCommand.java
Log:
When iterating over a map's keys and values, it seems better to iterate the 
entrySet

Modified: 
mmbase/trunk/applications/vpro-wizards/src/main/java/org/mmbase/applications/vprowizards/spring/BasicCommand.java
===================================================================
--- 
mmbase/trunk/applications/vpro-wizards/src/main/java/org/mmbase/applications/vprowizards/spring/BasicCommand.java
   2010-03-18 10:18:28 UTC (rev 41492)
+++ 
mmbase/trunk/applications/vpro-wizards/src/main/java/org/mmbase/applications/vprowizards/spring/BasicCommand.java
   2010-03-18 10:23:01 UTC (rev 41493)
@@ -42,10 +42,10 @@
                                HttpServletResponse response, ResultContainer 
resultContainer) {
         //we only iterate over the actions until there is an error
         ACTIONS:
-        for(String actionMapping :  actions.keySet()){
-            Map<String, Action> mappedActions = actions.get(actionMapping);
-            log.service(mappedActions.size() + " actions found for mapping '" 
+ actionMapping + "'");
-            for (Action action: mappedActions.values()){
+        for(Map.Entry<String, Map<String, Action>> entry  : 
actions.entrySet()){
+            Map<String, Action> mappedActions = entry.getValue();
+            log.debug(mappedActions.size() + " actions found for mapping '" + 
entry.getKey() + "': " + mappedActions);
+            for (Action action: mappedActions.values()){ // Not entirely sure 
then where the key is good for then
                 action.process(resultContainer);
                 if(resultContainer.hasGlobalErrors() || 
resultContainer.hasFieldErrors()){
                     break ACTIONS;

_______________________________________________
Cvs mailing list
[email protected]
http://lists.mmbase.org/mailman/listinfo/cvs

Reply via email to