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.
Problem/Motivation
- taxonomy_autocomplete() documentation lacks proper API documentation.
- Multiple open issues need to update this docblock, and so far each issue includes its own variant.
See also: #1326644: Clean up API docs for taxonomy module
Proposed resolution
Stop the madness. :) Give this function a docblock sooner rather than later so that every bugfix doesn't write its own variant. Patch in #3 adds this docblock.
Remaining tasks
None.
User interface changes
None.
API changes
None.
Comment | File | Size | Author |
---|---|---|---|
#9 | taxonomy_autocomplete-d7-1337124-9.patch | 1.54 KB | xjm |
#6 | taxonomy_autocomplete-1337124-6.patch | 1.56 KB | xjm |
#3 | taxonomy_autocomplete-1337124-3.patch | 1.56 KB | xjm |
#2 | taxonomy_autocomplete_docblock.patch | 1.56 KB | xjm |
#1 | taxonomy_autocomplete_docblock.patch | 1.56 KB | xjm |
Comments
Comment #1
xjmComment #2
xjmAnd without the whitespace this time.
Comment #3
xjmksenzee pointed out it's an object, not an array. Curse you, JavaScript!
Comment #4
ksenzeeLooks good to me.
Comment #4.0
ksenzeeUpdated issue summary.
Comment #4.1
xjmUpdated issue summary.
Comment #4.2
xjmUpdated issue summary.
Comment #4.3
xjmUpdated issue summary.
Comment #4.4
xjmUpdated issue summary.
Comment #5
catchLooks like one too many line breaks?
Comment #6
xjmThat is entirely possible.
Comment #7
aspilicious CreditAttribution: aspilicious commentedRTBC again
Comment #8
catchCommitted/pushed to 8.x, will need a re-roll for 7.x.
Comment #9
xjmSame docblock, different branch.
Comment #10
xjmComment #11
Dries CreditAttribution: Dries commentedCommitted to 7.x.
Comment #12.0
(not verified) CreditAttribution: commentedMaking it a bit more legible. ;)