svn commit: r931425 - in /ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore: EbayEvents.java EbayStore.java EbayStoreAutoPreferences.java EbayStoreHelper.java EbayStoreInventoryServices.java EbayStoreOptions.java

[hansbak-2]

Author: hansbak
Date: Wed Apr  7 04:00:18 2010
New Revision: 931425

URL: http://svn.apache.org/viewvc?rev=931425&view=rev
Log:
Ebaystore: fixed error  in createErrorLogMessage method, call to insertErrorMessagesFromEbay service.

Modified:
    ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayEvents.java
    ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStore.java
    ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java
    ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreHelper.java
    ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreInventoryServices.java
    ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreOptions.java

Modified: ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayEvents.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayEvents.java?rev=931425&r1=931424&r2=931425&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayEvents.java (original)
+++ ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayEvents.java Wed Apr  7 04:00:18 2010
@@ -891,6 +891,8 @@ public class EbayEvents {
         Map<String,Object> requestParams = UtilHttp.getParameterMap(request);
         Locale locale = UtilHttp.getLocale(request);
         String productStoreId = (String) requestParams.get("productStoreId");
+        HttpSession session = request.getSession(true);
+        GenericValue userLogin = (GenericValue) session.getAttribute("userLogin");
 
         try {
             ApiContext apiContext = EbayStoreHelper.getApiContext(productStoreId, locale, delegator);
@@ -924,7 +926,7 @@ public class EbayEvents {
                                 feesummary = feesummary + dfee;
                             }
                         } else {
-                            EbayStoreHelper.createErrorLogMessage(dispatcher, productStoreId, resp.getAck().toString(), "Verify Item : verifyItemBeforeAdd", resp.getMessage());
+                            EbayStoreHelper.createErrorLogMessage(userLogin, dispatcher, productStoreId, resp.getAck().toString(), "Verify Item : verifyItemBeforeAdd", resp.getErrors(0).getLongMessage());
                         }
                     }
                 }

Modified: ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStore.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStore.java?rev=931425&r1=931424&r2=931425&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStore.java (original)
+++ ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStore.java Wed Apr  7 04:00:18 2010
@@ -633,6 +633,7 @@ public class EbayStore {
     /* Get store output */
     public static Map<String,Object> getEbayStoreOutput(DispatchContext dctx, Map<String,Object> context) {
         Locale locale = (Locale) context.get("locale");
+        GenericValue userLogin = (GenericValue) context.get("userLogin");
         Delegator delegator = dctx.getDelegator();
         Map<String,Object> result = FastMap.newInstance();
         StoreType returnedStoreType = null;
@@ -647,10 +648,10 @@ public class EbayStore {
                 List<GenericValue> productStoreRoles = delegator.findByAnd("ProductStoreRole", UtilMisc.toMap("productStoreId", context.get("productStoreId").toString(),"roleTypeId","EBAY_ACCOUNT"));
                 if (productStoreRoles.size() != 0) {
                     partyId=  (String)productStoreRoles.get(0).get("partyId");
-                    List<GenericValue> userLogin = delegator.findByAnd("UserLogin", UtilMisc.toMap("partyId", partyId));
-                    if (userLogin.size() != 0) {
-                        userLoginId = (String)userLogin.get(0).get("userLoginId");
-                        password = (String)userLogin.get(0).get("currentPassword");
+                    List<GenericValue> userLogins = delegator.findByAnd("UserLogin", UtilMisc.toMap("partyId", partyId));
+                    if (userLogins.size() != 0) {
+                        userLoginId = (String)userLogins.get(0).get("userLoginId");
+                        password = (String)userLogins.get(0).get("currentPassword");
                     }
                     
                 }
@@ -854,7 +855,7 @@ public class EbayStore {
 
                     result.put("ebayStore", ebayResp);
                 } else {
-                    EbayStoreHelper.createErrorLogMessage(dctx.getDispatcher(), context.get("productStoreId").toString(), resp.getAck().toString(), "Get store : getEbayStoreOutput", resp.getMessage());
+                    EbayStoreHelper.createErrorLogMessage(userLogin, dctx.getDispatcher(), context.get("productStoreId").toString(), resp.getAck().toString(), "Get store : getEbayStoreOutput", resp.getErrors(0).getLongMessage());
                     result.put(ModelService.RESPONSE_MESSAGE, ModelService.RESPOND_ERROR);
                     result.put(ModelService.ERROR_MESSAGE, resp.getAck().toString() +":"+ resp.getMessage());
                 }
@@ -890,7 +891,7 @@ public class EbayStore {
     }
 
     public static Map<String,Object> retrieveThemeColorSchemeByThemeId(DispatchContext dctx, Map<String,Object> context) {
-
+        GenericValue userLogin = (GenericValue) context.get("userLogin");
         Locale locale = (Locale) context.get("locale");
         Delegator delegator = dctx.getDelegator();
         Map<String,Object> result = FastMap.newInstance();
@@ -941,7 +942,7 @@ public class EbayStore {
                     }
                     result.put("storeFontScheme", storeColorSchemeMap);
                 } else {
-                    EbayStoreHelper.createErrorLogMessage(dctx.getDispatcher(), context.get("productStoreId").toString(), resp.getAck().toString(), "Get store option : retrieveThemeColorSchemeByThemeId", resp.getMessage());
+                    EbayStoreHelper.createErrorLogMessage(userLogin, dctx.getDispatcher(), context.get("productStoreId").toString(), resp.getAck().toString(), "Get store option : retrieveThemeColorSchemeByThemeId", resp.getErrors(0).getLongMessage());
                 }
             }
         } catch (ApiException e) {
@@ -955,6 +956,7 @@ public class EbayStore {
     }
 
     public static Map<String,Object> retrievePredesignedLogoOption(DispatchContext dctx, Map<String,Object> context) {
+        GenericValue userLogin = (GenericValue) context.get("userLogin");
         Map<String,Object> result = FastMap.newInstance();
         Locale locale = (Locale) context.get("locale");
         Delegator delegator = dctx.getDelegator();
@@ -985,7 +987,7 @@ public class EbayStore {
                     result = ServiceUtil.returnSuccess("load store logo data success..");
                     result.put("storeLogoOptList", logoList);
                 } else {
-                    EbayStoreHelper.createErrorLogMessage(dctx.getDispatcher(), context.get("productStoreId").toString(), resp.getAck().toString(), "Get store option : retrievePredesignedLogoOption", resp.getMessage());
+                    EbayStoreHelper.createErrorLogMessage(userLogin, dctx.getDispatcher(), context.get("productStoreId").toString(), resp.getAck().toString(), "Get store option : retrievePredesignedLogoOption", resp.getErrors(0).getLongMessage());
                 }
             }
         } catch (ApiException e) {
@@ -1000,6 +1002,7 @@ public class EbayStore {
 
     public static Map<String,Object> retrieveBasicThemeArray(DispatchContext dctx, Map<String,Object> context) {
         Map<String,Object> result = FastMap.newInstance();
+        GenericValue userLogin = (GenericValue) context.get("userLogin");
         Locale locale = (Locale) context.get("locale");
         Delegator delegator = dctx.getDelegator();
         GetStoreOptionsRequestType req = null;
@@ -1036,7 +1039,7 @@ public class EbayStore {
                     result = ServiceUtil.returnSuccess("load store Basic Theme option data success..");
                     result.put("storeThemeList", themeList);
                 } else {
-                    EbayStoreHelper.createErrorLogMessage(dctx.getDispatcher(), context.get("productStoreId").toString(), resp.getAck().toString(), "Get store option : retrieveBasicThemeArray", resp.getMessage());
+                    EbayStoreHelper.createErrorLogMessage(userLogin, dctx.getDispatcher(), context.get("productStoreId").toString(), resp.getAck().toString(), "Get store option : retrieveBasicThemeArray", resp.getErrors(0).getLongMessage());
                 }
             }
         } catch (ApiException e) {
@@ -1051,6 +1054,7 @@ public class EbayStore {
 
     public static Map<String,Object> retrieveAdvancedThemeArray(DispatchContext dctx, Map<String,Object> context) {
         Map<String,Object> result = FastMap.newInstance();
+        GenericValue userLogin = (GenericValue) context.get("userLogin");
         Locale locale = (Locale) context.get("locale");
         Delegator delegator = dctx.getDelegator();
         GetStoreOptionsRequestType req = null;
@@ -1092,7 +1096,7 @@ public class EbayStore {
                     }
                     result.put("storeAdvancedThemeColorOptList", themeColorList);
                 } else {
-                    EbayStoreHelper.createErrorLogMessage(dctx.getDispatcher(), context.get("productStoreId").toString(), resp.getAck().toString(), "Get store option : retrieveAdvancedThemeArray", resp.getMessage());
+                    EbayStoreHelper.createErrorLogMessage(userLogin, dctx.getDispatcher(), context.get("productStoreId").toString(), resp.getAck().toString(), "Get store option : retrieveAdvancedThemeArray", resp.getErrors(0).getLongMessage());
                 }
                 //this.returnedSubscriptionArray = resp.getSubscriptionArray();
             }
@@ -1108,6 +1112,7 @@ public class EbayStore {
 
     public static Map<String,Object> retrieveStoreFontTheme(DispatchContext dctx, Map<String,Object> context) {
         Map<String,Object> result = FastMap.newInstance();
+        GenericValue userLogin = (GenericValue) context.get("userLogin");
         Locale locale = (Locale) context.get("locale");
         Delegator delegator = dctx.getDelegator();
         GetStoreOptionsRequestType req = null;
@@ -1217,7 +1222,7 @@ public class EbayStore {
                     result = ServiceUtil.returnSuccess("load store Basic Theme option data success..");
                     result.put("advanceFontTheme", advanceFontTheme);
                 } else {
-                    EbayStoreHelper.createErrorLogMessage(dctx.getDispatcher(), context.get("productStoreId").toString(), resp.getAck().toString(), "Get store option : retrieveStoreFontTheme", resp.getMessage());
+                    EbayStoreHelper.createErrorLogMessage(userLogin, dctx.getDispatcher(), context.get("productStoreId").toString(), resp.getAck().toString(), "Get store option : retrieveStoreFontTheme", resp.getErrors(0).getLongMessage());
                 }
             }
         } catch (ApiException e) {

Modified: ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java?rev=931425&r1=931424&r2=931425&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java (original)
+++ ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java Wed Apr  7 04:00:18 2010
@@ -872,7 +872,7 @@ public class EbayStoreAutoPreferences {
                                             String orderId = resp.getOrderID();
                                             Debug.log(":: new order id is = " + orderId);
                                         } else {
-                                            EbayStoreHelper.createErrorLogMessage(dispatcher, productStoreId, resp.getAck().toString(), "Add order : runCombineOrders", resp.getMessage());
+                                            EbayStoreHelper.createErrorLogMessage(userLogin, dispatcher, productStoreId, resp.getAck().toString(), "Add order : runCombineOrders", resp.getErrors(0).getLongMessage());
                                         }
                                     }
                                 }
@@ -1048,6 +1048,7 @@ public class EbayStoreAutoPreferences {
 
     public static Map<String,Object> autoBlockItemsOutOfStock(DispatchContext dctx, Map<String,Object> context) {
         Locale locale = (Locale) context.get("locale");
+        GenericValue userLogin = (GenericValue) context.get("userLogin");
         Delegator delegator = dctx.getDelegator();
         Map<String,Object> result = FastMap.newInstance();
         GetSellingManagerInventoryRequestType req = new GetSellingManagerInventoryRequestType();
@@ -1085,7 +1086,7 @@ public class EbayStoreAutoPreferences {
                                    if (tresp != null && "SUCCESS".equals(tresp.getAck().toString())) {
                                       result = ServiceUtil.returnSuccess();
                                    } else {
-                                       EbayStoreHelper.createErrorLogMessage(dctx.getDispatcher(), context.get("productStoreId").toString(), tresp.getAck().toString(), "Delete selling manager template : autoBlockItemsOutOfStock", tresp.getMessage());
+                                       EbayStoreHelper.createErrorLogMessage(userLogin, dctx.getDispatcher(), context.get("productStoreId").toString(), tresp.getAck().toString(), "Delete selling manager template : autoBlockItemsOutOfStock", tresp.getErrors(0).getLongMessage());
                                    }
                                }
                            }
@@ -1093,7 +1094,7 @@ public class EbayStoreAutoPreferences {
                     }
                     result.put(ModelService.RESPONSE_MESSAGE, ModelService.RESPOND_SUCCESS);
                 } else {
-                    EbayStoreHelper.createErrorLogMessage(dctx.getDispatcher(), context.get("productStoreId").toString(), resp.getAck().toString(), "Get selling manager inventory : autoBlockItemsOutOfStock", resp.getMessage());
+                    EbayStoreHelper.createErrorLogMessage(userLogin, dctx.getDispatcher(), context.get("productStoreId").toString(), resp.getAck().toString(), "Get selling manager inventory : autoBlockItemsOutOfStock", resp.getErrors(0).getLongMessage());
                 }
                 result = ServiceUtil.returnSuccess();
             } catch (ApiException e) {

Modified: ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreHelper.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreHelper.java?rev=931425&r1=931424&r2=931425&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreHelper.java (original)
+++ ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreHelper.java Wed Apr  7 04:00:18 2010
@@ -624,6 +624,7 @@ public class EbayStoreHelper {
     public static Map<String, Object> uploadTrackingInfoBackToEbay(DispatchContext dctx, Map<String, Object> context) {
     Delegator delegator = dctx.getDelegator();
     Locale locale = (Locale) context.get("locale");
+    GenericValue userLogin = (GenericValue) context.get("userLogin");
     String productStoreId = (String) context.get("productStoreId");
     String orderId = (String) context.get("orderId");
     GetOrdersRequestType req = new GetOrdersRequestType();
@@ -693,12 +694,12 @@ public class EbayStoreHelper {
                             if (addResp != null && "SUCCESS".equals(addResp.getAck().toString())) {
                                 Debug.log("Upload tracking code to eBay success...");
                             } else {
-                                createErrorLogMessage(dctx.getDispatcher(), productStoreId, addResp.getAck().toString(), "Update order : uploadTrackingInfoBackToEbay", addResp.getMessage());
+                                createErrorLogMessage(userLogin, dctx.getDispatcher(), productStoreId, addResp.getAck().toString(), "Update order : uploadTrackingInfoBackToEbay", addResp.getErrors(0).getLongMessage());
                             }
                         }
                     }
                 } else {
-                    createErrorLogMessage(dctx.getDispatcher(), productStoreId, resp.getAck().toString(), "Get order : uploadTrackingInfoBackToEbay", resp.getMessage());
+                    createErrorLogMessage(userLogin, dctx.getDispatcher(), productStoreId, resp.getAck().toString(), "Get order : uploadTrackingInfoBackToEbay", resp.getErrors(0).getLongMessage());
                 }
             }
         }
@@ -708,7 +709,7 @@ public class EbayStoreHelper {
     return ServiceUtil.returnSuccess();
     }
 
-    public static void createErrorLogMessage(LocalDispatcher dispatcher, String productStoreId, String ack, String fuction, String errorMessage) {
+    public static void createErrorLogMessage(GenericValue userLogin, LocalDispatcher dispatcher, String productStoreId, String ack, String fuction, String errorMessage) {
         if (!"".equals(productStoreId) && (!"".equals(errorMessage))) {
             try {
                 Map<String, Object> newMap = FastMap.newInstance();
@@ -717,6 +718,7 @@ public class EbayStoreHelper {
                 newMap.put("functionName", fuction);
                 newMap.put("logMessage", errorMessage);
                 newMap.put("createDatetime", UtilDateTime.nowTimestamp());
+                newMap.put("userLogin", userLogin);
                 dispatcher.runSync("insertErrorMessagesFromEbay", newMap);
             } catch (Exception ex) {
                 Debug.log("Error from create error log messages : "+ex.getMessage());

Modified: ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreInventoryServices.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreInventoryServices.java?rev=931425&r1=931424&r2=931425&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreInventoryServices.java (original)
+++ ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreInventoryServices.java Wed Apr  7 04:00:18 2010
@@ -66,6 +66,7 @@ public class EbayStoreInventoryServices
     /*update inventory on ebay site*/
     public static Map<String,Object> updateEbayStoreInventory(DispatchContext dctx, Map<String,Object> context) {
         Map<String,Object> result = FastMap.newInstance();
+        GenericValue userLogin = (GenericValue) context.get("userLogin");
         Locale locale = (Locale) context.get("locale");
         Delegator delegator = dctx.getDelegator();
         GetSellingManagerInventoryRequestType invenReq = null;
@@ -100,7 +101,7 @@ public class EbayStoreInventoryServices
                         }
                     }
                 } else {
-                    EbayStoreHelper.createErrorLogMessage(dctx.getDispatcher(), context.get("productStoreId").toString(), invenResp.getAck().toString(), "GetSellingManagerInventoryCall : updateEbayStoreInventory", invenResp.getMessage());
+                    EbayStoreHelper.createErrorLogMessage(userLogin, dctx.getDispatcher(), context.get("productStoreId").toString(), invenResp.getAck().toString(), "GetSellingManagerInventoryCall : updateEbayStoreInventory", invenResp.getErrors(0).getLongMessage());
                 }
 
                 // checkProduct is true then update detail  but is false do create new one.
@@ -133,6 +134,7 @@ public class EbayStoreInventoryServices
 
     /* add new product and quantity to ebay inventory */
     public static boolean createNewProductInEbayInventoryFolder(DispatchContext dctx, Map<String,Object> context) {
+        GenericValue userLogin = (GenericValue) context.get("userLogin");
         Locale locale = (Locale) context.get("locale");
         Delegator delegator = dctx.getDelegator();
         AddSellingManagerProductRequestType productReq = null;
@@ -162,7 +164,7 @@ public class EbayStoreInventoryServices
                     ebayProductStoreInventory.put("folderId", folderId);
                     ebayProductStoreInventory.store();
                 } else {
-                    EbayStoreHelper.createErrorLogMessage(dctx.getDispatcher(), context.get("productStoreId").toString(), productResp.getAck().toString(), "AddSellingManagerProductCall : createNewProductInEbayInventoryFolder", productResp.getMessage());
+                    EbayStoreHelper.createErrorLogMessage(userLogin, dctx.getDispatcher(), context.get("productStoreId").toString(), productResp.getAck().toString(), "AddSellingManagerProductCall : createNewProductInEbayInventoryFolder", productResp.getErrors(0).getLongMessage());
                     Debug.logError("Fail to  create inventory product ".concat(productId).concat("in productStore ").concat(context.get("productStoreId").toString()).concat(" message from ebay : ").concat(productResp.getMessage()), module);
                 }
             }
@@ -180,6 +182,7 @@ public class EbayStoreInventoryServices
 
     /* update product and quantity to ebay inventory */
     public static boolean updateProductInEbayInventoryFolder(DispatchContext dctx, Map<String,Object> context) {
+        GenericValue userLogin = (GenericValue) context.get("userLogin");
         Locale locale = (Locale) context.get("locale");
         Delegator delegator = dctx.getDelegator();
         ReviseSellingManagerProductRequestType req = null;
@@ -217,7 +220,7 @@ public class EbayStoreInventoryServices
                     ebayProductStoreInventory.put("folderId", folderId);
                     ebayProductStoreInventory.store();
                 } else {
-                    EbayStoreHelper.createErrorLogMessage(dctx.getDispatcher(), context.get("productStoreId").toString(), resp.getAck().toString(), "ReviseSellingManagerProductCall : updateProductInEbayInventoryFolder", resp.getMessage());
+                    EbayStoreHelper.createErrorLogMessage(userLogin, dctx.getDispatcher(), context.get("productStoreId").toString(), resp.getAck().toString(), "ReviseSellingManagerProductCall : updateProductInEbayInventoryFolder", resp.getErrors(0).getLongMessage());
                     Debug.logError("Fail to  update inventory product ".concat(productId).concat("in productStore ").concat(context.get("productStoreId").toString()).concat(" message from ebay : ").concat(resp.getMessage()), module);
                 }
             }
@@ -235,6 +238,7 @@ public class EbayStoreInventoryServices
 
     public static Map<String,Object> getFolderInEbayStoreInventory(DispatchContext dctx, Map<String,Object> context) {
         Map<String,Object> result = FastMap.newInstance();
+        GenericValue userLogin = (GenericValue) context.get("userLogin");
         Locale locale = (Locale) context.get("locale");
         Delegator delegator = dctx.getDelegator();
         GetSellingManagerInventoryFolderRequestType req = null;
@@ -263,7 +267,7 @@ public class EbayStoreInventoryServices
                         folderId = createNewFolderInEbayStoreInventory(dctx,context);
                     }
                 } else {
-                    EbayStoreHelper.createErrorLogMessage(dctx.getDispatcher(), context.get("productStoreId").toString(), resp.getAck().toString(), "GetSellingManagerInventoryFolderCall : getFolderInEbayStoreInventory", resp.getMessage());
+                    EbayStoreHelper.createErrorLogMessage(userLogin, dctx.getDispatcher(), context.get("productStoreId").toString(), resp.getAck().toString(), "GetSellingManagerInventoryFolderCall : getFolderInEbayStoreInventory", resp.getErrors(0).getLongMessage());
                 }
                 result = ServiceUtil.returnSuccess("load ebay store folderId "+folderId+" success..");
             }
