report copy is not taking over all properties

Rello · Jan 6, 2025 · 44f4bd8 · 44f4bd8
1 parent 69ddea1
commit 44f4bd8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,7 @@
 
 ### Fixed
 - calculated columns: correct percentage formula in total row
+- report copy is not taking over all properties
 
 ## 5.2.2 - 2025-01-05
 ### Added

diff --git a/js/filter.js b/js/filter.js
@@ -768,8 +768,6 @@ OCA.Analytics.Filter.Backend = {
 
         if (typeof (OCA.Analytics.currentReportData.options.filteroptions) === 'undefined') {
             OCA.Analytics.currentReportData.options.filteroptions = {};
-        } else {
-            OCA.Analytics.currentReportData.options.filteroptions = JSON.stringify(OCA.Analytics.currentReportData.options.filteroptions);
         }
 
         OCA.Analytics.unsavedFilters = false;
@@ -786,9 +784,9 @@ OCA.Analytics.Filter.Backend = {
             body: JSON.stringify({
                 reportId: reportId,
                 chartoptions: JSON.stringify(OCA.Analytics.currentReportData.options.chartoptions),
-                dataoptions: OCA.Analytics.currentReportData.options.dataoptions,
-                filteroptions: OCA.Analytics.currentReportData.options.filteroptions,
-                tableoptions: OCA.Analytics.currentReportData.options.tableoptions,
+                dataoptions: JSON.stringify(OCA.Analytics.currentReportData.options.dataoptions),
+                filteroptions: JSON.stringify(OCA.Analytics.currentReportData.options.filteroptions),
+                tableoptions: JSON.stringify(OCA.Analytics.currentReportData.options.tableoptions),
             })
         })
             .then(response => response.json())