this is epic... by rendering the reports this way we can remove a lot of duplicated, unneeded code while having backwards compatible urls, developers are still able to completely overwrite the reports controller action, we fix some issues where CoreHome.renderWidget was reported in dispatch events instead of the actual module/action -> fixes also some security issues and most saved params for reports work again. Hope it works as expected

1 files changed, 1 insertions, 2 deletions

diff --git a/core/ViewDataTable/Factory.php b/core/ViewDataTable/Factory.php
index aa42c9de68..afd093c4cb 100644
--- a/core/ViewDataTable/Factory.php
+++ b/core/ViewDataTable/Factory.php
@@ -107,8 +107,7 @@ class Factory
             $params = array();
         } else {
             $login  = Piwik::getCurrentUserLogin();
-            // TODO this won't work as currently all reports use CoreHome.renderMenuReport
-            $params = Manager::getViewDataTableParameters($login, $controllerAction);
+            $params = Manager::getViewDataTableParameters($login, $apiAction);
         }
 
         $savedViewDataTable = false;