[wp-trac] Re: [WordPress Trac] #9452: Plugin Editor lists all files in plugin - should limit to text only

WordPress Trac wp-trac at lists.automattic.com
Tue May 5 09:37:44 GMT 2009


#9452: Plugin Editor lists all files in plugin - should limit to text only
----------------------------+-----------------------------------------------
 Reporter:  jbsil           |        Owner:  jbsil   
     Type:  defect (bug)    |       Status:  reopened
 Priority:  normal          |    Milestone:  2.8     
Component:  Administration  |      Version:  2.8     
 Severity:  normal          |   Resolution:          
 Keywords:  has-patch       |  
----------------------------+-----------------------------------------------
Changes (by hakre):

  * keywords:  needs-patch => has-patch


Comment:

 unneeded wrapper if no error then has been removed, wp_die() direct.

 while goind through I saw that you are not able to actually filter the
 list of extensions, you can only add new ones. this has been simplyfied to
 filter the actual list.

 the i asked myself if it makes sense to use a regex to get the file
 extension. isn't instrrev or similar faster? but this is backend and one
 call only so maybe not a need to put the finger on.

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


More information about the wp-trac mailing list