Closed m2-assistant[bot] closed 1 week ago
Hi @engcom-Dash. Thank you for working on this issue. In order to make sure that issue has enough information and ready for development, please read and check the following instruction: :point_down:
Area: XXXXX
label to the ticket, indicating the functional areas it may be related to.2.4-develop
branch@magento give me 2.4-develop instance
to deploy test instance on Magento infrastructure. 2.4-develop
branch, please, add the label Reproduced on 2.4.x
.Confirming this issue for further processing.
:white_check_mark: Jira issue https://jira.corp.adobe.com/browse/AC-12597 is successfully created for this GitHub issue.
:white_check_mark: Confirmed by @engcom-Hotel. Thank you for verifying the issue.
Issue Available: @engcom-Hotel, You will be automatically unassigned. Contributors/Maintainers can claim this issue to continue. To reclaim and continue work, reassign the ticket to yourself.
This issue is automatically created based on existing pull request: magento/magento2#38951: Remove references to files from extjs which got removed in e1ccdb…
…83e7e312572038c476d0695300700db849
Description (*)
While running Magento 2.4-develop locally, I noticed some errors both in my brower's console and the
var/log/system.log
file:After some searching around, it looks like the entire directory
lib/web/extjs
got removed in e1ccdb83e7e312572038c476d0695300700db849It looks like @Atul-glo35265 didn't notice there were still references to some of those files in the codebase, which were still being used in the backoffice.
This PR cleans up those references.
The removal of the phtml file is because it also referenced extjs, so it would also no longer work, and it seems the file is not being used (at least not in Magento OS). From the commit message that introduced the file, I think this may have been used in Adobe Commerce since they mention a Banner Rotation Widget, which doesn't exist in Magento OS. But since this commit is 9 years old, I have no idea if that still exists. And maybe it was also refactored recently in Adobe Commerce to no longer use the extjs files? Maybe somebody with access to Adobe Commerce source code can double check this?
Related Pull Requests
Fixed Issues (if relevant)
N/A
Manual testing scenarios (*)
var/log/system.log
if there are errors about empty less files from extjsQuestions or comments
Preferably, this PR should be included in whatever next Magento release (2.4.8?) together with the changes from e1ccdb83e7e312572038c476d0695300700db849
Contribution checklist (*)