@@ -283,6 +287,7 @@ public class EbayStoreInventoryServices
     /*create new folder for export product into inventory.*/
     public static String createNewFolderInEbayStoreInventory(DispatchContext dctx, Map<String,Object> context) {
         Locale locale = (Locale) context.get("locale");
+        GenericValue userLogin = (GenericValue) context.get("userLogin");
         Delegator delegator = dctx.getDelegator();
         AddSellingManagerInventoryFolderRequestType req = null;
         AddSellingManagerInventoryFolderResponseType resp = null;
@@ -296,7 +301,7 @@ public class EbayStoreInventoryServices
                 if (resp != null && "SUCCESS".equals(resp.getAck().toString())) {
                     folderId = String.valueOf(resp.getFolderID());
                 } else {
-                    EbayStoreHelper.createErrorLogMessage(dctx.getDispatcher(), context.get("productStoreId").toString(), resp.getAck().toString(), "AddSellingManagerInventoryFolderCall : createNewFolderInEbayStoreInventory", resp.getMessage());
+                    EbayStoreHelper.createErrorLogMessage(userLogin, dctx.getDispatcher(), context.get("productStoreId").toString(), resp.getAck().toString(), "AddSellingManagerInventoryFolderCall : createNewFolderInEbayStoreInventory", resp.getErrors(0).getLongMessage());
                     Debug.logError("The problem with create new folder on ebay site.", module);
                     return folderId;
                 }
@@ -314,6 +319,7 @@ public class EbayStoreInventoryServices
     /* update inventory status from ebay store inventory */
     public static Map<String,Object> updateEbayInventoryStatusByProductId(DispatchContext dctx, Map<String,Object> context) {
         Locale locale = (Locale) context.get("locale");
+        GenericValue userLogin = (GenericValue) context.get("userLogin");
         Map<String,Object> result = FastMap.newInstance();
         Delegator delegator = dctx.getDelegator();
         String productStoreId = (String)context.get("productStoreId");
@@ -350,7 +356,7 @@ public class EbayStoreInventoryServices
                         }
                     }
                 } else {
-                    EbayStoreHelper.createErrorLogMessage(dctx.getDispatcher(), context.get("productStoreId").toString(), resp.getAck().toString(), "GetSellingManagerInventoryCall : updateEbayInventoryStatusByProductId", resp.getMessage());
+                    EbayStoreHelper.createErrorLogMessage(userLogin, dctx.getDispatcher(), context.get("productStoreId").toString(), resp.getAck().toString(), "GetSellingManagerInventoryCall : updateEbayInventoryStatusByProductId", resp.getErrors(0).getLongMessage());
                     Debug.logError("The problem with get manage inventory detail from ebay site.", module);
                 }
             }

