Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invalid Argument Warning in coupon.php #70

Closed
jeffchannell opened this issue Dec 11, 2013 · 5 comments
Closed

Invalid Argument Warning in coupon.php #70

jeffchannell opened this issue Dec 11, 2013 · 5 comments
Labels
V2 V2 Client

Comments

@jeffchannell
Copy link

If $this->_links is empty in Recurly_Coupon a warning is displayed.

@drewish
Copy link

drewish commented Dec 11, 2013

Can you say more about ho the object was getting initialized without that?

@jeffchannell
Copy link
Author

I haven't nailed that down just yet, but will post more when I figure it out!

@drewish
Copy link

drewish commented Dec 11, 2013

Awesome, your fix made sense but if there's an upstream cause, I'd love to try to fix that.

@jeffchannell
Copy link
Author

It appears to be triggered by doing this (simplified):

$coupon = Recurly_Coupon::get('expired_coupon');
$coupon->redeemCoupon($account_code, 'USD');

I think this should trigger an Exception...

@drewish
Copy link

drewish commented Jan 6, 2014

You can see the test triggering this error now: https://travis-ci.org/recurly/recurly-client-php/jobs/16490175

@drewish drewish closed this as completed in 07d3ae4 Jan 7, 2014
@bhelx bhelx added the V2 V2 Client label Mar 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
V2 V2 Client
Projects
None yet
Development

No branches or pull requests

3 participants