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 of MAGETWO-65607 for Magento 2.1: [GitHub][PR] Check return … #10750

Merged
merged 1 commit into from Sep 4, 2017

Conversation

hostep
Copy link
Contributor

@hostep hostep commented Sep 2, 2017

…value for getProduct() in getPrice(). #7794

(cherry picked from commit 6752968)

Description

This is a backport of #7794 for Magento 2.1

Fixed Issues (if relevant)

  1. Getting PHP Fatal Error on getPrice()  #5519: Getting PHP Fatal Error on getPrice()
  2. Getting PHP Fatal Error on getPrice() #10206: Getting PHP Fatal Error on getPrice()

Watch out @heldchen points out in #10206 (comment) that there are other places where this problem can still occur.

…value for getProduct() in getPrice(). magento#7794

 - Merge Pull Request magento#7794 from pbaylies/magento2:patch-2

(cherry picked from commit 6752968)
@ishakhsuvarov ishakhsuvarov self-assigned this Sep 2, 2017
@ishakhsuvarov ishakhsuvarov added this to the September 2017 milestone Sep 2, 2017
@magento-team magento-team merged commit ad07888 into magento:2.1-develop Sep 4, 2017
magento-team pushed a commit that referenced this pull request Sep 4, 2017
magento-team pushed a commit that referenced this pull request Sep 4, 2017
[EngCom] Public Pull Requests - 2.1
 - MAGETWO-72252: Backport of MAGETWO-52577 for Magento 2.1: [GitHub] Set Product as Ne… #10751
 - MAGETWO-72251: Backport of MAGETWO-65607 for Magento 2.1: [GitHub][PR] Check return … #10750
 - MAGETWO-71643: [BUGFIX] Flat Category reindexList of AllChildren if the url_key of t… #10557
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.

None yet

4 participants