WordPress.org
Get WordPress

Make WordPress Core

Opened 15 years ago

Closed 15 years ago

Last modified 10 years ago

#8704 closed defect (bug) (fixed)

Missing Filter in function wp_get_object_terms()

Reported by: miqrogroove's profile miqrogroove Owned by: ryan's profile ryan
Milestone: 2.8 Priority: normal
Severity: normal Version: 2.7
Component: Taxonomy Keywords: has-patch tested commit
Focuses: Cc:

Description

The qTranslate plugin author has asked me to forward this issue to WP dev.

in wp-includes/taxonomy.php function wp_get_object_terms()

find 

   return $terms;

replace with 

   $terms = apply_filters('wp_get_object_terms', $terms);
   return $terms;

Original discussion at ​http://www.qianqin.de/qtranslate/forum/viewtopic.php?f=3&t=305

Enjoy

Attachments (1)

8704.diff​ (472 bytes) - added by Denis-de-Bernardy 15 years ago.

Download all attachments as: .zip

Change History (10)

@Denis-de-Bernardy
15 years ago

#1 @Denis-de-Bernardy
15 years ago

  • Keywords has-patch dev-feedback added

patch adds a filter that passes the full set of parameters.

#2 @Denis-de-Bernardy
15 years ago

  • Keywords tested added; dev-feedback removed
  • Milestone changed from 2.7.2 to 2.8

#4 @Denis-de-Bernardy
15 years ago

  • Keywords commit added

commit, wontfix?

#5 @westi
15 years ago

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

(In [11253]) Add a filter to the result of wp_get_object_terms(). Fixes #8704 props Denis-de-Bernardy.

#6 follow-up: @Denis-de-Bernardy
15 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

shouldn't the empty terms list be filtered too, as in the initial patch?

#7 in reply to: ↑ 6 @westi
15 years ago

Replying to Denis-de-Bernardy:

shouldn't the empty terms list be filtered too, as in the initial patch?

Thanks for the typo catch.

Bad manual application of the patch :-(

#8 @westi
15 years ago

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

(In [11255]) Fix typo in [11253] fixes #8704 props Denis-de-Bernardy.

#9 @johnbillion
10 years ago

#9719 was marked as a duplicate.

Note: See TracTickets for help on using tickets.