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

Adding to wishlist doesn't work when not logged in #11908

Closed
mikelevy300 opened this issue Oct 31, 2017 · 2 comments
Closed

Adding to wishlist doesn't work when not logged in #11908

mikelevy300 opened this issue Oct 31, 2017 · 2 comments
Labels
duplicate Fixed in 2.1.x The issue has been fixed in 2.1 release line Fixed in 2.2.x The issue has been fixed in 2.2 release line Fixed in 2.3.x The issue has been fixed in 2.3 release line Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed

Comments

@mikelevy300
Copy link

Preconditions

  1. Apache 2.4
  2. PHP 7.0.16
  3. MySQL 5.7
  4. Magento 2.1.9 w/ sample data installed

Steps to reproduce

  1. Make sure you're not logged in
  2. Navigate to any category or product details page
  3. Click the Add to Wishlist button

Expected result

  1. The user get's redirected to the login page
  2. After logging in the user is redirected to the wishlist page where the item has been added to his or her wishlist

Actual result

  1. The user is redirected to the wishlist page, but the item has not been added to his or her wishlist

I traced it out and the form_key is failing validation in Magento\Wishlist\Controller\Index\Add.php after the user has been logged in. It appears that the form_key is used during both the login controller call as well as the wishlist controller call. The form_key passes validation just fine during the login controller call, but then fails when it reaches the wishlist controller.

@magento-engcom-team magento-engcom-team added Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed duplicate labels Oct 31, 2017
@magento-engcom-team
Copy link
Contributor

@mikelevy300, thank you for your report.
This is duplicate of #11825

@okorshenko
Copy link
Contributor

The issue has been fixed and delivered to 2.2-develop branch. Will be available with upcoming patch release

@okorshenko okorshenko added Fixed in 2.2.x The issue has been fixed in 2.2 release line Fixed in 2.1.x The issue has been fixed in 2.1 release line Fixed in 2.3.x The issue has been fixed in 2.3 release line labels Dec 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate Fixed in 2.1.x The issue has been fixed in 2.1 release line Fixed in 2.2.x The issue has been fixed in 2.2 release line Fixed in 2.3.x The issue has been fixed in 2.3 release line Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed
Projects
None yet
Development

No branches or pull requests

3 participants