Opened 14 years ago

Closed 14 years ago

Last modified 13 years ago

#14330 closed (fixed)

Test failure with get_image_dimensions since [13715]

Reported by: Gabriel Hurley Owned by: Gabriel Hurley
Component: File uploads/storage Version: dev
Severity: Keywords: get_image_dimensions PIL mode
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

On my windows xp machine (not on my mac os x box) the file_storage.InconsistentGetImageDimensionsBug fails because PIL is unable to read the data from the already-open ImageFile. The problem is that in the test case the ImageFile is created by calling open() on the file path without an explicit mode. This defaults to mode 'r', and PIL on Windows seems to only parse the data from the ImageFile if the mode is 'rb'.

I've attached a tiny patch that adds the mode flag to the test case, but when dealing with images the mode flag really ought to always be 'rb', right? Maybe there ought to be a larger fix here...

Attachments (1)

image_storage_read_binary.diff​ (712 bytes ) - added by Gabriel Hurley 14 years ago.
sets the mode flag to 'rb' for the InconsistentGetImageDimensions test case.

Download all attachments as: .zip

Change History (5)

by Gabriel Hurley, 14 years ago

Attachment: image_storage_read_binary.diff​ added

sets the mode flag to 'rb' for the InconsistentGetImageDimensions test case.

comment:1 by Gabriel Hurley, 14 years ago

Has patch: set
Owner: changed from nobody to Gabriel Hurley
Status: new → assigned

comment:2 by Alex Gaynor, 14 years ago

Triage Stage: Unreviewed → Ready for checkin

comment:3 by Alex Gaynor, 14 years ago

Resolution: → fixed
Status: assigned → closed

Fixed by [14001].

comment:4 by Jacob, 13 years ago

milestone: 1.3

Milestone 1.3 deleted

Note: See TracTickets for help on using tickets.
Back to Top