Safe way to use thumbnails in case file no longer exists

OpenUpSA · Dec 12, 2024 · c030704 · c030704
1 parent a5a84de
commit c030704
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/pombola/core/templatetags/safe_image.py b/pombola/core/templatetags/safe_image.py
@@ -0,0 +1,9 @@
+from django import template
+
+register = template.Library()
+
+@register.filter
+def safe_image_width(image):
+    if image and hasattr(image, 'width'):
+        return image.width
+    return None
diff --git a/pombola/south_africa/templates/core/organisation_people.html b/pombola/south_africa/templates/core/organisation_people.html
@@ -2,6 +2,7 @@
 {% load pagination_tags %}
 {% load staticfiles %}
 {% load thumbnail %}
+{% load safe_image %}
 
 
 {% block title %}{{ object.name }} People{% endblock %}
@@ -45,7 +46,11 @@ <h3>Show</h3>
 
                   <a href="{{ person.get_absolute_url }}">
                       {% thumbnail person.primary_image "58x78" crop="center" as im %}
+                      {% if im|safe_image_width %}
                       <img src="{{ im.url }}" alt="{{ person.name }}" width="{{ im.width }}" height="{{ im.height }}" />
+                      {% else %}
+                      <img src="{% static 'images/person-90x90.jpg' %}" height="58" width="58"/>
+                      {% endif %}
                       {% empty %}
                       <img src="{% static 'images/person-90x90.jpg' %}" height="58" width="58"/>
                       {% endthumbnail %}