Skip to content

Solve problem adding product from wishlist when these require more of one unit #12025

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

Conversation

PedroBlascoInteractiv4
Copy link

@PedroBlascoInteractiv4 PedroBlascoInteractiv4 commented Nov 4, 2017

Solve problem adding product from wishlist when these require more of one unit

Description

Problem encountered when obtaining the qty parameter. In this case, the qty is sent by mail and query string. The solution is to verify if the qty param is sent by post and take this value; in another case obtain param from querystring.

Fixed Issues (if relevant)

  1. Adding product from wishlist not adding to cart showing warning message. #9155: Adding product from wishlist not adding to cart showing warning message.

Manual testing scenarios

1.add a simple product from admin with Minimum Qty Allowed in Shopping Cart 10
2.login in front end
3.Add product to wishlist
4.Click on wishlist
5.Try to add product with qty 1 it will redirect to product detail page
6.try to add product with 10 qty in the same page..

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)

iivashchenko and others added 30 commits September 25, 2017 12:19
… of item in basket, original item quantity is being held in the quantity field on the update item page - for 2.1
[TSG] Backporting for 2.1 (pr31) (2.1.10) (FT fix)
…g email then leaving checkout page - for 2.1
…amount

  - Don't use price index for product grid on backend
  - Add integration test
…amount

  - Add test namespace
  - Add empty line at end of file
…g email then leaving checkout page - for 2.1
[TSG] Backporting for 2.1 (pr30) (2.1.10)
# Conflicts:
#	app/code/Magento/Bundle/view/base/web/js/price-bundle.js
ishakhsuvarov and others added 25 commits October 27, 2017 15:59
[EngCom] Public Pull Requests - 2.1-develop
 - MAGETWO-82527: Backport MAGETWO-52856 into Magento 2.1.x magento#11640
Public Pull Requests

magento#11611 FR#6891_21 Add-to-cart checkbox still visible when = false [Backport 2.1 develop] by @mrodespin

Fixed Public Issues

magento#6891 Add-to-cart checkbox still visible when $canItemsAddToCart = false
… EavAttribute.php magento#11728

 - Merge Pull Request magento#11728 from angelo983/magento2:patch-1
 - Merged commits:
   1. 84e649e
magento#11758

 - Merge Pull Request magento#11758 from hauso/magento2:patch-3
 - Merged commits:
   1. 0c660ce
…est fail randomly magento#11606

 - Merge Pull Request magento#11606 from adrian-martinez-interactiv4/magento2:FR21#ACCOUNTMANAGEMENTTEST-RANDOM-FAIL
 - Merged commits:
   1. db010ca
…aces causes exception magento#11848

 - Merge Pull Request magento#11848 from ihor-sviziev/magento2:patch-12
 - Merged commits:
   1. 96cc453
…2.1] magento#11844

 - Merge Pull Request magento#11844 from raumatbel/magento2:FR#Save0PriceType21
 - Merged commits:
   1. ccf4962
   2. b4b7635
[EngCom] Public Pull Requests - 2.1-develop
 - MAGETWO-82980: [Backport 2.1-develop] Fix AcountManagementTest unit test fail randomly magento#11606
 - MAGETWO-82959: Fix magento#11729 - negative value in excel export[M2.1] magento#11758
Public Pull Requests

magento#11844 Save the price 0 as price in custom options [backport 2.1] by @raumatbel
magento#11728 Check variable existence in prepareOptionIds(array) in EavAttribute.php by @angelo983

Fixed Public Issues

magento#4808 The price of product custom option can't be set to 0.
…by default magento#11932

 - Merge Pull Request magento#11932 from ihor-sviziev/magento2:2.1-develop-order-grid-sort-by-purchase-date
 - Merged commits:
   1. 22e05d1
Public Pull Requests

magento#11848 [2.1-develop] X-Magento-Tags header containing whitespaces causes exception by @ihor-sviziev

Fixed Public Issues

magento#7640 X-Magento-Tags header containing whitespaces causes exception
Public Pull Requests

magento#11932 [2.1-develop] Order grid - Sort by Purchase Date Desc by default by @ihor-sviziev

Fixed Public Issues

magento#10185 New Orders are not in Order grid after data migration from M 1.7.0.2 to M 2.1.7
@magento-cicd2
Copy link
Contributor

magento-cicd2 commented Nov 4, 2017

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 11 committers have signed the CLA.

✅ joni-jones
❌ iivashchenko
❌ RuslanKostiv1
❌ Dmytro Poperechnyy
❌ vzabaznov
❌ ameysar
❌ svitja
❌ OlgaVasyltsun
❌ igrybkov
❌ viktym
❌ magento-cicd2


Dmytro Poperechnyy seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.

@PedroBlascoInteractiv4 PedroBlascoInteractiv4 changed the title Pr#issue 9155 Solve problem adding product from wishlist when these require more of one unit Nov 4, 2017
@vrann vrann closed this Nov 5, 2017
@vrann vrann added the mm17es label Nov 5, 2017
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

Successfully merging this pull request may close these issues.