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

Latest merge broke the Invoices (0165a64) #60

Closed
lucacri opened this issue Oct 29, 2013 · 6 comments
Closed

Latest merge broke the Invoices (0165a64) #60

lucacri opened this issue Oct 29, 2013 · 6 comments

Comments

@lucacri
Copy link

lucacri commented Oct 29, 2013

Hello,
after the latest merge,
$invoices = \Stripe_Invoice::all(["count" => 100]);

will return the error

[ErrorException] array_keys() expects parameter 1 to be array, null given
in stripe-php/lib/Stripe/Object.php:138

Only way to have it working, at the moment, is to revert to a previous version.

@lexx27
Copy link

lexx27 commented Nov 1, 2013

I have the same problem here!

@what4893
Copy link

what4893 commented Nov 5, 2013

I'm having the same problem as well.

@BrianJM
Copy link

BrianJM commented Nov 6, 2013

Same problem. I reported this to customer support, and ti has been escalated internally. For the time being, I downgraded to version 1.8.3 which resolved the problem.

@akuzemchak
Copy link

Same issue here.

@iteratelance
Copy link

I think that this might be due to the "metadata": null in some events. I have some json that doesn't not have this field and I do not receive this error.

@michelle
Copy link
Contributor

michelle commented Nov 8, 2013

Thanks for reporting--this was indeed due to metadata being added to invoice items. Subscription line items in invoices don't have metadata and thus return null rather than {} as its metadata. These warnings should no longer show up in the latest version.

Sorry for any inconvenience!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants