Undefined function url() error when "Rebuild theme registry on every page is enabled" [#2843011]

When "Rebuild Theme Registry on every page" is enabled, no pages will load. You will instead get an error that the url() function is undefined.

I don't have time to submit a patch at the moment, but the relevant line to replace is line 27 of zen-internals/theme-registry.inc.

This will get things to load:
drupal_set_message(t('For easier theme development, the theme registry is being rebuilt on every page request. It is extremely important to turn off this feature on production websites.', array('!link' => \Drupal\Core\Url::fromUri('base://admin/appearance/settings/' . $active_theme))), 'warning', FALSE);

Note that the !link placeholder still needs to be replaced. I'll do something about that if I find time to come back and create the patch before someone else gets there.

Comment	File	Size	Author
#7	zen-undefined_url_function-2843011-7.patch	2.28 KB	pakmanlh
#7	interdiff_3-7.txt	1.19 KB	pakmanlh
#3	2843011-resolve-url-function-issue.patch	2.2 KB	rhuffstedtler

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

12 January 2017 at 14:11

rhuffstedtler created an issue. See original summary.

Comment #2

rhuffstedtler CreditAttribution: rhuffstedtler as a volunteer and at ICF commented 12 January 2017 at 14:24

Regarding the !link placeholder, apparently the format string for that kind of placeholder changed, and it should now be :link rather than !link. Similar change needs to be made in theme-settings.php

Comment #3

rhuffstedtler CreditAttribution: rhuffstedtler as a volunteer and at ICF commented 12 January 2017 at 22:50

File	Size
2843011-resolve-url-function-issue.patch	2.2 KB

This patch should get it.

Comment #4

rhuffstedtler CreditAttribution: rhuffstedtler as a volunteer and at ICF commented 12 January 2017 at 22:51

Status:

Active

» Needs review

Comment #5

caschbre CreditAttribution: caschbre commented 13 January 2017 at 03:51

@rob... I think you want to use Url::fromRoute instead of Url::fromUri to generate the link from a Drupal route.

drupal_set_message(t('For easier theme development, the theme registry is being rebuilt on every page request. It is <em>extremely</em> important to <a href=":link">turn off this feature</a> on production websites.', array(':link' => \Drupal\Core\Url::fromRoute('system.theme_settings_theme', ['theme' => $active_theme]))), 'warning', FALSE);

I haven't tested that code yet though.