From e2779c006fd9477151a5873a181b09425dd13277 Mon Sep 17 00:00:00 2001
From: Ohad Lutzky <ohad@lutzky.net>
Date: Mon, 2 Aug 2021 21:32:49 +0100
Subject: [PATCH] Hide child column if only one child is shown

Fixes #233
---
 babybuddy/views.py                         |  9 +++++++++
 core/templates/core/diaperchange_list.html |  8 ++++++--
 core/templates/core/feeding_list.html      |  8 ++++++--
 core/templates/core/note_list.html         |  8 ++++++--
 core/templates/core/sleep_list.html        |  8 ++++++--
 core/templates/core/temperature_list.html  |  8 ++++++--
 core/templates/core/timer_list.html        | 16 ++++++++++------
 core/templates/core/tummytime_list.html    |  8 ++++++--
 core/templates/core/weight_list.html       |  8 ++++++--
 9 files changed, 61 insertions(+), 20 deletions(-)

diff --git a/babybuddy/views.py b/babybuddy/views.py
index 58c48284..c089f889 100644
--- a/babybuddy/views.py
+++ b/babybuddy/views.py
@@ -38,6 +38,15 @@ class BabyBuddyFilterView(FilterView):
     # TODO Figure out the correct way to use this.
     strict = False
 
+    def get_context_data(self, **kwargs):
+        context = super().get_context_data(**kwargs)
+        children = {
+            o.child for o in context['object_list'] if hasattr(o, "child")
+        }
+        if len(children) == 1:
+            context['unique_child'] = True
+        return context
+
 
 class UserList(StaffOnlyMixin, BabyBuddyFilterView):
     model = User
diff --git a/core/templates/core/diaperchange_list.html b/core/templates/core/diaperchange_list.html
index 664ca139..796981ee 100644
--- a/core/templates/core/diaperchange_list.html
+++ b/core/templates/core/diaperchange_list.html
@@ -22,7 +22,9 @@
             <thead class="thead-inverse">
                 <tr>
                     <th>{% trans "Time" %}</th>
-                    <th>{% trans "Child" %}</th>
+                    {% if not unique_child %}
+                        <th>{% trans "Child" %}</th>
+                    {% endif %}
                     <th class="text-center">{% trans "Contents" %}</th>
                     <th>{% trans "Color" %}</th>
                     <th>{% trans "Amount" %}</th>
@@ -34,7 +36,9 @@
                     {% cycle "odd" "even" as row_class silent %}
                     <tr class="{{ row_class }}">
                         <th scope="row">{{ change.time|datetime_short }}</th>
-                        <td><a href="{% url 'core:child' change.child.slug %}">{{ change.child }}</a></td>
+                        {% if not unique_child %}
+                            <td><a href="{% url 'core:child' change.child.slug %}">{{ change.child }}</a></td>
+                        {% endif %}
                         <td class="text-center">
                             {% if change.wet %}<span title="{% trans "Wet" %}">💦</span>{% endif %}
                             {% if change.solid %}<span title="{% trans "Solid" %}">💩</span>{% endif %}
diff --git a/core/templates/core/feeding_list.html b/core/templates/core/feeding_list.html
index b21caefd..b2cfae3a 100644
--- a/core/templates/core/feeding_list.html
+++ b/core/templates/core/feeding_list.html
@@ -22,7 +22,9 @@
             <thead class="thead-inverse">
                 <tr>
                     <th>{% trans "Date" %}</th>
-                    <th>{% trans "Child" %}</th>
+                    {% if not unique_child %}
+                        <th>{% trans "Child" %}</th>
+                    {% endif %}
                     <th>{% trans "Method" %}</th>
                     <th>{% trans "Type" %}</th>
                     {% comment %}Abbreviation of "Amount"{% endcomment %}
@@ -36,7 +38,9 @@
                     {% cycle "odd" "even" as row_class silent %}
                     <tr class="{{ row_class }}">
                         <th scope="row">{{ feeding.start|datetime_short }}</th>
