Show full name in patch history, not just username

Noted by Stefan - we did this for everything except the patch history, so
let's make it consistent.

diff --git a/pgcommitfest/commitfest/models.py b/pgcommitfest/commitfest/models.py
index 124af720aa4e4d646a814124aec195494475d785..e1b83b4e60318271872a4c59b16ae99e083cb8ac 100644 (file)
--- a/pgcommitfest/commitfest/models.py
+++ b/pgcommitfest/commitfest/models.py
@@ -183,6 +183,10 @@ class PatchHistory(models.Model):
        by = models.ForeignKey(User, blank=False, null=False)
        what = models.CharField(max_length=500, null=False, blank=False)
 
+       @property
+       def by_string(self):
+               return "%s %s (%s)" % (self.by.first_name, self.by.last_name, self.by.username)
+
        def __unicode__(self):
                return "%s - %s" % (self.patch.name, self.date)
 

diff --git a/pgcommitfest/commitfest/templates/patch.html b/pgcommitfest/commitfest/templates/patch.html
index 97ac9b52910511a9513780427135abe18e113fef..ab66df49af3ae7e3e6fc7d12e02e57a21f0ad72a 100644 (file)
--- a/pgcommitfest/commitfest/templates/patch.html
+++ b/pgcommitfest/commitfest/templates/patch.html
@@ -95,7 +95,7 @@
            {%for h in patch.history %}   
            <tr>
              <td style="white-space: nowrap;">{{h.date}}</td>
-             <td style="white-space: nowrap;">{{h.by}}</td>
+             <td style="white-space: nowrap;">{{h.by_string}}</td>
              <td width="99%">{{h.what}}</td>
            </tr>
            {%endfor%}