Skip to content

Trying to get data from non existent products #12321

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

Merged
merged 3 commits into from
Dec 6, 2017
Merged

Trying to get data from non existent products #12321

merged 3 commits into from
Dec 6, 2017

Conversation

angelo983
Copy link
Member

@angelo983 angelo983 commented Nov 17, 2017

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

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

Thank you @angelo983 for your PR. Could you please add some manual test cases and mention any related issue. See our guidelines for more information https://github.com/magento/magento2/blob/2.2-develop/PULL_REQUEST_TEMPLATE.md

@ishakhsuvarov
Copy link
Contributor

Hi @angelo983
Would be really nice if you could provide a similar fix for 2.2-develop and possibly 2.3-develop as well.
Thanks

@angelo983
Copy link
Member Author

@ishakhsuvarov have I to open new pull requests to provide a similar fix for 2.2-develop and possibly 2.3-develop?

@ishakhsuvarov
Copy link
Contributor

@angelo983 That would be really helpful

@angelo983
Copy link
Member Author

Now I have applied the same fix for 2.2-develop and 2.3-develop but I had to open other two pull requests.

@ishakhsuvarov
Copy link
Contributor

@angelo983 Thank you

@magento-team magento-team merged commit f9855e7 into magento:2.1-develop Dec 6, 2017
magento-team pushed a commit that referenced this pull request Dec 6, 2017
[EngCom] Public Pull Requests - 2.1-develop
 - MAGETWO-84476: Trying to get data from non existent products #12321
@angelo983 angelo983 deleted the patch-1 branch December 11, 2017 17:04
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