-                        <td><a href="{% url 'core:child' feeding.child.slug %}">{{ feeding.child }}</a></td>
+                        {% if not unique_child %}
+                            <td><a href="{% url 'core:child' feeding.child.slug %}">{{ feeding.child }}</a></td>
+                        {% endif %}
                         <td>{{ feeding.get_method_display }}</td>
                         <td>{{ feeding.get_type_display }}</td>
                         <td>
diff --git a/core/templates/core/note_list.html b/core/templates/core/note_list.html
index f65fa478..46b7ba19 100644
--- a/core/templates/core/note_list.html
+++ b/core/templates/core/note_list.html
@@ -22,7 +22,9 @@
             <thead class="thead-inverse">
                 <tr>
                     <th>{% trans "Time" %}</th>
-                    <th>{% trans "Child" %}</th>
+                    {% if not unique_child %}
+                        <th>{% trans "Child" %}</th>
+                    {% endif %}
                     <th>{% trans "Note" %}</th>
                     <th class="text-center">{% trans "Actions" %}</th>
                 </tr>
@@ -31,7 +33,9 @@
                 {% for note in object_list %}
                     <tr>
                         <th scope="row">{{ note.time|datetime_short }}</th>
-                        <td><a href="{% url 'core:child' note.child.slug %}">{{ note.child }}</a></td>
+                        {% if not unique_child %}
+                            <td><a href="{% url 'core:child' note.child.slug %}">{{ note.child }}</a></td>
+                        {% endif %}
                         <td>{{ note.note }}</td>
                         <td class="text-center">
                             <div class="btn-group btn-group-sm" role="group" aria-label="{% trans "Actions" %}">
diff --git a/core/templates/core/sleep_list.html b/core/templates/core/sleep_list.html
index ab3b2f43..3bd7cce0 100644
--- a/core/templates/core/sleep_list.html
+++ b/core/templates/core/sleep_list.html
@@ -23,7 +23,9 @@
                 <tr>
                     <th>{% trans "Start" %}</th>
                     <th>{% trans "End" %}</th>
-                    <th>{% trans "Child" %}</th>
+                    {% if not unique_child %}
+                        <th>{% trans "Child" %}</th>
+                    {% endif %}
                     <th>{% trans "Duration" %}</th>
                     <th class="text-center">{% trans "Nap" %}</th>
                     <th class="text-center">{% trans "Actions" %}</th>
@@ -35,7 +37,9 @@
                     <tr class="{{ row_class }}">
                         <th scope="row">{{ sleep.start|datetime_short }}</th>
                         <td>{{ sleep.end|datetime_short }}</td>
-                        <td><a href="{% url 'core:child' sleep.child.slug %}">{{ sleep.child }}</a></td>
+                        {% if not unique_child %}
+                            <td><a href="{% url 'core:child' sleep.child.slug %}">{{ sleep.child }}</a></td>
+                        {% endif %}
                         <td>{{ sleep.duration|duration_string }}</td>
                         <td class="text-center">{{ sleep.nap|bool_icon }}</td>
                         <td class="text-center">
diff --git a/core/templates/core/temperature_list.html b/core/templates/core/temperature_list.html
index 046a293b..74997636 100644
--- a/core/templates/core/temperature_list.html
+++ b/core/templates/core/temperature_list.html
@@ -22,7 +22,9 @@
             <thead class="thead-inverse">
                 <tr>
                     <th>{% trans "Time" %}</th>
-                    <th>{% trans "Child" %}</th>
+                    {% if not unique_child %}
+                        <th>{% trans "Child" %}</th>
+                    {% endif %}
                     <th>{% trans "Temperature" %}</th>
                     <th class="text-center">{% trans "Actions" %}</th>
                 </tr>
@@ -32,7 +34,9 @@
                     {% cycle "odd" "even" as row_class silent %}
                     <tr class="{{ row_class }}">
                         <th scope="row">{{ temperature.time|datetime_short }}</th>
-                        <td><a href="{% url 'core:child' temperature.child.slug %}">{{ temperature.child }}</a></td>
+                        {% if not unique_child %}
+                            <td><a href="{% url 'core:child' temperature.child.slug %}">{{ temperature.child }}</a></td>
+                        {% endif %}
                         <td>{{ temperature.temperature }}</td>
                         <td class="text-center">
                             <div class="btn-group btn-group-sm" role="group" aria-label="{% trans "Actions" %}">
