Closed (fixed)
Project:
Ubercart Addresses
Version:
6.x-1.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
23 May 2009 at 14:47 UTC
Updated:
26 Dec 2009 at 01:40 UTC
Jump to comment: Most recent file
Comments
Comment #1
scott m. sanders commentedIf you look at your Drupal database and find uc_addresses, do the address fields happen to be prefixed with "delivery_"?
If so, manually edit the structure of these database fields such that all those "delivery_" prefixes are removed, and try again.
I gained them somehow and suspected they appeared while upgrading from Drupal 5 to 6 and the subsequent uc_addresses module but have yet to see this confirmed. Otherwise I have the same configuration as you on two production sites but do not have this issue of which I'm aware.
Comment #2
strestus commentedI have just checked this table and everything seems normal without any prefixes (see image)
Comment #3
freixas commentedHi,
I checked my test system and it uses PayPal Website Payments Pro, so I haven't tested with PayPal Standard.
There is a similar issue reported for 5.x-2.1: #467646: After an order is submitted and the user doesn't get the new address saved.
If anyone with PayPal Standard could record the values of $op and $order->order_status during a PayPal Standard transaction, I might be able to help fix this. The idea would be to add this code right before the "if ($op == 'submit' && $order->order_status == 'in_checkout') {" line:
file_put_contents('some filename', "op = $op order->order_status = {$order->order_status}");
Then place an order and follow it through to a successful completion. It might be useful to also place an order and then cancel it, to see how that differs. After performing the transaction, the file whose name you selected will contain the results.
Comment #4
freixas commentedI've changed the code to use hook_uc_checkout_complete() to determine when I can capture the address. This should fix the problem.
Look for a dev release dated 12/11/09 or later.
Comment #5
dwwI don't see a commit for this at http://drupal.org/project/cvs/262984
Comment #6
freixas commented