Upload file types should be checked BEFORE uploading.

[ipstenu]

Wanna get frustrated? Try uploading a 50 meg video and realizing you forgot to set your Upload file types to allow m4v.

Is there any way this can be checked BEFORE the file gets uploaded? As it stands today, you upload THEN it checks and deletes. Arg :)

[markoheijnen]

Is this still the case. I would say this is fixed by using Plupload?

[Ipstenu]

This is NOT fixed by Plupload and is still a problem today.

[ericlewis]

Yes, we can do something similar to the early image detection in the FilesAdded callback in wp.Uploader.

[chriscct7]

If its not an allowable filetype, it seems more like a bug that it doesn't do this detection earlier

[polevaultweb]

Patch added that tells plupload the allowed mime types, and throws the error early if the file upload is not allowed. To test, the file would need to be dragged and dropped onto uploader, as plupload restricts the selection of those file types with this implementation.

[ocean90]

@azaozz Can you take a look at 14244.diff​ please?

[ocean90]

In 37727:

Media: Pass allowed file extensions to Plupload.

Plupload's mime_types filter (​http://www.plupload.com/docs/Options#filters.mime_types) allows us to inform a user about an unsupported file before the file gets uploaded.

Props polevaultweb.
Fixes #14244.

[ocean90]

In 37728:

Media: Use the correct variable for the file object.

See #14244.