Fix functionality "open all" / "close all" GUI buttons

Author: linev

changes.md

@@ -4,6 +4,7 @@
 1. Fix zooming in color palette
 2. Fix interactive update of TGraph painting on time scale
 3. Fix I/O error in reading std::map (#204)
+4. Fix functionality "open all" / "close all" GUI buttons
 
 
 ## Changes in 5.9.0

scripts/JSRootCore.js

@@ -95,7 +95,7 @@
 
    "use strict";
 
-   JSROOT.version = "5.9.x 9/10/2020";
+   JSROOT.version = "5.9.x 27/11/2020";
 
    JSROOT.source_dir = "";
    JSROOT.source_min = false;

scripts/JSRootPainter.hierarchy.js

@@ -1456,6 +1456,13 @@
       find_next();
    }
 
+   HierarchyPainter.prototype.canExpand = function(item) {
+      if (!item) return false;
+      if (item._expand) return true;
+      var handle = JSROOT.getDrawHandle(item._kind, "::expand");
+      return handle && (handle.expand_item || handle.expand);
+   }
+
    HierarchyPainter.prototype.expand = function(itemname, call_back, d3cont, silent) {
       var hpainter = this, hitem = this.Find(itemname);
 

scripts/JSRootPainter.jquery.js

@@ -743,7 +743,11 @@
       var hitem = h ? h : this.h;
 
       if (!('_childs' in hitem)) {
-         if (!isopen || this.with_icons || (!hitem._expand && (hitem._more !== true))) return false;
+         if (!isopen) return false;
+         if (this.with_icons) {
+            // in normal hierarchy check precisely if item can be expand
+            if (!hitem._more && !hitem._expand && !this.canExpand(hitem)) return false;
+         }
          this.expand(this.itemFullName(hitem));
          if (hitem._childs) hitem._isopen = true;
          return true;