-
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
[Issue] Add default empty array in constructor #38165
Comments
@magento give me 2.4-develop instance - upcoming 2.4.x release |
Hi @alpesh475. Thank you for your request. I'm working on Magento instance for you. |
Hi @alpesh475, here is your Magento Instance: https://173b0b5acc7ed3f56e8c4dcbfca1d2eb.instances-prod.magento-community.engineering |
Hi @engcom-Hotel. Thank you for working on this issue.
|
Hello @jissereitsma, Thanks for the report and collaboration! We have tried to reproduce the issue in the 2.4-develop branch and the issue is reproducible for us after following the below steps:
Hence confirming the issue. Thanks |
✅ Jira issue https://jira.corp.adobe.com/browse/AC-10583 is successfully created for this GitHub issue. |
✅ Confirmed by @engcom-Hotel. Thank you for verifying the issue. |
give me 2.4-develop instance - upcoming 2.4.x release |
This issue is automatically created based on existing pull request: #38163: Add default empty array in constructor
Description (*)
This class its constructor argument
$tagGenerators
is empty by default in PHP, so that via DI argument types indi.xml
files the array is filled automatically. This actually means that - without any runtime configuration of this argument - the PHP class has a required argument$tagGenerators
.In my cases, this is breaking DI compile when the modules
Magento_DirectoryGraphQl
andMagento_StoreGraphQl
are disabled (and actually a lot of people will disable this module if they are not building a headless shop). This PR is a simple but required fix.Related Pull Requests
None
Fixed Issues (if relevant)
None
Manual testing scenarios (*)
Install Magento 2. Next, add a composer replacement of
magento/module-store-graph-ql
andmagento/module-directory-graph-ql
and runcomposer update
to remove these dependencies. Alternatively, just disable the modulesMagento_DirectoryGraphQl
andMagento_StoreGraphQl
. Next, runbin/magento setup:di:compile
. It should fail. With this patch, it should work.Contribution checklist (*)
The text was updated successfully, but these errors were encountered: