fix(analytics): detect type of fields for XLSX export (#2729)

projectcaluma · Jul 4, 2024 · 6ea49aa · 6ea49aa
1 parent 86eb035
commit 6ea49aa
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 2 deletions.
diff --git a/packages/analytics/addon/components/ca-report-preview.hbs b/packages/analytics/addon/components/ca-report-preview.hbs
@@ -30,7 +30,7 @@
         {{#each this.data.value.fields as |row|}}
           <tr>
             {{#each row as |entry|}}
-              <td>
+              <td data-t={{this.getXLSXType entry.value}}>
                 {{entry.value}}
               </td>
             {{/each}}
@@ -39,7 +39,7 @@
         {{#if this.data.value.summary}}
           <tr class="uk-text-bold">
             {{#each this.data.value.summary as |summary|}}
-              <td>
+              <td data-t={{this.getXLSXType summary.value}}>
                 {{summary.value}}
               </td>
             {{/each}}

diff --git a/packages/analytics/addon/components/ca-report-preview.js b/packages/analytics/addon/components/ca-report-preview.js
@@ -74,4 +74,19 @@ export default class CaReportPreviewComponent extends Component {
       );
     });
   }
+
+  getXLSXType(input) {
+    // Check if it's a number
+    if (!isNaN(input) && input.trim() !== "") {
+      return "n";
+    }
+
+    // Check if it looks like a date and can be parsed as a date
+    if (/^\d{4}-\d{2}-\d{2}$/.test(input) && !isNaN(Date.parse(input))) {
+      return "d";
+    }
+
+    // Default to string
+    return "s";
+  }
 }