Make WordPress Core

Opened 4 years ago

Last modified 2 months ago

#20386 new defect (bug)

Year permalinks "win" against category permalinks

Reported by: markjaquith Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Canonical Keywords: needs-patch needs-unit-tests
Focuses: Cc:


We need to decide whether category permalinks should take priority over year permalinks.

e.g. /2008/?category_name=cat-a

Should that stay the same, or redirect to:


We have a unit test which is failing.

Attachments (1)

20386-tests.patch (929 bytes) - added by boonebgorges 13 months ago.
Removing from trunk as per #30284

Download all attachments as: .zip

Change History (7)

#1 @nacin
4 years ago

We've had somewhat similar prioritization issues where this has come up for post types versus taxonomies.

I think it should go like this: post types > taxonomies > date filters.

#2 @duck_
4 years ago

Similarly with the author_name vs. year query variables, see #17661. We should take one of these tickets and define the priority of all query variables.

Also related: #18385.

#3 @scribu
4 years ago

Agree that date filters should have the lowest priority.

#4 @wonderboymusic
3 years ago

IMO, canonical URL should always win over query string, and both always win over tax_query.

I addressed the problem with tax_query winning here: http://core.trac.wordpress.org/ticket/20767

Right now, tax is beating tag and category, which to me makes no sense because tax_query is a sub-request. Query string terms get tax=term treatment in WP_Query but are really only altering context of the request, not completely changing it.

Precedence really only affects theme functions since is_year, is_category will both be true and the year and category are publicly available in the request. Whoever "wins" gets precedence in which template is picked or which title is displayed, etc.

Version 0, edited 3 years ago by wonderboymusic (next)

13 months ago

Removing from trunk as per #30284

#5 @chriscct7
3 months ago

  • Keywords needs-patch added

#6 @johnbillion
2 months ago

  • Keywords needs-unit-tests added
Note: See TracTickets for help on using tickets.