Add ID column to commitfest page

The ID of a CF entry is the only stable identifier (people can change
the name). That's why tooling such as CFbot uses the ID of the patch for
a lot of things (including showing it on the cfbot overview page).
Having it visible on the page is quite useful for debugging purposes

In eee60a5 the ID was added to the title of the entry, but that made the
the title of the page itself less prominent. So this is an attempt to
have the information available, but not too prominently visible.

Author: JelteF

pgcommitfest/commitfest/templates/commitfest.html

@@ -61,6 +61,7 @@ <h3>{{p.is_open|yesno:"Active patches,Closed patches"}}</h3>
  <thead>
   <tr>
    <th>{%if p.is_open%}<a href="#" style="color:#333333;" onclick="return sortpatches(0);">Patch</a>{%if sortkey == 0%}<div style="float:right;"><i class="glyphicon glyphicon-arrow-down"></i></div>{%endif%}{%endif%}</th>
+   <th>{%if p.is_open%}<a href="#" style="color:#333333;" onclick="return sortpatches(4);">ID</a>{%if sortkey == 4%}<div style="float:right;"><i class="glyphicon glyphicon-arrow-down"></i></div>{%endif%}{%else%}ID{%endif%}</th>
    <th>Status</th>
    <th>Ver</th>
    <th>CI status</th>
@@ -85,6 +86,7 @@ <h3>{{p.is_open|yesno:"Active patches,Closed patches"}}</h3>
 {%endif%}
   <tr>
    <td><a href="{{p.id}}/">{{p.name}}</a></td>
+   <td>{{p.id}}</td>
    <td><span class="label label-{{p.status|patchstatuslabel}}">{{p.status|patchstatusstring}}</span></td>
    <td>{%if p.targetversion%}<span class="label label-default">{{p.targetversion}}</span>{%endif%}</td>
    <td class="cfbot-summary">

pgcommitfest/commitfest/views.py

@@ -187,6 +187,8 @@ def commitfest(request, cfid):
             orderby_str = 'lastmail, created'
         elif sortkey == 3:
             orderby_str = 'num_cfs DESC, modified, created'
+        elif sortkey == 4:
+            orderby_str = 'p.id'
         else:
             orderby_str = 'p.id'
             sortkey = 0