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.
Attached one-line patch fixes what I am sure is a simple typographical error:
--- a/includes/theme.inc
+++ b/includes/theme.inc
@@ -275,7 +275,7 @@ function _theme_registry_callback($callback = NULL, array $argume
* @param $base_theme
* An array of loaded $theme objects representing the ancestor themes in
* oldest first order.
- * @param theme_engine
+ * @param $theme_engine
* The name of the theme engine.
*/
function _theme_load_registry($theme, $base_theme = NULL, $theme_engine = NULL) {
Comment | File | Size | Author |
---|---|---|---|
#1 | _theme_load_registry-fix_missing_dollar_sign.patch | 909 bytes | pillarsdotnet |
Comments
Comment #1
pillarsdotnet CreditAttribution: pillarsdotnet commentedFound another one in _theme_build_registry:
Attaching the patch for real this time.
Comment #2
TR CreditAttribution: TR commentedYes, this needs to be done.
Comment #3
TR CreditAttribution: TR commentedTagging for backport to D7. Applies cleanly to D7 and D8.
Comment #4
pillarsdotnet CreditAttribution: pillarsdotnet commentedFor now...
Comment #5
TR CreditAttribution: TR commentedWell, yes. It's always a moving target. Sometimes moving more than usual :-)
Comment #6
webchickCommitted and pushed to 8.x and 7.x. Thanks!
For future reference, I'm fine with patches like these being all combined into one. It makes reviewing and committing easier.