WordPress.org

Make WordPress Core

Opened 5 months ago

Last modified 2 months ago

#40158 reviewing defect (bug)

Plupload bug with Firefox and Mac OS where the user can't select .jpeg files

Reported by: stiofansisland Owned by: azaozz
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: External Libraries Keywords: has-patch needs-testing
Focuses: javascript Cc:

Description

NOTE: Core currently uses Plupload version 2.1.8, this bug is solved in version 2.2.1 + (latest stable is 2.3.1).

The bug
If plupload is limited to jpg/jpeg images and a user using FireFox and Mac OS tries to select images, all the .jpeg images are grayed out and not selectable, a user can select .jpg files but not .jpeg ones.

Bug Example
(You need to be running FireFox on Mac OS to recreate)
http://jsfiddle.net/sr8jarsc/1/

Solution
Update the Plupload version to 2.2.1 or greater
Example: http://jsfiddle.net/mr31zw6L/1/

Attachments (1)

40158.diff (283.9 KB) - added by bor0 5 months ago.

Download all attachments as: .zip

Change History (20)

#1 follow-up: @bor0
5 months ago

I (almost) have a patch ready, but can you please share the 2 images you're trying to upload so that I can test?

#2 in reply to: ↑ 1 @stiofansisland
5 months ago

Replying to bor0:

I (almost) have a patch ready, but can you please share the 2 images you're trying to upload so that I can test?

It seems to just go by extension so any .jpg and any .jpeg images will do.

Thanks,

Stiofan

@bor0
5 months ago

#3 @bor0
5 months ago

  • Keywords has-patch added

Was able to reproduce this. Added diff that updates the version of the library. Feel free to use https://gist.github.com/bor0/c1f47e46bf9824746ac06a5a4ac76b6f for testing. Note that this issue is only reproducible on Firefox.

#4 @swissspidy
5 months ago

  • Component changed from Upload to External Libraries
  • Milestone changed from Awaiting Review to 4.8
  • Version trunk deleted

#5 @swissspidy
5 months ago

#40207 was marked as a duplicate.

#6 @swissspidy
5 months ago

  • Milestone changed from 4.8 to 4.7.4

This ticket was mentioned in Slack in #core by swissspidy. View the logs.


5 months ago

This ticket was mentioned in Slack in #core-media by joemcgill. View the logs.


5 months ago

#9 @azaozz
5 months ago

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

This ticket was mentioned in Slack in #core by swissspidy. View the logs.


5 months ago

#11 @swissspidy
5 months ago

  • Keywords needs-testing added

#12 @azaozz
4 months ago

  • Milestone changed from 4.7.4 to 4.7.5

Don't think we can merge this in time for 4.7.4-beta.

#13 @Otto42
4 months ago

#40207 was marked as a duplicate.

This ticket was mentioned in Slack in #core by desrosj. View the logs.


3 months ago

#15 @desrosj
3 months ago

  • Milestone changed from 4.7.5 to 4.8

This ticket was mentioned in Slack in #core by obenland. View the logs.


3 months ago

#17 @obenland
3 months ago

  • Milestone changed from 4.8 to Future Release

#18 @wpcachepress
2 months ago

@obenland why so? this bug is related to other bug - chrome + non-standard TLDs too - media select bug https://core.trac.wordpress.org/ticket/40207

#19 @swissspidy
2 months ago

@wpcachepress The milestone was changed because WordPress 4.8 will probably be released today and this patch still needs testing.

Note: See TracTickets for help on using tickets.