You use once function
$('input.autocomplete-deluxe-form').once('attachAutocompleteDeluxe')
but don't add a necessary dependency for jquery/once plugin.

CommentFileSizeAuthor
#3 jquery-once-2931412-3.patch561 bytesoriol_e9g
#2 autocomplete_deluxe-2931412-1-add_once_plugin.patch357 byteshoter
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hoter created an issue. See original summary.

hoter’s picture

hoter CreditAttribution: hoter at ADCI Solutions commented
FileSize
autocomplete_deluxe-2931412-1-add_once_plugin.patch357 bytes
oriol_e9g’s picture

oriol_e9g
Primary language Catalan
Location Barcelona
CreditAttribution: oriol_e9g at Diputació de Barcelona commented
Status: Active » Needs review
FileSize
jquery-once-2931412-3.patch561 bytes

Like do other modules it's a best practice add also jquery dependency and order dependencies alphabetically.

edwardchiapet’s picture

edwardchiapet
Location New York, NY
CreditAttribution: edwardchiapet at Mediacurrent commented

Thanks, @hoter and @oriol_e9g!

Merging @oriol_e9g's patch - as it also adds the jQuery dependency and alphabetizes it.

Merging into 8.x-1.x-dev.

edwardchiapet’s picture

edwardchiapet
Location New York, NY
CreditAttribution: edwardchiapet at Mediacurrent commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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