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

[Backport] Issue in adding the wishlist of "zero price" product #2

Closed
wants to merge 126 commits into from

Conversation

sreichel
Copy link
Owner

@sreichel sreichel commented Aug 6, 2018

Original Pull Request

magento#16537

Description

Fixed add to wishtlist item which price is zero.

Fixed Issues (if relevant)

  1. Issue in adding the wishlist of "zero price" product.  magento/magento2#16479:Issue in adding the wishlist of "zero price" product

Manual testing scenarios

  1. Login as a customer in front end of the website.
  2. Open product with zero price
  3. Click on the wishlist button in the product.

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)

ronak2ram and others added 30 commits July 8, 2018 14:28
magento#16911

 - Merge Pull Request magento#16911 from gelanivishal/magento2:2.1-develop-PR-port-14395
 - Merged commits:
   1. 3d27dff
… the Subtotal column in the Creditmemo email magento#17155

 - Merge Pull Request magento#17155 from TomashKhamlai/magento2:fix-creditmemo-email-less-backport
 - Merged commits:
   1. 48e9b1b
…e database when the quote is empty magento#16898

 - Merge Pull Request magento#16898 from mage2pratik/magento2:2.1-develop-PR-port-16675
 - Merged commits:
   1. f9ccb51
…ontains default date "1/1/1970" instead of empty field.
…ries to register with sa magento#17182

 - Merge Pull Request magento#17182 from gelanivishal/magento2:2.1-develop-PR-port-15272
 - Merged commits:
   1. ff550b4
   2. 158e977
   3. 7359a16
   4. f49cf3d
   5. f08d932
…onfig fixture. Set the schedule for every minute so that the job is scheduled regardless of the time when you run the test.
Uniform jquery variable to "$" as this javascript is partially using "jquery" and partially "$".

The line 62 will cause an error in Internet Explorer if $ is not declared.
Stanislav Idolov and others added 11 commits August 2, 2018 22:57
Accepted Public Pull Requests:
 - magento#16619: [Backport] Login with wishlist raise report after logout (by @ronak2ram)
 - magento#17179: [Backport] Fix newsletter subscription behaviour for registered customer.  (by @mage2pratik)
 - magento#16911: [Backport] Fixed extends and removed unnecessary variables (by @gelanivishal)
 - Merge Pull Request magento#17335 from DanielRuf/magento2:fix/remove-unused-id-2-1
 - Merged commits:
   1. 2ddec42
…y validator magento#17108

 - Merge Pull Request magento#17108 from gelanivishal/magento2:2.1-develop-PR-port-13509
 - Merged commits:
   1. 727503d
   2. 38e2133
   3. 4148751
Accepted Public Pull Requests:
 - magento#17335: fix: remove unused ID (by @DanielRuf)
removed extraneous margin and compensated padding and set both to 0, which creates the same effect
…to#17379

 - Merge Pull Request magento#17379 from gelanivishal/magento2:2.1-develop-PR-port-15936
 - Merged commits:
   1. ebdc7c3
Accepted Public Pull Requests:
 - magento#17379: [Backport] magento#15308 removed extraneous margin (by @gelanivishal)
 - magento#17108: [Backport] Use constant time string comparison in FormKey validator (by @gelanivishal)


Fixed GitHub Issues:
 - magento#15308: extraneous margins on product list and product list items (reported by @DanielRuf) has been fixed in magento#17379 by @gelanivishal in 2.1-develop branch
   Related commits:
     1. ebdc7c3
sreichel pushed a commit that referenced this pull request Aug 6, 2018
@sreichel sreichel closed this Aug 10, 2018
sreichel pushed a commit that referenced this pull request Sep 16, 2018
sreichel pushed a commit that referenced this pull request Sep 16, 2018
@sreichel sreichel deleted the 2.1-develop-PR-port-16537 branch September 28, 2018 13:41
sreichel pushed a commit that referenced this pull request Oct 17, 2018
sreichel pushed a commit that referenced this pull request Oct 17, 2018
sreichel pushed a commit that referenced this pull request Oct 17, 2018
sreichel pushed a commit that referenced this pull request Oct 29, 2018
sreichel pushed a commit that referenced this pull request Nov 15, 2018
sreichel pushed a commit that referenced this pull request Nov 20, 2018
sreichel pushed a commit that referenced this pull request Nov 22, 2018
sreichel pushed a commit that referenced this pull request Dec 13, 2018
sreichel pushed a commit that referenced this pull request Dec 13, 2018
sreichel pushed a commit that referenced this pull request Dec 13, 2018
sreichel pushed a commit that referenced this pull request Dec 13, 2018
sreichel added a commit that referenced this pull request Dec 13, 2018
Fixes phpstan where 1st parameter is entity_id and 2nd $product->getId()

  115    Parameter #2 $value of method
         Magento\Catalog\Model\AbstractModel::loadByAttribute() expects
         array|string|null, int|null given.
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.