#14244 closed defect (bug) (fixed)
Upload file types should be checked BEFORE uploading.
Reported by: | ipstenu | Owned by: | ocean90 |
---|---|---|---|
Milestone: | 4.6 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Media | Keywords: | has-patch needs-testing |
Focuses: | Cc: |
Description
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 :)
Attachments (1)
Change History (14)
#1
@
14 years ago
- Component changed from General to Media
- Milestone changed from Awaiting Review to Future Release
#4
@
10 years ago
- Keywords needs-patch added
- Type changed from defect (bug) to enhancement
Yes, we can do something similar to the early image detection in the FilesAdded callback in wp.Uploader.
#5
@
9 years ago
- Type changed from enhancement to defect (bug)
If its not an allowable filetype, it seems more like a bug that it doesn't do this detection earlier
#6
@
9 years ago
- Keywords has-patch needs-testing added; needs-patch removed
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.
This ticket was mentioned in Slack in #core-images by joemcgill. View the logs.
9 years ago
This ticket was mentioned in Slack in #core-images by joemcgill. View the logs.
8 years ago
#10
@
8 years ago
@azaozz Can you take a look at 14244.diff please?
#11
@
8 years ago
- Owner set to ocean90
- Resolution set to fixed
- Status changed from new to closed
In 37727:
Is this still the case. I would say this is fixed by using Plupload?