WordPress.org
Get WordPress

Make WordPress Core

Opened 16 years ago

Closed 16 years ago

#9293 closed enhancement (fixed)

Clarify the args passed into wp_update_term

Reported by: simonwheatley's profile simonwheatley Owned by: ryan's profile 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 16 years ago.
Patch to clarify the args for wp_update_term
9293.patch​ (1.0 KB) - added by johnbillion 16 years ago.
Refreshed patch

Download all attachments as: .zip

Change History (6)

@simonwheatley
16 years ago

Patch to clarify the args for wp_update_term

#1 @filosofo
16 years ago

Your patch should also update the phpdoc for that function.

#2 @FFEMTcJ
16 years ago

  • Milestone changed from Unassigned to 2.8

@johnbillion
16 years ago

Refreshed patch

#3 @johnbillion
16 years ago

Refreshed patch. Simple stuff.

#4 @ryan
16 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.