You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by ja...@apache.org on 2009/05/30 23:16:07 UTC
svn commit: r780330 -
/ofbiz/trunk/specialpurpose/googlecheckout/src/org/ofbiz/googlecheckout/GoogleCheckoutResponseEvents.java
Author: jaz
Date: Sat May 30 21:16:02 2009
New Revision: 780330
URL: http://svn.apache.org/viewvc?rev=780330&view=rev
Log:
now sending error message back to google
Modified:
ofbiz/trunk/specialpurpose/googlecheckout/src/org/ofbiz/googlecheckout/GoogleCheckoutResponseEvents.java
Modified: ofbiz/trunk/specialpurpose/googlecheckout/src/org/ofbiz/googlecheckout/GoogleCheckoutResponseEvents.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/googlecheckout/src/org/ofbiz/googlecheckout/GoogleCheckoutResponseEvents.java?rev=780330&r1=780329&r2=780330&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/googlecheckout/src/org/ofbiz/googlecheckout/GoogleCheckoutResponseEvents.java (original)
+++ ofbiz/trunk/specialpurpose/googlecheckout/src/org/ofbiz/googlecheckout/GoogleCheckoutResponseEvents.java Sat May 30 21:16:02 2009
@@ -56,10 +56,10 @@
document = Utils.newDocumentFromInputStream(request.getInputStream());
} catch (CheckoutException e) {
Debug.logError(e, module);
- sendResponse(response, null, true);
+ sendResponse(response, null, e);
} catch (IOException e) {
Debug.logError(e, module);
- sendResponse(response, null, true);
+ sendResponse(response, null, e);
}
// check the document type and process
@@ -71,10 +71,10 @@
String serialNumber = info.getSerialNumber();
try {
helper.createOrder(info, ProductStoreWorker.getProductStoreId(request), ProductStoreWorker.getStoreLocale(request));
- sendResponse(response, serialNumber, false);
+ sendResponse(response, serialNumber, null);
} catch (GeneralException e) {
Debug.logError(e, module);
- sendResponse(response, serialNumber, true);
+ sendResponse(response, serialNumber, e);
return null;
}
} else if ("order-state-change-notification".equals(nodeValue)) {
@@ -82,10 +82,10 @@
String serialNumber = info.getSerialNumber();
try {
helper.processStateChange(info);
- sendResponse(response, serialNumber, false);
+ sendResponse(response, serialNumber, null);
} catch (GeneralException e) {
Debug.logError(e, module);
- sendResponse(response, serialNumber, true);
+ sendResponse(response, serialNumber, e);
return null;
}
} else if ("risk-information-notification".equals(nodeValue)) {
@@ -93,10 +93,10 @@
String serialNumber = info.getSerialNumber();
try {
helper.processRiskNotification(info);
- sendResponse(response, serialNumber, false);
+ sendResponse(response, serialNumber, null);
} catch (GeneralException e) {
Debug.logError(e, module);
- sendResponse(response, serialNumber, true);
+ sendResponse(response, serialNumber, e);
return null;
}
} else if ("authorization-amount-notification".equals(nodeValue)) {
@@ -104,10 +104,10 @@
String serialNumber = info.getSerialNumber();
try {
helper.processAuthNotification(info);
- sendResponse(response, serialNumber, false);
+ sendResponse(response, serialNumber, null);
} catch (GeneralException e) {
Debug.logError(e, module);
- sendResponse(response, serialNumber, true);
+ sendResponse(response, serialNumber, e);
return null;
}
} else if ("charge-amount-notification".equals(nodeValue)) {
@@ -115,10 +115,10 @@
String serialNumber = info.getSerialNumber();
try {
helper.processChargeNotification(info);
- sendResponse(response, serialNumber, false);
+ sendResponse(response, serialNumber, null);
} catch (GeneralException e) {
Debug.logError(e, module);
- sendResponse(response, serialNumber, true);
+ sendResponse(response, serialNumber, e);
return null;
}
} else if ("chargeback-amount-notification".equals(nodeValue)) {
@@ -126,10 +126,10 @@
String serialNumber = info.getSerialNumber();
try {
helper.processChargeBackNotification(info);
- sendResponse(response, serialNumber, false);
+ sendResponse(response, serialNumber, null);
} catch (GeneralException e) {
Debug.logError(e, module);
- sendResponse(response, serialNumber, true);
+ sendResponse(response, serialNumber, e);
return null;
}
} else if ("refund-amount-notification".equals(nodeValue)) {
@@ -137,10 +137,10 @@
String serialNumber = info.getSerialNumber();
try {
helper.processRefundNotification(info);
- sendResponse(response, serialNumber, false);
+ sendResponse(response, serialNumber, null);
} catch (GeneralException e) {
Debug.logError(e, module);
- sendResponse(response, serialNumber, true);
+ sendResponse(response, serialNumber, e);
return null;
}
} else {
@@ -151,10 +151,10 @@
return null;
}
- private static void sendResponse(HttpServletResponse response, String serialNumber, boolean error) {
- if (error) {
+ private static void sendResponse(HttpServletResponse response, String serialNumber, Exception error) {
+ if (error != null) {
try {
- response.sendError(500);
+ response.sendError(HttpServletResponse.SC_INTERNAL_SERVER_ERROR, error.getMessage());
} catch (IOException e) {
Debug.logError(e, module);
}