[wp-trac] [WordPress Trac] #23367: Remove message parameters from admin URl's in the browser address bar

WordPress Trac noreply at wordpress.org
Wed Mar 11 02:43:05 UTC 2015


#23367: Remove message parameters from admin URl's in the browser address bar
----------------------------------------+-----------------------------
 Reporter:  mark-k                      |       Owner:  morganestes
     Type:  enhancement                 |      Status:  assigned
 Priority:  low                         |   Milestone:  4.2
Component:  Administration              |     Version:
 Severity:  minor                       |  Resolution:
 Keywords:  good-first-bug needs-patch  |     Focuses:  administration
----------------------------------------+-----------------------------
Changes (by johnbillion):

 * keywords:  good-first-bug needs-unit-tests has-patch => good-first-bug
     needs-patch


Comment:

 [attachment:23367.5.diff] looks solid to me. I agree that this can go into
 one function because of its narrow use.

 The filter name should be changed to `removable_query_args` so it's more
 inline with the `add|remove_query_arg()` function names.

 We can also add `update`, `enabled`, and `disabled` to the list of
 removable query args ([https://github.com/johnbillion/user-
 switching/blob/1.0.3/user-switching.php#L586-L588 ref]).

--
Ticket URL: <https://core.trac.wordpress.org/ticket/23367#comment:32>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list