Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Alex Bukach created an issue. See original summary.

Alex Bukach’s picture

Assigned: Alex Bukach » Unassigned
Status: Active » Needs review
FileSize
2.64 KB

Here's the patch that provides such integration.

Alex Bukach’s picture

Added update hook.

Alex Bukach’s picture

Fixed fallback behaviour.

LNakamura’s picture

Hi, @alex-bukach - thanks for the patch!

Ironically, it doesn't apply completely right now due to your patch for #2745707: Autocomplete not handling special characters being merged in :-)

Could you take a look and provide an update to the first part of the patch to the .js file?

Thanks!

LNakamura’s picture

Status: Needs review » Needs work
Alex Bukach’s picture

Status: Needs work » Needs review
FileSize
3.94 KB

@LNakamura here you are.

LNakamura’s picture

Status: Needs review » Needs work

Hi, @alex-bukach - thanks for the update!

#7 applied successfully, so I ran updb. However, when I click on an Autocomplete Deluxe widget, I see no suggestions in the dropdown, and in the console I see a 404 error on the GET of http://drupalvm-d7.dev/autocomplete_deluxe/taxonomy/field_multi_term_ref...

I think I know what the problem is. If I'm looking in the right place, it appears that your update hook (which did indeed run) didn't make the change you wanted it to make. This is what I see in the field_config_instance table for the field which is generating the 404 error:

| 24 |       17 | field_multi_term_ref_2      | node                  | test_ct                     | a:7:{s:5:"label";s:16:"Multi term ref 2";s:6:"widget";a:5:{s:6:"weight";s:1:"3";s:4:"type";s:28:"autocomplete_deluxe_taxonomy";s:6:"module";s:19:"autocomplete_deluxe";s:6:"active";i:1;s:8:"settings";a:6:{s:4:"size";s:2:"60";s:5:"limit";s:2:"10";s:10:"min_length";s:1:"0";s:9:"delimiter";s:0:"";s:17:"not_found_message";s:31:"The term '@term' will be added.";s:24:"autocomplete_deluxe_path";s:28:"autocomplete_deluxe/taxonomy";}}s:8:"settings";a:1:{s:18:"user_register_form";b:0;}s:7:"display";a:1:{s:7:"default";a:5:{s:5:"label";s:6:"inline";s:4:"type";s:28:"taxonomy_term_reference_link";s:6:"weight";s:1:"3";s:8:"settings";a:0:{}s:6:"module";s:8:"taxonomy";}}s:8:"required";i:0;s:11:"description";s:0:"";s:13:"default_value";N;}

Finally, one very minor point - a typo in comment at the top of autocomplete_deluxe_taxonomy_callback():

in it's bootstrap process

should be:

in its bootstrap process

quotesBro’s picture

Status: Needs work » Needs review
FileSize
3.87 KB
jstoller’s picture

Fix coding standards in autocomplete_deluxe.install and "file does not exist" error when applying the patch.