The following error happens on page load: Uncaught TypeError: Cannot read property 'currentPathIsAdmin' of undefined

CommentFileSizeAuthor
#4 core-js-fix-escapeAdmin-regression-rollback-2169299-4.patch467 bytesnod_
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Sam152’s picture

Sam152 CreditAttribution: Sam152 commented

Related to: #8349445

Sam152’s picture

Sam152 CreditAttribution: Sam152 commented
Priority: Critical » Normal
Related issues: +#1979468: ".active" from linkGenerator(), l() and theme_links() forces an upper limit of per-page caching for all content containing links
martin107’s picture

martin107 CreditAttribution: martin107 commented

+1
can confirm this is new and is seen in browsers console log when visiting the main page after a fresh install

nod_’s picture

nod_
Primary language French
Location Lille
CreditAttribution: nod_ commented
Priority: Normal » Critical
Status: Needs work » Needs review
Issue tags: +JavaScript
FileSize
core-js-fix-escapeAdmin-regression-rollback-2169299-4.patch467 bytes

dumb error let's fix it now since it doesn't look like anyone wants to fix the other issue.

Sam152’s picture

Sam152 CreditAttribution: Sam152 commented
Status: Needs review » Reviewed & tested by the community

Looks good. Applies cleanly. Resolves issue.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Priority: Critical » Normal

Can we please use priorities appropriately?

nod_’s picture

nod_
Primary language French
Location Lille
CreditAttribution: nod_ commented

Regression cause by the roll back of a critical task, and pretty much the reverse of the critical bug #2167611: Back to site button broken follow-up of the critical task.

Back button is currently broken, this patch fixes it. I thought critical was fair, every time I broke the overlay that was critical soooo…
#1772466: Broke overlay with the dependency patch
#2004098: Overlay forms broken in IE10 for Windows 7
#1740958: overlay is broken

nod_’s picture

nod_
Primary language French
Location Lille
CreditAttribution: nod_ commented
Status: Reviewed & tested by the community » Closed (duplicate)

It'll get fixed in #1979468: ".active" from linkGenerator(), l() and theme_links() forces an upper limit of per-page caching for all content containing links.