From: Kohei Yoshino <kohei.yoshino@gmail.com>
Date: Mon, 15 Jul 2019 22:31:43 +0000 (-0400)
Subject: Bug 1566079 - User autocomplete doesn’t work for requestee for certain flags
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=379f49b9987c79a07bc38d34f7f06ef9129b6e00;p=thirdparty%2Fbugzilla.git

Bug 1566079 - User autocomplete doesn’t work for requestee for certain flags
---

diff --git a/js/field.js b/js/field.js
index 639484ac1..5631e6cf9 100644
--- a/js/field.js
+++ b/js/field.js
@@ -738,28 +738,27 @@ $(function() {
         },
         formatResult: function(suggestion) {
             const $input = this;
-            const user = suggestion.data;
+            const { email, real_name, requests, gravatar } = suggestion.data;
             const request_type = $input.getAttribute('data-request-type');
-            const blocked = user.requests && request_type ? user.requests[request_type].blocked : false;
-            const pending = user.requests && request_type ? user.requests[request_type].pending : 0;
-            const image = user.gravatar ? `<img itemprop="image" alt="" src="${user.gravatar}">` : '';
+            const { blocked, pending } = requests ? (requests[request_type] || {}) : {};
+            const image = gravatar ? `<img itemprop="image" alt="" src="${gravatar}">` : '';
             const description = blocked ? '<span class="icon" aria-hidden="true"></span> Requests blocked' :
                 pending ? `${pending} pending ${request_type}${pending === 1 ? '' : 's'}` : '';
 
             return `<div itemscope itemtype="http://schema.org/Person">${image} ` +
-                `<span itemprop="name">${user.real_name.htmlEncode()}</span> ` +
-                `<span class="minor" itemprop="email">${user.email.htmlEncode()}</span> ` +
+                `<span itemprop="name">${real_name.htmlEncode()}</span> ` +
+                `<span class="minor" itemprop="email">${email.htmlEncode()}</span> ` +
                 `<span class="minor${blocked ? ' blocked' : ''}" itemprop="description">${description}</span></div>`;
         },
         onSelect: function (suggestion) {
             const $input = this;
-            const user = suggestion.data;
+            const { real_name, requests } = suggestion.data;
             const is_multiple = !!$input.getAttribute('data-multiple');
             const request_type = $input.getAttribute('data-request-type');
-            const blocked = user.requests && request_type ? user.requests[request_type].blocked : false;
+            const { blocked } = requests ? (requests[request_type] || {}) : {};
 
             if (blocked) {
-                window.alert(`${user.real_name} is not accepting ${request_type} requests at this time. ` +
+                window.alert(`${real_name} is not accepting ${request_type} requests at this time. ` +
                     'If you’re in a hurry, ask someone else for help.');
             } else if (is_multiple) {
                 const _values = $input.value.split(',').map(value => value.trim());