Opened 12 years ago

Closed 12 years ago

#404 closed defect (fixed)

Don't save a "medium" image unless we have to

Reported by: Christopher Allan Webber Owned by: Bassam Kurdali
Priority: major Milestone: 0.3.0
Component: component1 Keywords:
Cc: Parent Tickets:


It used to be that if the image was smaller than the maximum width/height, it would just not create a "medium" image. But when the exif rotation code was added, it was switched so that it would create a medium image no matter what.

Do something along the lines of:

if needs_exif_rotation or greater_than_maxsize:
    medium creating code here

Change History (4)

comment:1 by Christopher Allan Webber, 12 years ago

Owner: changed from somebody to Bassam Kurdali
Status: newassigned

comment:2 by Bassam Kurdali, 12 years ago

Ok, I've fixed this bug afaik, I'm posting here to ask if *not* creating the medium dict entry in media_files_dict will have any sideeffects that we don't want?
earlier I tried creating it with just the same filepath as original in the value, that backfired when deleting images because it would try to delete the same file twice, and fail the second time (perhaps deleting files could be made more fault tolerant?)

comment:4 by Christopher Allan Webber, 12 years ago

Resolution: fixed
Status: assignedclosed

Merged! Thanks, Bassam!

Note: See TracTickets for help on using tickets.