From 43198a2a0c38b15e834c1685cd75a6162a79d725 Mon Sep 17 00:00:00 2001
From: Matthieu Napoli <matthieu@mnapoli.fr>
Date: Wed, 11 Feb 2015 15:27:22 +1300
Subject: [PATCH] Fixes #7168 include_aggregate_rows was still enabled in
 exports when disabled previously

"0" (the string) was casted to a boolean in the `if` which lead to it being `true`
---
 plugins/CoreHome/javascripts/dataTable.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/plugins/CoreHome/javascripts/dataTable.js b/plugins/CoreHome/javascripts/dataTable.js
index 2ce28c7ebf..6746663b7e 100644
--- a/plugins/CoreHome/javascripts/dataTable.js
+++ b/plugins/CoreHome/javascripts/dataTable.js
@@ -1098,7 +1098,7 @@ $.extend(DataTable.prototype, UIControl.prototype, {
 
                 if (typeof self.param.flat != "undefined") {
                     str += '&flat=' + (self.param.flat == 0 ? '0' : '1');
-                    if (typeof self.param.include_aggregate_rows != "undefined" && self.param.include_aggregate_rows) {
+                    if (typeof self.param.include_aggregate_rows != "undefined" && self.param.include_aggregate_rows == '1') {
                         str += '&include_aggregate_rows=1';
                     }
                     if (!self.param.flat
-- 
GitLab