[wp-trac] [WordPress Trac] #40070: Cannot remove theme with Javascript if folder name contains periods

WordPress Trac noreply at wordpress.org
Tue Mar 14 10:37:28 UTC 2017


#40070: Cannot remove theme with Javascript if folder name contains periods
-------------------------------------+-------------------------
 Reporter:  svanlooy                 |       Owner:
     Type:  defect (bug)             |      Status:  new
 Priority:  normal                   |   Milestone:  4.8
Component:  Themes                   |     Version:  4.7.3
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch needs-testing  |     Focuses:  javascript
-------------------------------------+-------------------------

Comment (by subrataemfluence):

 @MattyRob,

 I tested by modifying regex pattern in `function wp_ajax_delete_theme()`
 with your suggestion (`[^A-z0-9._\-]/`) but it did not work. The original
 error message ("Deletion failed: The requested theme does not exist")
 disappeared but the following new message is now coming up:

 "'''Deletion failed: Could not fully remove the theme theme.period.'''"
 and as expected the theme was neither removed from admin UI nor physically
 from the disk.

 `$theme_dir` variable however outputs the correct path.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/40070#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list