yireo / magento2-theme-override-checker
N/A
Installs: 2 097
Dependents: 0
Suggesters: 0
Security: 0
Stars: 16
Watchers: 2
Forks: 2
Type:magento2-module
Requires
- ext-dom: *
- magento/framework: ^101.0.1|^101.1|^102.0|^103.0
- sebastian/diff: ^1.0 || ^2.0 || ^3.0 || ^4.0 || ^5.0
- symfony/finder: ^4.0 || ^5.0 || ^6.0
This package is auto-updated.
Last update: 2024-11-12 12:58:10 UTC
README
Magento module to find all files of a given Magento theme (for example Magento/luma
), compare the found files with the parent theme (and/or modules) and return whether there are differences or not.
Current status: Beta, ready for trying out
Installation
composer require yireo/magento2-theme-override-checker:@dev bin/magento module:enable Yireo_ThemeOverrideChecker
Usage
List all possible overrides, with a warning for unneeded overrides (with no changes) and a red warning when lines are different:
bin/magento yireo:theme-overrides:check Magento/luma
List all specific override details for a specific file:
bin/magento yireo:theme-overrides:show Magento/luma web/jquery.js bin/magento yireo:theme-overrides:show Magento/blank web/css/source/_theme.less
List a diff between the current version and the parent theme version:
bin/magento yireo:theme-overrides:diff Magento/luma web/css/source/_theme.less
FAQ
How does this work?
AmpersandHQ developed an excellent tool ampersand-magento2-upgrade-patch-helper to check the status of a current project. Part of this is that it also scans for override files in the theme. However, it does not report what overriding files actually means. For instance, is overriding an XML layout file good or bad? How many lines of overriding a JavaScript file classifies as a bad practice?
Todo
- Support for multiple parent themes
- Make
contextLines
in DifferFactory configurable - Improve adviser
- An XML layout file with dozens of line differences is ok
- An XML override file is dangerous
- An PHTML / JS / CSS file with dozens of line differences is dangerous
- Markdown and text files can be the same
- SVG files can be totally different