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.

Sign up for GitHub

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

Jump to bottom

Can add to cart out of stock product #2

Closed
marionf opened this issue Apr 25, 2018 · 0 comments
Closed

Can add to cart out of stock product #2

marionf opened this issue Apr 25, 2018 · 0 comments
Labels
1.7.3.1 Affect version Classic Component FO Label Major Priority productpage Label QA_Automation Label regression Label stock Label
Milestone
1.7.3.2

Comments

@marionf
Copy link
Owner

marionf commented Apr 25, 2018

I have a product with 3 quantity in stock. I have not allowed order of out of stock product.
I am in the product page and try to add 4 products to the cart.

I have well the out of stock message under the add to cart button but the button isn't disabled and I can click on it.

It was working on 1.7.3.0, the bug was introduced by this PR: PrestaShop#8607

It's working for packs but not for standard products and products with combinations
@marionf marionf added 1.7.3.1 Affect version Classic Component QA_Automation Label Major Priority FO Label productpage Label regression Label stock Label labels Apr 25, 2018
@marionf marionf added this to the 1.7.3.2 milestone Apr 25, 2018
@marionf marionf mentioned this issue Apr 25, 2018
Closed
@marionf marionf mentioned this issue Apr 26, 2018
Closed
2 tasks
marionf pushed a commit that referenced this issue Jul 3, 2018
@sLorenzini @eternoendless
Clean header backoffice design - more clear for merchant #2
bcfd1dd
@marionf marionf closed this as completed Aug 8, 2019
marionf pushed a commit that referenced this issue Oct 14, 2019
@dheerajwebkul
Merge pull request #2 from dheerajwebkul/customer-message-mail-typo
33d2eab 
Fixed {firstname} typo in order customer comment mail
marionf added a commit that referenced this issue Sep 22, 2020
@marionf
Merge pull request #2 from marionf/patch-1
873e6bf 
Catalog-Products-add-edit-elements-in-all-tabs.md
marionf pushed a commit that referenced this issue Oct 8, 2021
@Quetzacoalt91
Apply fixes from phpstan level 5 review
1eaca08 
 ------ ----------------------------------------------------------------------
  Line   /web/module/classes/Cookie.php
 ------ ----------------------------------------------------------------------
  71     Parameter #2 $value of function setcookie expects string, int given.
  73     Parameter #1 $string of method
         PrestaShop\Module\AutoUpgrade\Cookie::encrypt() expects string, int
         given.
 ------ ----------------------------------------------------------------------

 ------ -------------------------------------------------------------------
  Line   /web/module/classes/ErrorHandler.php
 ------ -------------------------------------------------------------------
  110    Parameter #2 $line of method
         PrestaShop\Module\AutoUpgrade\ErrorHandler::report() expects int,
         string given.
 ------ -------------------------------------------------------------------

 ------ -------------------------------------------------------------------
  Line   /web/module/classes/TaskRunner/AbstractTask.php
 ------ -------------------------------------------------------------------
  95     Parameter #2 $logger of class
         PrestaShop\Module\AutoUpgrade\AjaxResponse constructor expects
         PrestaShop\Module\AutoUpgrade\Log\Logger, Psr\Log\LoggerInterface
         given.
 ------ -------------------------------------------------------------------

 ------ -------------------------------------------------------------------
  Line   /web/module/classes/UpgradeTools/CoreUpgrader/CoreUpgrader16.php
 ------ -------------------------------------------------------------------
  118    Parameter #2 $p_compress of class Archive_Tar constructor expects
         string|null, true given.
 ------ -------------------------------------------------------------------

 ------ --------------------------------------------------------------------
  Line   /web/module/classes/UpgradeTools/Translation.php
 ------ --------------------------------------------------------------------
  173    Parameter #1 $fp of function fwrite expects resource, false given.
  179    Parameter #1 $fp of function fwrite expects resource, false given.
  181    Parameter #1 $fp of function fwrite expects resource, false given.
  184    Parameter #1 $fp of function fwrite expects resource, false given.
  185    Parameter #1 $fp of function fclose expects resource, false given.
 ------ --------------------------------------------------------------------


 [ERROR] Found 10 errors
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
1.7.3.1 Affect version Classic Component FO Label Major Priority productpage Label QA_Automation Label regression Label stock Label
Projects
None yet
Milestone
1.7.3.2
Development

No branches or pull requests
1 participant
@marionf