WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#6280 closed defect (bug) (fixed)

Attachment permalinks with int slugs fail

Reported by: andy Owned by: 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 6 years ago.
6280screen2.jpg (77.1 KB) - added by MtDewVirus 6 years ago.
Notice the same content is loaded and displayed for both attachment URLs
6280.diff (767 bytes) - added by andy 6 years ago.
use explicit attachment permalink marker for int names

Download all attachments as: .zip

Change History (5)

MtDewVirus6 years ago

MtDewVirus6 years ago

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

andy6 years ago

use explicit attachment permalink marker for int names

comment:1 andy6 years ago

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

comment:2 ryan6 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.