Closed (fixed)
Project:
Commerce Stripe
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
14 Apr 2017 at 04:41 UTC
Updated:
12 May 2017 at 10:24 UTC
Jump to comment: Most recent
Comments
Comment #3
vasikeindeed.
Commited a fix for this : Do not save commerce_remote_id for anonymous.
Please re-open if it's not fixed.
Thank you
Comment #4
quocnht commentedApplied patch and it does not save commerce_remote_id for anonymous anymore. But it throw an exception "Invalid parameters were supplied to Stripe's API" at createPayment:
Comment #5
vasikei can't replicate this new issue.
@quocnht : could you, please, provide more details on this?
How exactly could be reproduced?
Comment #6
quocnht commented@vasike
Step to reproduced:
- Install new commerce project, with commerce_stripe, add a product for testing.
- Add to cart, check out as guest, provide a card, go to review it is ok, purchase it show error "We encountered an unexpected error processing your payment method. Please try again later."
- Check log message there are 2 error:
-- commerce_stripe "No such token: card_1AAGEKJbl4MR0znism5CB2JH"
-- commerce_payment "Invalid parameters were supplied to Stripe's API."
Check at Stripe backend, a customer still create with a card card_1AAGEKJbl4MR0znism5CB2JH.
I do some debug and find the $transaction_data have customer=>NULL.
I follow the stripe api at https://stripe.com/docs/api#create_charge . if you do not pass a customer ID, the source you provide must either be a token
*This also error when create a new account, and then login and checkout. It seem that the code at line 417 never reach, because of return on line 410.
That's all I have for now. Thank you
Comment #8
vasikeindeed it seems Stripe payment is broken for Anonymous.
So the issue here is not to use customer in the Anonymous case.
So we need to use the card token for anonymous payment methods.
Here is a new commit that should fix or at least help with this issue.
Needs review and re-titled.
Comment #9
quocnht commentedIt works. Thank you.
Comment #10
vasikei think we can close this, as it seems there are some people helping with the (new) issues.
@quocnht : Actually, RTBC status is for patches before commit.
Anyway it was unusual also put the on "Needs review" after commit.
But, i needed support, and i got it ;)
thanks a lot!