Opened 14 years ago
Last modified 6 years ago
#17170 new defect (bug)
Attachment slugs are not unique
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | major | Version: | 2.8 |
Component: | Permalinks | Keywords: | has-patch 2nd-opinion |
Focuses: | Cc: |
Description
From wp_unique_post_slug() and #9726:
Attachment slugs must be unique across all types.
It's true that attachment slugs are checked for uniqueness against all other object types, however, when inserting an object of another type any uniqueness checks do not include attachments. This means that #9539 isn't actually fixed if you attempt the steps in the order described (insert attachment first followed by conflicting page).
Fix would be to add 'attachment' to the post_type IN
clause for hierarchical types. See attachments-hierarchically-unique.diff.
Still puzzling out why/if attachments need to be unique against non-hierarchical types since there shouldn't be collisions as attachment URLs are subpage style. However, if attachment slugs do need to be globally unique then see attachments-globally-unique.diff (not tested so much).
Looks like the change for hierarchically-unique attachment slugs was made about nine months ago in [30629]. Refreshed the patch as 17170.diff for the regular unique attachment slugs.
Since @boonebgorges was the author for [30629], I thought perhaps he might weigh in here on finishing this out (or not).