[wp-trac] [WordPress Trac] #24612: Permalink conflict: image attachment page and post when permalink setting is /%postname%/
WordPress Trac
noreply at wordpress.org
Thu Oct 15 09:18:12 UTC 2015
#24612: Permalink conflict: image attachment page and post when permalink setting
is /%postname%/
------------------------------------------------+--------------------------
Reporter: augustas | Owner:
Type: defect (bug) | SergeyBiryukov
Priority: normal | Status: accepted
Component: Permalinks | Milestone: 4.4
Severity: normal | Version: 3.3
Keywords: 4.4-early has-patch has-unit-tests | Resolution:
| Focuses:
------------------------------------------------+--------------------------
Changes (by SergeyBiryukov):
* keywords: needs-unit-tests 4.4-early needs-patch => 4.4-early has-patch
has-unit-tests
Comment:
@dmchale, thanks for a [comment:8 thorough comment].
> created "Permalink" PAGE (system gave it a slug of /permalink/)
> created "Permalink" POST (system gave it a slug of /permalink/)
> Already have a conflict. Why can POSTS create a shared slug as PAGES?
#13459 should resolve that.
> uploaded permalink.pdf (system gave it a slug of /permalink-2/)
> Side note, the "permalink" shown at the top of the page still shows the
native /attachment_id=N format, which obviously does not match my
permalink setting. Is this by design, or something that should be
addressed? It's misleading at best.
Should be fixed in #1914, see [34690] (currently broken in trunk though
due to [35195]).
> Bottom line, my belief is that the post/page/attachment post types
should all respect each other out of the box and never create naming
conflicts with each other.
Let's try to fix that in #13459, left a comment there.
[attachment:24612.2.patch] is a refreshed patch with a unit test.
For reference, some of this functionality may (or may not) have changed in
#18962.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/24612#comment:15>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list