# HG changeset patch
# User Mads Kiilerich <mads@kiilerich.com>
# Date 1505437590 -7200
# Node ID 44ed1c6497ce6d9f847ba3754689b238149a6147
# Parent  c47ab93fdc72656120bb7f664a7c05a3754c3f59
autocomplete: simplify code

Avoid unnecessary temporary variables and checking.

diff -r c47ab93fdc72 -r 44ed1c6497ce kallithea/public/js/base.js
--- a/kallithea/public/js/base.js	Fri Sep 15 03:06:30 2017 +0200
+++ b/kallithea/public/js/base.js	Fri Sep 15 03:06:30 2017 +0200
@@ -1105,36 +1105,30 @@
         query = sResultMatch.term.toLowerCase();
 
     // group
-    if (oResultData.grname != undefined) {
-        var grname = oResultData.grname;
-        var grmembers = oResultData.grmembers;
-        var grprefix = "{0}: ".format(_TM['Group']);
-        var grsuffix = " ({0} {1})".format(grmembers, _TM['members']);
-
-        return autocompleteGravatar(grprefix + autocompleteHighlightMatch(grname, query) + grsuffix, null, null, true);
+    if (oResultData.grname) {
+        return autocompleteGravatar(
+            "{0}: {1} ({2} {3})".format(
+                _TM['Group'],
+                autocompleteHighlightMatch(oResultData.grname, query),
+                oResultData.grmembers,
+                _TM['members']),
+            null, null, true);
+    }
 
     // users
-    } else if (oResultData.nname != undefined) {
-        var fname = oResultData.fname || "";
-        var lname = oResultData.lname || "";
-        var nname = oResultData.nname;
-
-        // Guard against null value
-        var displayfname = autocompleteHighlightMatch(fname, query);
-
-        var displaylname = autocompleteHighlightMatch(lname, query);
-
-        var displaynname = autocompleteHighlightMatch(nname, query);
-
-        var displayname = displaynname;
-        if (displayfname && displaylname) {
-            displayname = "{0} {1} ({2})".format(displayfname, displaylname, displayname);
+    if (oResultData.nname) {
+        var displayname = autocompleteHighlightMatch(oResultData.nname, query);
+        if (oResultData.fname && oResultData.lname) {
+            displayname = "{0} {1} ({2})".format(
+                autocompleteHighlightMatch(oResultData.fname, query),
+                autocompleteHighlightMatch(oResultData.lname, query),
+                displayname);
         }
 
         return autocompleteGravatar(displayname, oResultData.gravatar_lnk, oResultData.gravatar_size);
-    } else {
-        return '';
     }
+
+    return '';
 };
 
 // Generate a basic autocomplete instance that can be tweaked further by the caller