Author: mrisaliti
Date: Sat Jan 22 20:40:52 2011
New Revision: 1062264

URL: http://svn.apache.org/viewvc?rev=1062264&view=rev
Log:
Avoid a null pointer exception

Modified:
    
ofbiz/trunk/applications/content/src/org/ofbiz/content/data/DataServices.java

Modified: 
ofbiz/trunk/applications/content/src/org/ofbiz/content/data/DataServices.java
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/applications/content/src/org/ofbiz/content/data/DataServices.java?rev=1062264&r1=1062263&r2=1062264&view=diff
==============================================================================
--- 
ofbiz/trunk/applications/content/src/org/ofbiz/content/data/DataServices.java 
(original)
+++ 
ofbiz/trunk/applications/content/src/org/ofbiz/content/data/DataServices.java 
Sat Jan 22 20:40:52 2011
@@ -213,6 +213,9 @@ public class DataServices {
 
         // obtain a reference to the file
         File file = null;
+        if (UtilValidate.isEmpty(objectInfo)) {
+            return ServiceUtil.returnError(UtilProperties.getMessage(resource, 
"ContentUnableObtainReferenceToFile", UtilMisc.toMap("objectInfo", ""), 
locale));
+        }
         if (UtilValidate.isEmpty(dataResourceTypeId) || 
dataResourceTypeId.equals("LOCAL_FILE") || 
dataResourceTypeId.equals("LOCAL_FILE_BIN")) {
             file = new File(objectInfo);
             if (!file.isAbsolute()) {
@@ -264,7 +267,7 @@ public class DataServices {
             return ServiceUtil.returnError(UtilProperties.getMessage(resource, 
"ContentNoContentFilePassed", UtilMisc.toMap("fileName", 
file.getAbsolutePath()), locale));
         }
 
-        Map result = ServiceUtil.returnSuccess();
+        Map<String, Object> result = ServiceUtil.returnSuccess();
         return result;
     }
 


Reply via email to