WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#9293 closed enhancement (fixed)

Clarify the args passed into wp_update_term

Reported by: simonwheatley Owned by: ryan
Milestone: 2.8 Priority: low
Severity: normal Version:
Component: Taxonomy Keywords: has-patch
Focuses: 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 5 years ago.
Patch to clarify the args for wp_update_term
9293.patch (1.0 KB) - added by johnbillion 5 years ago.
Refreshed patch

Download all attachments as: .zip

Change History (6)

simonwheatley5 years ago

Patch to clarify the args for wp_update_term

comment:1 filosofo5 years ago

Your patch should also update the phpdoc for that function.

comment:2 FFEMTcJ5 years ago

  • Milestone changed from Unassigned to 2.8

johnbillion5 years ago

Refreshed patch

comment:3 johnbillion5 years ago

Refreshed patch. Simple stuff.

comment:4 ryan5 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.