Make WordPress Core

Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#16728 closed enhancement (invalid)

Canonical redirect ?category_name= to ?cat=

Reported by: greuben's profile greuben Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.1
Component: Canonical Keywords: has-patch close
Focuses: Cc:

Description

See #16644

Attachments (2)

16728.diff (1.1 KB) - added by greuben 14 years ago.
16728.2.diff (1.1 KB) - added by greuben 14 years ago.

Download all attachments as: .zip

Change History (10)

@greuben
14 years ago

#1 follow-up: @scribu
14 years ago

  • Owner set to scribu
  • Status changed from new to reviewing

Shouldn't there also be a check for wether pretty permalinks are on?

#2 @scribu
14 years ago

  • Owner scribu deleted

@greuben
14 years ago

#3 in reply to: ↑ 1 @greuben
14 years ago

Replying to scribu:

Shouldn't there also be a check for wether pretty permalinks are on?

Yes added it.

#4 @greuben
14 years ago

  • Type changed from defect (bug) to enhancement

#5 follow-up: @scribu
14 years ago

  • Keywords close added

I've just learned from ryan that ?cat= doesn't in fact include children, while ?category_name does.

So ?cat= is different from ?category_name=. Therefore, there shouldn't be a canonical redirect.

#6 @scribu
14 years ago

Also, if permalinks are enabled, ?cat= shouldn't redirect to /category/ either.

#7 in reply to: ↑ 5 @greuben
14 years ago

  • Resolution set to invalid
  • Status changed from reviewing to closed

Replying to scribu:

I've just learned from ryan that ?cat= doesn't in fact include children, while ?category_name does.

So ?cat= is different from ?category_name=. Therefore, there shouldn't be a canonical redirect.

Okay, closing this as invalid.

#8 @scribu
14 years ago

  • Milestone Awaiting Review deleted
Note: See TracTickets for help on using tickets.