svn commit: r1060261 - /ofbiz/trunk/specialpurpose/pos/src/org/ofbiz/pos/PosTransaction.java

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r1060261 - /ofbiz/trunk/specialpurpose/pos/src/org/ofbiz/pos/PosTransaction.java

jleroux@apache.org
Author: jleroux
Date: Tue Jan 18 09:18:19 2011
New Revision: 1060261

URL: http://svn.apache.org/viewvc?rev=1060261&view=rev
Log:
Fix an issue reported by Paul Foxworthy in OFBIZ-4114

As it's not a service but a public method we should not use return ServiceUtil.returnError here but return null (return was missing Paul reported)

Modified:
    ofbiz/trunk/specialpurpose/pos/src/org/ofbiz/pos/PosTransaction.java

Modified: ofbiz/trunk/specialpurpose/pos/src/org/ofbiz/pos/PosTransaction.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/pos/src/org/ofbiz/pos/PosTransaction.java?rev=1060261&r1=1060260&r2=1060261&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/pos/src/org/ofbiz/pos/PosTransaction.java (original)
+++ ofbiz/trunk/specialpurpose/pos/src/org/ofbiz/pos/PosTransaction.java Tue Jan 18 09:18:19 2011
@@ -1274,7 +1274,7 @@ public class PosTransaction implements S
             shoppingLists = delegator.findList("ShoppingList", null, null, null, null, false);
         } catch (GenericEntityException e) {
             Debug.logError(e, module);
-            ServiceUtil.returnError("Error running initLowLevelCode: " + e.getMessage());
+            return null;
         }
 
         if (shoppingLists == null) {