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) {
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pillarsdotnet’s picture

Found another one in _theme_build_registry:

@@ -519,7 +519,7 @@ function _theme_process_registry(&$cache, $name, $type, $theme, $path) {
  * @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_build_registry($theme, $base_theme, $theme_engine) {

Attaching the patch for real this time.

TR’s picture

Status: Needs review » Reviewed & tested by the community

Yes, this needs to be done.

TR’s picture

Issue tags: +Needs backport to D7

Tagging for backport to D7. Applies cleanly to D7 and D8.

pillarsdotnet’s picture

Applies cleanly to D7 and D8.

For now...

TR’s picture

Well, yes. It's always a moving target. Sometimes moving more than usual :-)

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed 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.

Automatically closed -- issue fixed for 2 weeks with no activity.