-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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] Fixed wrong proxing in the inventory observer #21793
[Backport] Fixed wrong proxing in the inventory observer #21793
Conversation
Hi @VitaliyBoyko. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
Hi @dmytro-ch, thank you for the review. |
Not for manual testing |
Hi @VitaliyBoyko, thank you for your contribution! |
Original Pull Request
#21731
Description (*)
FIxed the wrong proxy
resourceStock
argument for the\Magento\CatalogInventory\Observer\UpdateItemsStockUponConfigChangeObserver
.Module di.xml
<argument name="resourceStock" xsi:type="object">Magento\CatalogInventory\Model\ResourceModel\Stock\Proxy</argument>
changed to
<argument name="resourceStockItem" xsi:type="object">Magento\CatalogInventory\Model\ResourceModel\Stock\Item\Proxy</argument>
Fixed Issues (if relevant)
These changes are needed for
magento/graphql-ce#167: Add support for '@magentoConfigFixture' annotation on GraphQL API-functional tests
PR: magento/graphql-ce#351
Manual testing scenarios (*)
No testing scenarios.
Contribution checklist (*)