Skip to content

Use cartId from REST URL #14161

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

Closed

Conversation

elzekool
Copy link
Contributor

Resolution for Guest cart API ignoring cartId in url for some methods

Description

For /V1/carts/mine/items and /V1/carts/mine/items the quoteId from the
quote item is used. For guest cart this is confusing as it makes the
cartId in the URL useless. For logged in carts it makes it unsafe as it
allows adding products to any active cart.

Fixed Issues

  1. Guest cart API ignoring cartId in url for some methods #14086: Guest cart API ignoring cartId in url for some methods

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

@elzekool elzekool force-pushed the 14086-Use-cartId-in-REST-URL branch 2 times, most recently from 8ead162 to a9d757b Compare March 17, 2018 14:53
@rogyar rogyar self-assigned this Mar 18, 2018
@rogyar
Copy link
Contributor

rogyar commented Mar 18, 2018

Hello, @elzekool. Thank you for your contribution.
In order to preserve backward compatibility, adding new methods to the existing interfaces is prohibited. A new interface should be created instead.
Please, take a look at the http://devdocs.magento.com/guides/v2.0/contributor-guide/backward-compatible-development/

For /V1/carts/mine/items and /V1/carts/mine/items the quoteId from the
quote item is used. For guest cart this is confusing as it makes the
cartId in the URL useless. For logged in carts it makes it unsafe as it
allows adding products to any active cart.

Relates to: magento#14086
@elzekool elzekool force-pushed the 14086-Use-cartId-in-REST-URL branch from a9d757b to f06ed69 Compare March 19, 2018 19:37
@rogyar
Copy link
Contributor

rogyar commented Apr 6, 2018

Hello @elzekool. Are you going to proceed working on this PR?

@rogyar
Copy link
Contributor

rogyar commented Apr 29, 2018

I am closing this PR now due to inactivity.
Please reopen and update if you wish to continue.
Thank you for the contribution

@rogyar rogyar closed this Apr 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants