Merge branch '5-name-favicons-dynamically' into 'master'

Resolve "Name favicons dynamically"

Closes #5

See merge request !5

favicon/models.py

@@ -7,6 +7,7 @@ from django.core.files.storage import default_storage as storage
 from django.core.files.uploadedfile import InMemoryUploadedFile
 from django.db import models
 from django.db.models import Q, signals
+from django.utils.text import slugify
 
 use_sites = hasattr(settings, "SITE_ID")
 
@@ -138,7 +139,8 @@ class FaviconImg(models.Model):
 
         tmp_io = BytesIO()
         tmp.save(tmp_io, format='PNG')
-        tmp_file = InMemoryUploadedFile(tmp_io, None, f"fav-{self.size}s.png", 'image/png', sys.getsizeof(tmp_io), None)
+        file_name = f"{slugify(self.faviconFK.title)}-{self.size}s.png"
+        tmp_file = InMemoryUploadedFile(tmp_io, None, file_name, 'image/png', sys.getsizeof(tmp_io), None)
 
         self.faviconImage = tmp_file