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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

basvanderheijden’s picture

Status: Needs work » Needs review
FileSize
770 bytes
Dave Reid’s picture

widukind’s picture

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

Works as intended for me.

Dave Reid’s picture

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

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

Dave Reid’s picture

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.