I agree with this. If a configuration is missing that is vital for operation but easy to miss (like these callbacks) and error is better than the data getting sent where it shouldn't be sent...
-David On Dec 15, 2007, at 5:08 PM, BJ Freeman wrote:
my thinking is it is better to error because it has not been configuredproperly.when the config file is read, the configuration can be check for [] andthrow an error. Jacques Le Roux sent the following on 12/15/2007 3:45 PM:Yes I asked myself the same question. I gived up thinking that after all so far it's the official demo server...Jacques De : "BJ Freeman" <[EMAIL PROTECTED]>is this correct to use the demo for a return address. should it not be just localhost till the person configures it? Should not the gateways be in a sandbox configurations? [EMAIL PROTECTED] sent the following on 12/14/2007 1:57 PM:Author: mrisaliti Date: Fri Dec 14 13:57:29 2007 New Revision: 604311 URL: http://svn.apache.org/viewvc?rev=604311&view=rev Log: Applied fix from trunk for revision: 604030 Modified:ofbiz/branches/release4.0/applications/accounting/config/ payment.properties ofbiz/branches/release4.0/applications/accounting/src/org/ ofbiz/accounting/thirdparty/paypal/PayPalEvents.javaModified: ofbiz/branches/release4.0/applications/accounting/ config/payment.propertiesURL:http://svn.apache.org/viewvc/ofbiz/branches/release4.0/applications/accounting/config/payment.properties?rev=604311&r1=604310&r2=604311&view=diff= = = = = = = = = = = =================================================================== --- ofbiz/branches/release4.0/applications/accounting/config/ payment.properties (original) +++ ofbiz/branches/release4.0/applications/accounting/config/ payment.properties Fri Dec 14 13:57:29 2007@@ -239,25 +239,29 @@ [EMAIL PROTECTED] # PayPal Notify URL -payment.paypal.notify=http://www.ofbiz.org:8080/ecommerce/control/payPalNotify +payment.paypal.notify=http://demo.hotwaxmedia.com/ecommerce/control/payPalNotify # PayPal Return URL -payment.paypal.return=http://www.ofbiz.org:8080/ecommerce/control/orderhistory +payment.paypal.return=http://demo.hotwaxmedia.com/ecommerce/control/orderhistory # PayPal Return On Cancel URL -payment.paypal.cancelReturn=http://www.ofbiz.org:8080/ecommerce/control/payPalCancel/main +payment.paypal.cancelReturn=http://demo.hotwaxmedia.com/ecommerce/control/payPalCancel/main # Image To Use On PayPal -payment.paypal.image=http://www.ofbiz:org:8080/images/ofb_logo.gif +payment.paypal.image=http://demo.hotwaxmedia.com/images/ofbiz_logo.jpg # Thank-You / Confirm Order Template (rendered via Freemarker) payment.paypal.confirmTemplate=/order/emailconfirmation.ftl -# PayPal Redirect URL -payment.paypal.redirect=https://www.paypal.com/cgi-bin/webscr - -# PayPal Confirm URL (JSSE must be configured to use SSL) -payment.paypal.confirm=http://www.paypal.com/cgi-bin/webscr +# Production PayPal Redirect URL +#payment.paypal.redirect=https://www.paypal.com/cgi-bin/webscr +# Sandbox PayPal Redirect URL +payment.paypal.redirect=https://www.sandbox.paypal.com/us/cgi-bin/webscr ++# Production PayPal Confirm URL (JSSE must be configured to use SSL)+#payment.paypal.confirm=http://www.paypal.com/cgi-bin/webscr +# Sandbox PayPal Confirm URL (JSSE must be configured to use SSL) +payment.paypal.confirm=https://www.sandbox.paypal.com/us/cgi-bin/webscr ############################################ # PCCharge ConfigurationModified: ofbiz/branches/release4.0/applications/accounting/src/ org/ofbiz/accounting/thirdparty/paypal/PayPalEvents.javaURL:http://svn.apache.org/viewvc/ofbiz/branches/release4.0/applications/accounting/src/org/ofbiz/accounting/thirdparty/paypal/PayPalEvents.java?rev=604311&r1=604310&r2=604311&view=diff= = = = = = = = = = = =================================================================== --- ofbiz/branches/release4.0/applications/accounting/src/org/ ofbiz/accounting/thirdparty/paypal/PayPalEvents.java (original) +++ ofbiz/branches/release4.0/applications/accounting/src/org/ ofbiz/accounting/thirdparty/paypal/PayPalEvents.java Fri Dec 1413:57:29 2007@@ -186,7 +186,11 @@ // get the confirm URLString confirmUrl = UtilProperties.getPropertyValue(configString, "payment.paypal.confirm");- if (confirmUrl == null) { + + // get the redirect URL+ String redirectUrl = UtilProperties.getPropertyValue(configString, "payment.paypal.redirect");+ + if (confirmUrl == null || redirectUrl == null) {Debug.logError("Payment properties is not configured properly, no confirm URL defined!", module); request.setAttribute("_ERROR_MESSAGE_", "PayPal has not been configured, please contact customer service.");return "error"; @@ -201,7 +205,7 @@ try { String str = UtilHttp.urlEncodeArgs(parametersMap);- URL u = new URL("http://www.paypal.com/cgi-bin/ webscr");+ URL u = new URL(redirectUrl); URLConnection uc = u.openConnection(); uc.setDoOutput(true);uc.setRequestProperty("Content-Type", "application/x- www-form-urlencoded");@@ -279,7 +283,7 @@String paymentStatus = request.getParameter("payment_status");// attempt to start a transaction - boolean okay = false; + boolean okay = true; boolean beganTransaction = false; try { beganTransaction = TransactionUtil.begin(); @@ -424,6 +428,8 @@paymentPreference.set("maxAmount", new Double(paymentAmount));if (paymentStatus.equals("Completed")) { paymentPreference.set("statusId", "PAYMENT_RECEIVED"); + } else if (paymentStatus.equals("Pending")) {+ paymentPreference.set("statusId", "PAYMENT_NOT_RECEIVED");} else { paymentPreference.set("statusId", "PAYMENT_CANCELLED"); } @@ -450,6 +456,13 @@ response.set("transactionDate", authDate); toStore.add(response); + try { + delegator.storeAll(toStore); + } catch (GenericEntityException e) {+ Debug.logError(e, "Cannot set payment preference/ payment info", module);+ return false; + } + // create a payment record too Map results = null; try { @@ -467,12 +480,6 @@ return false; } - try { - delegator.storeAll(toStore); - } catch (GenericEntityException e) {- Debug.logError(e, "Cannot set payment preference/ payment info", module);- return false; - } return true; }
smime.p7s
Description: S/MIME cryptographic signature
