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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

xjm’s picture

Status: Active » Needs review
FileSize
1.56 KB
xjm’s picture

And without the whitespace this time.

xjm’s picture

ksenzee pointed out it's an object, not an array. Curse you, JavaScript!

ksenzee’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me.

ksenzee’s picture

Issue summary: View changes

Updated issue summary.

xjm’s picture

Issue summary: View changes

Updated issue summary.

xjm’s picture

Issue summary: View changes

Updated issue summary.

xjm’s picture

Issue summary: View changes

Updated issue summary.

xjm’s picture

Issue summary: View changes

Updated issue summary.

catch’s picture

Status: Reviewed & tested by the community » Needs review
+ * @code
+ *   {
+ *     "red fish, blue fish": "blue fish",
+ *     "red fish, blue moon": "blue moon",
+ *   };
+ * @endcode
+ *
+ *
+ * @param $field_name

Looks like one too many line breaks?

xjm’s picture

That is entirely possible.

aspilicious’s picture

Status: Needs review » Reviewed & tested by the community

RTBC again

catch’s picture

Version: 8.x-dev » 7.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Committed/pushed to 8.x, will need a re-roll for 7.x.

xjm’s picture

Status: Patch (to be ported) » Reviewed & tested by the community
FileSize
1.54 KB

Same docblock, different branch.

xjm’s picture

Assigned: xjm » Unassigned
Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 7.x.

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

Anonymous’s picture

Issue summary: View changes

Making it a bit more legible. ;)