WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 3 years ago

#13081 closed enhancement (fixed)

Make $taxonomy optional in get_term_link()

Reported by: scribu Owned by: scribu
Priority: low Milestone: 3.1
Component: Taxonomy Version:
Severity: normal Keywords: has-patch commit
Cc:

Description

When get_term_link() receives an object as the first argument, it shouldn't require the second $taxonomy argument, since the $term object already has it.

Attachments (1)

get_term_link.diff (1.9 KB) - added by scribu 3 years ago.

Download all attachments as: .zip

Change History (7)

scribu3 years ago

comment:1 follow-up: nacin3 years ago

if !is_object($term) we would need to then return WP_Error if we don't get a $taxonomy either.

comment:2 filosofo3 years ago

  • Owner changed from filosofo to scribu
  • Status changed from new to assigned

comment:3 nacin3 years ago

  • Milestone changed from 3.0 to 3.1

comment:4 in reply to: ↑ 1 scribu3 years ago

  • Keywords commit added
  • Milestone changed from Awaiting Triage to 3.1

Replying to nacin:

if !is_object($term) we would need to then return WP_Error if we don't get a $taxonomy either.

We would get a WP_Error from get_term() or get_term_by() so it's taken care of.

Patch still applies clean.

comment:5 scribu3 years ago

Related: #14156

comment:6 markjaquith3 years ago

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

(In [15586]) Make the second (taxonomy) parameter for get_term_link() optional if you pass in a term object as the first argument (which has the taxonomy in it). fixes #13081. props scribu

Note: See TracTickets for help on using tickets.