WordPress.org

Making WordPress.org

Opened 2 years ago

Last modified 5 months ago

#2939 assigned defect

Themes older than 2 years cannot be found by an exact slug match

Reported by: SergeyBiryukov Owned by:
Milestone: Priority: normal
Component: Theme Directory Keywords:
Cc:

Description (last modified by SergeyBiryukov)

Looks like themes older than 2 years cannot be found even by an exact slug match:

Some details from @otto42:

Seems like there is no exact-match check in there at all, and the limiter that cuts off at 2 years doesn't look at searches or expects an exact match check to be done elsewhere.
https://meta.trac.wordpress.org/browser/sites/trunk/wordpress.org/public_html/wp-content/plugins/theme-directory/query-modifications.php?rev=2920#L110
also perhaps there used to be a dropdown to search for author or theme name rather than just a general search? Because a lot of the code makes the assumption that you can search by several fields.

Change History (8)

#1 @SergeyBiryukov
2 years ago

  • Description modified (diff)
  • Summary changed from Themes older than 2 years cannot be found by a direct slug match to Themes older than 2 years cannot be found by an exact slug match

#2 @SergeyBiryukov
21 months ago

Just stumbled upon this again.

Since the 2-year search limit does not exist in the new Plugin Directory, perhaps now is the time to remove it from the Theme Directory as well?

Switching the banner to mention the latest 3 major WordPress releases instead of the arbitrary 2 years check (#3010) might also be a good idea, but it looks like themes don't have a Tested up to: field (or a standardized readme.txt file at all), so that's something for another time.

This ticket was mentioned in Slack in #themereview by poena. View the logs.


21 months ago

This ticket was mentioned in Slack in #meta by sergey. View the logs.


21 months ago

#5 @dingo_d
5 months ago

  • Priority changed from normal to high

@SergeyBiryukov could you implement the fix you had in mind, since the TRT now requires a valid readme.txt (which means that the Tested up to field should be present in it).

https://make.wordpress.org/themes/handbook/review/required/#readme-txt-file

#6 @dd32
5 months ago

  • Owner set to dd32
  • Priority changed from high to normal
  • Status changed from new to accepted

The Tested Up to value for Themes can't be relied upon, for historical reasons. Maybe in two years time when all themes in that timeframe have valid readme's (Except new updates today still don't).

The original request here, for direct requests to a theme to bypass the limit seems reasonable still though.

#7 @dd32
5 months ago

  • Owner dd32 deleted
  • Status changed from accepted to assigned

Actually, I misread the ticket.
Direct links to themes work, searches for exact matches don't.
That seems reasonable enough to me, which explains why the tickets still open.

This is in the too-hard basket for now, as WP_Query isn't flexible enough for this scenario and altering the WP_Query results to force an exact-match is likely to have knock on effects in the caching plugin and API that's reliant upon the same code.

IMHO this ticket is a wontfix or maybelater for when the Theme Directory search is ElasticSearch powered.

#8 @dingo_d
5 months ago

IMHO this ticket is a wontfix or maybelater for when the Theme Directory search is ElasticSearch powered.

Any info when this might be?

Note: See TracTickets for help on using tickets.