Make WordPress Core

Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#14935 closed defect (bug) (worksforme)

Taxonomy Permalinks Broken (Matt's Community Tags)

Reported by: ryan Owned by: scribu
Priority: normal Milestone:
Component: Taxonomy Version: 3.1
Severity: normal Keywords:
Cc: Focuses:

Description

Taxonomy permalinks broke in trunk sometime since r15490. Example from Matt's blog:

http://ma.tt/person/stephane-daury/

Test with this plugin:

http://wordpress.org/extend/plugins/matts-community-tags/

Change History (14)

#1 @demetris
16 years ago

For me they stop working at r15613.

#2 @scribu
16 years ago

  • Owner set to scribu
  • Status newaccepted

#3 @ryan
16 years ago

Here's $wp->query_vars for the /person/stephane-daury/ request.

array(3) { ["people"]=>  string(14) "stephane-daury" ["taxonomy"]=>  string(6) "people" ["term"]=>  string(14) "stephane-daury" } 

#4 @scribu
16 years ago

The problem is that it's supposed to look for attachments, so these extra args would be necessary:

'post_type' => 'attachment',
'post_status' => 'inherit'

I have no idea why it worked previously.

#5 @scribu
16 years ago

Part of the problem was fixed in [15650].

#6 @scribu
16 years ago

This is the SQL generated by the 3.0 branch:

SELECT SQL_CALC_FOUND_ROWS wp_posts . *
FROM wp_posts
WHERE 1=1
AND wp_posts.ID
IN ( 42447 )
AND wp_posts.post_type
IN (
'post', 'page', 'attachment'
)
AND (
wp_posts.post_status = 'publish'
OR wp_posts.post_author =1
AND wp_posts.post_status = 'private'
)
ORDER BY wp_posts.post_date DESC
LIMIT 0 , 10

and this is in trunk:

SELECT SQL_CALC_FOUND_ROWS wp_posts . *
FROM wp_posts
WHERE 1 =1
AND wp_posts.ID
IN ( 42447 )
AND wp_posts.post_type
IN (
'post', 'page', 'attachment'
)
AND (
wp_posts.post_status = 'publish'
OR wp_posts.post_status = 'private'
)
ORDER BY wp_posts.post_date DESC
LIMIT 0 , 10

neither of which work with the Matt's Community Tags plugin out of the box. This leads me to believe that Matt has some additional code running on his site, which doesn't work in trunk, for some reason.

#7 @scribu
16 years ago

  • Summary Taxonomy Permalinks BrokenTaxonomy Permalinks Broken (Matt's Community Tags)

#8 @jane
16 years ago

  • Keywords needs-patch added

#9 @scribu
16 years ago

  • Keywords reporter-feedback added; needs-patch removed

#10 @markmcwilliams
16 years ago

It looks good at my end, whatever it was must have been fixed?

#11 @markjaquith
16 years ago

  • Resolutionworksforme
  • Status acceptedclosed

#12 @nacin
16 years ago

  • Milestone 3.1

#13 @scribu
16 years ago

Probably fixed in #15982

#14 @markmcwilliams
16 years ago

  • Keywords reporter-feedback removed

We can remove the reporter-feedback keyword from here! :)

Note: See TracTickets for help on using tickets.