diff --git a/core/templates/core/timer_list.html b/core/templates/core/timer_list.html
index 12c3eee1..570328ab 100644
--- a/core/templates/core/timer_list.html
+++ b/core/templates/core/timer_list.html
@@ -23,7 +23,9 @@
             <tr>
                 <th>{% trans "Start" %}</th>
                 <th>{% trans "Name" %}</th>
-                <th>{% trans "Child" %}</th>
+                {% if not unique_child %}
+                    <th>{% trans "Child" %}</th>
+                {% endif %}
                 <th>{% trans "Duration" %}</th>
                 <th>{% trans "End" %}</th>
                 <th>{% trans "Active" %}</th>
@@ -35,11 +37,13 @@
                 <tr>
                     <th scope="row">{{ timer.start|datetime_short }}</th>
                     <td><a href="{% url 'core:timer-detail' timer.id %}">{{ timer }}</a></td>
-                    <td>
-                        {% if timer.child %}
-                            <a href="{% url 'core:child' timer.child.slug %}">{{ timer.child }}</a>
-                        {% endif %}
-                    </td>
+                    {% if not unique_child %}
+                        <td>
+                            {% if timer.child %}
+                                <a href="{% url 'core:child' timer.child.slug %}">{{ timer.child }}</a>
+                            {% endif %}
+                        </td>
+                    {% endif %}
                     <td>{{ timer.duration|duration_string }}</td>
                     <td>
                         {% if timer.end %}
diff --git a/core/templates/core/tummytime_list.html b/core/templates/core/tummytime_list.html
index f05d5c98..c9f50113 100644
--- a/core/templates/core/tummytime_list.html
+++ b/core/templates/core/tummytime_list.html
@@ -22,7 +22,9 @@
                 <tr>
                     <th>{% trans "Start" %}</th>
                     <th>{% trans "End" %}</th>
-                    <th>{% trans "Child" %}</th>
+                    {% if not unique_child %}
+                        <th>{% trans "Child" %}</th>
+                    {% endif %}
                     <th>{% trans "Duration" %}</th>
                     <th>{% trans "Milestone" %}</th>
                     <th class="text-center">{% trans "Actions" %}</th>
@@ -33,7 +35,9 @@
                     <tr>
                         <th scope="row">{{ tummytime.start|datetime_short }}</th>
                         <td>{{ tummytime.end|datetime_short }}</td>
-                        <td><a href="{% url 'core:child' tummytime.child.slug %}">{{ tummytime.child }}</a></td>
+                        {% if not unique_child %}
+                            <td><a href="{% url 'core:child' tummytime.child.slug %}">{{ tummytime.child }}</a></td>
+                        {% endif %}
                         <td>{{ tummytime.duration|duration_string }}</td>
                         <td>{{ tummytime.milestone }}</td>
                         <td class="text-center">
diff --git a/core/templates/core/weight_list.html b/core/templates/core/weight_list.html
index 464a7748..d89a79cd 100644
--- a/core/templates/core/weight_list.html
+++ b/core/templates/core/weight_list.html
@@ -22,7 +22,9 @@
             <thead class="thead-inverse">
                 <tr>
                     <th>{% trans "Date" %}</th>
-                    <th>{% trans "Child" %}</th>
+                    {% if not unique_child %}
+                        <th>{% trans "Child" %}</th>
+                    {% endif %}
                     <th>{% trans "Weight" %}</th>
                     <th class="text-center">{% trans "Actions" %}</th>
                 </tr>
@@ -32,7 +34,9 @@
                     {% cycle "odd" "even" as row_class silent %}
                     <tr class="{{ row_class }}">
                         <th scope="row">{{ weight.date }}</th>
-                        <td><a href="{% url 'core:child' weight.child.slug %}">{{ weight.child }}</a></td>
+                        {% if not unique_child %}
+                            <td><a href="{% url 'core:child' weight.child.slug %}">{{ weight.child }}</a></td>
+                        {% endif %}
                         <td>{{ weight.weight }}</td>
                         <td class="text-center">
                             <div class="btn-group btn-group-sm" role="group" aria-label="{% trans "Actions" %}">