-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
Missing required argument $productAvailabilityChecks of Magento\Sales\Model\Order\Reorder\OrderedProductAvailabilityChecker. #20825
Comments
Hi @ianitsky. Thank you for your report.
Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:
For more details, please, review the Magento Contributor Assistant documentation. @ianitsky do you confirm that you was able to reproduce the issue on vanilla Magento instance following steps to reproduce?
|
Hi @dominicfernando. Thank you for working on this issue.
|
@magento-engcom-team give me 2.2-develop instance |
Hi @dominicfernando. Thank you for your request. I'm working on Magento 2.2-develop instance for you |
Hi @shikhamis11. Thank you for working on this issue.
|
@magento-engcom-team give me 2.2-develop instance |
Hi @shikhamis11. Thank you for your request. I'm working on Magento 2.2-develop instance for you |
@magento-engcom-team give me 2.2-develop instance |
Hi @nainesh2jcommerce. Thank you for your request. I'm working on Magento 2.2-develop instance for you |
Hi @nainesh2jcommerce. Thank you for working on this issue.
|
Hi @ianitsky I checked the issue at my local environment with current 2.2- develop branch code . This issue is not reproducable so I am closing this. You can share screenshots here if you still get issue in 2.2-develop branch |
I came across this issue when Magento_ConfigurableProductSales was disabled on a 2.2.3 installation. The productAvailabilityChecks argument isn't defined in di.xml in Magento_Sales. The sole definition comes from Magento_ConfigurableProductSales's di.xml when its adding its own check to the array (and in fact defining the argument as an array). If you disable Magento_ConfigurableProductSales, you'll get the error. As the Sales module shouldn't depend on ConfigurableProductSales, the argument should really be defined in the Sales module and I'd classify this as a core bug. This just needs to be added to the Sales di.xml:
I added that via patch and it fixed the error, even with ConfigurableProductSales still disabled. |
@zaximus84 thanks for explanation I will recheck and reopen the issue if it exists |
✅ Confirmed by @shikhamis11 Issue Available: @shikhamis11, You will be automatically unassigned. Contributors/Maintainers can claim this issue to continue. To reclaim and continue work, reassign the ticket to yourself. |
@zaximus84 yes found the issue .. yes Sales module shouldn't depend on ConfigurableProductSales . If you have solution for the issue please create Pull request . |
Hi @kisroman. Thank you for working on this issue.
|
… Magento\Sales\Model\Order\Reorder\OrderedProductAvailabilityChecker.
… Magento\Sales\Model\Order\Reorder\OrderedProductAvailabilityChecker.
Hi @ianitsky. Thank you for your report.
The fix will be available with the upcoming 2.3.2 release. |
… Magento\Sales\Model\Order\Reorder\OrderedProductAvailabilityChecker.
… Magento\Sales\Model\Order\Reorder\OrderedProductAvailabilityChecker.
Hi @ianitsky. Thank you for your report. The fix will be available with the upcoming 2.2.9 release. |
…labilityChecks of Magent… #21920
Preconditions (*)
Steps to reproduce (*)
Expected result (*)
Actual result (*)
1 exception(s):
Exception #0 (BadMethodCallException): Missing required argument $productAvailabilityChecks of Magento\Sales\Model\Order\Reorder\OrderedProductAvailabilityChecker.
Exception #0 (BadMethodCallException): Missing required argument $productAvailabilityChecks of Magento\Sales\Model\Order\Reorder\OrderedProductAvailabilityChecker.
#0 /var/www/magento/vendor/magento/framework/ObjectManager/Factory/Dynamic/Developer.php(34): Magento\Framework\ObjectManager\Factory\AbstractFactory->resolveArgumentsInRuntime('Magento\Sales\M...', Array, Array)
#1 /var/www/magento/vendor/magento/framework/ObjectManager/Factory/Dynamic/Developer.php(59): Magento\Framework\ObjectManager\Factory\Dynamic\Developer->_resolveArguments('Magento\Sales\M...', Array, Array)
#2 /var/www/magento/vendor/magento/framework/ObjectManager/ObjectManager.php(70): Magento\Framework\ObjectManager\Factory\Dynamic\Developer->create('Magento\Sales\M...')
#3 /var/www/magento/vendor/magento/framework/ObjectManager/Factory/AbstractFactory.php(144): Magento\Framework\ObjectManager\ObjectManager->get('Magento\Sales\M...')
#4 /var/www/magento/vendor/magento/framework/ObjectManager/Factory/AbstractFactory.php(230): Magento\Framework\ObjectManager\Factory\AbstractFactory->resolveArgument(Array, 'Magento\Sales\M...', NULL, 'orderedProductA...', 'Magento\Sales\M...')
#5 /var/www/magento/vendor/magento/framework/ObjectManager/Factory/Dynamic/Developer.php(34): Magento\Framework\ObjectManager\Factory\AbstractFactory->resolveArgumentsInRuntime('Magento\Sales\M...', Array, Array)
#6 /var/www/magento/vendor/magento/framework/ObjectManager/Factory/Dynamic/Developer.php(59): Magento\Framework\ObjectManager\Factory\Dynamic\Developer->_resolveArguments('Magento\Sales\M...', Array, Array)
#7 /var/www/magento/vendor/magento/framework/ObjectManager/ObjectManager.php(70): Magento\Framework\ObjectManager\Factory\Dynamic\Developer->create('Magento\Sales\M...')
#8 /var/www/magento/vendor/magento/framework/ObjectManager/Factory/AbstractFactory.php(144): Magento\Framework\ObjectManager\ObjectManager->get('Magento\Sales\M...')
#9 /var/www/magento/vendor/magento/framework/ObjectManager/Factory/AbstractFactory.php(230): Magento\Framework\ObjectManager\Factory\AbstractFactory->resolveArgument(Array, 'Magento\Sales\M...', NULL, 'unavailableProd...', 'Magento\Sales\C...')
#10 /var/www/magento/vendor/magento/framework/ObjectManager/Factory/Dynamic/Developer.php(34): Magento\Framework\ObjectManager\Factory\AbstractFactory->resolveArgumentsInRuntime('Magento\Sales\C...', Array, Array)
#11 /var/www/magento/vendor/magento/framework/ObjectManager/Factory/Dynamic/Developer.php(59): Magento\Framework\ObjectManager\Factory\Dynamic\Developer->_resolveArguments('Magento\Sales\C...', Array, Array)
#12 /var/www/magento/vendor/magento/framework/ObjectManager/ObjectManager.php(56): Magento\Framework\ObjectManager\Factory\Dynamic\Developer->create('Magento\Sales\C...', Array)
#13 /var/www/magento/vendor/magento/framework/App/ActionFactory.php(40): Magento\Framework\ObjectManager\ObjectManager->create('Magento\Sales\C...')
#14 /var/www/magento/vendor/magento/framework/App/Router/Base.php(297): Magento\Framework\App\ActionFactory->create('Magento\Sales\C...')
#15 /var/www/magento/vendor/magento/framework/App/Router/Base.php(158): Magento\Framework\App\Router\Base->matchAction(Object(Magento\Framework\App\Request\Http), Array)
#16 /var/www/magento/vendor/magento/framework/App/FrontController.php(50): Magento\Framework\App\Router\Base->match(Object(Magento\Framework\App\Request\Http))
#17 /var/www/magento/vendor/magento/framework/Interception/Interceptor.php(58): Magento\Framework\App\FrontController->dispatch(Object(Magento\Framework\App\Request\Http))
#18 /var/www/magento/vendor/magento/framework/Interception/Interceptor.php(138): Magento\Framework\App\FrontController\Interceptor->___callParent('dispatch', Array)
#19 /var/www/magento/vendor/magento/framework/Interception/Interceptor.php(153): Magento\Framework\App\FrontController\Interceptor->Magento\Framework\Interception{closure}(Object(Magento\Framework\App\Request\Http))
#20 /var/www/magento/generated/code/Magento/Framework/App/FrontController/Interceptor.php(26): Magento\Framework\App\FrontController\Interceptor->___callPlugins('dispatch', Array, Array)
#21 /var/www/magento/vendor/magento/framework/App/Http.php(135): Magento\Framework\App\FrontController\Interceptor->dispatch(Object(Magento\Framework\App\Request\Http))
#22 /var/www/magento/vendor/magento/framework/Interception/Interceptor.php(58): Magento\Framework\App\Http->launch()
#23 /var/www/magento/vendor/magento/framework/Interception/Interceptor.php(138): Magento\Framework\App\Http\Interceptor->___callParent('launch', Array)
#24 /var/www/magento/vendor/msp/apienhancer/Plugin/AppInterfacePlugin.php(75): Magento\Framework\App\Http\Interceptor->Magento\Framework\Interception{closure}()
#25 /var/www/magento/vendor/magento/framework/Interception/Interceptor.php(135): MSP\APIEnhancer\Plugin\AppInterfacePlugin->aroundLaunch(Object(Magento\Framework\App\Http\Interceptor), Object(Closure))
#26 /var/www/magento/vendor/magento/framework/Interception/Interceptor.php(153): Magento\Framework\App\Http\Interceptor->Magento\Framework\Interception{closure}()
#27 /var/www/magento/generated/code/Magento/Framework/App/Http/Interceptor.php(26): Magento\Framework\App\Http\Interceptor->___callPlugins('launch', Array, NULL)
#28 /var/www/magento/vendor/magento/framework/App/Bootstrap.php(257): Magento\Framework\App\Http\Interceptor->launch()
#29 /var/www/magento/pub/index.php(37): Magento\Framework\App\Bootstrap->run(Object(Magento\Framework\App\Http\Interceptor))
#30 {main}
@magento-engcom-team give me 2.2.7 instance
The text was updated successfully, but these errors were encountered: