Contextual link JavaScript error when viewing a page such as /admin/* that doesn't contain any contextual links

There already is a follow-up patch on the issue that introduced this: #1971108-36: Convert contextual.js to use Backbone (and support dynamic contextual links). It solves it very differently though. This is slightly more complex, but also better for front-end perf.

Let's figure out the best solution here.

I'm more for something like #1 than adding the dep to the script.

Same fix as in #1 with a nicer syntax.

If it wasn't for #1639012: Guard against broken behaviors it'd be a critical bug :p

:)

I only found it because I run hacked core JS to expose stuff like this.

Needs reroll...

curl https://drupal.org/files/core-js-contextual-error-admin-1988328-3.patch | git a
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  1970  100  1970    0     0  37175      0 --:--:-- --:--:-- --:--:-- 42826
error: patch failed: core/modules/contextual/contextual.toolbar.js:46
error: core/modules/contextual/contextual.toolbar.js: patch does not apply

turns out #914382: Contextual links incompatible with render cache fixed it so it's all good.

#9: actually, the solution applied in #914382: Contextual links incompatible with render cache is suboptimal, as you already said in #3. So this patch should still be rerolled, and should remove the dep introduced by #914382: Contextual links incompatible with render cache.

Umm, sure, but in contextual_page_build there is $page['#attached']['library'][] = array('contextual', 'drupal.contextual-links');

Which means that it'll be on the page regardless.

Anyway, here is the patch. Made into an anonymous function otherwise a js checking tool complains about a function defined in a if (because of hoisting and that sort of things).

You're right. But it's possible that somebody removes the library in hook_page_alter() for some reason, and then thanks to #11 we still won't break. So it's still better :)

Thanks!

Committed to 8.x. Thanks!

.