You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jo...@apache.org on 2007/09/03 12:38:40 UTC

svn commit: r572275 - /ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceDispatcher.java

Author: jonesde
Date: Mon Sep  3 03:38:34 2007
New Revision: 572275

URL: http://svn.apache.org/viewvc?rev=572275&view=rev
Log:
Fixed NPE caused by passing in a null context, now creates an empty one for use throughout the call

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

Modified: ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceDispatcher.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceDispatcher.java?rev=572275&r1=572274&r2=572275&view=diff
==============================================================================
--- ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceDispatcher.java (original)
+++ ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceDispatcher.java Mon Sep  3 03:38:34 2007
@@ -242,6 +242,10 @@
                 "/" + modelService.invoke + "] (" + modelService.engineName + ")", module);
         }
 
+        if (context == null) {
+            context = FastMap.newInstance();
+        }
+
         // setup the result map
         Map result = FastMap.newInstance();
         boolean isFailure = false;
@@ -482,6 +486,10 @@
         if (Debug.verboseOn()) {
             Debug.logVerbose("[ServiceDispatcher.runAsync] : prepareing service " + service.name + " [" + service.location + "/" + service.invoke +
                 "] (" + service.engineName + ")", module);
+        }
+        
+        if (context == null) {
+            context = FastMap.newInstance();
         }
 
         // setup the result map