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) {
Files: 
CommentFileSizeAuthor
#1 _theme_load_registry-fix_missing_dollar_sign.patch909 bytespillarsdotnet
PASSED: [[SimpleTest]]: [MySQL] 33,136 pass(es).
[ View ]

Comments

StatusFileSize
new909 bytes
PASSED: [[SimpleTest]]: [MySQL] 33,136 pass(es).
[ View ]

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.

Status:Needs review» Reviewed & tested by the community

Yes, this needs to be done.

Issue tags:+needs backport to D7

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

Applies cleanly to D7 and D8.

For now...

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

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.