404 error and redirect when using url with query string

[pmwp]

When a query var and the query_var of non-public CPT are the same:

• You will get a 404 error even when the permalink exists.
• redirect_guess_404_permalink() returns the permalink of other post which has slug like {query value}%.

Here an example:

• CPT foo has 'public' => false.
• The permalink /a123 exists.
• The permalink /bar/456 exists.

When you visit /a123?foo=4, You will redirect to other url bar/456?foo=4 and get a 404 error.
I want to get /a123 page and $_GET[ 'foo' ] which is not the CTP query_var.

I think CTP query_var parameter must be set to false, if public parameter is false.

Related tickets: #17609, #19693

[jorbin]

@markjaquith - do you think this is canonical related? Is there a change here that might make sense?

[wonderboymusic]

30018.diff​ fixes it. It's in WP::parse_request() - this seems like a reasonable fix, curious if it would break anything?

[wonderboymusic]

In 30018.2.diff​, non-viewable post types shouldn't have their query vars parsed in WP::parse_request(). Unit test (surprise, surprise) demonstrates this behavior. If you parse their query vars, and the var does not actually exist to represent that item, the request goes 404. The default value for query_var is true, which means it will be set to $post_type if you don't specify something else. register_post_type() then indiscriminately adds it to WP::public_query_vars

[wonderboymusic]

In 34215:

In WP::parse_request(), don't add query vars of non-viewable post types to WP::public_query_vars. In register_post_type(), don't add query vars of non-viewable post types to WP::public_query_vars.
In _unregister_post_type() (unit tests), don't add query vars of non-viewable post types to WP::public_query_vars.

Adds unit test.

Fixes #30018.