Egg on face. :D just shows I should not code at night.
Jacques Le Roux sent the following on 12/21/2007 5:58 AM: > From: "BJ Freeman" <[EMAIL PROTECTED]> >> please explain IIRC. >> not sure were +129 is, I am guessing UtilValidate.isEmpty(payPalAccount) >> UtilValidate.isEmpty(payPalAccount) is not in my patch. >> Don't see payPalAccount in the properties file > > Trunk and release > # PayPal E-Mail address > [EMAIL PROTECTED] > payment.paypal.business= > > String payPalAccount = UtilProperties.getPropertyValue(configString, > "payment.paypal.business"); > > Jacques > >> this is minor >>>>>> [EMAIL PROTECTED] >>>>>> [EMAIL PROTECTED] >> should be >>>>>> [EMAIL PROTECTED] >>>>>> +#payment.paypal.business=[your emailaddress at paypal like >> [EMAIL PROTECTED] >> >> so letme go thru my patch and see if I can make it work for you. >> you can revert what is done. >> :) >> >> Jacques Le Roux sent the following on 12/21/2007 2:09 AM: >>> From: "BJ Freeman" <[EMAIL PROTECTED]> >>> [...] >>>> This is not only for paypal but any property (method) that send data >>>> erroneously to somewhere and not make the user aware it is happening >>> >>> Yes, but IIRC there are currently only code for dealing with paypal. See >>> UtilValidate.isEmpty at the end of this message >>> (PayPalEvents.java[129+...] >>> >>> Jacques >>> >>>> Jacopo Cappellato sent the following on 12/20/2007 9:31 PM: >>>>> Premise: I don't know much about PayPalEvents.java and I didn't really >>>>> reviewed BJ Freeman's patch. >>>>> >>>>> That said, I have a question: I've noticed that the change to the >>>>> PayPalEvents.java slightly modify the logic of the event... are we >>>>> sure >>>>> that it is something we need? What is it supposed to fix/improve >>>>> exactly? I cannot understand this (due to my limited knowledge of >>>>> PayPal >>>>> events) simply reading the comments in Jira and the svn log. >>>>> >>>>> Thanks, >>>>> >>>>> Jacopo >>>>> >>>>> >>>>> [EMAIL PROTECTED] wrote: >>>>>> Author: jleroux >>>>>> Date: Thu Dec 20 13:21:47 2007 >>>>>> New Revision: 606031 >>>>>> >>>>>> URL: http://svn.apache.org/viewvc?rev=606031&view=rev >>>>>> Log: >>>>>> Done by hand from an unusable patch of BJ Freeman "Authorize.NET >>>>>> Payment Gateway Implementation" >>>>>> (https://issues.apache.org/jira/browse/OFBIZ-274) - OFBIZ-274 >>>>>> >>>>>> Modified: >>>>>> ofbiz/trunk/applications/accounting/config/payment.properties >>>>>> >>>>>> ofbiz/trunk/applications/accounting/src/org/ofbiz/accounting/thirdparty/paypal/PayPalEvents.java >>>>>> >>>>>> >>>>>> >>>>>> >>>>>> Modified: >>>>>> ofbiz/trunk/applications/accounting/config/payment.properties >>>>>> URL: >>>>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/accounting/config/payment.properties?rev=606031&r1=606030&r2=606031&view=diff >>>>>> >>>>>> >>>>>> >>>>>> ============================================================================== >>>>>> >>>>>> >>>>>> >>>>>> --- ofbiz/trunk/applications/accounting/config/payment.properties >>>>>> (original) >>>>>> +++ ofbiz/trunk/applications/accounting/config/payment.properties Thu >>>>>> Dec 20 13:21:47 2007 >>>>>> @@ -240,33 +240,39 @@ >>>>>> >>>>>> # PayPal E-Mail address >>>>>> [EMAIL PROTECTED] >>>>>> [EMAIL PROTECTED] >>>>>> +payment.paypal.business= >>>>>> >>>>>> # PayPal Notify URL >>>>>> -payment.paypal.notify=http://[yourServerName]/ecommerce/control/payPalNotify >>>>>> >>>>>> >>>>>> >>>>>> +#payment.paypal.notify=http://[yourServerName]/ecommerce/control/payPalNotify >>>>>> >>>>>> >>>>>> >>>>>> +payment.paypal.notify= >>>>>> >>>>>> # PayPal Return URL >>>>>> -payment.paypal.return=http://[yourServerName]/ecommerce/control/orderhistory >>>>>> >>>>>> >>>>>> >>>>>> +#payment.paypal.return=http://[yourServerName]/ecommerce/control/orderhistory >>>>>> >>>>>> >>>>>> >>>>>> +payment.paypal.return= >>>>>> >>>>>> # PayPal Return On Cancel URL >>>>>> -payment.paypal.cancelReturn=http://[yourServerName]/ecommerce/control/payPalCancel/main >>>>>> >>>>>> >>>>>> >>>>>> +#payment.paypal.cancelReturn=http://[yourServerName]/ecommerce/control/payPalCancel/main >>>>>> >>>>>> >>>>>> >>>>>> +payment.paypal.cancelReturn= >>>>>> >>>>>> # Image To Use On PayPal >>>>>> #payment.paypal.image=http://[yourServerName]/images/ofbiz_logo.jpg >>>>>> -payment.paypal.image=http://[yourServerName]/images/[your_logo].jpg >>>>>> +payment.paypal.image= >>>>>> >>>>>> # Thank-You / Confirm Order Template (rendered via Freemarker) >>>>>> -payment.paypal.confirmTemplate=/order/emailconfirmation.ftl >>>>>> +#payment.paypal.confirmTemplate=/order/emailconfirmation.ftl >>>>>> +payment.paypal.confirmTemplate= >>>>>> >>>>>> # 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 >>>>>> >>>>>> >>>>>> +#payment.paypal.redirect=https://www.sandbox.paypal.com/us/cgi-bin/webscr >>>>>> >>>>>> >>>>>> >>>>>> +payment.paypal.redirect= >>>>>> >>>>>> # 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 >>>>>> >>>>>> >>>>>> +#payment.paypal.confirm=https://www.sandbox.paypal.com/us/cgi-bin/webscr >>>>>> >>>>>> >>>>>> +payment.paypal.confirm= >>>>>> >>>>>> ############################################ >>>>>> # PCCharge Configuration >>>>>> >>>>>> Modified: >>>>>> ofbiz/trunk/applications/accounting/src/org/ofbiz/accounting/thirdparty/paypal/PayPalEvents.java >>>>>> >>>>>> >>>>>> >>>>>> URL: >>>>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/accounting/src/org/ofbiz/accounting/thirdparty/paypal/PayPalEvents.java?rev=606031&r1=606030&r2=606031&view=diff >>>>>> >>>>>> >>>>>> >>>>>> ============================================================================== >>>>>> >>>>>> >>>>>> >>>>>> --- >>>>>> ofbiz/trunk/applications/accounting/src/org/ofbiz/accounting/thirdparty/paypal/PayPalEvents.java >>>>>> >>>>>> >>>>>> (original) >>>>>> +++ >>>>>> ofbiz/trunk/applications/accounting/src/org/ofbiz/accounting/thirdparty/paypal/PayPalEvents.java >>>>>> >>>>>> >>>>>> Thu Dec 20 13:21:47 2007 >>>>>> @@ -129,7 +129,12 @@ >>>>>> // get the paypal account >>>>>> String payPalAccount = >>>>>> UtilProperties.getPropertyValue(configString, >>>>>> "payment.paypal.business"); >>>>>> - if (UtilValidate.isEmpty(notifyUrl) || >>>>>> UtilValidate.isEmpty(returnUrl) || >>>>>> UtilValidate.isEmpty(cancelReturnUrl)) { >>>>>> + if (UtilValidate.isEmpty(redirectUrl) + || >>>>>> UtilValidate.isEmpty(notifyUrl) + || >>>>>> UtilValidate.isEmpty(returnUrl) + || >>>>>> UtilValidate.isEmpty(cancelReturnUrl) + || >>>>>> UtilValidate.isEmpty(imageUrl) >>>>>> + || UtilValidate.isEmpty(payPalAccount) ) { >>>>>> Debug.logError("Payment properties is not configured >>>>>> properly, some notify URL from PayPal is not correctly defined!", >>>>>> module); >>>>>> request.setAttribute("_ERROR_MESSAGE_", >>>>>> UtilProperties.getMessage(resource, >>>>>> "payPalEvents.problemsGettingMerchantConfiguration", locale)); >>>>>> return "error"; >>>>>> >>>>> >>>>> >>>>> >>>>> >>>> >>>> >>> >>> >>> >>> >> >> > > > >