WordPress.org

Make WordPress Core

Opened 8 years ago

Closed 7 years ago

Last modified 7 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 8 years ago.
Diff to avoid filtering out '/' from theme filenames.

Download all attachments as: .zip

Change History (9)

@zedlander
8 years ago

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

#1 @DD32
8 years ago

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

#2 @ryan
7 years ago

  • Milestone changed from 2.8 to 2.7.1

#3 @ryan
7 years ago

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

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

#4 @ryan
7 years ago

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

#5 @chaoskaizer
7 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 .

#6 @FFEMTcJ
7 years ago

  • Milestone changed from 2.7.1 to 2.7.2

See #9244 for patch to new problem.

#7 @automattor
7 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

#8 @ryan
7 years ago

[10676] for trunk

Note: See TracTickets for help on using tickets.