WordPress.org

Making WordPress.org

Opened 8 months ago

Closed 6 months ago

Last modified 6 months ago

#4364 closed defect (fixed)

Mixed case URLs in the theme directory break the display

Reported by: jonoaldersonwp Owned by: dd32
Milestone: Priority: high
Component: Theme Directory Keywords: seo analytics
Cc:

Description

See, e.g., https://wordpress.org/themes/twentyninetEEn/ vs https://wordpress.org/themes/twentynineteen/.

Looks like the JS process (see https://meta.trac.wordpress.org/ticket/4276) isn't handling that well. Hopefully, fixing #4276 resolves this.

Regardless, I'd like us to 301 redirect the mixed/uppercase version to the lowercase version. To contain the complexity of that, I'd be happy for it to just run on ^/themes/{{path}}/$.

Change History (6)

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


7 months ago

#2 @dd32
7 months ago

  • Owner set to dd32
  • Status changed from new to accepted

#3 @dd32
6 months ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 8920:

Theme Directory: Add canonical redirects for upper-case requests for Themes or section filters to their lower-case variant. The JS is case-sensitive.

Fixes #4364.

#4 @dd32
6 months ago

In 8921:

Theme Directory: Tighten a regular expression.

Amends [8920].
See #4364.

#5 @dd32
6 months ago

In 8934:

Theme Directory: Make the canonical redirects for uppercase urls a little less strict.

This uses path comparisons instead to reduce the complexity.

See #4364.

#6 @dd32
6 months ago

In 8935:

Theme Directory: Fix a typo that didn't make it to the previous commit.

Ammends [8934].
See #4364.

Note: See TracTickets for help on using tickets.