WordPress.org
Get WordPress

Make WordPress Core

Opened 18 years ago

Closed 18 years ago

#6280 closed defect (bug) (fixed)

Attachment permalinks with int slugs fail

Reported by: andy's profile andy Owned by: ryan's profile ryan
Milestone: 2.5 Priority: high
Severity: normal Version: 2.5
Component: General Keywords: has-patch
Focuses: Cc:

Description

​http://www.gamesgorilla.com/2008/03/18/testing/

Attachment slugs should not be allowed to be ints.

Possible fix: if type is attachment and $slug is int, make it "attachment-$slug"

Attachments (3)

6280screen1.jpg​ (78.9 KB) - added by MtDewVirus 18 years ago.
6280screen2.jpg​ (77.1 KB) - added by MtDewVirus 18 years ago.
Notice the same content is loaded and displayed for both attachment URLs
6280.diff​ (767 bytes) - added by andy 18 years ago.
use explicit attachment permalink marker for int names

Download all attachments as: .zip

Change History (5)

@MtDewVirus
18 years ago

@MtDewVirus
18 years ago

Notice the same content is loaded and displayed for both attachment URLs

@andy
18 years ago

use explicit attachment permalink marker for int names

#1 @andy
18 years ago

  • Keywords has-patch added
  • Owner changed from anonymous to ryan

Clean solution: use ​http://trac.wordpress.org/browser/trunk/wp-includes/rewrite.php?rev=6964#L717

#2 @ryan
18 years ago

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

(In [7437]) use explicit attachment permalink marker for int names. Props andy. fixes #6280

Note: See TracTickets for help on using tickets.