You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by at...@apache.org on 2019/01/31 13:05:48 UTC

[ambari] branch branch-2.7 updated: AMBARI-25102 Dasboard metrics will not load for ambari user which has dot in their username.

This is an automated email from the ASF dual-hosted git repository.

atkach pushed a commit to branch branch-2.7
in repository https://gitbox.apache.org/repos/asf/ambari.git


The following commit(s) were added to refs/heads/branch-2.7 by this push:
     new c89b1a9  AMBARI-25102 Dasboard metrics will not load for ambari user which has dot in their username.
c89b1a9 is described below

commit c89b1a9b9c573d601ffe93e0efc743e69ee4b5ff
Author: Andrii Tkach <at...@apache.org>
AuthorDate: Mon Jan 14 16:26:34 2019 +0200

    AMBARI-25102 Dasboard metrics will not load for ambari user which has dot in their username.
---
 ambari-web/app/utils/db.js | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/ambari-web/app/utils/db.js b/ambari-web/app/utils/db.js
index 2faffb4..78bc99a 100644
--- a/ambari-web/app/utils/db.js
+++ b/ambari-web/app/utils/db.js
@@ -139,6 +139,10 @@ App.db.get = function (namespace, key) {
   App.db.data = localStorage.getObject('ambari');
   Em.assert('`namespace` should be defined', !!namespace);
   checkNamespace(namespace);
+  if (key.contains('user-pref')) {
+    // username may contain "." which is the part of "user-pref-*" key so Em.set should be avoided
+    return Em.get(App.db.data, namespace)[key];
+  }
   return Em.get(Em.get(App.db.data, namespace), key);
 };
 
@@ -165,7 +169,12 @@ App.db.set = function (namespace, key, value) {
   App.db.data = localStorage.getObject('ambari');
   Em.assert('`namespace` should be defined', !!namespace);
   checkNamespace(namespace);
-  Em.set(Em.get(App.db.data, namespace), key, value);
+  if (key.contains('user-pref')) {
+    // username may contain "." which is the part of "user-pref-*" key so Em.set should be avoided
+    Em.get(App.db.data, namespace)[key] = value;
+  } else {
+    Em.set(Em.get(App.db.data, namespace), key, value);
+  }
   localStorage.setObject('ambari', App.db.data);
 };