[PATCH 48/49] filters: Fix submitter display when we only have an email address

Damien Lespiau damien.lespiau at intel.com
Fri Oct 2 00:12:53 AEST 2015


I'm fairly sure I fixed that already, but here we go again. It's totally
valid to only have an email address for the submitter, so handle that
case properly.

Signed-off-by: Damien Lespiau <damien.lespiau at intel.com>
Acked-by: Stephen Finucane <stephen.finucane at intel.com>
---
 patchwork/templates/patchwork/filters.html | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/patchwork/templates/patchwork/filters.html b/patchwork/templates/patchwork/filters.html
index fe0c9b8..65fd6bf 100644
--- a/patchwork/templates/patchwork/filters.html
+++ b/patchwork/templates/patchwork/filters.html
@@ -54,11 +54,15 @@ $(document).ready(function() {
         },
         render: {
             option: function(item, escape) {
-                return '<div>' + escape(item.name) + ' <' +
-                                 escape(item.email) + '>' + '</div>';
+                if (item.name)
+                    return '<div>' + escape(item.name) + ' <' +
+                                     escape(item.email) + '>' + '</div>';
+                return '<div>' + escape(item.email) + '</div>';
             },
             item: function(item, escape) {
-                return '<div>' + escape(item.name) + '</div>';
+                if (item.name)
+                    return '<div>' + escape(item.name) + '</div>';
+                return '<div>' + escape(item.email) + '</div>';
             }
         },
         load: function(query, callback) {
-- 
2.1.0



More information about the Patchwork mailing list