Problem/Motivation

In released Drupal 7.39 new function Drupal.urlIsLocal to misc/drupal.js file was added. This function is missed in LABjs replace/drupal.js file, so all AJAX behavior fails due to the "Uncaught TypeError: Drupal.urlIsLocal is not a function" error.

Proposed resolution

Add missed Drupal.urlIsLocal and additional Drupal.absoluteUrl function to the LABjs version of drupal.js file.

CommentFileSizeAuthor
#2 labjs-drupal_js_file_upgrade-2561797-1.patch2.41 KBOleksiy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Oleksiy created an issue. See original summary.

Oleksiy’s picture

Oleksiy
Primary language Ukrainian
Location Lutsk 🇺🇦
CreditAttribution: Oleksiy at Skilld commented
Assigned: Oleksiy » Unassigned
Status: Active » Needs review
FileSize
labjs-drupal_js_file_upgrade-2561797-1.patch2.41 KB
ogursoy’s picture

ogursoy CreditAttribution: ogursoy commented

#2 works. Thanks alot.

cristianalcaraz’s picture

cristianalcaraz CreditAttribution: cristianalcaraz commented

Can confirm #2 works. Thanks!!!

vyasamit2007’s picture

vyasamit2007 CreditAttribution: vyasamit2007 as a volunteer and at Blisstering Solutions commented
Status: Needs review » Reviewed & tested by the community

I've applied the patch on 7.x-1.6 and the patch works perfectly and resolves the issue.

Thanks!!
~Amit

drupalpal’s picture

drupalpal CreditAttribution: drupalpal commented

I had a problem in the ajax popup modal in views administration; It didn't show up. I have been searching for 5 days for a solution until I found this. I have now disabled the Labjs module and it is now working!

Thank you!

himanshupathak3’s picture

himanshupathak3 CreditAttribution: himanshupathak3 as a volunteer and at Srijan | A Material+ Company commented

@drupalpal
Thanks for your comment

#6 worked for me. Disable lab JS

jcisio’s picture

jcisio CreditAttribution: jcisio commented
Status: Reviewed & tested by the community » Fixed

The bug was found several months ago and committed a month ago (slightly different from the patch above). I couldn't say anything here because due to a security release. It was delayed because we had to synchronize the release of core and several similar contrib modules. Now it's done. Thanks all.

Status: Fixed » Closed (fixed)

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

henrikakselsen’s picture

henrikakselsen CreditAttribution: henrikakselsen as a volunteer commented

If someone should stumble on this issue: Turning ON javascript aggregation on admin/config/development/performance/ actually solved my issue with this ajax error. In my case we're not using the LABjs module, so it seems like it can be triggered by other issues too.

I suspect it could be a mismatch in variable names when using a jquery update and the aggregation is not 'aligned'.