Incorrect redirect after deleting a term from a taxonomy of a custom post type

[johnbillion]

Given a taxonomy attached to a custom post type, deleting a term from the individual term editing screen redirects the user to the term listing screen without the post_type query var, which is incorrect and means the Posts menu is highlighted.

Steps to reproduce on a site with a taxonomy attached to a custom post type:

• Navigate to the term listing screen for the taxonomy
• Click the Edit link below a term
• Click the Delete link at the bottom of the screen and confirm the prompt

Observe that the URL you're redirected to is missing the post_type query var and that the Posts admin menu is highlighted.

[donmhico]

Nice find @johnbillion. I can't believe we've missed this up until now.

In my attached patch, 47858.diff, you'll notice that I removed esc_url() in

wp_redirect( $sendback );

This is because esc_url() will transform & to & which is not what we wanted in our url query args. Also, the base url is defined as $sendback = admin_url( 'edit-tags.php' ); and the url query args are added using add_query_arg() so I think it's safe to remove the esc_url().

[mukesh27]

Updated patch.

[mukesh27]

Updated patch 47858.2.diff with different code change.

@SergeyBiryukov, @donmhico Can you please check it.

[donmhico]

@mukesh27 IMHO

if ( 'post' !== get_current_screen()->post_type )

is better than

if ( ! empty( $typenow ) ) 

because the former is easier to understand.

[mukesh27]

@donmhico As per $taxnow variable we can access $typenow from parent code.

Check ​https://github.com/WordPress/WordPress/blob/master/wp-admin/term.php#L15

[johnbillion]

In 46263:

Taxonomy: Redirect to the correct URL after deleting a taxonomy term attached to a post type other than Post.

Props donmhico, mukesh27

Fixes #47858