drillthrough correct handling fix

Author: nikitaeverywhere

source/js/DataController.js

@@ -80,6 +80,8 @@ DataController.prototype.getData = function () {
 
 DataController.prototype.setData = function (data) {
 
+    var _ = this;
+
     if (!this.isValidData(data)) {
         console.error("Invalid data to set.", data);
         return;
@@ -91,6 +93,13 @@ DataController.prototype.setData = function (data) {
     this.resetConditionalFormatting();
     this.resetRawData();
 
+    if (data.info.mdxType === "drillthrough") {
+        this.setDrillThroughHandler(function (params) {
+            _.controller.pivotView.displayMessage(params["cellData"]["value"] || "", true);
+            return false;
+        });
+    }
+
     this._trigger();
     return data;
 

source/js/DataSource.js

@@ -161,7 +161,7 @@ DataSource.prototype.getCurrentData = function (callback) {
 
         var data = ready.data;
 
-        console.log("Retrieved data:", ready);
+        //console.log("Retrieved data:", ready);
 
         if (mdxType === "drillthrough") {
             callback((function (data) {
@@ -187,7 +187,8 @@ DataSource.prototype.getCurrentData = function (callback) {
                         cubeName: "No cube name",
                         leftHeaderColumnsNumber: 0,
                         rowCount: arr.length,
-                        topHeaderRowsNumber: headers.length
+                        topHeaderRowsNumber: headers.length,
+                        mdxType: mdxType
                     }
                 };
 

source/js/LightPivotTable.js

@@ -210,10 +210,6 @@ LightPivotTable.prototype.tryDrillThrough = function (filters) {
             });
             _.dataController.pushData();
             _.dataController.setData(data);
-            _.dataController.setDrillThroughHandler(function (params) {
-                _.pivotView.displayMessage(params["cellData"]["value"] || "", true);
-                return false;
-            });
             if (typeof _.CONFIG.triggers["drillThrough"] === "function") {
                 _.CONFIG.triggers["drillThrough"].call(_, {
                     level: _.DRILL_LEVEL,