Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#8548 closed defect (bug) (fixed)

Theme preview fails when theme in subdirectory

Reported by: zedlander Owned by:
Priority: normal Milestone: 2.7.2
Component: Themes Version: 2.7
Severity: normal Keywords:
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 4 years ago.
Diff to avoid filtering out '/' from theme filenames.

Download all attachments as: .zip

Change History (9)

zedlander — 4 years ago

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

comment:1   DD32 — 4 years ago

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

comment:2   ryan — 4 years ago

  • Milestone changed from 2.8 to 2.7.1

comment:3   ryan — 4 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   ryan — 4 years ago

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

comment:5   chaoskaizer — 4 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   FFEMTcJ — 4 years ago

  • Milestone changed from 2.7.1 to 2.7.2

See #9244 for patch to new problem.

comment:7   automattor — 4 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   ryan — 4 years ago

[10676] for trunk

Note: See TracTickets for help on using tickets.