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

Trying to get data from non existent products #12539

Merged
merged 1 commit into from
Dec 7, 2017
Merged

Trying to get data from non existent products #12539

merged 1 commit into from
Dec 7, 2017

Conversation

angelo983
Copy link
Member

As requested in #12321

When calling Products in Cart report, it called data of deleted products resulting in exception trying to access $productData[$item->getProductId()]

Preconditions

  1. Magento 2.1.10

Steps to reproduce

  1. Delete a product recently sold
  2. Open Reports > Marketing > Products in Cart

Expected result

  1. List of products

Actual result

  1. Blank list

@ishakhsuvarov ishakhsuvarov self-assigned this Dec 4, 2017
@ishakhsuvarov ishakhsuvarov added this to the December 2017 milestone Dec 4, 2017
@magento-team magento-team merged commit ee37eb3 into magento:2.2-develop Dec 7, 2017
magento-team pushed a commit that referenced this pull request Dec 7, 2017
[EngCom] Public Pull Requests - 2.2-develop
 - MAGETWO-84981: Trying to get data from non existent products #12539
 - MAGETWO-84979: [Backport 2.2-develop] Fix swagger-ui on instances of Magento running on a non-standard port #12541
 - MAGETWO-84903: Added namespace to product videos fotorama events #12469
 - MAGETWO-84862: [Backport 2.2-develop] #11409: Too many password reset requests even when disabled in settings #11435
 - MAGETWO-84856: Issue 12506: Fixup typo getDispretionPath -> getDispersionPath #12507
 - MAGETWO-84808: 12110: Missing cascade into attribute set deletion. #12167
 - MAGETWO-83503: [2.2] - Add command to view mview state and queue #12122
 - MAGETWO-80223: Fix syntax of expectException() calls #11099
@angelo983 angelo983 deleted the 2.2-develop branch December 11, 2017 17:05
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.

5 participants