Remove removable query vars from certain redirect URLs

[johnbillion]

Scenario:

1. Edit a post and hit 'Update' or 'Publish'
2. Navigate to Appearance -> Customize
3. Do all the things
4. Close the Customizer

Note that you are returned to the post editing screen, but the 'Post updated' message persists because the message query var is still present.

#23367 introduced a list of removable query vars which get hidden from the page's URL once the page loads. This list of query vars should be abstracted into its own function so the list can be reused. It can then be used to remove these query vars from the redirect destination URL after closing the Customizer, and maybe also in other situations (logging in?).

My ​User Switching plugin already does this when switching between users, but it has to implement its own list of removable query vars.

[Mte90]

I'm exploring your plugin i see this:

add_filter( 'removable_query_args',            array( $this, 'filter_removable_query_args' ) );

So filter_removable_query_args return new values in the array and elsewhere is used to add new parameters

add_query_arg( $args, $redirect_to )

to add this parameters.
In wp-admin/customize.php i see two add_query_arg to two different url, home_url and wp_login_url.
Can you help me to find the correct file to edit for fix this bug?

[swissspidy]

add_query_arg isn't the problem here.

First, the removable query vars inside wp_admin_canonical_url should be moved to their own function (pretty easy). Then you need to use that function in wp-admin/customize.php, just at the beginning where $return is set.

I will try to create a patch for that at the WordCamp Europe Contributor Day on Sunday.

[Mte90]

wp_admin_canonical_url with two parameters

[Mte90]

I've changed wp_admin_canonical_url, now have 2 parameters, one for the url to filter and one for print.
In this way i can get the url cleaned in customize.php for $return.

But after your comment i think that a function that wrap wp_admin_canonical_url and to the same thing is better.

[Mte90]

correct patch

[swissspidy]

Another approach like in the original suggestion.

[swissspidy]

Oops, wrong file extension, but you'll get the point. My patch introduces a new wp_removable_query_args function.

[johnbillion]

32692.diff​ is exactly what I was thinking.

[swissspidy]

32692.2.diff​ is an updated patch for trunk. I also simplified the wp_removable_query_args function a bit. IMHO a good candidate for 4.4.

[johnbillion]

In 33849:

Introduce wp_removable_query_args(), which returns an array of single-use query variables which can be removed from a URL.

Also applies the function to the return URL when the Customizer is closed.

Fixes #32692
Props swissspidy, Mte90

[westonruter]

In 40313:

Customize: Prevent links to customize.php from being generated which have query vars from wp_removable_query_args() present.

Props dlh.
See #23367, #32692.
Fixes #31850.

[swissspidy]

In 40331:

Customize: Prevent links to customize.php from being generated which have query vars from wp_removable_query_args() present.

Props dlh.
See #23367, #32692.
Fixes #31850.

Merges [40313] to the 4.7 branch.