[wp-meta] [Making WordPress.org] #7085: Correct various "plugin type" issues
Making WordPress.org
noreply at wordpress.org
Fri Jun 23 02:38:01 UTC 2023
#7085: Correct various "plugin type" issues
------------------------------+---------------------
Reporter: jonoaldersonwp | Owner: (none)
Type: defect (bug) | Status: new
Priority: high | Milestone:
Component: Plugin Directory | Resolution:
Keywords: seo performance |
------------------------------+---------------------
Comment (by dd32):
Replying to [comment:2 jonoaldersonwp]:
> We'll still need canonical URLs, then, which include the business model
string.
Yep, I agree.
Some notes while looking at this:
> Filtered states omit a canonical URL tag
https://wordpress.org/plugins/browse/blocks/?plugin_business_model=community
doesn't contain a canonical link, as it's a multi-taxonomy query which we
don't currently support canonical links for right now.
Because from a code point of view, it's not possible to determine which
taxonomy is the "primary" one and which should appear as query vars. We
can probably add minimal support by doing something like "if multi
taxonomy, and only one has a pretty-permalink, we can assume that's
primary".
> Requests to out-of-bounds filtered ranges
https://wordpress.org/plugins/browse/blocks/page/2/?plugin_business_model=community
is redirected by `Plugin_Directory->custom_redirects:1278`, same thing
which redirects urls like `/plugins/browse/somethingrandom/` or
`/plugins/browse/blocks/sillyendpoint/`.
I don't think a redirect is actually the proper thing here, probably a 404
is more appropriate.
The actual problem I think is that
https://wordpress.org/plugins/browse/blocks/page/2/ links to this
problematic URL, the Community/Commercial links here should likely remove
the pagination state.
> The 'all' link on paginated+filtered states
Again, these urls should be omitting any pagination data I think, which I
think would resolve this.
--
Ticket URL: <https://meta.trac.wordpress.org/ticket/7085#comment:3>
Making WordPress.org <https://meta.trac.wordpress.org/>
Making WordPress.org
More information about the wp-meta
mailing list