When using the function callback 'drupalSetSummary', the context is given as a firm parameter. This context should subsequently be used to setup the jQuery variables.
Also, it is cleaner to use 'else if' instead of 'if', but this is just a minor code style issue.

CommentFileSizeAuthor
#1 pathauto-context-2351205-1.patch770 bytesbasvanderheijden

Comments

basvanderheijden’s picture

basvanderheijden commented
Status: Needs work » Needs review
StatusFileSize
new pathauto-context-2351205-1.patch770 bytes
dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Parent issue: » #2305515: Plan for Pathauto 7.x-1.3 release
widukind’s picture

widukind commented

Reviewed and click-tested with latest FF and Chrome on Linux via simpletest.me.

Works as intended for me.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Version: 7.x-1.x-dev » 8.x-1.x-dev
Status: Needs review » Patch (to be ported)

Committed to 7.x-1.x.

deanflory’s picture

deanflory commented

Did that 7x commit make it to the project page as a download?

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Version: 8.x-1.x-dev » 7.x-1.x-dev
Status: Patch (to be ported) » Fixed

Status: Fixed » Closed (fixed)

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