WordPress.org

Making WordPress.org

Opened 8 months ago

Last modified 6 weeks ago

#4415 reopened defect

Invalid theme browse categories/types should 404

Reported by: jonoaldersonwp Owned by: dd32
Milestone: Priority: normal
Component: General Keywords: seo analytics
Cc:

Change History (6)

#1 @dd32
8 weeks 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
8 weeks ago

In 9189:

Theme Directory: Cache bump after r9188.

See #4415.

#3 follow-up: @jonoaldersonwp
8 weeks 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
8 weeks 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
6 weeks 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
6 weeks ago

In 9245:

Themes: Bump JS cache after r9244.

See #4415.

Note: See TracTickets for help on using tickets.