[wp-trac] [WordPress Trac] #6531: Recursively search for files in theme and plugin editors

[WordPress Trac]

#6531: Recursively search for files in theme and plugin editors
-------------------------------------+-----------------------------
 Reporter:  torbens                  |       Owner:  chsxf
     Type:  defect (bug)             |      Status:  assigned
 Priority:  normal                   |   Milestone:  Future Release
Component:  Template                 |     Version:  2.5
 Severity:  minor                    |  Resolution:
 Keywords:  has-patch needs-testing  |
-------------------------------------+-----------------------------
Changes (by Daedalon):

 * cc: daedalon@… (added)
 * type:  enhancement => defect (bug)


Comment:

 Thanks Mike for the patch! Verified, works. Also improves the coding style
 to match http://make.wordpress.org/core/handbook/coding-standards/php/.

 I'd like to see this included in 3.6 after two improvements:

 1. Use curly braces with all ifs. Avoids hassle and bugs in later edits. A
 decent percentage of bugs are caused by editing code like this:

 {{{
 } else {
         if ( plugin_basename("$dir/$file") != $plugin )
                 $plugin_files[] = plugin_basename("$dir/$file");
 }
 }}}


 2. Theme editor could use the same functionality.

 Marking ticket as defect (bug) instead of enhancement. More appropriate
 for the file list not displaying all files of a plugin or theme.
 Milestone, anyone?

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/6531#comment:17>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software