[checkmk-commits] 3551 FIX Now able to open BI aggregations containing a few hundred hosts

Andreas Boesl ab at mathias-kettner.de
Mon Jul 4 09:39:12 CEST 2016


Module: check_mk
Branch: master
Commit: 1ac72a3e2cf972ddb430caf66d622003decb7832
URL:    http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=1ac72a3e2cf972ddb430caf66d622003decb7832

Author: Andreas Boesl <ab at mathias-kettner.de>
Date:   Thu Jun 30 17:11:56 2016 +0200

3551 FIX Now able to open BI aggregations containing a few hundred hosts

A BI in the GUI could not get unfolded correctly if it had too much hosts.
This has been fixed.

---

 .werks/3551              |   10 ++++++++++
 ChangeLog                |    1 +
 web/htdocs/js/checkmk.js |   24 ++++++++++++++++++------
 3 files changed, 29 insertions(+), 6 deletions(-)

diff --git a/.werks/3551 b/.werks/3551
new file mode 100644
index 0000000..d47518f
--- /dev/null
+++ b/.werks/3551
@@ -0,0 +1,10 @@
+Title: Now able to open BI aggregations containing a few hundred hosts
+Level: 1
+Component: bi
+Compatible: compat
+Version: 1.4.0i1
+Date: 1467299366
+Class: fix
+
+A BI in the GUI could not get unfolded correctly if it had too much hosts.
+This has been fixed.
diff --git a/ChangeLog b/ChangeLog
index f4f4edf..7f59a79 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -493,6 +493,7 @@
     * 3439 FIX: Fixed processing of BI aggregations containing umlauts
     * 3440 FIX: Fixed possible BI exception "Unknown BI aggregation group ..."...
     * 3574 FIX: Fixed broken aggregations snapin when using aggregations based on hard states...
+    * 3551 FIX: Now able to open BI aggregations containing a few hundred hosts...
 
     Reporting & Availability:
     * 3267 New availability option for reclassifying host state "unreachable"...
diff --git a/web/htdocs/js/checkmk.js b/web/htdocs/js/checkmk.js
index c967dd8..55036ba 100644
--- a/web/htdocs/js/checkmk.js
+++ b/web/htdocs/js/checkmk.js
@@ -524,12 +524,24 @@ function get_url_sync(url)
     return AJAX.responseText;
 }
 
-function post_url(url, params)
+function post_url(url, post_params, responseHandler, handler_data, errorHandler)
 {
-    call_ajax(url, {
+    var args = {
         method: "POST",
-        post_data: params
-    });
+        post_data: post_params
+    }
+
+    if (typeof responseHandler !== 'undefined') {
+        args.response_handler = responseHandler;
+    }
+
+    if (typeof handler_data !== 'undefined')
+        args.handler_data = handler_data;
+
+    if (typeof errorHandler !== 'undefined')
+        args.error_handler = errorHandler;
+
+    call_ajax(url, args);
 }
 
 function bulkUpdateContents(ids, codes)
@@ -2958,8 +2970,8 @@ function bi_update_tree(oImg, code)
     while (oDiv.className != "bi_tree_container") {
         oDiv = oDiv.parentNode;
     }
-    var url = "bi_render_tree.py?" + oDiv.id;
-    get_url(url, bi_update_tree_response, oDiv);
+
+    post_url("bi_render_tree.py", oDiv.id, bi_update_tree_response, oDiv);
 }
 
 function bi_update_tree_response(oDiv, code) {



More information about the checkmk-commits mailing list