From 18ada1e73ca9a129436e920abfaf9d1d0d618e98 Mon Sep 17 00:00:00 2001
From: Lova Andriarimalala <43842786+Xpirix@users.noreply.github.com>
Date: Mon, 15 Jul 2024 10:50:05 +0300
Subject: [PATCH] Add conversations count to feedback icons (#442)

* Add conversations count to feedback icons

* Reset the feedback button color when all tasks are completed
---
 .../templates/plugins/plugin_detail.html        | 17 ++++++++++++++---
 qgis-app/plugins/templatetags/plugin_utils.py   |  4 ++++
 2 files changed, 18 insertions(+), 3 deletions(-)

diff --git a/qgis-app/plugins/templates/plugins/plugin_detail.html b/qgis-app/plugins/templates/plugins/plugin_detail.html
index c85b2c04..2448114e 100644
--- a/qgis-app/plugins/templates/plugins/plugin_detail.html
+++ b/qgis-app/plugins/templates/plugins/plugin_detail.html
@@ -329,10 +329,21 @@ <h2>{{ object.name }}
                                 <td>{{ version.created_on|local_timezone }}</td>
                                 {% if user.is_staff or user in version.plugin.approvers or user in version.plugin.editors %}<td><form method="post" action="{% url "version_manage" object.package_name version.version %}">{% csrf_token %}
                                 {% if user.is_staff or user in version.plugin.approvers %}
-                                    {% if not version.approved %}<button class="btn btn-success btn-mini" type="submit" name="version_approve" id="version_approve"title="{% trans "Approve" %}"><i class="icon-thumbs-up icon-white"></i></button>{% else %}<button class="btn btn-warning btn-mini" type="submit" name="version_unapprove" id="version_unapprove" title="{% trans "Unapprove" %}"><i class="icon-thumbs-down icon-white"></i></button>{% endif %}
+                                    {% if not version.approved %}
+                                    <button class="btn btn-success btn-mini" type="submit" name="version_approve" id="version_approve"title="{% trans "Approve" %}">
+                                        <i class="icon-thumbs-up icon-white"></i>
+                                    </button>
+                                    {% else %}
+                                    <button class="btn btn-warning btn-mini" type="submit" name="version_unapprove" id="version_unapprove" title="{% trans "Unapprove" %}">
+                                        <i class="icon-thumbs-down icon-white"></i></button>
+                                    {% endif %}
                                 {% endif %}
-                                <a class="btn {% if version.feedback.exists %}btn-warning{% else %}btn-primary{% endif %} btn-mini"
-                                   href="{% url "version_feedback" object.package_name version.version %}" title="{% trans "Feedback" %}"><i class="icon-comments icon-white"></i>
+                                <a class="btn {% if version.feedback|feedbacks_not_completed|length >= 1 %}btn-warning{% else %}btn-primary{% endif %} btn-mini"
+                                   href="{% url "version_feedback" object.package_name version.version %}" title="{% trans "Feedback" %}">
+                                   <i class="icon-comments icon-white"></i>
+                                    {% if version.feedback|feedbacks_not_completed|length >= 2 %}
+                                        {{ version.feedback|feedbacks_not_completed|length }}
+                                    {% endif %}
                                 </a>
                                 {% if user.is_staff or user in version.plugin.editors %}
                                 <a class="btn btn-primary btn-mini" href="{% url "version_update" object.package_name version.version %}" title="{% trans "Edit" %}"><i class="icon-pencil icon-white"></i></a>&nbsp;<a class="btn btn-danger btn-mini delete" href="{% url "version_delete" object.package_name version.version %}" title="{% trans "Delete" %}"><i class="icon-remove icon-white"></i></a>{% endif %}</form>
diff --git a/qgis-app/plugins/templatetags/plugin_utils.py b/qgis-app/plugins/templatetags/plugin_utils.py
index 0bd4ca49..3daf1722 100755
--- a/qgis-app/plugins/templatetags/plugin_utils.py
+++ b/qgis-app/plugins/templatetags/plugin_utils.py
@@ -56,3 +56,7 @@ def _validate_image(file_path):
         return True
     except (FileNotFoundError, UnidentifiedImageError):
         return False
+
+@register.filter
+def feedbacks_not_completed(feedbacks):
+    return feedbacks.filter(is_completed=False)