Context Navigation

← Previous Ticket
Next Ticket →

#4286 closed defect (bug) (fixed)

Plugin review URLs reference the raw 'p' value

Reported by:	jonoaldersonwp	Owned by:	dd32
Priority:	lowest	Milestone:
Component:	Plugin Directory	Keywords:	seo
Cc:

Description

Opening the 'Reviews' tab on a plugin page (such as https://wordpress.org/plugins/akismet/#reviews) reveals, when present, a list of reviews.

For each of these, the title of the review is linked to the page for that URL. However, it looks via a short URL, which redirects to the final URL - e.g., https://wordpress.org/support/plugin/?p=11306755 to https://wordpress.org/support/topic/good-but-147/.

We should link directly to the final URL instead.

Change History (4)

#1 @Otto42
7 years ago

Generating that final URL from inside the plugin directory is actually not the easiest thing in the world to do, since the plugin directory isn't running bbPress. So the P method was used because it simplifies the code a great deal, and the redirects are considered acceptable for the speed improvements.

#2 @jonoaldersonwp
7 years ago

No problem, we have bigger priorities. Let's leave this for future consideration when we've made more speed/performance improvements elsewhere to trade against.

#3 @dd32
7 years ago

Owner set to dd32
Resolution → fixed
Status new → closed

In 8474:

Plugin Directory: Link directly to the Reviews rather than using the ?p= syntax.

Fixes #4286.

#4 @dd32
7 years ago

In 8917:

Plugin Directory: Always link to the 'pretty permalink' of reviews, now that the caches will have cleared.

Amends [8474].
See #4286.

Note: See TracTickets for help on using tickets.

Download in other formats:

Making WordPress.org