Opened 2 years ago
Closed 23 months ago
#16220 closed defect (bug) (duplicate)
Media Upload from custom tab using Flash Uploader is broken
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | Upload | Version: | 3.0.4 |
| Severity: | normal | Keywords: | |
| Cc: |
Description
If you create a custom tab in the Media Uploader, and try to upload using the Flash Upload method (as opposed to the Browser Upload method), any hooks you have in place to intercept the upload (media_upload_$tab) will not be triggered.
Reason: The upload function are not aware of the tab. Even though the Flash Uploader passes the $tab variable, the upload functions are not receiving it.
Cause: The $tab variable is set from the GET string ($_GET['tab']). This only works with the Browser Upload method. Flash passes variables with POST($_POST['tab']).
Fix: Inside /wp-admin/media-upload.php and /wp-admin/includes/media.php replace all $_GET['tab'] with $_REQUEST['tab'].
Also, the global $tab variable must be set.
Add it to /wp-admin/includes/media.php around line #101 after the 'else{'
global $tab;
Change History (4)
comment:2
jackcrosby
— 2 years ago
That should be '/wp-admin/media-upload.php' that needs 'global $tab;' after the else on line #104
comment:3
jackcrosby
— 2 years ago
Hmmm... WP 3.1 changed something. Now the $tab isn't getting passed at all.
Related: #15655