Making WordPress.org

Opened 6 years ago

Closed 5 years ago

#4415 closed defect (bug) (fixed)

Invalid theme browse categories/types should 404

Reported by: jonoaldersonwp's profile jonoaldersonwp Owned by: dd32's profile dd32
Milestone: Priority: normal
Component: Theme Directory Keywords: seo analytics
Cc:

Change History (9)

#1 @dd32
5 years ago

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

In 9188:

Theme Directory: 404 on invalid browse requests.

Fixes #4415.

#2 @dd32
5 years ago

In 9189:

Theme Directory: Cache bump after r9188.

See #4415.

#3 follow-up: @jonoaldersonwp
5 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Can we get that to return the 404 template?

If that's a pain, could we fix/disable the JS logic which tries to set (and accidentally strips) the page title?

#4 in reply to: ↑ 3 @dd32
5 years ago

Replying to jonoaldersonwp:

Can we get that to return the 404 template?

It does, what are you seeing? (The 404 template is, and will stay, a "no themes found" page)
https://cloudup.com/cr1o2UtW-KX

If that's a pain, could we fix/disable the JS logic which tries to set (and accidentally strips) the page title?

Uhh.. I'll see if I can get it to at least avoid clearing it.

#5 @dd32
5 years ago

In 9244:

Themes: Don't query for themes on the 404 page.

This was causing the 404 page to still query for themes and select the improper filter element.

See #4415.

#6 @dd32
5 years ago

In 9245:

Themes: Bump JS cache after r9244.

See #4415.

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


5 years ago

#8 @SergeyBiryukov
5 years ago

  • Component changed from General to Theme Directory

https://wordpress.org/themes/browse/non-existing-theme/ appears to return a 404 error now. What's left here?

#9 @jonoaldersonwp
5 years ago

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

Ah, let's call this done! :)

Note: See TracTickets for help on using tickets.