-
-
Notifications
You must be signed in to change notification settings - Fork 436
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
[preview] get rid of "Adminhtml" #2533
Conversation
I would definitely support this, but question: if we are changing the block class names, are we updating these in all the layout xml files and in any PHP where we createBlock manually? Or do we continue to call adminhtml/blockname ? |
This is why it is on draft. I dont want to change anything for layouts/xml ... so class names have to be resolved in another way. Edit: 1st step would be to move code to correct place and deprecate classes. It had no impact to nothing. Drop that classes is something else ... |
This PR will make it easier to uninstall non-required modules, ref PR #2455. With an uninstall script, the files in On that perspective, we can keep the code for the required modules in |
Why did you close this draft? Any solid reason? I liked the idea. |
I'll reopen later. |
Description (*)
I'd like to get rid off
Adminhtml
. It is just a collection of classes that belong to a specific module.My idea is to move all code to correct place and keep empty/deprecated adminhtml-classes for BC (like mysql4-classes).
Contribution checklist (*)