Make WordPress Core

Opened 12 years ago

Closed 10 years ago

Last modified 10 years ago

#22403 closed defect (bug) (fixed)

Prevent default browser behavior with Plupload disallowed file extensions

Reported by: nacin's profile nacin Owned by: azaozz's profile azaozz
Milestone: 4.4 Priority: normal
Severity: normal Version: 3.3
Component: Upload Keywords: has-patch
Focuses: Cc:

Description

See #22149. When Plupload is set to accept only certain file types, the drag event falls back to the browser's default behavior. That means HTML files replace the entire viewport, ZIP files get re-downloaded, etc. Ideally, our own handlers intercept such a drag event and prevents anything else from happening.

Attachments (1)

22403.diff (580 bytes) - added by antpb 10 years ago.

Download all attachments as: .zip

Change History (7)

#1 @kovshenin
12 years ago

  • Cc kovshenin added

#2 @chriscct7
10 years ago

  • Keywords needs-patch added
  • Priority changed from low to normal
  • Severity changed from minor to normal

#3 @antpb
10 years ago

  • Keywords has-patch added; needs-patch removed
  • Milestone changed from Future Release to 4.4

We could add some jQuery to the admin-header file to disallow drag and drop events. The following patch fixes it. I've tested the media upload page's drag and drop feature after applying the patch and this does not overwrite the functionality.

@antpb
10 years ago

#4 @wonderboymusic
10 years ago

  • Owner set to azaozz
  • Status changed from new to assigned

#5 @azaozz
10 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 35019:

Plupload: update to 2.1.8. Changelog: https://github.com/moxiecode/plupload/releases.

Hide the editor uploader overlay on all drop events in the window, not only when dropping on the editor uploader element.

Fixes #34251, #22403.

#6 @azaozz
10 years ago

Fixed by updating to Plupload 2.1.8. It properly fires the error event and prevents the default when a file extension is rejected.

Last edited 10 years ago by azaozz (previous) (diff)
Note: See TracTickets for help on using tickets.