[wp-trac] [WordPress Trac] #44047: The link you followed has expired. - Export / Erasure admin screens
WordPress Trac
noreply at wordpress.org
Sat Apr 6 11:05:42 UTC 2019
#44047: The link you followed has expired. - Export / Erasure admin screens
------------------------------+---------------------------------
Reporter: xkon | Owner: garrett-eclipse
Type: defect (bug) | Status: accepted
Priority: normal | Milestone: 5.2
Component: Privacy | Version: 4.9.6
Severity: normal | Resolution:
Keywords: has-patch commit | Focuses: ui, administration
------------------------------+---------------------------------
Comment (by xkon):
@garrett-eclipse / @birgire ,
I'd still be more comfortable by using `tools.php?page=` than just
`?page=` and we could change that to anything when the pages get moved, as
well most forms use actions similar to `tools.php?params` or even with
`admin_url()` and `self_admin_url()`. Maybe it would be better to be in-
line with other files within the admin area :) .
I'm ok with either choice and [attachment:"44047.commit.diff"] works as
expected. I'd love an opinion from a committer though before this goes in
( even if this is a really minor thing ).
--
Ticket URL: <https://core.trac.wordpress.org/ticket/44047#comment:36>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list