fix: replace the old XLS generation with the new one

src/controllers/project.controller.js

@@ -179,10 +179,8 @@ export const findAll = async (req, res) => {
         createXlsFromSequelizeResults({
           rows: response,
           model: Project,
-          hex: false,
           toStructuredCsv: false,
           excludeOrgUid: true,
-          isUserFriendlyFormat: true,
         }),
         res,
       );

src/controllers/units.controller.js

@@ -188,10 +188,8 @@ export const findAll = async (req, res) => {
         createXlsFromSequelizeResults({
           rows: response,
           model: Unit,
-          hex: false,
           toStructuredCsv: false,
           excludeOrgUid: true,
-          isUserFriendlyFormat: true,
         }),
         res,
       );

src/models/projects/projects.model.js

@@ -293,28 +293,22 @@ class Project extends Model {
     const insertXslsSheets = createXlsFromSequelizeResults({
       rows: insertRecords,
       model: Project,
-      hex: false,
       toStructuredCsv: true,
       excludeOrgUid: false,
-      isUserFriendlyFormat: false,
     });
 
     const updateXslsSheets = createXlsFromSequelizeResults({
       rows: updateRecords,
       model: Project,
-      hex: false,
       toStructuredCsv: true,
       excludeOrgUid: false,
-      isUserFriendlyFormat: false,
     });
 
     const deleteXslsSheets = createXlsFromSequelizeResults({
       rows: deletedRecords,
       model: Project,
-      hex: false,
       toStructuredCsv: true,
       excludeOrgUid: false,
-      isUserFriendlyFormat: false,
     });
 
     const insertChangeList = await transformFullXslsToChangeList(

src/models/units/units.model.js

@@ -307,28 +307,22 @@ class Unit extends Model {
     const insertXslsSheets = createXlsFromSequelizeResults({
       rows: insertRecords,
       model: Unit,
-      hex: false,
       toStructuredCsv: true,
       excludeOrgUid: false,
-      isUserFriendlyFormat: false,
     });
 
     const updateXslsSheets = createXlsFromSequelizeResults({
       rows: updateRecords,
       model: Unit,
-      hex: false,
       toStructuredCsv: true,
       excludeOrgUid: false,
-      isUserFriendlyFormat: false,
     });
 
     const deleteXslsSheets = createXlsFromSequelizeResults({
       rows: deletedRecords,
       model: Unit,
-      hex: false,
       toStructuredCsv: true,
       excludeOrgUid: false,
-      isUserFriendlyFormat: false,
     });
 
     const insertChangeList = await transformFullXslsToChangeList(

src/utils/xls.js

@@ -54,12 +54,14 @@ export const encodeValue = (value, hex = false) => {
  * @param toStructuredCsv {Boolean} - Whether to generate an XLS/CSV
  * @param excludeOrgUid {Boolean} - Whether to exclude the organization id from the result
  */
-export function createXlsFromResults({
+export function createXlsFromSequelizeResults({
   rows,
   model,
   toStructuredCsv = false,
   excludeOrgUid = false,
 }) {
+  // TODO MariusD: Test with null values
+
   // Unsure if this is need, therefore just left the semi-deep-clone here. The assumption is that it wants to remove all functions from the prototypes.
   const rowsClone = JSON.parse(JSON.stringify(rows)); // Sadly this is the best way to simplify sequelize's return shape
 
@@ -285,24 +287,14 @@ function buildObjectXlsData({
   return aggregatedData;
 }
 
-export const createXlsFromSequelizeResults = ({
+export const createXlsFromSequelizeResults_old = ({
   rows,
   model,
   hex = false,
   toStructuredCsv = false,
   excludeOrgUid = false,
   isUserFriendlyFormat = true,
 }) => {
-  return createXlsFromResults({
-    rows,
-    model,
-    toStructuredCsv,
-    excludeOrgUid,
-  });
-  // TODO MariusD: Test with null values
-
-  /* eslint-disable no-unreachable */
-
   rows = JSON.parse(JSON.stringify(rows)); // Sadly this is the best way to simplify sequelize's return shape
 
   let columnsInResults = [];
@@ -466,8 +458,6 @@ export const createXlsFromSequelizeResults = ({
   } else {
     return xlsData;
   }
-
-  /* eslint-enable no-unreachable */
 };
 
 export const tableDataFromXlsx = (xlsx, model) => {