From 1089e1f4de2442ef06c3f47bbaa2a51de5421b9e Mon Sep 17 00:00:00 2001 From: winkidney Date: Sat, 18 Aug 2018 07:31:58 -0700 Subject: [PATCH 1/3] Feauture: Add AUTO-DELETE option in settings For django-images, the AUTO-DELETE mark is required to remove files automatically. --- pinry/settings/base.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/pinry/settings/base.py b/pinry/settings/base.py index f342ecf..e90825a 100644 --- a/pinry/settings/base.py +++ b/pinry/settings/base.py @@ -133,3 +133,6 @@ IMAGE_SIZES = { 'standard': {'size': [600, 0]}, 'square': {'crop': True, 'size': [125, 125]}, } + +# User custom settings +IMAGE_AUTO_DELETE = True From 701b4dd581313959597218380e55d0a0a20d2762 Mon Sep 17 00:00:00 2001 From: winkidney Date: Sat, 18 Aug 2018 08:20:55 -0700 Subject: [PATCH 2/3] Fix: Add workaround for image object deletion after pin deleted --- core/models.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/core/models.py b/core/models.py index 6cbdc4e..7755866 100644 --- a/core/models.py +++ b/core/models.py @@ -5,6 +5,7 @@ from io import BytesIO from django.conf import settings from django.core.files.uploadedfile import InMemoryUploadedFile from django.db import models, transaction +from django.dispatch import receiver from django_images.models import Image as BaseImage, Thumbnail from taggit.managers import TaggableManager @@ -49,3 +50,7 @@ class Pin(models.Model): def __unicode__(self): return '%s - %s' % (self.submitter, self.published) + +@receiver(models.signals.post_delete, sender=Pin) +def delete_pin_images(sender, instance, **kwargs): + instance.image.delete() From f8268c3a9d64f4f9bb2d38e8fc4caa890249adf1 Mon Sep 17 00:00:00 2001 From: winkidney Date: Sat, 18 Aug 2018 08:38:16 -0700 Subject: [PATCH 3/3] Feature: Use original-image url instead in light-box Also add css to restrict original-image size --- pinry/static/css/lightbox.css | 1 + pinry/templates/includes/lightbox.html | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/pinry/static/css/lightbox.css b/pinry/static/css/lightbox.css index a55f160..528eaa4 100644 --- a/pinry/static/css/lightbox.css +++ b/pinry/static/css/lightbox.css @@ -22,6 +22,7 @@ } .lightbox-wrapper img { + max-width: 600px; display: none; -webkit-border-top-right-radius: 3px; -webkit-border-top-left-radius: 3px; diff --git a/pinry/templates/includes/lightbox.html b/pinry/templates/includes/lightbox.html index 46f0283..ba7dd1e 100644 --- a/pinry/templates/includes/lightbox.html +++ b/pinry/templates/includes/lightbox.html @@ -3,7 +3,7 @@