Opened 4 years ago

Closed 4 years ago

#9293 closed enhancement (fixed)

Clarify the args passed into wp_update_term

Reported by: simonwheatley Owned by: ryan
Priority: low Milestone: 2.8
Component: Taxonomy Version:
Severity: normal Keywords: has-patch
Cc:

Description

The args for wp_update_term are currently:

wp_update_term( $term, $taxonomy, $args = array() )

The first argument is not actually the term, but the term ID. Clearer code might be:

wp_update_term( $term_id, $taxonomy, $args = array() )

Attachments (2)

clarify_taxonomy_wp_update_term.diff​ (884 bytes) - added by simonwheatley 4 years ago.
Patch to clarify the args for wp_update_term
9293.patch​ (1.0 KB) - added by johnbillion 4 years ago.
Refreshed patch

Download all attachments as: .zip

Change History (6)

simonwheatley — 4 years ago

Patch to clarify the args for wp_update_term

comment:1   filosofo — 4 years ago

Your patch should also update the phpdoc for that function.

comment:2   FFEMTcJ — 4 years ago

  • Milestone changed from Unassigned to 2.8

johnbillion — 4 years ago

Refreshed patch

comment:3   johnbillion — 4 years ago

Refreshed patch. Simple stuff.

comment:4   ryan — 4 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [11068]) Clarify args to wp_update_term(). Props simonwheatley, johnbillion. fixes #9293

Note: See TracTickets for help on using tickets.