Closed (fixed)
Project:
Ubercart
Version:
7.x-3.x-dev
Component:
Payment
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
1 Feb 2011 at 07:50 UTC
Updated:
27 Apr 2011 at 16:01 UTC
Jump to comment: Most recent file
Comments
Comment #1
essbee commentedAppears to be caused by the Bank Transfer module. Will re-post in that queue
Comment #2
essbee commentedStill displaying this symptom. Not related to Bank Transfer. Only have uc_paypal and uc_payment enabled.
Cant see whats causing it.
Comment #3
longwaveWhat payment methods were used in the affected orders, or does this happen on all orders? Haven't seen this in the brief testing I've done.
Comment #4
essbee commentedHappening on all orders, paypal is the only enabled payment method.
Comment #5
ndewhurstencountered same issue. "edit order" link works w/o trouble.
Comment #6
Meriial commentedThe problem is in uc_paypal.module, l. 532
will return the string, not the array. It should be something like:
I'm new to helping out. Should I make a patch?
Comment #7
tr commented@Meriial: Yes, it would be great if you can make a patch, preferably against 7.x-3.x-dev. That way people who experience this issue can test the *exact* changes you're proposing and verify the patch fixes that issue (and doesn't cause any others either!). See http://drupal.org/patch/create to see how it's done, if you don't already know.
Comment #8
Meriial commentedOK, here's my patch, done against a CVS checkout with the HEAD tag. Let me know if it could be improved.
Comment #9
tr commentedPatch looks fine - I don't know if it works to fix the problem (I don't currently have a PayPal setup to test against). Perhaps @essbee can check, since he was the one having the problem in the first place?
Comment #10
longwaveTested and committed. Meriial, thanks for submitting the patch!
Comment #12
n8o commentedThis problem also affects the Express Checkout payment method. I have attached a patch for it, as well against 7.x-3.x-dev . Hope it works out!
Comment #13
longwaveCommitted, thanks.