Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#4286 closed defect (bug) (fixed)

Plugin review URLs reference the raw 'p' value

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


Opening the 'Reviews' tab on a plugin page (such as 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., to

We should link directly to the final URL instead.

Change History (4)

#1 @Otto42
5 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
5 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
5 years 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
5 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.