When I use the Paypal Express checkout button on cart page (thus bypassing the UC checkout process) and after I return from Paypal website to complete the review process, there's no address in billing data. Only first_name, last_name and payer_email are saved with order.

I think the complete address returned through IPN should be saved to billing info also.
This is crucial if you want to apply taxes based on billing country just before submitting the order.

I don't know if there'll be any side effect when you simply copy all the shipping info to billing info right away after receiving the IPN. But simple checkbox on 'Review payment' page that is similar to 'My billing information is the same as my delivery information' checkbox from checkout pane could solve the problem also.

CommentFileSizeAuthor
#1 508794-paypalEC_address_copy.patch837 byteslop
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lop’s picture

lop CreditAttribution: lop commented
Status: Active » Needs review
FileSize
508794-paypalEC_address_copy.patch837 bytes

So here's the patch that solves this issue.
It simply copies the returned shipping address to billing address fields at Paypal EC review stage.

I see no reason of storing the payer email to billing_street_1 field or any other fields (billing_company, billing_phone), so it's not storing it anymore.

itsmahitha’s picture

itsmahitha CreditAttribution: itsmahitha commented
Assigned: Unassigned » itsmahitha
Status: Needs review » Reviewed & tested by the community

Reviewed.

TR’s picture

TR CreditAttribution: TR commented
Assigned: itsmahitha » Unassigned
Status: Reviewed & tested by the community » Needs review
TR’s picture

TR CreditAttribution: TR commented
Version: 6.x-2.0-rc3 » 6.x-2.x-dev
Status: Needs review » Needs work

Let's see if we can get this tested.

TR’s picture

TR CreditAttribution: TR commented
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 508794-paypalEC_address_copy.patch, failed testing.

TR’s picture

TR CreditAttribution: TR commented
Version: 6.x-2.x-dev » 7.x-3.x-dev
Component: Code » Payment
Status: Needs work » Active

New features should go into 7.x-3.x at this point.

Anyone interested in writing a patch for this?

longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave commented
Status: Active » Closed (won't fix)

I am not convinced this is the correct thing to do. We cannot assume the billing address is the same as the delivery address, as this is not always the case - and this can make a difference where things like taxes are concerned. PayPal usually only tell you the delivery address via IPN, unless you have a special agreement with them to get access to the billing address as well.