Making WordPress.org

Opened 13 months ago

Closed 13 months ago

Last modified 10 months ago

#4286 closed defect (fixed)

Plugin review URLs reference the raw 'p' value

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


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
13 months 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
13 months 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
13 months ago

  • Owner set to dd32
  • Resolution set to fixed
  • Status changed from new to closed

In 8474:

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

Fixes #4286.

#4 @dd32
10 months 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.