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

nixon pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/atlas.git

commit 1ccb7c715cb0ff0f60f27d21c77a810e6640e01c
Author: kevalbhatt <kbh...@apache.org>
AuthorDate: Thu Mar 12 21:50:36 2020 +0530

    ATLAS-3598 : Beta UI: Issues with single click functionality in left hand 
panel.
    
    Signed-off-by: nixonrodrigues <ni...@apache.org>
---
 .../js/views/search/SearchDefaultLayoutView.js     |  6 +--
 .../search/tree/ClassificationTreeLayoutView.js    | 46 ++++++++++++----------
 .../js/views/search/tree/EntityTreeLayoutView.js   | 38 ++++++++++--------
 3 files changed, 49 insertions(+), 41 deletions(-)

diff --git a/dashboardv3/public/js/views/search/SearchDefaultLayoutView.js 
b/dashboardv3/public/js/views/search/SearchDefaultLayoutView.js
index 3c0d813..d0946c4 100644
--- a/dashboardv3/public/js/views/search/SearchDefaultLayoutView.js
+++ b/dashboardv3/public/js/views/search/SearchDefaultLayoutView.js
@@ -214,7 +214,8 @@ define(["require", "backbone", "utils/Globals", 
"hbs!tmpl/search/SearchDefaultLa
             renderGlobalSearch: function() {
                 var that = this;
                 require(["views/search/GlobalSearchLayoutView"], 
function(GlobalSearchLayoutView) {
-                    that.RGlobalSearchLayoutView.show(new 
GlobalSearchLayoutView(_.extend({ closeOnSubmit: true }, that.options)));
+
+                    that.RGlobalSearchLayoutView.show(new 
GlobalSearchLayoutView(_.extend({ closeOnSubmit: true }, _.omit(that.options, 
"value"))));
                 });
             },
             renderSearchResult: function() {
@@ -303,8 +304,7 @@ define(["require", "backbone", "utils/Globals", 
"hbs!tmpl/search/SearchDefaultLa
                         this.renderQueryBuilder(_.extend({}, obj, {
                             tag: false,
                             type: true,
-                            attrObj: attrTypeObj,
-                            applicableType: that.options.value ? 
that.options.value.type : null
+                            attrObj: attrTypeObj
                         }), this.RQueryBuilderEntity);
 
                         this.ui.entityName.html(that.options.value.type);
diff --git 
a/dashboardv3/public/js/views/search/tree/ClassificationTreeLayoutView.js 
b/dashboardv3/public/js/views/search/tree/ClassificationTreeLayoutView.js
index dd8161e..87f0b9a 100644
--- a/dashboardv3/public/js/views/search/tree/ClassificationTreeLayoutView.js
+++ b/dashboardv3/public/js/views/search/tree/ClassificationTreeLayoutView.js
@@ -258,28 +258,32 @@ define([
                 this.ui.classificationSearchTree.jstree(true).deselect_all();
                 this.tagId = null;
             } else {
-                if ((that.options.value.tag.indexOf('*') != -1)) {
-                    
that.ui.classificationSearchTree.jstree(true).deselect_all();
-                    that.ui.wildCardValue.val(that.options.value.tag);
-                }
-                var dataFound = 
this.classificationDefCollection.fullCollection.find(function(obj) {
-                    return obj.get("name") === that.options.value.tag
-                });
-                if (dataFound) {
-                    if ((this.tagId && this.tagId !== dataFound.get("guid")) 
|| this.tagId === null) {
-                        if (this.tagId) {
-                            
this.ui.classificationSearchTree.jstree(true).deselect_node(this.tagId);
+                if (that.options.value.tag === "_ALL_CLASSIFICATION_TYPES" && 
this.tagId !== "_ALL_CLASSIFICATION_TYPES") {
+                    this.fromManualRender = true;
+                    if (this.tagId) {
+                        
this.ui.classificationSearchTree.jstree(true).deselect_node(this.tagId);
+                    }
+                    this.tagId = Globals[that.options.value.tag].guid;
+                    
this.ui.classificationSearchTree.jstree(true).select_node(this.tagId);
+                } else if (this.tagId !== "_ALL_CLASSIFICATION_TYPES" && 
that.options.value.tag !== this.tagId) {
+                    if ((that.options.value.tag.indexOf('*') != -1)) {
+                        
that.ui.classificationSearchTree.jstree(true).deselect_all();
+                        that.ui.wildCardValue.val(that.options.value.tag);
+                    }
+                    var dataFound = 
this.classificationDefCollection.fullCollection.find(function(obj) {
+                        return obj.get("name") === that.options.value.tag
+                    });
+                    if (dataFound) {
+                        if ((this.tagId && this.tagId !== 
dataFound.get("guid")) || this.tagId === null) {
+                            if (this.tagId) {
+                                
this.ui.classificationSearchTree.jstree(true).deselect_node(this.tagId);
+                            }
+                            this.fromManualRender = true;
+                            this.tagId = dataFound.get("guid");
+                            
this.ui.classificationSearchTree.jstree(true).select_node(dataFound.get("guid"));
                         }
-                        this.fromManualRender = true;
-                        this.tagId = dataFound.get("guid");
-                        
this.ui.classificationSearchTree.jstree(true).select_node(dataFound.get("guid"));
                     }
                 }
-                if (!dataFound && Globals[that.options.value.tag]) {
-                    this.fromManualRender = true;
-                    this.typeId = Globals[that.options.value.tag].guid;
-                    
this.ui.classificationSearchTree.jstree(true).select_node(this.typeId);
-                }
             }
         },
         onNodeSelect: function(options) {
@@ -516,8 +520,8 @@ define([
                 name: rootClassification.name,
                 type: rootClassification.category,
                 gType: "Classification",
-                guid: "root-classification",
-                id: "root-classification",
+                guid: rootClassification.guid,
+                id: rootClassification.guid,
                 model: rootClassification,
                 children: [],
                 icon: "fa fa-tag",
diff --git a/dashboardv3/public/js/views/search/tree/EntityTreeLayoutView.js 
b/dashboardv3/public/js/views/search/tree/EntityTreeLayoutView.js
index 6df64ad..5b96e8f 100644
--- a/dashboardv3/public/js/views/search/tree/EntityTreeLayoutView.js
+++ b/dashboardv3/public/js/views/search/tree/EntityTreeLayoutView.js
@@ -171,23 +171,27 @@ define([
                 this.ui.entitySearchTree.jstree(true).deselect_all();
                 this.typeId = null;
             } else {
-                var dataFound = 
this.typeHeaders.fullCollection.find(function(obj) {
-                    return obj.get("name") === that.options.value.type
-                });
-                if (dataFound) {
-                    if ((this.typeId && this.typeId !== dataFound.get("guid")) 
|| this.typeId === null) {
-                        if (this.typeId) {
-                            
this.ui.entitySearchTree.jstree(true).deselect_node(this.typeId);
-                        }
-                        this.fromManualRender = true;
-                        this.typeId = dataFound.get("guid");
-                        
this.ui.entitySearchTree.jstree(true).select_node(dataFound.get("guid"));
-                    }
-                }
-                if (!dataFound && Globals[that.options.value.type]) {
+                if (that.options.value.type === "_ALL_ENTITY_TYPES" && 
this.typeId !== "_ALL_ENTITY_TYPES") {
                     this.fromManualRender = true;
+                    if (this.typeId) {
+                        
this.ui.entitySearchTree.jstree(true).deselect_node(this.typeId);
+                    }
                     this.typeId = Globals[that.options.value.type].guid;
                     
this.ui.entitySearchTree.jstree(true).select_node(this.typeId);
+                } else if (this.typeId !== "_ALL_ENTITY_TYPES" && 
that.options.value.type !== this.typeId) {
+                    var dataFound = 
this.typeHeaders.fullCollection.find(function(obj) {
+                        return obj.get("name") === that.options.value.type
+                    });
+                    if (dataFound) {
+                        if ((this.typeId && this.typeId !== 
dataFound.get("guid")) || this.typeId === null) {
+                            if (this.typeId) {
+                                
this.ui.entitySearchTree.jstree(true).deselect_node(this.typeId);
+                            }
+                            this.fromManualRender = true;
+                            this.typeId = dataFound.get("guid");
+                            
this.ui.entitySearchTree.jstree(true).select_node(dataFound.get("guid"));
+                        }
+                    }
                 }
             }
         },
@@ -301,7 +305,7 @@ define([
                                 text: _.escape(modelname),
                                 name: model.get("name"),
                                 type: model.get("category"),
-                                gType: "serviceType",
+                                gType: "Entity",
                                 guid: model.get("guid"),
                                 id: model.get("guid"),
                                 model: model,
@@ -336,7 +340,7 @@ define([
                         text: _.escape(rootEntity.name),
                         name: rootEntity.name,
                         type: rootEntity.category,
-                        gType: "serviceType",
+                        gType: "Entity",
                         guid: rootEntity.guid,
                         id: rootEntity.guid,
                         model: rootEntity,
@@ -389,7 +393,7 @@ define([
                             parent = {
                                 icon: "fa fa-folder-o",
                                 type: type,
-                                gType: "serviceType",
+                                gType: "ServiceType",
                                 children: getParrent.children,
                                 text: _.escape(textName),
                                 name: data[parents[i]].name,

Reply via email to