[wp-trac] Re: [WordPress Trac] #9918: Disable Flash Upload Option

WordPress Trac wp-trac at lists.automattic.com
Mon Jun 1 21:44:27 GMT 2009


#9918: Disable Flash Upload Option
-----------------------------+----------------------------------------------
 Reporter:  robertaccettura  |       Owner:  westi                         
     Type:  feature request  |      Status:  assigned                      
 Priority:  low              |   Milestone:  2.9                           
Component:  Upload           |     Version:  2.8                           
 Severity:  minor            |    Keywords:  reporter-feedback, needs-patch
-----------------------------+----------------------------------------------
Changes (by lloydbudd):

  * keywords:  reporter-feedback => reporter-feedback, needs-patch
  * priority:  normal => low
  * milestone:  2.8 => 2.9


Comment:

 v2.8 is sailing. Should only include regressions at this point.

 Also, patch does not meet coding standards -- I know, I know, WP does not
 meet its own coding style either, but your patch isn't even consider with
 the code around it or itself.

 {{{if(FORCE_BROWSER_UPLOAD)}}} should be {{{if ( FORCE_BROWSER_UPLOAD )}}}
 , etc.

 Actually, on further consideration, I think this should be WONTFIX.
 Already, seems well handled by plugin http://wordpress.org/extend/plugins
 /no-flash-uploader/ . Don't think we should move to defines for
 everything.

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


More information about the wp-trac mailing list