[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