WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 6 years ago

Last modified 6 years ago

#8548 closed defect (bug) (fixed)

Theme preview fails when theme in subdirectory

Reported by: zedlander Owned by:
Milestone: 2.7.2 Priority: normal
Severity: normal Version: 2.7
Component: Themes Keywords:
Focuses: Cc:

Description

I have a folder called dreamhost in my themes directory. All the themes display and can be activated properly, but the preview is broken, only showing a white page. This is because the preview_theme function is stripping out '/', changing 'dreamhost/classic' to 'dreamhostclassic', which then fails. Patch is attached.

Attachments (1)

themefix.diff (923 bytes) - added by zedlander 7 years ago.
Diff to avoid filtering out '/' from theme filenames.

Download all attachments as: .zip

Change History (9)

@zedlander7 years ago

Diff to avoid filtering out '/' from theme filenames.

comment:1 @DD327 years ago

  • Component changed from General to Themes
  • Keywords has-patch added
  • Owner anonymous deleted
  • Version set to 2.7

comment:2 @ryan6 years ago

  • Milestone changed from 2.8 to 2.7.1

comment:3 @ryan6 years ago

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

(In [10483]) Fix previewing of themes in subdirs. Props zedlander. fixes #8548

comment:4 @ryan6 years ago

(In [10484]) Fix previewing of themes in subdirs. Props zedlander. fixes #8548 for 2.7

comment:5 @chaoskaizer6 years ago

  • Keywords has-patch removed
  • Resolution fixed deleted
  • Status changed from closed to reopened

current fixes broke all themes that used "-" character for a directory name.

example

  • themes\wp-theme
  • themes\wp-theme-27 .

comment:6 @FFEMTcJ6 years ago

  • Milestone changed from 2.7.1 to 2.7.2

See #9244 for patch to new problem.

comment:7 @automattor6 years ago

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

(In [10677]) Escape hyphen in regex. Props chrisbliss18. fixes #8548 #9244 for 2.7

comment:8 @ryan6 years ago

[10676] for trunk

Note: See TracTickets for help on using tickets.