Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.drupal_implode_autocomplete() is quite unnecessary now that we have a generic drupal_to_js() mechanism. Less code is better.
After this patch, any code that used drupal_implode_autocomplete() can use drupal_to_js() instead, with no other changes.
Tasks:
- Update contrib modules
- Update Ajax docs in the handbook
| Comment | File | Size | Author |
|---|---|---|---|
| autocomplete_17.patch | 5 KB | Steven | |
Comments
Comment #1
Jaza CreditAttribution: Jaza commentedLooks like a move in the right direction to me. I'd be happy to update the activeselect module (http://drupal.org/node/49976) to use JSON as well.
When I wrote activeselect, I just copied the technique used by autocomplete - I didn't really think about any other alternatives. Despite having written an AJAX module, I still consider myself a newbie to JavaScript - before now, I didn't know what JSON was, and I'm still scared of actually putting the 'X' (for XML) in AJAX. ;-)
Comment #2
robertDouglass CreditAttribution: robertDouglass commentedApplies and does what it did before, with less code.
@Jaza it was exactly your module that I had in mind when I suggested this change =)
Comment #3
Steven CreditAttribution: Steven commentedCommitted to HEAD. Updated docs. Updated all contrib modules tagged as 4.7.
Comment #4
(not verified) CreditAttribution: commented