commit 75bd7fcc91d60976d014902b7fe4063216c4164b
author Sven Selberg <svense@axis.com> Wed Aug 28 15:07:30 2019 +0200
committer David Pursehouse <dpursehouse@collab.net> Mon Sep 02 10:11:20 2019 +0900
tree 4795e9df27fe31ba7a62d84d5464b241c32904a1
parent 2c1ae694fafcd22cc0c594ad1f03555df4b13da3

Allow to display change-list for inactive users

Accounts for inactive users can only be resolved by account id (and
'self' but 'self' is not useful in this context) by the back-end.
Use account id when available to resolve accounts.

Bug: Issue 11367
Change-Id: I99d4acb9e0a7708912b718aa1104f98af82fb608

polygerrit-ui/app/elements/change-list/gr-change-list-view/gr-change-list-view.js

diff --git a/polygerrit-ui/app/elements/change-list/gr-change-list-view/gr-change-list-view.js b/polygerrit-ui/app/elements/change-list/gr-change-list-view/gr-change-list-view.js
index d546fe3..e525600 100644
--- a/polygerrit-ui/app/elements/change-list/gr-change-list-view/gr-change-list-view.js
+++ b/polygerrit-ui/app/elements/change-list/gr-change-list-view/gr-change-list-view.js
@@ -244,9 +244,13 @@
       if (!changes || !changes.length) {
         return;
       }
-      if (USER_QUERY_PATTERN.test(this._query) && changes[0].owner.email) {
-        this._userId = changes[0].owner.email;
-        return;
+      if (USER_QUERY_PATTERN.test(this._query)) {
+        const owner = changes[0].owner;
+        const userId = owner._account_id ? owner._account_id : owner.email;
+        if (userId) {
+          this._userId = userId;
+          return;
+        }
       }
       if (REPO_QUERY_PATTERN.test(this._query)) {
         this._repo = changes[0].project;