[wp-trac] [WordPress Trac] #18206: Upgrade the file uploader

WordPress Trac wp-trac at lists.automattic.com
Mon Sep 5 20:42:53 UTC 2011


#18206: Upgrade the file uploader
---------------------------------------+------------------
 Reporter:  azaozz                     |       Owner:
     Type:  task (blessed)             |      Status:  new
 Priority:  normal                     |   Milestone:  3.3
Component:  Upload                     |     Version:
 Severity:  normal                     |  Resolution:
 Keywords:  needs-testing needs-patch  |
---------------------------------------+------------------

Comment (by SergeyBiryukov):

 Since [18482], there's a notice on the uploader page:
 {{{
 Notice: Undefined variable: plupload in wp-admin/includes/media.php on
 line 1591
 }}}
 To reproduce:
 * Visit "Media" > "Add New" with Debug Bar enabled.
 * Or "Edit Post" > "Add an Image", open Firebug, search for "undefined" in
 the thickbox window (the notice is hidden by `hide-if-js` class).

 Code reference:[[BR]]
 http://core.trac.wordpress.org/browser/trunk/wp-
 admin/includes/media.php?rev=18596#L1591

 There used to be `media_upload_html_bypass()` function hooked to `post-
 html-upload-ui`, which accepted `$flash` as a parameter. The function is
 now gone, so perhaps we can remove `$plupload` parameter from there, or
 just pass `false`.

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


More information about the wp-trac mailing list