diff mbox

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

Message ID 1443708774-26996-49-git-send-email-damien.lespiau@intel.com
State Accepted
Headers show

Commit Message

Damien Lespiau Oct. 1, 2015, 2:12 p.m. UTC
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@intel.com>
Acked-by: Stephen Finucane <stephen.finucane@intel.com>
---
 patchwork/templates/patchwork/filters.html | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)
diff mbox

Patch

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) + ' &lt;' +
-                                 escape(item.email) + '&gt;' + '</div>';
+                if (item.name)
+                    return '<div>' + escape(item.name) + ' &lt;' +
+                                     escape(item.email) + '&gt;' + '</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) {