svn commit: r815916 - /ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceUtil.java

[doogie-3]

Author: doogie
Date: Wed Sep 16 18:10:49 2009
New Revision: 815916

URL: http://svn.apache.org/viewvc?rev=815916&view=rev
Log:
Apply patch from OFBIZ-1023, adding returnSuccess(List); the javadoc
from the patch wasn't used, however.

Modified:
    ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceUtil.java

Modified: ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceUtil.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceUtil.java?rev=815916&r1=815915&r2=815916&view=diff
==============================================================================
--- ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceUtil.java (original)
+++ ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceUtil.java Wed Sep 16 18:10:49 2009
@@ -161,6 +161,13 @@
         return returnMessage(ModelService.RESPOND_SUCCESS, null);
     }
 
+    /** A small routine used all over to improve code efficiency, make a result map with the message and the success response code */
+    public static Map<String, Object> returnSuccess(List<String> successMessageList) {
+        Map result = returnMessage(ModelService.RESPOND_SUCCESS, null);
+        result.put(ModelService.SUCCESS_MESSAGE_LIST, successMessageList);
+        return result;
+    }
+
     /** A small routine to make a result map with the message and the response code
      * NOTE: This brings out some bad points to our message convention: we should be using a single message or message list
      *  and what type of message that is should be determined by the RESPONSE_MESSAGE (and there's another annoyance, it should be RESPONSE_CODE)