django-versatileimagefield icon indicating copy to clipboard operation
django-versatileimagefield copied to clipboard

Image warmer doesn't handle empty image fields

Open Bartvds opened this issue 8 years ago • 2 comments

The image warmer assumes every image field has a path, so with the FileSystem storage it naively tries using the empty string and we get a IsADirectoryError on the MEDIA_ROOT.

Bartvds avatar Sep 19 '17 13:09 Bartvds

i got this on each setup call in tests :c because we had signal connected to instance save method to warm images. so I've added if instance.image condition before calling .warm() method to handle it

paxapy avatar Nov 12 '19 12:11 paxapy

i got this on each setup call in tests :c because we had signal connected to instance save method to warm images. so I've added if instance.image condition before calling .warm() method to handle it

thanks, pasha!

sadman4eg avatar Jul 31 '20 09:07 sadman4eg