Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hass’s picture

hass’s picture

D8 patch

BarisW’s picture

Status: Needs review » Reviewed & tested by the community

Patch looks good, although an invalid theme hook call still results in this error:

Warning: call_user_func_array() expects parameter 1 to be a valid callback, no array or string given in theme_get_registry() (line 268 of core/includes/theme.inc).

But that should be another issue. This patch is fine ;)

catch’s picture

Version: 8.x-dev » 7.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Thanks, theme hook is better here. Committed/pushed to 8.x.

hass’s picture

D7 Patch is in #1.

hass’s picture

Status: Patch (to be ported) » Needs review
hass’s picture

What is holding this patch back?

dcam’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
7.58 KB

The patch in #1 contains the same changes that were applied to D8. The attached image shows the error log messages before and after the patch was applied.

theme_error.png

The patch looks good to me. Marking as RTBC.

David_Rothstein’s picture

Status: Reviewed & tested by the community » Fixed
Issue tags: +7.22 release notes

I'm not sure why we're getting rid of the quotes, but this is an obscure enough log message that it doesn't matter much. Also, it's a string change, so I probably should have committed this earlier in the release cycle... but again, it's an obscure string - and also one that is completely wrong and confusing in its current form, so I think untranslated but correct might be better than translated but incorrect in this case :)

Therefore, committed to 7.x - thanks! http://drupalcode.org/project/drupal.git/commit/815faef

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