Context Navigation

← Previous Ticket
Next Ticket →

#9436 closed defect (bug) (fixed)

'the_tags' filter not applied on the_tags() function

Reported by:	scribu	Owned by:
Priority:	normal	Milestone:	2.8
Component:	Taxonomy	Version:	2.5
Severity:	normal	Keywords:	has-patch
Cc:		Focuses:

Description

When you call the_tags() this is what happens:

the_tags() => the_terms() => get_the_term_list() => apply_filters( "term_links-$taxonomy")

But 'the_tags' filter is called in get_tags_list().

This is how it should be:

the_tags() => get_tags_list()

Attachments (1)

the_tags.patch (751 bytes ) - added by scribu 17 years ago.: Also pass the arguments to the filter

Download all attachments as: .zip

Change History (8)

#1 @scribu
17 years ago

Cc scribu@… added

@scribu
17 years ago

Attachment the_tags.patch added

Also pass the arguments to the filter

#2 @scribu
17 years ago

Component General → Taxonomy
Owner changed from anonymous to ryan

#3 @scribu
17 years ago

Owner ryan removed

#4 @jbsil
17 years ago

+1 But I believe this goes back as far as 2.5, maybe further.

#5 @scribu
17 years ago

Version 2.8 → 2.5

You're right. All the more reason to fix it. :)

#6 @jbsil
17 years ago

I came up with an almost identical fix to the patch scribu posted before I knew this ticket existed. I've tested his patch on 2.5.1 and 2.8-svn, but nothing in between. Can someone qualified tag this as (tested|needs-testing|commit)?

#7 @ryan
17 years ago

Resolution → fixed
Status new → closed

(In [10888]) Invoke the_tags filter in the_tags(). Props scribu. fixes #9436

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#9436 closed defect (bug) (fixed)

'the_tags' filter not applied on the_tags() function

Description

Attachments (1)

Change History (8)

#1 @scribu 17 years ago

@scribu 17 years ago

#2 @scribu 17 years ago

#3 @scribu 17 years ago

#4 @jbsil 17 years ago

#5 @scribu 17 years ago

#6 @jbsil 17 years ago

#7 @ryan 17 years ago