[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