Make sure ecommerceOrder/ecommerceAbandonedCart isn't used as viewDat…

…aTable anywhere.
d19dotca · Dec 3, 2014 · a3c8a9e · a3c8a9e
1 parent 8a8a5e3
commit a3c8a9e
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 2 deletions.
diff --git a/plugins/CoreHome/javascripts/dataTable.js b/plugins/CoreHome/javascripts/dataTable.js
@@ -1897,7 +1897,12 @@ var switchToEcommerceView = function (dataTable, viewDataTable) {
         dataTable.param.abandonedCarts = '1';
     }
 
-    switchToHtmlTable(dataTable, dataTable.param.viewDataTable);
+    var viewDataTable = dataTable.param.viewDataTable;
+    if (viewDataTable == 'ecommerceOrder' || viewDataTable == 'ecommerceAbandonedCart') {
+        viewDataTable = 'table';
+    }
+
+    switchToHtmlTable(dataTable, viewDataTable);
 };
 
 DataTable.registerFooterIconHandler('table', switchToHtmlTable);

diff --git a/plugins/CoreHome/templates/_dataTableFooter.twig b/plugins/CoreHome/templates/_dataTableFooter.twig
@@ -33,7 +33,7 @@
                     {% for footerIcon in footerIconGroup.buttons %}
                         {% set isActiveEcommerceView = clientSideParameters.abandonedCarts is defined and
                                 ((footerIcon.id == 'ecommerceOrder' and clientSideParameters.abandonedCarts == 0) or
-                                (footerIcon.id == 'ecommerceAbandonedCart' and clientSideParameters.abandonedCarts == 1)) %}
+                                 (footerIcon.id == 'ecommerceAbandonedCart' and clientSideParameters.abandonedCarts == 1)) %}
                         <span>
                             {% if properties.show_active_view_icon and (clientSideParameters.viewDataTable == footerIcon.id or isActiveEcommerceView) %}
                                 <img src="plugins/Morpheus/images/data_table_footer_active_item.png" class="dataTableFooterActiveItem"/>

diff --git a/plugins/Goals/Reports/BaseEcommerceItem.php b/plugins/Goals/Reports/BaseEcommerceItem.php
@@ -105,6 +105,17 @@ public function configureView(ViewDataTable $view)
 
         $view->config->addTranslations($translations);
         $view->config->columns_to_display = $columnsOrdered;
+
+        // handle old case where viewDataTable is set to ecommerceOrder/ecommerceAbandonedCart. in this case, we
+        // set abandonedCarts accordingly and remove the ecommerceOrder/ecommerceAbandonedCart as viewDataTable.
+        $viewDataTable = Common::getRequestVar('viewDataTable', '');
+        if ($viewDataTable == 'ecommerceOrder') {
+            $view->config->custom_parameters['abandonedCarts'] = '0';
+            $view->config->custom_parameters['viewDataTable'] = 'table';
+        } else if ($viewDataTable == 'ecommerceAbandonedCart') {
+            $view->config->custom_parameters['abandonedCarts'] = '1';
+            $view->config->custom_parameters['viewDataTable'] = 'table';
+        }
     }
 
     private function isAbandonedCart()