Make WordPress Core

Opened 9 years ago

Closed 9 years ago

#33664 closed defect (bug) (duplicate)

Create pretty permalinks for unlinked attachments

Reported by: pfefferle's profile pfefferle Owned by: swissspidy's profile swissspidy
Milestone: Priority: normal
Severity: normal Version: 4.4
Component: Permalinks Keywords:
Focuses: Cc:


The URL of an unlinked attachments always uses query params (for example: if pretty permalinks are enabled or not.

This causes problems with feed links (for example: and any other plugins that are using add_rewrite_endpoint (for example:

Change History (4)

#1 @swissspidy
9 years ago

  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to Future Release
  • Owner set to swissspidy
  • Status changed from new to assigned

I can confirm this. The error lies in get_post_comments_feed_link which just appends /feed/ to the permalink if pretty permalinks are enabled. It should check for unattached attachments though.

#2 @pfefferle
9 years ago

I think it is more consistent to use a simple pretty permalink for the attachment (for example because a lot of plugins might make the same mistake (like we did with the oEmbed API plugin). It is recommended on several codex pages to check the permalink structure like this:

<?php if ( get_option('permalink_structure') ) { echo 'permalinks enabled'; } ?>

#3 @swissspidy
9 years ago

  • Keywords needs-unit-tests added
  • Summary changed from Missing mod_rewrite rule for unlinked attachments to Create pretty permalinks for unlinked attachments

I created #33693 for the feed link problem, leaving this ticket for discussing the permalink stuff.

#4 @SergeyBiryukov
9 years ago

  • Keywords needs-patch needs-unit-tests removed
  • Milestone Future Release deleted
  • Resolution set to duplicate
  • Status changed from assigned to closed

Duplicate of #1914.

Note: See TracTickets for help on using tickets.