Delete a child theme will delete parent theme

[Wupperpirat]

I'm one of the maintainers and supporters of Thematic testing a lot of child themes. Today I wanted to clean up my theme directory using the backend. Deleting one of my child themes will delete the parent theme instead.

I'm running the current 2.8 SVN version 10912 on my local machine.

Let me know if you need more details.

Thanks,

Chris

[DD32]

So the child theme remains? Just the Parent theme gets deleted? (Looks like someone got Template vs Stylesheet mixed up)

[ionfish]

Can you test the attached patch?

[DD32]

ionfish: Did you test that at all? It doesnt look like it'd do anything at all to me.

$_GETstylesheet? isnt passed at all, Line 197 of themes.php passes in $template, All occurences of $template related to the deletion should probably be renamed to $stylesheet

[ionfish]

The Preview and Activate URIs both include the template and the stylesheet (now I look, the Delete one doesn't), so I based my patch on that. Here's an updated one.

[Wupperpirat]

Thanks a lot for your help .. the last 9519.1.diff is working .. I checked deletion of child themes: worked .. and I checked deletion of standard themes: worked.

Again .. thanks a lot for your help & Happy Easter!

Cheers,

Chris

[ryan]

(In [10915]) Use stylesheet directory when deleting a theme so we don't accidentally delete a parent theme. Props ionfish. fixes #9519