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

Stripe\Tax\Settings::update requires an ID to be passed #1645

Closed
JakeBooher opened this issue Feb 12, 2024 · 2 comments · Fixed by #1647
Closed

Stripe\Tax\Settings::update requires an ID to be passed #1645

JakeBooher opened this issue Feb 12, 2024 · 2 comments · Fixed by #1647
Assignees
Labels

Comments

@JakeBooher
Copy link

Describe the bug

Stripe\Tax\Settings::update requires an ID to be passed, which is not compatible with the Stripe API

To Reproduce

\Stripe\Tax\Settings::update(null, $taxSettings);

triggers the following error:

  Could not determine which URL to request: Stripe\Tax\Settings instance has invalid ID:

and sending a value of '' (empty string) for the ID triggers the following error:

Unrecognized request URL (POST: /v1/tax/settings/). If you are trying to list objects, remove the trailing slash. If you are trying to retrieve an object, make sure you passed a valid (non-empty) identifier in your code. Please see https://stripe.com/docs or we can help at https://support.stripe.com/.

Expected behavior

URL should be generated correctly without an ID, and error should not be triggered. Ideally the $id argument should not exist on this method

Code snippets

No response

OS

N/A

PHP version

PHP 8.2

Library version

13.10.0

API version

2023-10-16

Additional context

No response

@JakeBooher JakeBooher added the bug label Feb 12, 2024
@remi-stripe remi-stripe self-assigned this Feb 12, 2024
@remi-stripe
Copy link
Contributor

@JakeBooher Thanks you for flagging and sorry about the trouble. This is a surprising one, we'll investigate the right fix but it might take a bit of time on that one.

@richardm-stripe
Copy link
Contributor

Hello @JakeBooher. Thanks again for flagging. This was fixed in #1647 and released in https://github.com/stripe/stripe-php/releases/tag/v13.11.0.

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

Successfully merging a pull request may close this issue.

3 participants