Opened 12 months ago
Closed 12 months ago
#20807 closed defect (bug) (fixed)
EXIF ISOSpeedRatings can be an array
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 3.4 |
| Component: | Upload | Version: | |
| Severity: | normal | Keywords: | has-patch commit |
| Cc: |
Description
This trips wp_read_image_metadata(). Seems ISOSpeedRatings may include ISO Speed and ISO Latitude and is returned as array on Android devices.
Attachments (5)
Change History (13)
Also hide the drag-drop area on mobile devices (this actually should be in plupload, uploader.features.dragdrop should be false for mobile).
(error)
Rather than array_shift(), we should use reset().
Looking into this patch now and testing on my Android device.
- Keywords has-patch commit added
Looks good. 20807.diff doesn't shuffle around code, and makes the array check a bit cleaner.
Tested and confirmed as working for an Android device. Suggesting commit.
Obviously that should be ! hasClass('mobile') -- that was me doing some testing.

20807.patch was tested on an Android phone (yes, images can be uploaded from there nicely).