Project:
UbercartCategory:
bug reportPriority:
criticalAssigned:
UnassignedStatus:
fixedToday, after updating to Alpha-7c i have got a Paypal Express Checkout Order, which seems to fail to calculate the shipping cost.
This is when i realised that the paypal module has not been updated to the latest panels code.
I assumed the guys will be busy for 7d release. Here is the patch that seems to be working for me. (Took lot of time, but all part of the learning)
The code needs review.
| Attachment | Size |
|---|---|
| uc_paypal.diff | 2.25 KB |
Re: Paypal Express Checkout Shipping Quote
Oof... this reminds me I need to update PayPal in other areas, too. Would you mind explaining your changes to the validate/submit functions? You seem to have done more than just update for the new pane system.
Re: Re: Paypal Express Checkout Shipping Quote
I also had to change this around line 780:
<?php// Redirects if a customer selects PayPal Express Checkout as a payment method.
function uc_paypal_ec_checkout($form_id, $form_values) {
if ($form_values['panes']['payment']['pane']['payment_method'] != 'paypal_ec') {
return;
}
?>
Should remove the extra ['pane']:
<?php// Redirects if a customer selects PayPal Express Checkout as a payment method.
function uc_paypal_ec_checkout($form_id, $form_values) {
if ($form_values['panes']['payment']['payment_method'] != 'paypal_ec') {
return;
}
?>
There are a few other changes, too... you can get them when you update to Alpha 7d.
Should that be ??
<?php// Redirects if a customer selects PayPal Express Checkout as a payment method.
function uc_paypal_ec_checkout($form_id, $form_values) {
if ($form_values['panes']['payment']['payment_method'] != 'paypal_ec') {
return;
}
?>
Re: Re: Paypal Express Checkout Shipping Quote
The thing was strange for me too..
The quote method also changed from the previous code. SO i had to figure out the things. Maybe Lyle could bring in more light whether i have done it right .
Re: Re: Re: Paypal Express Checkout Shipping Quote
Honestly, it works fine for me so I'm just going to test a few more times and let it be.
Re: Re: Re: Re: Paypal Express Checkout Shipping Quote
Do i need to make any changes to any of the other paypal code(i suppose not..) ??
I had few orders this morning and i think everything went smooth.
I didnt understand ur concenrn abt the pane??
Re: Re: Re: Re: Re: Paypal Express Checkout Shipping Quote
I see it now..
Re: Re: Re: Paypal Express Checkout Shipping Quote
Heh, I usually stop paying attention when I see "PayPal", so it took me a while to go look at your diff.
It looks like it should work. I should break this habit of just changing things without telling anybody. But I guess we're still in alpha for a little while longer.
Re: Re: Re: Re: Paypal Express Checkout Shipping Quote
Or just grep the Uber-source to see where your changes might apply...