Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
Follow-up from #2638150: Move _system_default_theme_features() to theme.inc which has a patch.
The global private function _system_default_theme_features()
should be provided from within it's related class ThemeSettings, for context of what the default array applies towards.
Proposed resolution
Move the array provided by _system_default_theme_features
onto the ThemeSettings object.
Remaining tasks
Upload the patch from the other issue here and continue.
API changes
None, this deprecates a private global function denoted by it's underscore prefix.
Comment | File | Size | Author |
---|---|---|---|
#4 | 2804453-1-_system_default_theme_features-refactor-reroll.patch | 3.13 KB | joelpittet |
#2 | 2804453-1-_system_default_theme_features-refactor.patch | 3.13 KB | joelpittet |
Comments
Comment #2
joelpittetReposting patch here.
Comment #4
joelpittetReroll, assigning to lauriii to review because it's a bit of a low level refactor that he may care about.
Comment #5
lauriiiIt would be great to have an issue summary update here since I'm not sure if I understand the problem we're fixing here. In general, I think it's great to move things away from the theme.inc, but it would be nice to understand why theme system should be responsible for facilitating the default values for the system module.
Comment #6
joelpittetThanks for the review, hopefully the issue summary helps clarify why this needed.
Comment #9
markhalliwell_system_default_theme_features()
should trigger a deprecation error.