Modified: ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreOptions.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreOptions.java?rev=931425&r1=931424&r2=931425&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreOptions.java (original)
+++ ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreOptions.java Wed Apr  7 04:00:18 2010
@@ -70,6 +70,8 @@ public class EbayStoreOptions {
     
     public static String retrieveThemeColorSchemeByThemeId(HttpServletRequest request, HttpServletResponse response) {
         Locale locale = UtilHttp.getLocale(request);
+        HttpSession session = request.getSession(true);
+        GenericValue userLogin = (GenericValue) session.getAttribute("userLogin");
         Delegator delegator = (Delegator) request.getAttribute("delegator");
         LocalDispatcher dispatcher = (LocalDispatcher) request.getAttribute("dispatcher");
         GetStoreOptionsRequestType req = null;
@@ -129,7 +131,7 @@ public class EbayStoreOptions {
                         i++;
                     }
                 } else {
-                    EbayStoreHelper.createErrorLogMessage(dispatcher, paramMap.get("productStoreId").toString(), resp.getAck().toString(), "GetStoreOptionsCall : retrieveThemeColorSchemeByThemeId", resp.getMessage());
+                    EbayStoreHelper.createErrorLogMessage(userLogin, dispatcher, paramMap.get("productStoreId").toString(), resp.getAck().toString(), "GetStoreOptionsCall : retrieveThemeColorSchemeByThemeId", resp.getErrors(0).getLongMessage());
                 }
             }
         } catch (ApiException e) {