Fix simcore.s3 tree loading issue (#947)

Show "Loading" spinner also of simcore.s3

Author: odeimaiz

services/web/client/source/class/qxapp/component/widget/logger/LoggerView.js

@@ -78,7 +78,7 @@ qx.Class.define("qxapp.component.widget.logger.LoggerView", {
 
     this.__createInitMsg();
 
-    this.__textfield.addListener("changeValue", this.__applyFilters, this);
+    this.__textFilterField.addListener("changeValue", this.__applyFilters, this);
   },
 
   events: {},
@@ -137,7 +137,7 @@ qx.Class.define("qxapp.component.widget.logger.LoggerView", {
 
   members: {
     __currentNodeButton: null,
-    __textfield: null,
+    __textFilterField: null,
     __logModel: null,
     __logView: null,
     __messengerColors: null,
@@ -155,12 +155,12 @@ qx.Class.define("qxapp.component.widget.logger.LoggerView", {
       toolbar.add(currentNodeButton);
 
       toolbar.add(new qx.ui.toolbar.Separator());
-      this.__textfield = new qx.ui.form.TextField().set({
+      const textFilterField = this.__textFilterField = new qx.ui.form.TextField().set({
         appearance: "toolbar-textfield",
         liveUpdate: true,
         placeholder: this.tr("Filter")
       });
-      toolbar.add(this.__textfield, {
+      toolbar.add(textFilterField, {
         flex: 1
       });
 
@@ -239,9 +239,9 @@ qx.Class.define("qxapp.component.widget.logger.LoggerView", {
       const workbench = this.getWorkbench();
       const node = workbench.getNode(nodeId);
       if (node) {
-        this.__textfield.setValue(node.getLabel());
+        this.__textFilterField.setValue(node.getLabel());
       } else {
-        this.__textfield.setValue("");
+        this.__textFilterField.setValue("");
       }
     },
 
@@ -321,7 +321,7 @@ qx.Class.define("qxapp.component.widget.logger.LoggerView", {
         return;
       }
 
-      this.__logModel.setFilterString(this.__textfield.getValue());
+      this.__logModel.setFilterString(this.__textFilterField.getValue());
       this.__logModel.setFilterLogLevel(this.getLogLevel());
       this.__logModel.reloadData();
     },

services/web/client/source/class/qxapp/desktop/StudyEditor.js

@@ -89,15 +89,15 @@ qx.Class.define("qxapp.desktop.StudyEditor", {
     initDefault: function() {
       const study = this.getStudy();
 
-      const treeView = this.__nodesTree = new qxapp.component.widget.NodesTree(study.getName(), study.getWorkbench());
-      treeView.addListener("addNode", () => {
+      const nodesTree = this.__nodesTree = new qxapp.component.widget.NodesTree(study.getName(), study.getWorkbench());
+      nodesTree.addListener("addNode", () => {
         this.__addNode();
       }, this);
-      treeView.addListener("removeNode", e => {
+      nodesTree.addListener("removeNode", e => {
         const nodeId = e.getData();
         this.__removeNode(nodeId);
       }, this);
-      this.__sidePanel.addOrReplaceAt(new qxapp.desktop.PanelView(this.tr("Service tree"), treeView), 0);
+      this.__sidePanel.addOrReplaceAt(new qxapp.desktop.PanelView(this.tr("Service tree"), nodesTree), 0);
 
       const extraView = this.__extraView = new qxapp.component.metadata.StudyInfo(study);
       this.__sidePanel.addOrReplaceAt(new qxapp.desktop.PanelView(this.tr("Study information"), extraView), 1);
@@ -176,15 +176,15 @@ qx.Class.define("qxapp.desktop.StudyEditor", {
       });
 
       const workbenchUI = this.__workbenchUI;
-      const treeView = this.__nodesTree;
-      treeView.addListener("changeSelectedNode", e => {
+      const nodesTree = this.__nodesTree;
+      nodesTree.addListener("changeSelectedNode", e => {
         const node = workbenchUI.getNodeUI(e.getData());
         if (node && node.classname.includes("NodeUI")) {
           node.setActive(true);
         }
       });
       workbenchUI.addListener("changeSelectedNode", e => {
-        treeView.nodeSelected(e.getData());
+        nodesTree.nodeSelected(e.getData());
       });
     },
 
@@ -233,7 +233,7 @@ qx.Class.define("qxapp.desktop.StudyEditor", {
         }
       }
 
-      this.__treeView.nodeSelected(nodeId, openNodeAndParents);
+      this.__nodesTree.nodeSelected(nodeId, openNodeAndParents);
       this.__loggerView.setCurrentNodeId(nodeId);
     },
 

services/web/client/source/class/qxapp/file/FilesTreePopulator.js

@@ -96,7 +96,7 @@ qx.Class.define("qxapp.file.FilesTreePopulator", {
     },
 
     populateMyLocation: function(locationId = null) {
-      if (locationId) {
+      if (locationId !== null) {
         const locationModel = this.__getLocationModel(locationId);
         if (locationModel) {
           locationModel.getChildren().removeAll();