-
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
Argument 6 missing in SaveImageInformation #31167
Comments
Hi @anvanza. 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. Please, add a comment to assign the issue:
🕙 You can find the schedule on the Magento Community Calendar page. 📞 The triage of issues happens in the queue order. If you want to speed up the delivery of your contribution, please join the Community Contributions Triage session to discuss the appropriate ticket. 🎥 You can find the recording of the previous Community Contributions Triage on the Magento Youtube Channel ✏️ Feel free to post questions/proposals/feedback related to the Community Contributions Triage process to the corresponding Slack Channel |
Hi @anvanza, |
Hi @engcom-Hotel. Thank you for working on this issue.
|
Confirmed bug and tested PR. PR fixes the issue (import image via cli). |
@PascalBrouwers @anvanza, could you provide some example of how exactly we can reproduce the issue? |
Unfortunately, not enough information was provided to create a Jira ticket. Please make sure you added the following label(s): Once all required labels are present, please add |
✅ Jira issue https://jira.corp.magento.com/browse/AC-993 is successfully created for this GitHub issue. |
✅ Confirmed by @engcom-Lima. Thank you for verifying the issue. |
❌ Cannot export the issue. This GitHub issue is already linked to Jira issue(s): https://jira.corp.magento.com/browse/AC-993 |
1 similar comment
❌ Cannot export the issue. This GitHub issue is already linked to Jira issue(s): https://jira.corp.magento.com/browse/AC-993 |
I can reproduce this on v2.4.3-p2 |
I can also reproduce on 2.4.3, but with a custom plugin that was built in-house. Same thing - when importing products using cron job - no problem. When using CLI command throws errors. Interestingly, if you extend the class that does the import, it throws error. If you just use ObjectManager (yes I know), it does not throw these errors. |
@anvanza Are you saying just throwing di.xml into src\vendor\magento\module-media-gallery-integration\etc\di.xml with
will work? UPDATE: yes it did. There is a commit to fix this issue: But it looks like some heavy rebuild instead of just adding new file to /etc folder. I think there might be some additional issues with the import after that (mine appears to be super slow compared to a cronjob). And thats why that import rebuild was so heavy-handed. |
This fixed my import issue thanks a lot! |
Preconditions (*)
Steps to reproduce (*)
Expected result (*)
Actual result (*)
Type Error occurred when creating object: Magento\MediaGalleryIntegration\Plugin\SaveImageInformation, Argument 6 passed to Magento\MediaGalleryIntegration\Plugin\SaveImageInformation::__construct() must be of the type array, null given, called in /var/www/html/vendor/magento/framework/ObjectManager/Factory/AbstractFactory.php on line 121
Cause
#29399
Solution
The imageExtensions argument is defined in the Magento/MediaGalleryIntegration/etc/adminhtml/di.xml file while the SaveImageInformation class could also be used in CLI. Therefore the imageExtensions argument should be moved to Magento/MediaGalleryIntegration/etc/di.xml (without the adminhtml)
CC @firebearstudio
The text was updated successfully, but these errors were encountered: