PIL.Image.save() need format parameter when using file object

[宋文武]

Hi! when upload a jpg file, I got:

[...]
  File "...src/mediagoblin/mediagoblin/media_types/image/processing.py", 
line 125, in resize_tool
    quality, filter)
  File "...src/mediagoblin/mediagoblin/media_types/image/processing.py", 
line 81, in resize_image
    resized.save(resized_file, quality=quality)
  File "...lib/python3.4/site-packages/Pillow-3.0.0-py3.4-linux-x86_64.egg/PIL/Image.py", line 1649, in save
    format = EXTENSION[ext]
KeyError: ''
[...]

According to ​https://pillow.readthedocs.org/en/3.0.x/reference/Image.html?highlight=save#PIL.Image.Image.save
when use file object instead of filename, the format parameter should always be used.

[Loic Dachary]

The modification looks good and is covered by the tests.

./runtests.sh --cov-report=term-missing --cov=mediagoblin mediagoblin/tests

mediagoblin/media_types/image/processing.py 190 65 66% 21-22, 71-72, 105-107, 137-151, 163-164, 245-246, 269, 282-307, 311, 337-342, 350-373, 377, 381-388, 398-404, 408-418, 422, 425-426, 436-446

Reviewed-by: Loic Dachary <loic@…>

[宋文武]

I meet another one in media_types/ascii/processing.py.
Instead of pass file object, we can pass filename and PIL will figure out the format itself.
New patch attached.

[Christopher Allan Webber]

Okay, fine by me, if PIL can do just fine reading it on its own!

[Christopher Allan Webber]

All 0.8.2 tickets are being rolled over to 0.9.0