From 803d728271fa46411e1536a3d872e37d0a28ccd4 Mon Sep 17 00:00:00 2001
From: Michael Watters <michael.watters@dart.biz>
Date: Dec 04 2018 08:18:11 +0000
Subject: Add build status to pull requests page


This change causes pagure to show the status of the most recent flags
from each third party application.

Merges #4073

---

diff --git a/pagure/lib/model.py b/pagure/lib/model.py
index 79b7ce6..1e6d9f4 100644
--- a/pagure/lib/model.py
+++ b/pagure/lib/model.py
@@ -2028,6 +2028,24 @@ class PullRequest(BASE):
         return [comment for comment in self.comments if not comment.commit_id]
 
     @property
+    def flags_stats(self):
+        """ Return some stats about the flags associated with this PR.
+        """
+        flags = self.flags
+        flags.reverse()
+
+        # Only keep the last flag from each service
+        tmp = {}
+        for flag in flags:
+            tmp[flag.username] = flag
+
+        output = collections.defaultdict(list)
+        for flag in tmp.values():
+            output[flag.status].append(flag)
+
+        return output
+
+    @property
     def score(self):
         """ Return the review score of the pull-request by checking the
         number of +1, -1, :thumbup: and :thumbdown: in the comment of the
diff --git a/pagure/static/pagure.css b/pagure/static/pagure.css
index 2206f02..dac79e7 100644
--- a/pagure/static/pagure.css
+++ b/pagure/static/pagure.css
@@ -274,7 +274,8 @@ th[data-sort] {
   overflow:auto;
 }
 
-.attachment_list p {
+.attachment_list p,
+.prs_flag_list p {
   margin-bottom: 0;
 }
 
diff --git a/pagure/templates/_render_pullrequests.html b/pagure/templates/_render_pullrequests.html
index f886ce2..1aa79d9 100644
--- a/pagure/templates/_render_pullrequests.html
+++ b/pagure/templates/_render_pullrequests.html
@@ -46,17 +46,17 @@
                             {% endif %}
                         </span>
                         {% endif %}
-            
+
                         <span title="branch to" class="text-muted ml-3" data-toggle="tooltip">
                             <span class="fa fa-fw fa-random pr-2"></span>
                               {{request.branch}}
                           </span>
-            
+
                         <span title="{{request.user_comments|count}} comments" class="text-muted ml-3" data-toggle="tooltip">
                           <span class="fa fa-fw fa-comment pr-2"></span>
                             {{request.user_comments|count}}
                         </span>
-                        
+
                 </span>
             </div>
                 {% if subtitle %}
@@ -87,7 +87,7 @@
                         </small>
                     {% else %}
                     <small>
-                        <span class="text-success font-weight-bold">Opened</span> 
+                        <span class="text-success font-weight-bold">Opened</span>
                         <span data-toggle="tooltip" title="{{
                         request.date_created | format_datetime
                       }}">{{ request.date_created | humanize}}</span> by <span title="{{
@@ -132,15 +132,43 @@
                       {% endif %}
                 </div>
                 <div>
-                        {% for tag in request.tags %}
-                            <span id="tag-{{ tag.tag }}" title="{{ tag.tag_description }}"
-                                  style="background-color:{{ tag.tag_color}}; font-size:90%"
-                                  class="badge badge-secondary"
-                                  >
-                                {{ tag.tag }}
-                            </span>
-                        {% endfor %}
+                  {% for tag in request.tags %}
+                      <span id="tag-{{ tag.tag }}" title="{{ tag.tag_description }}"
+                            style="background-color:{{ tag.tag_color}}; font-size:90%"
+                            class="badge badge-secondary">
+                          {{ tag.tag }}
+                      </span>
+                  {% endfor %}
                 </div>
+
+                {% if request.flags %}
+                  <div class="d-flex prs_flag_list">
+                    {% set flags_stats = request.flags_stats %}
+                    {% if flags_stats | length == 1 %}
+                      {% for status in flags_stats %}
+                        <p>{{ flags_stats[status][0].username }}:
+                          <span class="badge {{
+                            flags_stats[status][0] | flag2label }} font-size-09">
+                              {{ flags_stats[status][0].status }}
+                          </span>
+                        </p>
+                      {% endfor %}
+                    {% else %}
+                      <p>Flags:
+                      {% for status in flags_stats %}
+                        <span class="badge {{
+                          flags_stats[status][0] | flag2label }} font-size-09"
+                          title="{% for f in flags_stats[status] %}{{
+                          f.username}}{% if not loop.last %}, {% endif %}{% endfor %}">
+                            {{ flags_stats[status] | length }} {{ status }}
+                        </span>
+                      </p>
+                      {% endfor %}
+                    {% endif %}
+                  </div>
+                {% endif %}
+
+                </span>
                 {% endif %}
             </div>
         </div>