_config.yml

@@ -1,2 +1,2 @@
 theme: jekyll-theme-cayman
-title: Sunrise Cemetery Management System (CMS)
+title: Sunrise Cemetery Management System

bin/www.js

@@ -12,7 +12,7 @@ import { getConfigProperty } from '../helpers/config.helpers.js';
 import version from '../version.js';
 const debug = Debug(`${DEBUG_NAMESPACE}:www:${process.pid}`);
 // INITIALIZE THE DATABASE
-initializeDatabase();
+await initializeDatabase();
 const directoryName = path.dirname(fileURLToPath(import.meta.url));
 const processCount = Math.min(getConfigProperty('application.maximumProcesses'), os.cpus().length * 2);
 const applicationName = getConfigProperty('application.applicationName');

bin/www.ts

@@ -17,7 +17,7 @@ import version from '../version.js'
 const debug = Debug(`${DEBUG_NAMESPACE}:www:${process.pid}`)
 
 // INITIALIZE THE DATABASE
-initializeDatabase()
+await initializeDatabase()
 
 const directoryName = path.dirname(fileURLToPath(import.meta.url))
 

bin/wwwProcess.js

@@ -10,7 +10,7 @@ import { getConfigProperty } from '../helpers/config.helpers.js';
 const debug = Debug(`${DEBUG_NAMESPACE}:wwwProcess:${process.pid.toString().padEnd(5)}`);
 if (process.send === undefined) {
     // INITIALIZE THE DATABASE
-    initializeDatabase();
+    await initializeDatabase();
 }
 function onError(error) {
     if (error.syscall !== 'listen') {

bin/wwwProcess.ts

@@ -15,7 +15,7 @@ const debug = Debug(`${DEBUG_NAMESPACE}:wwwProcess:${process.pid.toString().padE
 
 if (process.send === undefined) {
   // INITIALIZE THE DATABASE
-  initializeDatabase()
+  await initializeDatabase()
 }
 
 interface ServerError extends Error {

data/base.config.js

@@ -6,7 +6,6 @@ export const config = {
     settings: {
         adminCleanup: {},
         burialSites: {},
-        burialSiteTypes: {},
         cemeteries: {},
         contracts: {},
         dynamicsGP: {

data/base.config.ts

@@ -8,7 +8,6 @@ export const config: Config = {
   settings: {
     adminCleanup: {},
     burialSites: {},
-    burialSiteTypes: {},
     cemeteries: {},
     contracts: {},
     dynamicsGP: {

data/configDefaults.d.ts

@@ -33,8 +33,6 @@ export declare const configDefaultValues: {
     'settings.longitudeMax': number;
     'settings.longitudeMin': number;
     'settings.cemeteries.refreshImageChanges': boolean;
-    'settings.burialSiteTypes.bodyCapacityMaxDefault': number;
-    'settings.burialSiteTypes.crematedCapacityMaxDefault': number;
     'settings.burialSites.burialSiteNameSegments': ConfigBurialSiteNameSegments;
     'settings.burialSites.burialSiteNameSegments.includeCemeteryKey': boolean;
     'settings.burialSites.refreshImageChanges': boolean;

data/configDefaults.js

@@ -32,10 +32,6 @@ export const configDefaultValues = {
     'settings.longitudeMax': 180,
     'settings.longitudeMin': -180,
     'settings.cemeteries.refreshImageChanges': false,
-    // eslint-disable-next-line no-secrets/no-secrets
-    'settings.burialSiteTypes.bodyCapacityMaxDefault': 2,
-    // eslint-disable-next-line no-secrets/no-secrets
-    'settings.burialSiteTypes.crematedCapacityMaxDefault': 6,
     'settings.burialSites.burialSiteNameSegments': {
         includeCemeteryKey: false,
         separator: '-',

data/configDefaults.ts

@@ -1,6 +1,7 @@
-import { hoursToMillis } from '@cityssm/to-millis'
 import type { config as MSSQLConfig } from 'mssql'
 
+import { hoursToMillis } from '@cityssm/to-millis'
+
 import type {
   ConfigActiveDirectory,
   ConfigBurialSiteNameSegments,
@@ -51,12 +52,6 @@ export const configDefaultValues = {
 
   'settings.cemeteries.refreshImageChanges': false,
 
-  // eslint-disable-next-line no-secrets/no-secrets
-  'settings.burialSiteTypes.bodyCapacityMaxDefault': 2,
-  
-  // eslint-disable-next-line no-secrets/no-secrets
-  'settings.burialSiteTypes.crematedCapacityMaxDefault': 6,
-
   'settings.burialSites.burialSiteNameSegments': {
     includeCemeteryKey: false,
     separator: '-',

database/addBurialSite.d.ts

@@ -6,8 +6,6 @@ export interface AddBurialSiteForm {
     burialSiteNameSegment5?: string;
     burialSiteStatusId: number | string;
     burialSiteTypeId: number | string;
-    bodyCapacity?: number | string;
-    crematedCapacity?: number | string;
     burialSiteImage?: string;
     cemeteryId: number | string;
     cemeterySvgId?: string;

database/addBurialSite.js

@@ -10,7 +10,6 @@ import getCemetery from './getCemetery.js';
  * @returns The new burial site's id.
  * @throws If an active burial site with the same name already exists.
  */
-// eslint-disable-next-line complexity
 export default function addBurialSite(burialSiteForm, user) {
     const database = sqlite(sunriseDB);
     const rightNowMillis = Date.now();
@@ -39,22 +38,17 @@ export default function addBurialSite(burialSiteForm, user) {
         burialSiteNameSegment5,
         burialSiteName,
         burialSiteTypeId, burialSiteStatusId,
-        bodyCapacity, crematedCapacity,
         cemeteryId, cemeterySvgId, burialSiteImage,
         burialSiteLatitude, burialSiteLongitude,
 
         recordCreate_userName, recordCreate_timeMillis,
         recordUpdate_userName, recordUpdate_timeMillis) 
-        values (?, ?, ?, ?, ?,
+        values (?, ?, ?,
           ?, ?, ?, ?, ?, ?, ?, ?, ?, ?,
           ?, ?, ?, ?)`)
         .run(burialSiteForm.burialSiteNameSegment1 ?? '', burialSiteForm.burialSiteNameSegment2 ?? '', burialSiteForm.burialSiteNameSegment3 ?? '', burialSiteForm.burialSiteNameSegment4 ?? '', burialSiteForm.burialSiteNameSegment5 ?? '', burialSiteName, burialSiteForm.burialSiteTypeId, burialSiteForm.burialSiteStatusId === ''
         ? undefined
-        : burialSiteForm.burialSiteStatusId, burialSiteForm.bodyCapacity === ''
+        : burialSiteForm.burialSiteStatusId, burialSiteForm.cemeteryId === '' ? undefined : burialSiteForm.cemeteryId, burialSiteForm.cemeterySvgId, burialSiteForm.burialSiteImage ?? '', burialSiteForm.burialSiteLatitude === ''
-        ? undefined
-        : burialSiteForm.bodyCapacity, burialSiteForm.crematedCapacity === ''
-        ? undefined
-        : burialSiteForm.crematedCapacity, burialSiteForm.cemeteryId === '' ? undefined : burialSiteForm.cemeteryId, burialSiteForm.cemeterySvgId, burialSiteForm.burialSiteImage ?? '', burialSiteForm.burialSiteLatitude === ''
         ? undefined
         : burialSiteForm.burialSiteLatitude, burialSiteForm.burialSiteLongitude === ''
         ? undefined

database/addBurialSite.ts

@@ -16,9 +16,6 @@ export interface AddBurialSiteForm {
   burialSiteStatusId: number | string
   burialSiteTypeId: number | string
 
-  bodyCapacity?: number | string
-  crematedCapacity?: number | string
-
   burialSiteImage?: string
   cemeteryId: number | string
   cemeterySvgId?: string
@@ -38,7 +35,6 @@ export interface AddBurialSiteForm {
  * @returns The new burial site's id.
  * @throws If an active burial site with the same name already exists.
  */
-// eslint-disable-next-line complexity
 export default function addBurialSite(
   burialSiteForm: AddBurialSiteForm,
   user: User
@@ -84,13 +80,12 @@ export default function addBurialSite(
         burialSiteNameSegment5,
         burialSiteName,
         burialSiteTypeId, burialSiteStatusId,
-        bodyCapacity, crematedCapacity,
         cemeteryId, cemeterySvgId, burialSiteImage,
         burialSiteLatitude, burialSiteLongitude,
 
         recordCreate_userName, recordCreate_timeMillis,
         recordUpdate_userName, recordUpdate_timeMillis) 
-        values (?, ?, ?, ?, ?,
+        values (?, ?, ?,
           ?, ?, ?, ?, ?, ?, ?, ?, ?, ?,
           ?, ?, ?, ?)`
     )
@@ -105,15 +100,6 @@ export default function addBurialSite(
       burialSiteForm.burialSiteStatusId === ''
         ? undefined
         : burialSiteForm.burialSiteStatusId,
-
-      burialSiteForm.bodyCapacity === ''
-        ? undefined
-        : burialSiteForm.bodyCapacity,
-
-      burialSiteForm.crematedCapacity === ''
-        ? undefined
-        : burialSiteForm.crematedCapacity,
-
       burialSiteForm.cemeteryId === '' ? undefined : burialSiteForm.cemeteryId,
       burialSiteForm.cemeterySvgId,
       burialSiteForm.burialSiteImage ?? '',

database/addBurialSiteType.d.ts

@@ -1,7 +0,0 @@
-export interface AddBurialSiteTypeForm {
-    burialSiteType: string;
-    bodyCapacityMax: number | string;
-    crematedCapacityMax: number | string;
-    orderNumber?: number | string;
-}
-export default function addBurialSiteType(addForm: AddBurialSiteTypeForm, user: User): number;

database/addBurialSiteType.js

@@ -1,20 +0,0 @@
-import sqlite from 'better-sqlite3';
-import { sunriseDB } from '../helpers/database.helpers.js';
-import { clearCacheByTableName } from '../helpers/functions.cache.js';
-export default function addBurialSiteType(addForm, user) {
-    const database = sqlite(sunriseDB);
-    const rightNowMillis = Date.now();
-    const result = database
-        .prepare(`insert into BurialSiteTypes (
-        burialSiteType, bodyCapacityMax, crematedCapacityMax,
-        orderNumber,
-        recordCreate_userName, recordCreate_timeMillis,
-        recordUpdate_userName, recordUpdate_timeMillis)
-        values (?, ?, ?, ?, ?, ?, ?, ?)`)
-        .run(addForm.burialSiteType, addForm.bodyCapacityMax === '' ? undefined : addForm.bodyCapacityMax, addForm.crematedCapacityMax === ''
-        ? undefined
-        : addForm.crematedCapacityMax, addForm.orderNumber ?? -1, user.userName, rightNowMillis, user.userName, rightNowMillis);
-    database.close();
-    clearCacheByTableName('BurialSiteTypes');
-    return result.lastInsertRowid;
-}

database/addBurialSiteType.ts

@@ -1,50 +0,0 @@
-import sqlite from 'better-sqlite3'
-
-import { sunriseDB } from '../helpers/database.helpers.js'
-import { clearCacheByTableName } from '../helpers/functions.cache.js'
-
-export interface AddBurialSiteTypeForm {
-  burialSiteType: string
-
-  bodyCapacityMax: number | string
-  crematedCapacityMax: number | string
-
-  orderNumber?: number | string
-}
-
-export default function addBurialSiteType(
-  addForm: AddBurialSiteTypeForm,
-  user: User
-): number {
-  const database = sqlite(sunriseDB)
-
-  const rightNowMillis = Date.now()
-
-  const result = database
-    .prepare(
-      `insert into BurialSiteTypes (
-        burialSiteType, bodyCapacityMax, crematedCapacityMax,
-        orderNumber,
-        recordCreate_userName, recordCreate_timeMillis,
-        recordUpdate_userName, recordUpdate_timeMillis)
-        values (?, ?, ?, ?, ?, ?, ?, ?)`
-    )
-    .run(
-      addForm.burialSiteType,
-      addForm.bodyCapacityMax === '' ? undefined : addForm.bodyCapacityMax,
-      addForm.crematedCapacityMax === ''
-        ? undefined
-        : addForm.crematedCapacityMax,
-      addForm.orderNumber ?? -1,
-      user.userName,
-      rightNowMillis,
-      user.userName,
-      rightNowMillis
-    )
-
-  database.close()
-
-  clearCacheByTableName('BurialSiteTypes')
-
-  return result.lastInsertRowid as number
-}

database/addBurialSiteTypeField.js

@@ -14,9 +14,7 @@ export default function addBurialSiteTypeField(addForm, user) {
         recordCreate_userName, recordCreate_timeMillis,
         recordUpdate_userName, recordUpdate_timeMillis)
         values (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`)
-        .run(addForm.burialSiteTypeId, addForm.burialSiteTypeField, addForm.fieldType ?? 'text', addForm.fieldValues ?? '', addForm.isRequired === '' ? 0 : 1, addForm.pattern ?? '', addForm.minLength ?? 0, 
+        .run(addForm.burialSiteTypeId, addForm.burialSiteTypeField, addForm.fieldType ?? 'text', addForm.fieldValues ?? '', addForm.isRequired === '' ? 0 : 1, addForm.pattern ?? '', addForm.minLength ?? 0, addForm.maxLength ?? 100, addForm.orderNumber ?? -1, user.userName, rightNowMillis, user.userName, rightNowMillis);
-    // eslint-disable-next-line @typescript-eslint/no-magic-numbers
-    addForm.maxLength ?? 100, addForm.orderNumber ?? -1, user.userName, rightNowMillis, user.userName, rightNowMillis);
     database.close();
     clearCacheByTableName('BurialSiteTypeFields');
     return result.lastInsertRowid;

database/addBurialSiteTypeField.ts

@@ -47,7 +47,6 @@ export default function addBurialSiteTypeField(
       addForm.isRequired === '' ? 0 : 1,
       addForm.pattern ?? '',
       addForm.minLength ?? 0,
-      // eslint-disable-next-line @typescript-eslint/no-magic-numbers
       addForm.maxLength ?? 100,
       addForm.orderNumber ?? -1,
       user.userName,

database/addContractTransaction.d.ts

@@ -1,8 +1,7 @@
-import { type DateString, type TimeString } from '@cityssm/utils-datetime';
 export interface AddTransactionForm {
     contractId: number | string;
-    transactionDateString?: '' | DateString;
+    transactionDateString?: string;
-    transactionTimeString?: '' | TimeString;
+    transactionTimeString?: string;
     externalReceiptNumber: string;
     transactionAmount: number | string;
     transactionNote: string;

database/addContractTransaction.js

@@ -15,12 +15,12 @@ export default function addContractTransaction(contractTransactionForm, user) {
         transactionIndex = maxIndexResult.transactionIndex + 1;
     }
     const rightNow = new Date();
-    const transactionDate = (contractTransactionForm.transactionDateString ?? '') === ''
+    const transactionDate = contractTransactionForm.transactionDateString
-        ? dateToInteger(rightNow)
+        ? dateStringToInteger(contractTransactionForm.transactionDateString)
-        : dateStringToInteger(contractTransactionForm.transactionDateString);
+        : dateToInteger(rightNow);
-    const transactionTime = (contractTransactionForm.transactionTimeString ?? '') === ''
+    const transactionTime = contractTransactionForm.transactionTimeString
-        ? dateToTimeInteger(rightNow)
+        ? timeStringToInteger(contractTransactionForm.transactionTimeString)
-        : timeStringToInteger(contractTransactionForm.transactionTimeString);
+        : dateToTimeInteger(rightNow);
     database
         .prepare(`insert into ContractTransactions (
         contractId, transactionIndex,

database/addContractTransaction.ts

@@ -1,6 +1,4 @@
 import {
-  type DateString,
-  type TimeString,
   dateStringToInteger,
   dateToInteger,
   dateToTimeInteger,
@@ -13,8 +11,8 @@ import { sunriseDB } from '../helpers/database.helpers.js'
 export interface AddTransactionForm {
   contractId: number | string
 
-  transactionDateString?: '' | DateString
+  transactionDateString?: string
-  transactionTimeString?: '' | TimeString
+  transactionTimeString?: string
 
   externalReceiptNumber: string
   transactionAmount: number | string
@@ -47,19 +45,13 @@ export default function addContractTransaction(
 
   const rightNow = new Date()
 
-  const transactionDate =
+  const transactionDate = contractTransactionForm.transactionDateString
-    (contractTransactionForm.transactionDateString ?? '') === ''
+    ? dateStringToInteger(contractTransactionForm.transactionDateString)
-      ? dateToInteger(rightNow)
+    : dateToInteger(rightNow)
-      : dateStringToInteger(
-          contractTransactionForm.transactionDateString as DateString
-        )
 
-  const transactionTime =
+  const transactionTime = contractTransactionForm.transactionTimeString
-    (contractTransactionForm.transactionTimeString ?? '') === ''
+    ? timeStringToInteger(contractTransactionForm.transactionTimeString)
-      ? dateToTimeInteger(rightNow)
+    : dateToTimeInteger(rightNow)
-      : timeStringToInteger(
-          contractTransactionForm.transactionTimeString as TimeString
-        )
 
   database
     .prepare(

database/addContractTypeField.d.ts

@@ -1,8 +1,8 @@
 export interface AddContractTypeFieldForm {
     contractTypeId?: number | string;
     contractTypeField: string;
-    fieldType?: string;
     fieldValues?: string;
+    fieldType?: string;
     isRequired?: string;
     maxLength?: number | string;
     minLength?: number | string;

database/addContractTypeField.js

@@ -13,9 +13,7 @@ export default function addContractTypeField(addForm, user) {
         recordCreate_userName, recordCreate_timeMillis,
         recordUpdate_userName, recordUpdate_timeMillis)
         values (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`)
-        .run(addForm.contractTypeId ?? undefined, addForm.contractTypeField, addForm.fieldType ?? 'text', addForm.fieldValues ?? '', addForm.isRequired === '' ? 0 : 1, addForm.pattern ?? '', addForm.minLength ?? 0, 
+        .run(addForm.contractTypeId ?? undefined, addForm.contractTypeField, addForm.fieldType ?? 'text', addForm.fieldValues ?? '', addForm.isRequired === '' ? 0 : 1, addForm.pattern ?? '', addForm.minLength ?? 0, addForm.maxLength ?? 100, addForm.orderNumber ?? -1, user.userName, rightNowMillis, user.userName, rightNowMillis);
-    // eslint-disable-next-line @typescript-eslint/no-magic-numbers
-    addForm.maxLength ?? 100, addForm.orderNumber ?? -1, user.userName, rightNowMillis, user.userName, rightNowMillis);
     database.close();
     clearCacheByTableName('ContractTypeFields');
     return result.lastInsertRowid;

database/addContractTypeField.ts

@@ -5,11 +5,9 @@ import { clearCacheByTableName } from '../helpers/functions.cache.js'
 
 export interface AddContractTypeFieldForm {
   contractTypeId?: number | string
-
   contractTypeField: string
-
-  fieldType?: string
   fieldValues?: string
+  fieldType?: string
   isRequired?: string
   maxLength?: number | string
   minLength?: number | string
@@ -45,7 +43,6 @@ export default function addContractTypeField(
       addForm.isRequired === '' ? 0 : 1,
       addForm.pattern ?? '',
       addForm.minLength ?? 0,
-      // eslint-disable-next-line @typescript-eslint/no-magic-numbers
       addForm.maxLength ?? 100,
       addForm.orderNumber ?? -1,
       user.userName,

database/addContractTypePrint.ts

@@ -6,7 +6,6 @@ import { clearCacheByTableName } from '../helpers/functions.cache.js'
 export interface AddContractTypePrintForm {
   contractTypeId: number | string
   printEJS: string
-
   orderNumber?: number
 }
 

database/addFuneralHome.d.ts

@@ -1,6 +1,6 @@
 export interface AddForm {
-    funeralHomeKey: string;
     funeralHomeName: string;
+    funeralHomeKey: string;
     funeralHomeAddress1: string;
     funeralHomeAddress2: string;
     funeralHomeCity: string;

database/addFuneralHome.ts

@@ -3,8 +3,8 @@ import sqlite from 'better-sqlite3'
 import { sunriseDB } from '../helpers/database.helpers.js'
 
 export interface AddForm {
-  funeralHomeKey: string
   funeralHomeName: string
+  funeralHomeKey: string
 
   funeralHomeAddress1: string
   funeralHomeAddress2: string

database/addRecord.d.ts

@@ -1,3 +1,3 @@
-type RecordTable = 'BurialSiteStatuses' | 'WorkOrderMilestoneTypes' | 'WorkOrderTypes';
+type RecordTable = 'BurialSiteStatuses' | 'BurialSiteTypes' | 'WorkOrderMilestoneTypes' | 'WorkOrderTypes';
 export default function addRecord(recordTable: RecordTable, recordName: string, orderNumber: number | string, user: User): number;
 export {};

database/addRecord.js

@@ -3,6 +3,7 @@ import { sunriseDB } from '../helpers/database.helpers.js';
 import { clearCacheByTableName } from '../helpers/functions.cache.js';
 const recordNameColumns = new Map([
     ['BurialSiteStatuses', 'burialSiteStatus'],
+    ['BurialSiteTypes', 'burialSiteType'],
     ['WorkOrderMilestoneTypes', 'workOrderMilestoneType'],
     ['WorkOrderTypes', 'workOrderType']
 ]);

database/addRecord.ts

@@ -5,11 +5,13 @@ import { clearCacheByTableName } from '../helpers/functions.cache.js'
 
 type RecordTable =
   | 'BurialSiteStatuses'
+  | 'BurialSiteTypes'
   | 'WorkOrderMilestoneTypes'
   | 'WorkOrderTypes'
 
 const recordNameColumns = new Map<RecordTable, string>([
   ['BurialSiteStatuses', 'burialSiteStatus'],
+  ['BurialSiteTypes', 'burialSiteType'],
   ['WorkOrderMilestoneTypes', 'workOrderMilestoneType'],
   ['WorkOrderTypes', 'workOrderType']
 ])

database/addWorkOrder.d.ts

@@ -1,9 +1,9 @@
 export interface AddWorkOrderForm {
-    workOrderDescription: string;
-    workOrderNumber?: string;
     workOrderTypeId: number | string;
-    workOrderCloseDateString?: string;
+    workOrderNumber?: string;
+    workOrderDescription: string;
     workOrderOpenDateString?: string;
+    workOrderCloseDateString?: string;
     contractId?: string;
 }
 export default function addWorkOrder(workOrderForm: AddWorkOrderForm, user: User): number;

database/addWorkOrder.js

@@ -25,8 +25,8 @@ export default function addWorkOrder(workOrderForm, user) {
     const workOrderId = result.lastInsertRowid;
     if ((workOrderForm.contractId ?? '') !== '') {
         addWorkOrderContract({
-            contractId: workOrderForm.contractId,
             workOrderId,
+            contractId: workOrderForm.contractId
         }, user, database);
     }
     database.close();

database/addWorkOrder.ts

@@ -11,14 +11,11 @@ import addWorkOrderContract from './addWorkOrderContract.js'
 import getNextWorkOrderNumber from './getNextWorkOrderNumber.js'
 
 export interface AddWorkOrderForm {
-  workOrderDescription: string
-  workOrderNumber?: string
-
   workOrderTypeId: number | string
-
+  workOrderNumber?: string
-  workOrderCloseDateString?: string
+  workOrderDescription: string
   workOrderOpenDateString?: string
-  
+  workOrderCloseDateString?: string
   contractId?: string
 }
 
@@ -70,8 +67,8 @@ export default function addWorkOrder(
   if ((workOrderForm.contractId ?? '') !== '') {
     addWorkOrderContract(
       {
-        contractId: workOrderForm.contractId as string,
         workOrderId,
+        contractId: workOrderForm.contractId as string
       },
       user,
       database

database/addWorkOrderBurialSite.d.ts

@@ -1,5 +1,5 @@
 export interface AddForm {
-    burialSiteId: number | string;
     workOrderId: number | string;
+    burialSiteId: number | string;
 }
 export default function addWorkOrderBurialSite(workOrderLotForm: AddForm, user: User): boolean;

database/addWorkOrderBurialSite.ts

@@ -3,8 +3,8 @@ import sqlite from 'better-sqlite3'
 import { sunriseDB } from '../helpers/database.helpers.js'
 
 export interface AddForm {
-  burialSiteId: number | string
   workOrderId: number | string
+  burialSiteId: number | string
 }
 
 export default function addWorkOrderBurialSite(

database/addWorkOrderComment.d.ts

@@ -1,5 +1,5 @@
 export interface AddWorkOrderCommentForm {
-    comment: string;
     workOrderId: string;
+    comment: string;
 }
 export default function addWorkOrderComment(workOrderCommentForm: AddWorkOrderCommentForm, user: User): number;

database/addWorkOrderComment.ts

@@ -4,8 +4,8 @@ import sqlite from 'better-sqlite3'
 import { sunriseDB } from '../helpers/database.helpers.js'
 
 export interface AddWorkOrderCommentForm {
-  comment: string
   workOrderId: string
+  comment: string
 }
 
 export default function addWorkOrderComment(

database/addWorkOrderMilestone.d.ts

@@ -1,11 +1,10 @@
-import { type DateString, type TimeString } from '@cityssm/utils-datetime';
 export interface AddWorkOrderMilestoneForm {
     workOrderId: number | string;
     workOrderMilestoneTypeId: number | string;
-    workOrderMilestoneDateString: '' | DateString;
+    workOrderMilestoneDateString: string;
-    workOrderMilestoneTimeString?: '' | TimeString;
+    workOrderMilestoneTimeString?: string;
     workOrderMilestoneDescription: string;
-    workOrderMilestoneCompletionDateString?: '' | DateString;
+    workOrderMilestoneCompletionDateString?: string;
-    workOrderMilestoneCompletionTimeString?: '' | TimeString;
+    workOrderMilestoneCompletionTimeString?: string;
 }
 export default function addWorkOrderMilestone(milestoneForm: AddWorkOrderMilestoneForm, user: User): number;

database/addWorkOrderMilestone.ts

@@ -1,6 +1,4 @@
 import {
-  type DateString,
-  type TimeString,
   dateStringToInteger,
   timeStringToInteger
 } from '@cityssm/utils-datetime'
@@ -12,13 +10,13 @@ export interface AddWorkOrderMilestoneForm {
   workOrderId: number | string
   workOrderMilestoneTypeId: number | string
 
-  workOrderMilestoneDateString: '' | DateString
+  workOrderMilestoneDateString: string
-  workOrderMilestoneTimeString?: '' | TimeString
+  workOrderMilestoneTimeString?: string
 
   workOrderMilestoneDescription: string
 
-  workOrderMilestoneCompletionDateString?: '' | DateString
+  workOrderMilestoneCompletionDateString?: string
-  workOrderMilestoneCompletionTimeString?: '' | TimeString
+  workOrderMilestoneCompletionTimeString?: string
 }
 
 export default function addWorkOrderMilestone(
@@ -50,19 +48,17 @@ export default function addWorkOrderMilestone(
         : dateStringToInteger(milestoneForm.workOrderMilestoneDateString),
       (milestoneForm.workOrderMilestoneTimeString ?? '') === ''
         ? 0
-        : timeStringToInteger(
+        : timeStringToInteger(milestoneForm.workOrderMilestoneTimeString!),
-            milestoneForm.workOrderMilestoneTimeString as TimeString
-          ),
       milestoneForm.workOrderMilestoneDescription,
       (milestoneForm.workOrderMilestoneCompletionDateString ?? '') === ''
         ? undefined
         : dateStringToInteger(
-            milestoneForm.workOrderMilestoneCompletionDateString as DateString
+            milestoneForm.workOrderMilestoneCompletionDateString!
           ),
       (milestoneForm.workOrderMilestoneCompletionTimeString ?? '') === ''
         ? undefined
         : timeStringToInteger(
-            milestoneForm.workOrderMilestoneCompletionTimeString as TimeString
+            milestoneForm.workOrderMilestoneCompletionTimeString!
           ),
       user.userName,
       rightNowMillis,

database/backupDatabase.d.ts

@@ -1 +0,0 @@
-export declare function backupDatabase(): Promise<false | string>;

database/backupDatabase.js

@@ -1,28 +0,0 @@
-import sqlite from 'better-sqlite3';
-import Debug from 'debug';
-import { sunriseDB } from '../helpers/database.helpers.js';
-const debug = Debug('sunrise:database:backupDatabase');
-const backupFolder = 'data/backups';
-export async function backupDatabase() {
-    const databasePathSplit = sunriseDB.split(/[/\\]/);
-    const backupDatabasePath = `${backupFolder}/${databasePathSplit.at(-1)}.${Date.now().toString()}`;
-    const database = sqlite(sunriseDB);
-    try {
-        const result = await database.backup(backupDatabasePath);
-        if (result.remainingPages === 0) {
-            debug('Database backup completed successfully:', backupDatabasePath);
-            return backupDatabasePath;
-        }
-        else {
-            debug('Database backup incomplete:', result.remainingPages, 'pages remaining');
-            return false;
-        }
-    }
-    catch (error) {
-        debug('Error backing up database:', error);
-        return false;
-    }
-    finally {
-        database.close();
-    }
-}

database/backupDatabase.ts

@@ -1,37 +0,0 @@
-import sqlite from 'better-sqlite3'
-import Debug from 'debug'
-
-import { sunriseDB } from '../helpers/database.helpers.js'
-
-const debug = Debug('sunrise:database:backupDatabase')
-
-const backupFolder = 'data/backups'
-
-export async function backupDatabase(): Promise<false | string> {
-  const databasePathSplit = sunriseDB.split(/[/\\]/)
-
-  const backupDatabasePath = `${backupFolder}/${databasePathSplit.at(-1)}.${Date.now().toString()}`
-
-  const database = sqlite(sunriseDB)
-
-  try {
-    const result = await database.backup(backupDatabasePath)
-
-    if (result.remainingPages === 0) {
-      debug('Database backup completed successfully:', backupDatabasePath)
-      return backupDatabasePath
-    } else {
-      debug(
-        'Database backup incomplete:',
-        result.remainingPages,
-        'pages remaining'
-      )
-      return false
-    }
-  } catch (error) {
-    debug('Error backing up database:', error)
-    return false
-  } finally {
-    database.close()
-  }
-}

database/cleanupDatabase.js

@@ -122,7 +122,7 @@ export default function cleanupDatabase(user) {
         .run(recordDeleteTimeMillisMin).changes;
     /*
      * Burial Site Contract Fees/Transactions
-     * - Maintain financial data, do not delete related.
+     * - Maintain financials, do not delete related.
      */
     purgedRecordCount += database
         .prepare('delete from ContractFees where recordDelete_timeMillis <= ?')

database/cleanupDatabase.ts

@@ -179,7 +179,7 @@ export default function cleanupDatabase(user: User): {
 
   /*
    * Burial Site Contract Fees/Transactions
-   * - Maintain financial data, do not delete related.
+   * - Maintain financials, do not delete related.
    */
 
   purgedRecordCount += database

database/closeWorkOrder.d.ts

@@ -1,6 +1,5 @@
-import { type DateString } from '@cityssm/utils-datetime';
 export interface CloseWorkOrderForm {
     workOrderId: number | string;
-    workOrderCloseDateString?: '' | DateString;
+    workOrderCloseDateString?: string;
 }
 export default function closeWorkOrder(workOrderForm: CloseWorkOrderForm, user: User): boolean;

database/closeWorkOrder.ts

@@ -1,8 +1,4 @@
-import {
+import { dateStringToInteger, dateToInteger } from '@cityssm/utils-datetime'
-  type DateString,
-  dateStringToInteger,
-  dateToInteger
-} from '@cityssm/utils-datetime'
 import sqlite from 'better-sqlite3'
 
 import { sunriseDB } from '../helpers/database.helpers.js'
@@ -10,7 +6,7 @@ import { sunriseDB } from '../helpers/database.helpers.js'
 export interface CloseWorkOrderForm {
   workOrderId: number | string
 
-  workOrderCloseDateString?: '' | DateString
+  workOrderCloseDateString?: string
 }
 
 export default function closeWorkOrder(

database/completeWorkOrderMilestone.d.ts

@@ -1,7 +1,6 @@
-import { type DateString, type TimeString } from '@cityssm/utils-datetime';
 export interface CompleteWorkOrderMilestoneForm {
     workOrderMilestoneId: number | string;
-    workOrderMilestoneCompletionDateString?: '' | DateString;
+    workOrderMilestoneCompletionDateString?: string;
-    workOrderMilestoneCompletionTimeString?: '' | TimeString;
+    workOrderMilestoneCompletionTimeString?: string;
 }
 export default function completeWorkOrderMilestone(milestoneForm: CompleteWorkOrderMilestoneForm, user: User): boolean;

database/completeWorkOrderMilestone.js

@@ -4,12 +4,6 @@ import { sunriseDB } from '../helpers/database.helpers.js';
 export default function completeWorkOrderMilestone(milestoneForm, user) {
     const rightNow = new Date();
     const database = sqlite(sunriseDB);
-    const completionDate = (milestoneForm.workOrderMilestoneCompletionDateString ?? '') === ''
-        ? dateToInteger(rightNow)
-        : dateStringToInteger(milestoneForm.workOrderMilestoneCompletionDateString);
-    const completionTime = (milestoneForm.workOrderMilestoneCompletionTimeString ?? '') === ''
-        ? dateToTimeInteger(rightNow)
-        : timeStringToInteger(milestoneForm.workOrderMilestoneCompletionTimeString);
     const result = database
         .prepare(`update WorkOrderMilestones
         set workOrderMilestoneCompletionDate = ?,
@@ -17,7 +11,11 @@ export default function completeWorkOrderMilestone(milestoneForm, user) {
         recordUpdate_userName = ?,
         recordUpdate_timeMillis = ?
         where workOrderMilestoneId = ?`)
-        .run(completionDate, completionTime, user.userName, rightNow.getTime(), milestoneForm.workOrderMilestoneId);
+        .run((milestoneForm.workOrderMilestoneCompletionDateString ?? '') === ''
+        ? dateToInteger(rightNow)
+        : dateStringToInteger(milestoneForm.workOrderMilestoneCompletionDateString), (milestoneForm.workOrderMilestoneCompletionTimeString ?? '') === ''
+        ? dateToTimeInteger(rightNow)
+        : timeStringToInteger(milestoneForm.workOrderMilestoneCompletionTimeString), user.userName, rightNow.getTime(), milestoneForm.workOrderMilestoneId);
     database.close();
     return result.changes > 0;
 }

database/completeWorkOrderMilestone.ts

@@ -1,6 +1,4 @@
 import {
-  type DateString,
-  type TimeString,
   dateStringToInteger,
   dateToInteger,
   dateToTimeInteger,
@@ -13,8 +11,8 @@ import { sunriseDB } from '../helpers/database.helpers.js'
 export interface CompleteWorkOrderMilestoneForm {
   workOrderMilestoneId: number | string
   
-  workOrderMilestoneCompletionDateString?: '' | DateString
+  workOrderMilestoneCompletionDateString?: string
-  workOrderMilestoneCompletionTimeString?: '' | TimeString
+  workOrderMilestoneCompletionTimeString?: string
 }
 
 export default function completeWorkOrderMilestone(
@@ -25,20 +23,6 @@ export default function completeWorkOrderMilestone(
 
   const database = sqlite(sunriseDB)
 
-  const completionDate =
-    (milestoneForm.workOrderMilestoneCompletionDateString ?? '') === ''
-      ? dateToInteger(rightNow)
-      : dateStringToInteger(
-          milestoneForm.workOrderMilestoneCompletionDateString as DateString
-        )
-
-  const completionTime =
-    (milestoneForm.workOrderMilestoneCompletionTimeString ?? '') === ''
-      ? dateToTimeInteger(rightNow)
-      : timeStringToInteger(
-          milestoneForm.workOrderMilestoneCompletionTimeString as TimeString
-        )
-
   const result = database
     .prepare(
       `update WorkOrderMilestones
@@ -49,8 +33,16 @@ export default function completeWorkOrderMilestone(
         where workOrderMilestoneId = ?`
     )
     .run(
-      completionDate,
+      (milestoneForm.workOrderMilestoneCompletionDateString ?? '') === ''
-      completionTime,
+        ? dateToInteger(rightNow)
+        : dateStringToInteger(
+            milestoneForm.workOrderMilestoneCompletionDateString!
+          ),
+      (milestoneForm.workOrderMilestoneCompletionTimeString ?? '') === ''
+        ? dateToTimeInteger(rightNow)
+        : timeStringToInteger(
+            milestoneForm.workOrderMilestoneCompletionTimeString!
+          ),
       user.userName,
       rightNow.getTime(),
       milestoneForm.workOrderMilestoneId

database/copyContract.ts

@@ -29,7 +29,6 @@ export default async function copyContract(
       funeralDirectorName: oldContract.funeralDirectorName,
       funeralHomeId: oldContract.funeralHomeId ?? '',
       funeralTimeString: oldContract.funeralTimeString ?? '',
-
       purchaserAddress1: oldContract.purchaserAddress1,
       purchaserAddress2: oldContract.purchaserAddress2,
       purchaserCity: oldContract.purchaserCity,

database/deleteBurialSite.js

@@ -30,6 +30,23 @@ export function deleteBurialSite(burialSiteId, user) {
         where burialSiteId = ?
         and recordDelete_timeMillis is null`)
         .run(user.userName, rightNowMillis, burialSiteId);
+    /*
+     * Delete fields and comments
+     */
+    database
+        .prepare(`update BurialSiteFields
+        set recordDelete_userName = ?,
+        recordDelete_timeMillis = ?
+        where burialSiteId = ?
+        and recordDelete_timeMillis is null`)
+        .run(user.userName, rightNowMillis, burialSiteId);
+    database
+        .prepare(`update BurialSiteComments
+        set recordDelete_userName = ?,
+        recordDelete_timeMillis = ?
+        where burialSiteId = ?
+        and recordDelete_timeMillis is null`)
+        .run(user.userName, rightNowMillis, burialSiteId);
     database.close();
     return true;
 }

database/deleteBurialSite.ts

@@ -44,6 +44,30 @@ export function deleteBurialSite(burialSiteId: number, user: User): boolean {
     )
     .run(user.userName, rightNowMillis, burialSiteId)
 
+  /*
+   * Delete fields and comments
+   */
+
+  database
+    .prepare(
+      `update BurialSiteFields
+        set recordDelete_userName = ?,
+        recordDelete_timeMillis = ?
+        where burialSiteId = ?
+        and recordDelete_timeMillis is null`
+    )
+    .run(user.userName, rightNowMillis, burialSiteId)
+
+  database
+    .prepare(
+      `update BurialSiteComments
+        set recordDelete_userName = ?,
+        recordDelete_timeMillis = ?
+        where burialSiteId = ?
+        and recordDelete_timeMillis is null`
+    )
+    .run(user.userName, rightNowMillis, burialSiteId)
+
   database.close()
 
   return true

database/deleteRecord.js

@@ -1,6 +1,6 @@
 import sqlite from 'better-sqlite3';
 import { sunriseDB } from '../helpers/database.helpers.js';
-import { cacheTableNames, clearCacheByTableName } from '../helpers/functions.cache.js';
+import { clearCacheByTableName } from '../helpers/functions.cache.js';
 const recordIdColumns = new Map([
     ['BurialSiteComments', 'burialSiteCommentId'],
     ['BurialSiteStatuses', 'burialSiteStatusId'],
@@ -55,9 +55,6 @@ export function deleteRecord(recordTable, recordId, user) {
             .run(user.userName, rightNowMillis, recordId);
     }
     database.close();
-    // Clear cache for tables that are cached
-    if (cacheTableNames.includes(recordTable)) {
     clearCacheByTableName(recordTable);
-    }
     return result.changes > 0;
 }

database/deleteRecord.ts

@@ -1,11 +1,7 @@
 import sqlite from 'better-sqlite3'
 
 import { sunriseDB } from '../helpers/database.helpers.js'
-import {
+import { clearCacheByTableName } from '../helpers/functions.cache.js'
-  CacheTableNames,
-  cacheTableNames,
-  clearCacheByTableName
-} from '../helpers/functions.cache.js'
 
 type RecordTable =
   | 'BurialSiteComments'
@@ -77,7 +73,7 @@ export function deleteRecord(
       `update ${recordTable}
         set recordDelete_userName = ?,
         recordDelete_timeMillis = ?
-        where ${recordIdColumns.get(recordTable)} = ?
+        where ${recordIdColumns.get(recordTable)!} = ?
         and recordDelete_timeMillis is null`
     )
     .run(user.userName, rightNowMillis, recordId)
@@ -88,7 +84,7 @@ export function deleteRecord(
         `update ${relatedTable}
           set recordDelete_userName = ?,
           recordDelete_timeMillis = ?
-          where ${recordIdColumns.get(recordTable)} = ?
+          where ${recordIdColumns.get(recordTable)!} = ?
           and recordDelete_timeMillis is null`
       )
       .run(user.userName, rightNowMillis, recordId)
@@ -96,10 +92,7 @@ export function deleteRecord(
 
   database.close()
 
-  // Clear cache for tables that are cached
+  clearCacheByTableName(recordTable)
-  if (cacheTableNames.includes(recordTable as CacheTableNames)) {
-    clearCacheByTableName(recordTable as CacheTableNames)
-  }
 
   return result.changes > 0
 }

database/getBurialSite.d.ts

@@ -1,3 +1,3 @@
 import type { BurialSite } from '../types/record.types.js';
-export default function getBurialSite(burialSiteId: number | string, includeDeleted?: boolean): Promise<BurialSite | undefined>;
+export default function getBurialSite(burialSiteId: number | string): Promise<BurialSite | undefined>;
-export declare function getBurialSiteByBurialSiteName(burialSiteName: string, includeDeleted?: boolean): Promise<BurialSite | undefined>;
+export declare function getBurialSiteByBurialSiteName(burialSiteName: string): Promise<BurialSite | undefined>;

database/getBurialSite.js

@@ -13,31 +13,25 @@ const baseSQL = `select l.burialSiteId,
   l.burialSiteName,
   l.burialSiteStatusId, s.burialSiteStatus,
 
-  l.bodyCapacity, l.crematedCapacity,
-  t.bodyCapacityMax, t.crematedCapacityMax,
-
   l.cemeteryId, m.cemeteryName,
   m.cemeteryLatitude, m.cemeteryLongitude,
   m.cemeterySvg, l.cemeterySvgId, l.burialSiteImage,
-  l.burialSiteLatitude, l.burialSiteLongitude,
+  l.burialSiteLatitude, l.burialSiteLongitude
-
-  l.recordDelete_userName, l.recordDelete_timeMillis
 
   from BurialSites l
   left join BurialSiteTypes t on l.burialSiteTypeId = t.burialSiteTypeId
   left join BurialSiteStatuses s on l.burialSiteStatusId = s.burialSiteStatusId
-  left join Cemeteries m on l.cemeteryId = m.cemeteryId`;
+  left join Cemeteries m on l.cemeteryId = m.cemeteryId
-export default async function getBurialSite(burialSiteId, includeDeleted = false) {
+  where l.recordDelete_timeMillis is null`;
-    return await _getBurialSite(`l.burialSiteId = ?`, burialSiteId, includeDeleted);
+export default async function getBurialSite(burialSiteId) {
+    return await _getBurialSite(`${baseSQL} and l.burialSiteId = ?`, burialSiteId);
 }
-export async function getBurialSiteByBurialSiteName(burialSiteName, includeDeleted = false) {
+export async function getBurialSiteByBurialSiteName(burialSiteName) {
-    return await _getBurialSite(`l.burialSiteName = ?`, burialSiteName, includeDeleted);
+    return await _getBurialSite(`${baseSQL} and l.burialSiteName = ?`, burialSiteName);
 }
-async function _getBurialSite(whereClausePiece, burialSiteIdOrLotName, includeDeleted = false) {
+async function _getBurialSite(sql, burialSiteIdOrLotName) {
     const database = sqlite(sunriseDB, { readonly: true });
-    const burialSite = database
+    const burialSite = database.prepare(sql).get(burialSiteIdOrLotName);
-        .prepare(`${baseSQL} ${includeDeleted ? 'where' : 'where l.recordDelete_timeMillis is null and'} ${whereClausePiece}`)
-        .get(burialSiteIdOrLotName);
     if (burialSite !== undefined) {
         const contracts = await getContracts({
             burialSiteId: burialSite.burialSiteId

database/getBurialSite.ts

@@ -17,55 +17,41 @@ const baseSQL = `select l.burialSiteId,
   l.burialSiteName,
   l.burialSiteStatusId, s.burialSiteStatus,
 
-  l.bodyCapacity, l.crematedCapacity,
-  t.bodyCapacityMax, t.crematedCapacityMax,
-
   l.cemeteryId, m.cemeteryName,
   m.cemeteryLatitude, m.cemeteryLongitude,
   m.cemeterySvg, l.cemeterySvgId, l.burialSiteImage,
-  l.burialSiteLatitude, l.burialSiteLongitude,
+  l.burialSiteLatitude, l.burialSiteLongitude
-
-  l.recordDelete_userName, l.recordDelete_timeMillis
 
   from BurialSites l
   left join BurialSiteTypes t on l.burialSiteTypeId = t.burialSiteTypeId
   left join BurialSiteStatuses s on l.burialSiteStatusId = s.burialSiteStatusId
-  left join Cemeteries m on l.cemeteryId = m.cemeteryId`
+  left join Cemeteries m on l.cemeteryId = m.cemeteryId
+  where l.recordDelete_timeMillis is null`
 
 export default async function getBurialSite(
-  burialSiteId: number | string,
+  burialSiteId: number | string
-  includeDeleted = false
 ): Promise<BurialSite | undefined> {
-  return await _getBurialSite(
+  return await _getBurialSite(`${baseSQL} and l.burialSiteId = ?`, burialSiteId)
-    `l.burialSiteId = ?`,
-    burialSiteId,
-    includeDeleted
-  )
 }
 
 export async function getBurialSiteByBurialSiteName(
-  burialSiteName: string,
+  burialSiteName: string
-  includeDeleted = false
 ): Promise<BurialSite | undefined> {
   return await _getBurialSite(
-    `l.burialSiteName = ?`,
+    `${baseSQL} and l.burialSiteName = ?`,
-    burialSiteName,
+    burialSiteName
-    includeDeleted
   )
 }
 
 async function _getBurialSite(
-  whereClausePiece: string,
+  sql: string,
-  burialSiteIdOrLotName: number | string,
+  burialSiteIdOrLotName: number | string
-  includeDeleted = false
 ): Promise<BurialSite | undefined> {
   const database = sqlite(sunriseDB, { readonly: true })
 
-  const burialSite = database
+  const burialSite = database.prepare(sql).get(burialSiteIdOrLotName) as
-    .prepare(
+    | BurialSite
-      `${baseSQL} ${includeDeleted ? 'where' : 'where l.recordDelete_timeMillis is null and'} ${whereClausePiece}`
+    | undefined
-    )
-    .get(burialSiteIdOrLotName) as BurialSite | undefined
 
   if (burialSite !== undefined) {
     const contracts = await getContracts(

database/getBurialSiteStatuses.d.ts

@@ -1,2 +1,2 @@
 import type { BurialSiteStatus } from '../types/record.types.js';
-export default function getBurialSiteStatuses(includeDeleted?: boolean): BurialSiteStatus[];
+export default function getBurialSiteStatuses(): BurialSiteStatus[];

database/getBurialSiteStatuses.js

@@ -1,16 +1,14 @@
 import sqlite from 'better-sqlite3';
 import { sunriseDB } from '../helpers/database.helpers.js';
 import { updateRecordOrderNumber } from './updateRecordOrderNumber.js';
-export default function getBurialSiteStatuses(includeDeleted = false) {
+export default function getBurialSiteStatuses() {
     const database = sqlite(sunriseDB);
-    const updateOrderNumbers = !includeDeleted;
     const statuses = database
         .prepare(`select burialSiteStatusId, burialSiteStatus, orderNumber
         from BurialSiteStatuses
-        ${includeDeleted ? '' : ' where recordDelete_timeMillis is null '}
+        where recordDelete_timeMillis is null
         order by orderNumber, burialSiteStatus`)
         .all();
-    if (updateOrderNumbers) {
     let expectedOrderNumber = 0;
     for (const status of statuses) {
         if (status.orderNumber !== expectedOrderNumber) {
@@ -19,7 +17,6 @@ export default function getBurialSiteStatuses(includeDeleted = false) {
         }
         expectedOrderNumber += 1;
     }
-    }
     database.close();
     return statuses;
 }

database/getBurialSiteStatuses.ts

@@ -5,23 +5,18 @@ import type { BurialSiteStatus } from '../types/record.types.js'
 
 import { updateRecordOrderNumber } from './updateRecordOrderNumber.js'
 
-export default function getBurialSiteStatuses(
+export default function getBurialSiteStatuses(): BurialSiteStatus[] {
-  includeDeleted = false
-): BurialSiteStatus[] {
   const database = sqlite(sunriseDB)
 
-  const updateOrderNumbers = !includeDeleted
-
   const statuses = database
     .prepare(
       `select burialSiteStatusId, burialSiteStatus, orderNumber
         from BurialSiteStatuses
-        ${includeDeleted ? '' : ' where recordDelete_timeMillis is null '}
+        where recordDelete_timeMillis is null
         order by orderNumber, burialSiteStatus`
     )
     .all() as BurialSiteStatus[]
 
-  if (updateOrderNumbers) {
   let expectedOrderNumber = 0
 
   for (const status of statuses) {
@@ -32,13 +27,11 @@ export default function getBurialSiteStatuses(
         expectedOrderNumber,
         database
       )
-
       status.orderNumber = expectedOrderNumber
     }
 
     expectedOrderNumber += 1
   }
-  }
 
   database.close()
 

database/getBurialSiteTypes.js

@@ -4,11 +4,9 @@ import getBurialSiteTypeFields from './getBurialSiteTypeFields.js';
 import { updateRecordOrderNumber } from './updateRecordOrderNumber.js';
 export default function getBurialSiteTypes(includeDeleted = false) {
     const database = sqlite(sunriseDB);
-    const updateOrderNumbers = !includeDeleted;
+    const updateOrderNumbers = !database.readonly && !includeDeleted;
     const burialSiteTypes = database
-        .prepare(`select burialSiteTypeId, burialSiteType,
+        .prepare(`select burialSiteTypeId, burialSiteType, orderNumber
-        bodyCapacityMax, crematedCapacityMax,
-        orderNumber
         from BurialSiteTypes
         ${includeDeleted ? '' : ' where recordDelete_timeMillis is null '}
         order by orderNumber, burialSiteType`)

database/getBurialSiteTypes.ts

@@ -11,13 +11,11 @@ export default function getBurialSiteTypes(
 ): BurialSiteType[] {
   const database = sqlite(sunriseDB)
 
-  const updateOrderNumbers = !includeDeleted
+  const updateOrderNumbers = !database.readonly && !includeDeleted
 
   const burialSiteTypes = database
     .prepare(
-      `select burialSiteTypeId, burialSiteType,
+      `select burialSiteTypeId, burialSiteType, orderNumber
-        bodyCapacityMax, crematedCapacityMax,
-        orderNumber
         from BurialSiteTypes
         ${includeDeleted ? '' : ' where recordDelete_timeMillis is null '}
         order by orderNumber, burialSiteType`

database/getBurialSites.d.ts

@@ -4,8 +4,8 @@ export interface GetBurialSitesFilters {
     burialSiteNameSearchType?: '' | 'endsWith' | 'startsWith';
     burialSiteName?: string;
     cemeteryId?: number | string;
-    burialSiteStatusId?: number | string;
     burialSiteTypeId?: number | string;
+    burialSiteStatusId?: number | string;
     contractStatus?: '' | 'occupied' | 'unoccupied';
     workOrderId?: number | string;
 }
@@ -16,6 +16,6 @@ export interface GetBurialSitesOptions {
     includeContractCount?: boolean;
 }
 export default function getBurialSites(filters: GetBurialSitesFilters, options: GetBurialSitesOptions, connectedDatabase?: sqlite.Database): {
-    burialSites: BurialSite[];
     count: number;
+    burialSites: BurialSite[];
 };

database/getBurialSites.js

@@ -4,7 +4,7 @@ import { sunriseDB } from '../helpers/database.helpers.js';
 import { getBurialSiteNameWhereClause } from '../helpers/functions.sqlFilters.js';
 export default function getBurialSites(filters, options, connectedDatabase) {
     const database = connectedDatabase ?? sqlite(sunriseDB, { readonly: true });
-    const { sqlParameters, sqlWhereClause } = buildWhereClause(filters);
+    const { sqlWhereClause, sqlParameters } = buildWhereClause(filters);
     const currentDate = dateToInteger(new Date());
     let count = 0;
     if (options.limit !== -1) {
@@ -20,8 +20,7 @@ export default function getBurialSites(filters, options, connectedDatabase) {
               group by burialSiteId
             ) o on l.burialSiteId = o.burialSiteId
             ${sqlWhereClause}`)
-            .pluck()
+            .get(sqlParameters).recordCount;
-            .get(sqlParameters);
     }
     let burialSites = [];
     if (options.limit === -1 || count > 0) {
@@ -38,7 +37,6 @@ export default function getBurialSites(filters, options, connectedDatabase) {
           l.burialSiteNameSegment5,
           l.burialSiteName,
           t.burialSiteType,
-          l.bodyCapacity, l.crematedCapacity,
           l.cemeteryId, m.cemeteryName, l.cemeterySvgId,
           l.burialSiteStatusId, s.burialSiteStatus
           ${includeContractCount
@@ -72,8 +70,8 @@ export default function getBurialSites(filters, options, connectedDatabase) {
         database.close();
     }
     return {
-        burialSites,
+        count,
-        count
+        burialSites
     };
 }
 function buildWhereClause(filters) {

database/getBurialSites.ts

@@ -9,9 +9,8 @@ export interface GetBurialSitesFilters {
   burialSiteNameSearchType?: '' | 'endsWith' | 'startsWith'
   burialSiteName?: string
   cemeteryId?: number | string
-
-  burialSiteStatusId?: number | string
   burialSiteTypeId?: number | string
+  burialSiteStatusId?: number | string
   contractStatus?: '' | 'occupied' | 'unoccupied'
   workOrderId?: number | string
 }
@@ -28,17 +27,18 @@ export default function getBurialSites(
   filters: GetBurialSitesFilters,
   options: GetBurialSitesOptions,
   connectedDatabase?: sqlite.Database
-): { burialSites: BurialSite[]; count: number } {
+): { count: number; burialSites: BurialSite[] } {
   const database = connectedDatabase ?? sqlite(sunriseDB, { readonly: true })
 
-  const { sqlParameters, sqlWhereClause } = buildWhereClause(filters)
+  const { sqlWhereClause, sqlParameters } = buildWhereClause(filters)
 
   const currentDate = dateToInteger(new Date())
 
   let count = 0
 
   if (options.limit !== -1) {
-    count = database
+    count = (
+      database
         .prepare(
           `select count(*) as recordCount
             from BurialSites l
@@ -52,8 +52,8 @@ export default function getBurialSites(
             ) o on l.burialSiteId = o.burialSiteId
             ${sqlWhereClause}`
         )
-      .pluck()
+        .get(sqlParameters) as { recordCount: number }
-      .get(sqlParameters) as number
+    ).recordCount
   }
 
   let burialSites: BurialSite[] = []
@@ -75,7 +75,6 @@ export default function getBurialSites(
           l.burialSiteNameSegment5,
           l.burialSiteName,
           t.burialSiteType,
-          l.bodyCapacity, l.crematedCapacity,
           l.cemeteryId, m.cemeteryName, l.cemeterySvgId,
           l.burialSiteStatusId, s.burialSiteStatus
           ${
@@ -119,8 +118,8 @@ export default function getBurialSites(
   }
 
   return {
-    burialSites,
+    count,
-    count
+    burialSites
   }
 }
 

database/getContract.js

@@ -16,7 +16,6 @@ export default async function getContract(contractId, connectedDatabase) {
         .prepare(`select o.contractId,
           o.contractTypeId, t.contractType, t.isPreneed,
           o.burialSiteId, l.burialSiteName, l.burialSiteTypeId,
-          case when l.recordDelete_timeMillis is null then 1 else 0 end as burialSiteIsActive,
           l.cemeteryId, m.cemeteryName,
           o.contractStartDate, userFn_dateIntegerToString(o.contractStartDate) as contractStartDateString,
           o.contractEndDate, userFn_dateIntegerToString(o.contractEndDate) as contractEndDateString,

database/getContract.ts

@@ -33,7 +33,6 @@ export default async function getContract(
       `select o.contractId,
           o.contractTypeId, t.contractType, t.isPreneed,
           o.burialSiteId, l.burialSiteName, l.burialSiteTypeId,
-          case when l.recordDelete_timeMillis is null then 1 else 0 end as burialSiteIsActive,
           l.cemeteryId, m.cemeteryName,
           o.contractStartDate, userFn_dateIntegerToString(o.contractStartDate) as contractStartDateString,
           o.contractEndDate, userFn_dateIntegerToString(o.contractEndDate) as contractEndDateString,

database/getContractTypes.d.ts

@@ -1,2 +1,2 @@
 import type { ContractType } from '../types/record.types.js';
-export default function getContractTypes(includeDeleted?: boolean): ContractType[];
+export default function getContractTypes(): ContractType[];

database/getContractTypes.js

@@ -3,20 +3,18 @@ import { sunriseDB } from '../helpers/database.helpers.js';
 import getContractTypeFields from './getContractTypeFields.js';
 import getContractTypePrints from './getContractTypePrints.js';
 import { updateRecordOrderNumber } from './updateRecordOrderNumber.js';
-export default function getContractTypes(includeDeleted = false) {
+export default function getContractTypes() {
     const database = sqlite(sunriseDB);
-    const updateOrderNumbers = !includeDeleted;
     const contractTypes = database
         .prepare(`select contractTypeId, contractType, isPreneed, orderNumber
         from ContractTypes
-        ${includeDeleted ? '' : ' where recordDelete_timeMillis is null '} 
+        where recordDelete_timeMillis is null
         order by orderNumber, contractType, contractTypeId`)
         .all();
     let expectedOrderNumber = -1;
     for (const contractType of contractTypes) {
         expectedOrderNumber += 1;
-        if (updateOrderNumbers &&
+        if (contractType.orderNumber !== expectedOrderNumber) {
-            contractType.orderNumber !== expectedOrderNumber) {
             updateRecordOrderNumber('ContractTypes', contractType.contractTypeId, expectedOrderNumber, database);
             contractType.orderNumber = expectedOrderNumber;
         }

database/getContractTypes.ts

@@ -7,18 +7,14 @@ import getContractTypeFields from './getContractTypeFields.js'
 import getContractTypePrints from './getContractTypePrints.js'
 import { updateRecordOrderNumber } from './updateRecordOrderNumber.js'
 
-export default function getContractTypes(
+export default function getContractTypes(): ContractType[] {
-  includeDeleted = false
-): ContractType[] {
   const database = sqlite(sunriseDB)
 
-  const updateOrderNumbers = !includeDeleted
-
   const contractTypes = database
     .prepare(
       `select contractTypeId, contractType, isPreneed, orderNumber
         from ContractTypes
-        ${includeDeleted ? '' : ' where recordDelete_timeMillis is null '} 
+        where recordDelete_timeMillis is null
         order by orderNumber, contractType, contractTypeId`
     )
     .all() as ContractType[]
@@ -28,10 +24,7 @@ export default function getContractTypes(
   for (const contractType of contractTypes) {
     expectedOrderNumber += 1
 
-    if (
+    if (contractType.orderNumber !== expectedOrderNumber) {
-      updateOrderNumbers &&
-      contractType.orderNumber !== expectedOrderNumber
-    ) {
       updateRecordOrderNumber(
         'ContractTypes',
         contractType.contractTypeId,

database/getContracts.d.ts

@@ -6,14 +6,13 @@ export interface GetContractsFilters {
     contractEffectiveDateString?: string;
     contractStartDateString?: DateString;
     contractTime?: '' | 'current' | 'future' | 'past';
-    cemeteryId?: number | string;
-    contractTypeId?: number | string;
     deceasedName?: string;
+    contractTypeId?: number | string;
+    cemeteryId?: number | string;
     burialSiteName?: string;
     burialSiteNameSearchType?: '' | 'endsWith' | 'startsWith';
     burialSiteTypeId?: number | string;
     funeralHomeId?: number | string;
-    funeralTime?: '' | 'upcoming';
     notWorkOrderId?: number | string;
     workOrderId?: number | string;
 }
@@ -21,7 +20,6 @@ export interface GetContractsOptions {
     /** -1 for no limit */
     limit: number | string;
     offset: number | string;
-    orderBy?: string;
     includeFees: boolean;
     includeInterments: boolean;
     includeTransactions: boolean;

database/getContracts.js

@@ -1,4 +1,4 @@
-import { dateIntegerToString, dateStringToInteger, dateToInteger, timeIntegerToPeriodString, timeIntegerToString } from '@cityssm/utils-datetime';
+import { dateIntegerToString, dateStringToInteger, timeIntegerToPeriodString, timeIntegerToString } from '@cityssm/utils-datetime';
 import sqlite from 'better-sqlite3';
 import { getConfigProperty } from '../helpers/config.helpers.js';
 import { sunriseDB } from '../helpers/database.helpers.js';
@@ -20,55 +20,48 @@ export default async function getContracts(filters, options, connectedDatabase)
     if (isLimited) {
         count = database
             .prepare(`select count(*) as recordCount
-          from Contracts c
+            from Contracts o
-          left join BurialSites l on c.burialSiteId = l.burialSiteId
+            left join BurialSites l on o.burialSiteId = l.burialSiteId
             left join Cemeteries m on l.cemeteryId = m.cemeteryId
             ${sqlWhereClause}`)
-            .pluck()
+            .get(sqlParameters).recordCount;
-            .get(sqlParameters);
     }
     let contracts = [];
     if (count !== 0) {
-        const sqlLimitClause = isLimited
-            ? ` limit ${options.limit} offset ${options.offset}`
-            : '';
         contracts = database
-            .prepare(`select c.contractId,
+            .prepare(`select o.contractId,
-            c.contractTypeId, t.contractType, t.isPreneed,
+          o.contractTypeId, t.contractType, t.isPreneed,
-            c.burialSiteId, lt.burialSiteType, l.burialSiteName,
+          o.burialSiteId, lt.burialSiteType, l.burialSiteName,
-            case when l.recordDelete_timeMillis is null then 1 else 0 end as burialSiteIsActive,
           l.cemeteryId, m.cemeteryName,
-            c.contractStartDate, userFn_dateIntegerToString(c.contractStartDate) as contractStartDateString,
+          o.contractStartDate, userFn_dateIntegerToString(o.contractStartDate) as contractStartDateString,
-            c.contractEndDate, userFn_dateIntegerToString(c.contractEndDate) as contractEndDateString,
+          o.contractEndDate, userFn_dateIntegerToString(o.contractEndDate) as contractEndDateString,
-            c.purchaserName, c.purchaserAddress1, c.purchaserAddress2,
+          o.purchaserName, o.purchaserAddress1, o.purchaserAddress2,
-            c.purchaserCity, c.purchaserProvince, c.purchaserPostalCode,
+          o.purchaserCity, o.purchaserProvince, o.purchaserPostalCode,
-            c.purchaserPhoneNumber, c.purchaserEmail, c.purchaserRelationship,
+          o.purchaserPhoneNumber, o.purchaserEmail, o.purchaserRelationship,
-            c.funeralHomeId, c.funeralDirectorName, f.funeralHomeName,
+          o.funeralHomeId, o.funeralDirectorName, f.funeralHomeName,
 
-            c.funeralDate, userFn_dateIntegerToString(c.funeralDate) as funeralDateString,
+          o.funeralDate, userFn_dateIntegerToString(o.funeralDate) as funeralDateString,
-            c.funeralTime,
+          o.funeralTime,
-            userFn_timeIntegerToString(c.funeralTime) as funeralTimeString,
+          userFn_timeIntegerToString(o.funeralTime) as funeralTimeString,
-            userFn_timeIntegerToPeriodString(c.funeralTime) as funeralTimePeriodString,
+          userFn_timeIntegerToPeriodString(o.funeralTime) as funeralTimePeriodString,
-            c.directionOfArrival,
+          o.directionOfArrival,
-            c.committalTypeId, cm.committalType
+          o.committalTypeId, c.committalType
-          from Contracts c
+          from Contracts o
-          left join ContractTypes t on c.contractTypeId = t.contractTypeId
+          left join ContractTypes t on o.contractTypeId = t.contractTypeId
-          left join CommittalTypes cm on c.committalTypeId = cm.committalTypeId
+          left join CommittalTypes c on o.committalTypeId = c.committalTypeId
-          left join BurialSites l on c.burialSiteId = l.burialSiteId
+          left join BurialSites l on o.burialSiteId = l.burialSiteId
           left join BurialSiteTypes lt on l.burialSiteTypeId = lt.burialSiteTypeId
           left join Cemeteries m on l.cemeteryId = m.cemeteryId
-          left join FuneralHomes f on c.funeralHomeId = f.funeralHomeId
+          left join FuneralHomes f on o.funeralHomeId = f.funeralHomeId
           ${sqlWhereClause}
-          ${options.orderBy !== undefined && options.orderBy !== ''
+          order by o.contractStartDate desc, ifnull(o.contractEndDate, 99999999) desc,
-            ? ` order by ${options.orderBy}`
-            : ` order by c.contractStartDate desc, ifnull(c.contractEndDate, 99999999) desc,
             l.burialSiteNameSegment1,
             l.burialSiteNameSegment2,
             l.burialSiteNameSegment3,
             l.burialSiteNameSegment4,
             l.burialSiteNameSegment5,
-                  c.burialSiteId, c.contractId desc`}
+            o.burialSiteId, o.contractId desc
-          ${sqlLimitClause}`)
+          ${isLimited ? ` limit ${options.limit} offset ${options.offset}` : ''}`)
             .all(sqlParameters);
         if (!isLimited) {
             count = contracts.length;
@@ -105,38 +98,38 @@ async function addInclusions(contract, options, database) {
 }
 // eslint-disable-next-line complexity
 function buildWhereClause(filters) {
-    let sqlWhereClause = ' where c.recordDelete_timeMillis is null';
+    let sqlWhereClause = ' where o.recordDelete_timeMillis is null';
     const sqlParameters = [];
     if ((filters.burialSiteId ?? '') !== '') {
-        sqlWhereClause += ' and c.burialSiteId = ?';
+        sqlWhereClause += ' and o.burialSiteId = ?';
         sqlParameters.push(filters.burialSiteId);
     }
     const burialSiteNameFilters = getBurialSiteNameWhereClause(filters.burialSiteName, filters.burialSiteNameSearchType ?? '', 'l');
     sqlWhereClause += burialSiteNameFilters.sqlWhereClause;
     sqlParameters.push(...burialSiteNameFilters.sqlParameters);
-    const deceasedNameFilters = getDeceasedNameWhereClause(filters.deceasedName, 'c');
+    const deceasedNameFilters = getDeceasedNameWhereClause(filters.deceasedName, 'o');
     if (deceasedNameFilters.sqlParameters.length > 0) {
-        sqlWhereClause += ` and c.contractId in (
+        sqlWhereClause += ` and o.contractId in (
-        select contractId from ContractInterments c
+        select contractId from ContractInterments o
         where recordDelete_timeMillis is null
         ${deceasedNameFilters.sqlWhereClause})`;
         sqlParameters.push(...deceasedNameFilters.sqlParameters);
     }
     if ((filters.contractTypeId ?? '') !== '') {
-        sqlWhereClause += ' and c.contractTypeId = ?';
+        sqlWhereClause += ' and o.contractTypeId = ?';
         sqlParameters.push(filters.contractTypeId);
     }
-    const contractTimeFilters = getContractTimeWhereClause(filters.contractTime ?? '', 'c');
+    const contractTimeFilters = getContractTimeWhereClause(filters.contractTime ?? '', 'o');
     sqlWhereClause += contractTimeFilters.sqlWhereClause;
     sqlParameters.push(...contractTimeFilters.sqlParameters);
     if ((filters.contractStartDateString ?? '') !== '') {
-        sqlWhereClause += ' and c.contractStartDate = ?';
+        sqlWhereClause += ' and o.contractStartDate = ?';
         sqlParameters.push(dateStringToInteger(filters.contractStartDateString));
     }
     if ((filters.contractEffectiveDateString ?? '') !== '') {
         sqlWhereClause += ` and (
-        c.contractEndDate is null
+        o.contractEndDate is null
-        or (c.contractStartDate <= ? and c.contractEndDate >= ?)
+        or (o.contractStartDate <= ? and o.contractEndDate >= ?)
       )`;
         sqlParameters.push(dateStringToInteger(filters.contractEffectiveDateString), dateStringToInteger(filters.contractEffectiveDateString));
     }
@@ -149,21 +142,17 @@ function buildWhereClause(filters) {
         sqlParameters.push(filters.burialSiteTypeId);
     }
     if ((filters.funeralHomeId ?? '') !== '') {
-        sqlWhereClause += ' and c.funeralHomeId = ?';
+        sqlWhereClause += ' and o.funeralHomeId = ?';
         sqlParameters.push(filters.funeralHomeId);
     }
-    if ((filters.funeralTime ?? '') === 'upcoming') {
-        sqlWhereClause += ' and c.funeralDate >= ?';
-        sqlParameters.push(dateToInteger(new Date()));
-    }
     if ((filters.workOrderId ?? '') !== '') {
         sqlWhereClause +=
-            ' and c.contractId in (select contractId from WorkOrderContracts where recordDelete_timeMillis is null and workOrderId = ?)';
+            ' and o.contractId in (select contractId from WorkOrderContracts where recordDelete_timeMillis is null and workOrderId = ?)';
         sqlParameters.push(filters.workOrderId);
     }
     if ((filters.notWorkOrderId ?? '') !== '') {
         sqlWhereClause +=
-            ' and c.contractId not in (select contractId from WorkOrderContracts where recordDelete_timeMillis is null and workOrderId = ?)';
+            ' and o.contractId not in (select contractId from WorkOrderContracts where recordDelete_timeMillis is null and workOrderId = ?)';
         sqlParameters.push(filters.notWorkOrderId);
     }
     return {

database/getContracts.ts

@@ -2,7 +2,6 @@ import {
   type DateString,
   dateIntegerToString,
   dateStringToInteger,
-  dateToInteger,
   timeIntegerToPeriodString,
   timeIntegerToString
 } from '@cityssm/utils-datetime'
@@ -29,17 +28,15 @@ export interface GetContractsFilters {
   contractStartDateString?: DateString
   contractTime?: '' | 'current' | 'future' | 'past'
 
-  cemeteryId?: number | string
-  contractTypeId?: number | string
   deceasedName?: string
+  contractTypeId?: number | string
+  cemeteryId?: number | string
 
   burialSiteName?: string
   burialSiteNameSearchType?: '' | 'endsWith' | 'startsWith'
 
   burialSiteTypeId?: number | string
-
   funeralHomeId?: number | string
-  funeralTime?: '' | 'upcoming'
 
   notWorkOrderId?: number | string
   workOrderId?: number | string
@@ -50,8 +47,6 @@ export interface GetContractsOptions {
   limit: number | string
   offset: number | string
 
-  orderBy?: string
-
   includeFees: boolean
   includeInterments: boolean
   includeTransactions: boolean
@@ -67,9 +62,7 @@ export default async function getContracts(
   database.function('userFn_dateIntegerToString', dateIntegerToString)
   database.function('userFn_timeIntegerToString', timeIntegerToString)
   database.function(
-    'userFn_timeIntegerToPeriodString',
+    'userFn_timeIntegerToPeriodString', timeIntegerToPeriodString)
-    timeIntegerToPeriodString
-  )
 
   const { sqlParameters, sqlWhereClause } = buildWhereClause(filters)
 
@@ -81,65 +74,59 @@ export default async function getContracts(
   const isLimited = options.limit !== -1
 
   if (isLimited) {
-    count = database
+    count = (
+      database
         .prepare(
           `select count(*) as recordCount
-          from Contracts c
+            from Contracts o
-          left join BurialSites l on c.burialSiteId = l.burialSiteId
+            left join BurialSites l on o.burialSiteId = l.burialSiteId
             left join Cemeteries m on l.cemeteryId = m.cemeteryId
             ${sqlWhereClause}`
         )
-      .pluck()
+        .get(sqlParameters) as { recordCount: number }
-      .get(sqlParameters) as number
+    ).recordCount
   }
 
   let contracts: Contract[] = []
 
   if (count !== 0) {
-    const sqlLimitClause = isLimited
-      ? ` limit ${options.limit} offset ${options.offset}`
-      : ''
-
     contracts = database
       .prepare(
-        `select c.contractId,
+        `select o.contractId,
-            c.contractTypeId, t.contractType, t.isPreneed,
+          o.contractTypeId, t.contractType, t.isPreneed,
-            c.burialSiteId, lt.burialSiteType, l.burialSiteName,
+          o.burialSiteId, lt.burialSiteType, l.burialSiteName,
-            case when l.recordDelete_timeMillis is null then 1 else 0 end as burialSiteIsActive,
           l.cemeteryId, m.cemeteryName,
-            c.contractStartDate, userFn_dateIntegerToString(c.contractStartDate) as contractStartDateString,
+          o.contractStartDate, userFn_dateIntegerToString(o.contractStartDate) as contractStartDateString,
-            c.contractEndDate, userFn_dateIntegerToString(c.contractEndDate) as contractEndDateString,
+          o.contractEndDate, userFn_dateIntegerToString(o.contractEndDate) as contractEndDateString,
-            c.purchaserName, c.purchaserAddress1, c.purchaserAddress2,
+          o.purchaserName, o.purchaserAddress1, o.purchaserAddress2,
-            c.purchaserCity, c.purchaserProvince, c.purchaserPostalCode,
+          o.purchaserCity, o.purchaserProvince, o.purchaserPostalCode,
-            c.purchaserPhoneNumber, c.purchaserEmail, c.purchaserRelationship,
+          o.purchaserPhoneNumber, o.purchaserEmail, o.purchaserRelationship,
-            c.funeralHomeId, c.funeralDirectorName, f.funeralHomeName,
+          o.funeralHomeId, o.funeralDirectorName, f.funeralHomeName,
 
-            c.funeralDate, userFn_dateIntegerToString(c.funeralDate) as funeralDateString,
+          o.funeralDate, userFn_dateIntegerToString(o.funeralDate) as funeralDateString,
-            c.funeralTime,
+          o.funeralTime,
-            userFn_timeIntegerToString(c.funeralTime) as funeralTimeString,
+          userFn_timeIntegerToString(o.funeralTime) as funeralTimeString,
-            userFn_timeIntegerToPeriodString(c.funeralTime) as funeralTimePeriodString,
+          userFn_timeIntegerToPeriodString(o.funeralTime) as funeralTimePeriodString,
-            c.directionOfArrival,
+          o.directionOfArrival,
-            c.committalTypeId, cm.committalType
+          o.committalTypeId, c.committalType
-          from Contracts c
+          from Contracts o
-          left join ContractTypes t on c.contractTypeId = t.contractTypeId
+          left join ContractTypes t on o.contractTypeId = t.contractTypeId
-          left join CommittalTypes cm on c.committalTypeId = cm.committalTypeId
+          left join CommittalTypes c on o.committalTypeId = c.committalTypeId
-          left join BurialSites l on c.burialSiteId = l.burialSiteId
+          left join BurialSites l on o.burialSiteId = l.burialSiteId
           left join BurialSiteTypes lt on l.burialSiteTypeId = lt.burialSiteTypeId
           left join Cemeteries m on l.cemeteryId = m.cemeteryId
-          left join FuneralHomes f on c.funeralHomeId = f.funeralHomeId
+          left join FuneralHomes f on o.funeralHomeId = f.funeralHomeId
           ${sqlWhereClause}
-          ${
+          order by o.contractStartDate desc, ifnull(o.contractEndDate, 99999999) desc,
-            options.orderBy !== undefined && options.orderBy !== ''
-              ? ` order by ${options.orderBy}`
-              : ` order by c.contractStartDate desc, ifnull(c.contractEndDate, 99999999) desc,
             l.burialSiteNameSegment1,
             l.burialSiteNameSegment2,
             l.burialSiteNameSegment3,
             l.burialSiteNameSegment4,
             l.burialSiteNameSegment5,
-                  c.burialSiteId, c.contractId desc`
+            o.burialSiteId, o.contractId desc
-          }
+          ${
-          ${sqlLimitClause}`
+            isLimited ? ` limit ${options.limit} offset ${options.offset}` : ''
+          }`
       )
       .all(sqlParameters) as Contract[]
 
@@ -204,11 +191,11 @@ function buildWhereClause(filters: GetContractsFilters): {
   sqlParameters: unknown[]
   sqlWhereClause: string
 } {
-  let sqlWhereClause = ' where c.recordDelete_timeMillis is null'
+  let sqlWhereClause = ' where o.recordDelete_timeMillis is null'
   const sqlParameters: unknown[] = []
 
   if ((filters.burialSiteId ?? '') !== '') {
-    sqlWhereClause += ' and c.burialSiteId = ?'
+    sqlWhereClause += ' and o.burialSiteId = ?'
     sqlParameters.push(filters.burialSiteId)
   }
 
@@ -222,30 +209,30 @@ function buildWhereClause(filters: GetContractsFilters): {
 
   const deceasedNameFilters = getDeceasedNameWhereClause(
     filters.deceasedName,
-    'c'
+    'o'
   )
   if (deceasedNameFilters.sqlParameters.length > 0) {
-    sqlWhereClause += ` and c.contractId in (
+    sqlWhereClause += ` and o.contractId in (
-        select contractId from ContractInterments c
+        select contractId from ContractInterments o
         where recordDelete_timeMillis is null
         ${deceasedNameFilters.sqlWhereClause})`
     sqlParameters.push(...deceasedNameFilters.sqlParameters)
   }
 
   if ((filters.contractTypeId ?? '') !== '') {
-    sqlWhereClause += ' and c.contractTypeId = ?'
+    sqlWhereClause += ' and o.contractTypeId = ?'
     sqlParameters.push(filters.contractTypeId)
   }
 
   const contractTimeFilters = getContractTimeWhereClause(
     filters.contractTime ?? '',
-    'c'
+    'o'
   )
   sqlWhereClause += contractTimeFilters.sqlWhereClause
   sqlParameters.push(...contractTimeFilters.sqlParameters)
 
   if ((filters.contractStartDateString ?? '') !== '') {
-    sqlWhereClause += ' and c.contractStartDate = ?'
+    sqlWhereClause += ' and o.contractStartDate = ?'
     sqlParameters.push(
       dateStringToInteger(filters.contractStartDateString as DateString)
     )
@@ -253,8 +240,8 @@ function buildWhereClause(filters: GetContractsFilters): {
 
   if ((filters.contractEffectiveDateString ?? '') !== '') {
     sqlWhereClause += ` and (
-        c.contractEndDate is null
+        o.contractEndDate is null
-        or (c.contractStartDate <= ? and c.contractEndDate >= ?)
+        or (o.contractStartDate <= ? and o.contractEndDate >= ?)
       )`
     sqlParameters.push(
       dateStringToInteger(filters.contractEffectiveDateString as DateString),
@@ -273,24 +260,19 @@ function buildWhereClause(filters: GetContractsFilters): {
   }
 
   if ((filters.funeralHomeId ?? '') !== '') {
-    sqlWhereClause += ' and c.funeralHomeId = ?'
+    sqlWhereClause += ' and o.funeralHomeId = ?'
     sqlParameters.push(filters.funeralHomeId)
   }
 
-  if ((filters.funeralTime ?? '') === 'upcoming') {
-    sqlWhereClause += ' and c.funeralDate >= ?'
-    sqlParameters.push(dateToInteger(new Date()))
-  }
-
   if ((filters.workOrderId ?? '') !== '') {
     sqlWhereClause +=
-      ' and c.contractId in (select contractId from WorkOrderContracts where recordDelete_timeMillis is null and workOrderId = ?)'
+      ' and o.contractId in (select contractId from WorkOrderContracts where recordDelete_timeMillis is null and workOrderId = ?)'
     sqlParameters.push(filters.workOrderId)
   }
 
   if ((filters.notWorkOrderId ?? '') !== '') {
     sqlWhereClause +=
-      ' and c.contractId not in (select contractId from WorkOrderContracts where recordDelete_timeMillis is null and workOrderId = ?)'
+      ' and o.contractId not in (select contractId from WorkOrderContracts where recordDelete_timeMillis is null and workOrderId = ?)'
     sqlParameters.push(filters.notWorkOrderId)
   }
 

database/getNextWorkOrderNumber.js

@@ -5,7 +5,6 @@ export default function getNextWorkOrderNumber(connectedDatabase) {
     const database = connectedDatabase ?? sqlite(sunriseDB, { readonly: true });
     const paddingLength = getConfigProperty('settings.workOrders.workOrderNumberLength');
     const currentYearString = new Date().getFullYear().toString();
-    // eslint-disable-next-line security/detect-non-literal-regexp
     const regex = new RegExp(`^${currentYearString}-\\d+$`);
     database.function(
     // eslint-disable-next-line no-secrets/no-secrets

database/getNextWorkOrderNumber.ts

@@ -13,7 +13,6 @@ export default function getNextWorkOrderNumber(
   )
   const currentYearString = new Date().getFullYear().toString()
 
-  // eslint-disable-next-line security/detect-non-literal-regexp
   const regex = new RegExp(`^${currentYearString}-\\d+$`)
 
   database.function(

database/getWorkOrderMilestoneTypes.d.ts

@@ -1,2 +1,2 @@
 import type { WorkOrderMilestoneType } from '../types/record.types.js';
-export default function getWorkOrderMilestoneTypes(includeDeleted?: boolean): WorkOrderMilestoneType[];
+export default function getWorkOrderMilestoneTypes(): WorkOrderMilestoneType[];

database/getWorkOrderMilestoneTypes.js

@@ -1,16 +1,14 @@
 import sqlite from 'better-sqlite3';
 import { sunriseDB } from '../helpers/database.helpers.js';
 import { updateRecordOrderNumber } from './updateRecordOrderNumber.js';
-export default function getWorkOrderMilestoneTypes(includeDeleted = false) {
+export default function getWorkOrderMilestoneTypes() {
     const database = sqlite(sunriseDB);
-    const updateOrderNumbers = !includeDeleted;
     const workOrderMilestoneTypes = database
         .prepare(`select workOrderMilestoneTypeId, workOrderMilestoneType, orderNumber
         from WorkOrderMilestoneTypes
-        ${includeDeleted ? '' : ' where recordDelete_timeMillis is null '}
+        where recordDelete_timeMillis is null
         order by orderNumber, workOrderMilestoneType`)
         .all();
-    if (updateOrderNumbers) {
     let expectedOrderNumber = 0;
     for (const workOrderMilestoneType of workOrderMilestoneTypes) {
         if (workOrderMilestoneType.orderNumber !== expectedOrderNumber) {
@@ -19,7 +17,6 @@ export default function getWorkOrderMilestoneTypes(includeDeleted = false) {
         }
         expectedOrderNumber += 1;
     }
-    }
     database.close();
     return workOrderMilestoneTypes;
 }

database/getWorkOrderMilestoneTypes.ts

@@ -5,23 +5,18 @@ import type { WorkOrderMilestoneType } from '../types/record.types.js'
 
 import { updateRecordOrderNumber } from './updateRecordOrderNumber.js'
 
-export default function getWorkOrderMilestoneTypes(
+export default function getWorkOrderMilestoneTypes(): WorkOrderMilestoneType[] {
-  includeDeleted = false
-): WorkOrderMilestoneType[] {
   const database = sqlite(sunriseDB)
 
-  const updateOrderNumbers = !includeDeleted
-
   const workOrderMilestoneTypes = database
     .prepare(
       `select workOrderMilestoneTypeId, workOrderMilestoneType, orderNumber
         from WorkOrderMilestoneTypes
-        ${includeDeleted ? '' : ' where recordDelete_timeMillis is null '}
+        where recordDelete_timeMillis is null
         order by orderNumber, workOrderMilestoneType`
     )
     .all() as WorkOrderMilestoneType[]
 
-  if (updateOrderNumbers) {
   let expectedOrderNumber = 0
 
   for (const workOrderMilestoneType of workOrderMilestoneTypes) {
@@ -38,7 +33,6 @@ export default function getWorkOrderMilestoneTypes(
 
     expectedOrderNumber += 1
   }
-  }
 
   database.close()
 

database/getWorkOrders.js

@@ -14,8 +14,7 @@ export async function getWorkOrders(filters, options, connectedDatabase) {
         .prepare(`select count(*) as recordCount
           from WorkOrders w
           ${sqlWhereClause}`)
-        .pluck()
+        .get(sqlParameters).recordCount;
-        .get(sqlParameters);
     let workOrders = [];
     if (count > 0) {
         workOrders = database

database/getWorkOrders.ts

@@ -48,14 +48,15 @@ export async function getWorkOrders(
 
   const { sqlParameters, sqlWhereClause } = buildWhereClause(filters)
 
-  const count: number = database
+  const count: number = (
+    database
       .prepare(
         `select count(*) as recordCount
           from WorkOrders w
           ${sqlWhereClause}`
       )
-    .pluck()
+      .get(sqlParameters) as { recordCount: number }
-    .get(sqlParameters) as number
+  ).recordCount
 
   let workOrders: WorkOrder[] = []
 

database/initializeDatabase.js

@@ -1,10 +1,9 @@
 // eslint-disable-next-line @eslint-community/eslint-comments/disable-enable-pair
-/* eslint-disable @typescript-eslint/no-magic-numbers, max-lines, no-secrets/no-secrets */
+/* eslint-disable max-lines, no-secrets/no-secrets */
 import sqlite from 'better-sqlite3';
 import Debug from 'debug';
 import { DEBUG_NAMESPACE } from '../debug.config.js';
 import { sunriseDB as databasePath } from '../helpers/database.helpers.js';
-import addBurialSiteType from './addBurialSiteType.js';
 import addCommittalType from './addCommittalType.js';
 import addContractType from './addContractType.js';
 import addFeeCategory from './addFeeCategory.js';
@@ -24,8 +23,6 @@ const createStatements = [
     `create table if not exists BurialSiteTypes (
     burialSiteTypeId integer not null primary key autoincrement,
     burialSiteType varchar(100) not null,
-    bodyCapacityMax smallint,
-    crematedCapacityMax smallint,
     orderNumber smallint not null default 0,
     ${recordColumns})`,
     `create index if not exists idx_BurialSiteTypes_orderNumber
@@ -97,9 +94,6 @@ const createStatements = [
     burialSiteNameSegment5 varchar(20) not null,
     burialSiteName varchar(200) not null,
 
-    bodyCapacity smallint,
-    crematedCapacity smallint,
-
     cemeteryId integer,
     cemeterySvgId varchar(100),
     burialSiteImage varchar(100) not null default '',
@@ -418,42 +412,12 @@ export function initializeDatabase() {
 }
 function initializeData() {
     debug('Initializing data...');
-    addBurialSiteType({
+    addRecord('BurialSiteTypes', 'Casket Grave', 1, initializingUser);
-        burialSiteType: 'In-Ground Grave',
+    addRecord('BurialSiteTypes', 'Columbarium', 2, initializingUser);
-        bodyCapacityMax: 2,
+    addRecord('BurialSiteTypes', 'Mausoleum', 2, initializingUser);
-        crematedCapacityMax: 6,
+    addRecord('BurialSiteTypes', 'Niche Wall', 2, initializingUser);
-        orderNumber: 1
+    addRecord('BurialSiteTypes', 'Urn Garden', 2, initializingUser);
-    }, initializingUser);
+    addRecord('BurialSiteTypes', 'Crematorium', 2, initializingUser);
-    addBurialSiteType({
-        burialSiteType: 'Columbarium',
-        bodyCapacityMax: 0,
-        crematedCapacityMax: '',
-        orderNumber: 2
-    }, initializingUser);
-    addBurialSiteType({
-        burialSiteType: 'Mausoleum',
-        bodyCapacityMax: 2,
-        crematedCapacityMax: 0,
-        orderNumber: 2
-    }, initializingUser);
-    addBurialSiteType({
-        burialSiteType: 'Niche Wall',
-        bodyCapacityMax: 0,
-        crematedCapacityMax: 1,
-        orderNumber: 2
-    }, initializingUser);
-    addBurialSiteType({
-        burialSiteType: 'Urn Garden',
-        bodyCapacityMax: 0,
-        crematedCapacityMax: 1,
-        orderNumber: 2
-    }, initializingUser);
-    addBurialSiteType({
-        burialSiteType: 'Crematorium',
-        bodyCapacityMax: 0,
-        crematedCapacityMax: 1,
-        orderNumber: 2
-    }, initializingUser);
     addRecord('BurialSiteStatuses', 'Available', 1, initializingUser);
     addRecord('BurialSiteStatuses', 'Reserved', 2, initializingUser);
     addRecord('BurialSiteStatuses', 'Taken', 3, initializingUser);

database/initializeDatabase.ts

@@ -1,5 +1,5 @@
 // eslint-disable-next-line @eslint-community/eslint-comments/disable-enable-pair
-/* eslint-disable @typescript-eslint/no-magic-numbers, max-lines, no-secrets/no-secrets */
+/* eslint-disable max-lines, no-secrets/no-secrets */
 
 import sqlite from 'better-sqlite3'
 import Debug from 'debug'
@@ -7,7 +7,6 @@ import Debug from 'debug'
 import { DEBUG_NAMESPACE } from '../debug.config.js'
 import { sunriseDB as databasePath } from '../helpers/database.helpers.js'
 
-import addBurialSiteType from './addBurialSiteType.js'
 import addCommittalType from './addCommittalType.js'
 import addContractType from './addContractType.js'
 import addFeeCategory from './addFeeCategory.js'
@@ -31,8 +30,6 @@ const createStatements = [
   `create table if not exists BurialSiteTypes (
     burialSiteTypeId integer not null primary key autoincrement,
     burialSiteType varchar(100) not null,
-    bodyCapacityMax smallint,
-    crematedCapacityMax smallint,
     orderNumber smallint not null default 0,
     ${recordColumns})`,
 
@@ -115,9 +112,6 @@ const createStatements = [
     burialSiteNameSegment5 varchar(20) not null,
     burialSiteName varchar(200) not null,
 
-    bodyCapacity smallint,
-    crematedCapacity smallint,
-
     cemeteryId integer,
     cemeterySvgId varchar(100),
     burialSiteImage varchar(100) not null default '',
@@ -488,71 +482,12 @@ export function initializeDatabase(): boolean {
 function initializeData(): void {
   debug('Initializing data...')
 
-  addBurialSiteType(
+  addRecord('BurialSiteTypes', 'Casket Grave', 1, initializingUser)
-    {
+  addRecord('BurialSiteTypes', 'Columbarium', 2, initializingUser)
-      burialSiteType: 'In-Ground Grave',
+  addRecord('BurialSiteTypes', 'Mausoleum', 2, initializingUser)
-
+  addRecord('BurialSiteTypes', 'Niche Wall', 2, initializingUser)
-      bodyCapacityMax: 2,
+  addRecord('BurialSiteTypes', 'Urn Garden', 2, initializingUser)
-      crematedCapacityMax: 6,
+  addRecord('BurialSiteTypes', 'Crematorium', 2, initializingUser)
-      orderNumber: 1
-    },
-    initializingUser
-  )
-
-  addBurialSiteType(
-    {
-      burialSiteType: 'Columbarium',
-
-      bodyCapacityMax: 0,
-      crematedCapacityMax: '',
-      orderNumber: 2
-    },
-    initializingUser
-  )
-
-  addBurialSiteType(
-    {
-      burialSiteType: 'Mausoleum',
-
-      bodyCapacityMax: 2,
-      crematedCapacityMax: 0,
-      orderNumber: 2
-    },
-    initializingUser
-  )
-
-  addBurialSiteType(
-    {
-      burialSiteType: 'Niche Wall',
-
-      bodyCapacityMax: 0,
-      crematedCapacityMax: 1,
-      orderNumber: 2
-    },
-    initializingUser
-  )
-
-  addBurialSiteType(
-    {
-      burialSiteType: 'Urn Garden',
-
-      bodyCapacityMax: 0,
-      crematedCapacityMax: 1,
-      orderNumber: 2
-    },
-    initializingUser
-  )
-
-  addBurialSiteType(
-    {
-      burialSiteType: 'Crematorium',
-
-      bodyCapacityMax: 0,
-      crematedCapacityMax: 1,
-      orderNumber: 2
-    },
-    initializingUser
-  )
 
   addRecord('BurialSiteStatuses', 'Available', 1, initializingUser)
   addRecord('BurialSiteStatuses', 'Reserved', 2, initializingUser)

database/restoreBurialSite.d.ts

@@ -1 +0,0 @@
-export declare function restoreBurialSite(burialSiteId: number, user: User): boolean;

database/restoreBurialSite.js

@@ -1,17 +0,0 @@
-import sqlite from 'better-sqlite3';
-import { sunriseDB } from '../helpers/database.helpers.js';
-export function restoreBurialSite(burialSiteId, user) {
-    const database = sqlite(sunriseDB);
-    const rightNowMillis = Date.now();
-    const result = database
-        .prepare(`update BurialSites
-        set recordDelete_userName = null,
-          recordDelete_timeMillis = null,
-          recordUpdate_userName = ?,
-          recordUpdate_timeMillis = ?
-        where burialSiteId = ?
-          and recordDelete_timeMillis is not null`)
-        .run(user.userName, rightNowMillis, burialSiteId);
-    database.close();
-    return result.changes > 0;
-}

database/restoreBurialSite.ts

@@ -1,25 +0,0 @@
-import sqlite from 'better-sqlite3'
-
-import { sunriseDB } from '../helpers/database.helpers.js'
-
-export function restoreBurialSite(burialSiteId: number, user: User): boolean {
-  const database = sqlite(sunriseDB)
-
-  const rightNowMillis = Date.now()
-
-  const result = database
-    .prepare(
-      `update BurialSites
-        set recordDelete_userName = null,
-          recordDelete_timeMillis = null,
-          recordUpdate_userName = ?,
-          recordUpdate_timeMillis = ?
-        where burialSiteId = ?
-          and recordDelete_timeMillis is not null`
-    )
-    .run(user.userName, rightNowMillis, burialSiteId)
-
-  database.close()
-
-  return result.changes > 0
-}

database/updateBurialSite.d.ts

@@ -7,8 +7,6 @@ export interface UpdateBurialSiteForm {
     burialSiteNameSegment5?: string;
     burialSiteStatusId: number | string;
     burialSiteTypeId: number | string;
-    bodyCapacity?: number | string;
-    crematedCapacity?: number | string;
     burialSiteImage: string;
     cemeteryId: number | string;
     cemeterySvgId: string;

database/updateBurialSite.js

@@ -11,7 +11,6 @@ import getCemetery from './getCemetery.js';
  * @returns True if the burial site was updated.
  * @throws If an active burial site with the same name already exists.
  */
-// eslint-disable-next-line complexity
 export default function updateBurialSite(updateForm, user) {
     const database = sqlite(sunriseDB);
     const cemetery = updateForm.cemeteryId === ''
@@ -41,8 +40,6 @@ export default function updateBurialSite(updateForm, user) {
         burialSiteName = ?,
         burialSiteTypeId = ?,
         burialSiteStatusId = ?,
-          bodyCapacity = ?,
-          crematedCapacity = ?,
         cemeteryId = ?,
         cemeterySvgId = ?,
         burialSiteImage = ?,
@@ -54,9 +51,7 @@ export default function updateBurialSite(updateForm, user) {
         and recordDelete_timeMillis is null`)
         .run(updateForm.burialSiteNameSegment1 ?? '', updateForm.burialSiteNameSegment2 ?? '', updateForm.burialSiteNameSegment3 ?? '', updateForm.burialSiteNameSegment4 ?? '', updateForm.burialSiteNameSegment5 ?? '', burialSiteName, updateForm.burialSiteTypeId, updateForm.burialSiteStatusId === ''
         ? undefined
-        : updateForm.burialSiteStatusId, updateForm.bodyCapacity === '' ? undefined : updateForm.bodyCapacity, updateForm.crematedCapacity === ''
+        : updateForm.burialSiteStatusId, updateForm.cemeteryId === '' ? undefined : updateForm.cemeteryId, updateForm.cemeterySvgId, updateForm.burialSiteImage, updateForm.burialSiteLatitude === ''
-        ? undefined
-        : updateForm.crematedCapacity, updateForm.cemeteryId === '' ? undefined : updateForm.cemeteryId, updateForm.cemeterySvgId, updateForm.burialSiteImage, updateForm.burialSiteLatitude === ''
         ? undefined
         : updateForm.burialSiteLatitude, updateForm.burialSiteLongitude === ''
         ? undefined

database/updateBurialSite.ts

@@ -19,9 +19,6 @@ export interface UpdateBurialSiteForm {
   burialSiteStatusId: number | string
   burialSiteTypeId: number | string
 
-  bodyCapacity?: number | string
-  crematedCapacity?: number | string
-
   burialSiteImage: string
   cemeteryId: number | string
   cemeterySvgId: string
@@ -40,7 +37,6 @@ export interface UpdateBurialSiteForm {
  * @returns True if the burial site was updated.
  * @throws If an active burial site with the same name already exists.
  */
-// eslint-disable-next-line complexity
 export default function updateBurialSite(
   updateForm: UpdateBurialSiteForm,
   user: User
@@ -83,8 +79,6 @@ export default function updateBurialSite(
         burialSiteName = ?,
         burialSiteTypeId = ?,
         burialSiteStatusId = ?,
-          bodyCapacity = ?,
-          crematedCapacity = ?,
         cemeteryId = ?,
         cemeterySvgId = ?,
         burialSiteImage = ?,
@@ -106,13 +100,6 @@ export default function updateBurialSite(
       updateForm.burialSiteStatusId === ''
         ? undefined
         : updateForm.burialSiteStatusId,
-
-      updateForm.bodyCapacity === '' ? undefined : updateForm.bodyCapacity,
-
-      updateForm.crematedCapacity === ''
-        ? undefined
-        : updateForm.crematedCapacity,
-
       updateForm.cemeteryId === '' ? undefined : updateForm.cemeteryId,
       updateForm.cemeterySvgId,
       updateForm.burialSiteImage,

database/updateBurialSiteType.d.ts

@@ -1,7 +0,0 @@
-export interface UpdateBurialSiteTypeForm {
-    burialSiteTypeId: number | string;
-    burialSiteType: string;
-    bodyCapacityMax: number | string;
-    crematedCapacityMax: number | string;
-}
-export default function updateBurialSiteType(updateForm: UpdateBurialSiteTypeForm, user: User): boolean;

database/updateBurialSiteType.js

@@ -1,23 +0,0 @@
-import sqlite from 'better-sqlite3';
-import { sunriseDB } from '../helpers/database.helpers.js';
-import { clearCacheByTableName } from '../helpers/functions.cache.js';
-export default function updateBurialSiteType(updateForm, user) {
-    const database = sqlite(sunriseDB);
-    const rightNowMillis = Date.now();
-    const result = database
-        .prepare(`update BurialSiteTypes
-        set burialSiteType = ?,
-          bodyCapacityMax = ?,
-          crematedCapacityMax = ?,
-          recordUpdate_userName = ?, recordUpdate_timeMillis = ?
-        where recordDelete_timeMillis is null
-          and burialSiteTypeId = ?`)
-        .run(updateForm.burialSiteType, updateForm.bodyCapacityMax === ''
-        ? undefined
-        : updateForm.bodyCapacityMax, updateForm.crematedCapacityMax === ''
-        ? undefined
-        : updateForm.crematedCapacityMax, user.userName, rightNowMillis, updateForm.burialSiteTypeId);
-    database.close();
-    clearCacheByTableName('BurialSiteTypes');
-    return result.changes > 0;
-}

database/updateBurialSiteType.ts

@@ -1,52 +0,0 @@
-import sqlite from 'better-sqlite3'
-
-import { sunriseDB } from '../helpers/database.helpers.js'
-import { clearCacheByTableName } from '../helpers/functions.cache.js'
-
-export interface UpdateBurialSiteTypeForm {
-  burialSiteTypeId: number | string
-
-  burialSiteType: string
-
-  bodyCapacityMax: number | string
-  crematedCapacityMax: number | string
-}
-
-export default function updateBurialSiteType(
-  updateForm: UpdateBurialSiteTypeForm,
-  user: User
-): boolean {
-  const database = sqlite(sunriseDB)
-
-  const rightNowMillis = Date.now()
-
-  const result = database
-    .prepare(
-      `update BurialSiteTypes
-        set burialSiteType = ?,
-          bodyCapacityMax = ?,
-          crematedCapacityMax = ?,
-          recordUpdate_userName = ?, recordUpdate_timeMillis = ?
-        where recordDelete_timeMillis is null
-          and burialSiteTypeId = ?`
-    )
-    .run(
-      updateForm.burialSiteType,
-      updateForm.bodyCapacityMax === ''
-        ? undefined
-        : updateForm.bodyCapacityMax,
-      updateForm.crematedCapacityMax === ''
-        ? undefined
-        : updateForm.crematedCapacityMax,
-
-      user.userName,
-      rightNowMillis,
-      updateForm.burialSiteTypeId
-    )
-
-  database.close()
-
-  clearCacheByTableName('BurialSiteTypes')
-
-  return result.changes > 0
-}

database/updateBurialSiteTypeField.ts

@@ -5,7 +5,6 @@ import { clearCacheByTableName } from '../helpers/functions.cache.js'
 
 export interface UpdateBurialSiteTypeFieldForm {
   burialSiteTypeFieldId: number | string
-  
   burialSiteTypeField: string
   isRequired: '0' | '1'
   

database/updateRecord.d.ts

@@ -1,3 +1,3 @@
-type RecordTable = 'BurialSiteStatuses' | 'CommittalTypes' | 'WorkOrderMilestoneTypes' | 'WorkOrderTypes';
+type RecordTable = 'BurialSiteStatuses' | 'BurialSiteTypes' | 'CommittalTypes' | 'WorkOrderMilestoneTypes' | 'WorkOrderTypes';
 export declare function updateRecord(recordTable: RecordTable, recordId: number | string, recordName: string, user: User): boolean;
 export {};

database/updateRecord.js

@@ -3,6 +3,7 @@ import { sunriseDB } from '../helpers/database.helpers.js';
 import { clearCacheByTableName } from '../helpers/functions.cache.js';
 const recordNameIdColumns = new Map([
     ['BurialSiteStatuses', ['burialSiteStatus', 'burialSiteStatusId']],
+    ['BurialSiteTypes', ['burialSiteType', 'burialSiteTypeId']],
     ['CommittalTypes', ['committalType', 'committalTypeId']],
     [
         'WorkOrderMilestoneTypes',
@@ -12,17 +13,13 @@ const recordNameIdColumns = new Map([
 ]);
 export function updateRecord(recordTable, recordId, recordName, user) {
     const database = sqlite(sunriseDB);
-    const columnNames = recordNameIdColumns.get(recordTable);
-    if (columnNames === undefined) {
-        throw new Error(`Invalid record table: ${recordTable}`);
-    }
     const result = database
         .prepare(`update ${recordTable}
-        set ${columnNames[0]} = ?,
+        set ${recordNameIdColumns.get(recordTable)[0]} = ?,
         recordUpdate_userName = ?,
         recordUpdate_timeMillis = ?
         where recordDelete_timeMillis is null
-        and ${columnNames[1]} = ?`)
+        and ${recordNameIdColumns.get(recordTable)[1]} = ?`)
         .run(recordName, user.userName, Date.now(), recordId);
     database.close();
     clearCacheByTableName(recordTable);

database/updateRecord.ts

@@ -5,12 +5,14 @@ import { clearCacheByTableName } from '../helpers/functions.cache.js'
 
 type RecordTable =
   | 'BurialSiteStatuses'
+  | 'BurialSiteTypes'
   | 'CommittalTypes'
   | 'WorkOrderMilestoneTypes'
   | 'WorkOrderTypes'
 
 const recordNameIdColumns = new Map<RecordTable, string[]>([
   ['BurialSiteStatuses', ['burialSiteStatus', 'burialSiteStatusId']],
+  ['BurialSiteTypes', ['burialSiteType', 'burialSiteTypeId']],
   ['CommittalTypes', ['committalType', 'committalTypeId']],
   [
     'WorkOrderMilestoneTypes',
@@ -27,20 +29,14 @@ export function updateRecord(
 ): boolean {
   const database = sqlite(sunriseDB)
 
-  const columnNames = recordNameIdColumns.get(recordTable)
-
-  if (columnNames === undefined) {
-    throw new Error(`Invalid record table: ${recordTable}`)
-  }
-
   const result = database
     .prepare(
       `update ${recordTable}
-        set ${columnNames[0]} = ?,
+        set ${recordNameIdColumns.get(recordTable)![0]} = ?,
         recordUpdate_userName = ?,
         recordUpdate_timeMillis = ?
         where recordDelete_timeMillis is null
-        and ${columnNames[1]} = ?`
+        and ${recordNameIdColumns.get(recordTable)![1]} = ?`
     )
     .run(recordName, user.userName, Date.now(), recordId)
 

handlers/admin-post/doAddBurialSiteType.d.ts

@@ -1,3 +1,5 @@
 import type { Request, Response } from 'express';
-import { type AddBurialSiteTypeForm } from '../../database/addBurialSiteType.js';
+export default function handler(request: Request<unknown, unknown, {
-export default function handler(request: Request<unknown, unknown, AddBurialSiteTypeForm>, response: Response): void;
+    burialSiteType: string;
+    orderNumber?: number | string;
+}>, response: Response): void;

handlers/admin-post/doAddBurialSiteType.js

@@ -1,7 +1,7 @@
-import addBurialSiteType from '../../database/addBurialSiteType.js';
+import addRecord from '../../database/addRecord.js';
 import { getBurialSiteTypes } from '../../helpers/functions.cache.js';
 export default function handler(request, response) {
-    const burialSiteTypeId = addBurialSiteType(request.body, request.session.user);
+    const burialSiteTypeId = addRecord('BurialSiteTypes', request.body.burialSiteType, request.body.orderNumber ?? -1, request.session.user);
     const burialSiteTypes = getBurialSiteTypes();
     response.json({
         success: true,

handlers/admin-post/doAddBurialSiteType.ts

@@ -1,16 +1,20 @@
 import type { Request, Response } from 'express'
 
-import addBurialSiteType, {
+import addRecord from '../../database/addRecord.js'
-  type AddBurialSiteTypeForm
-} from '../../database/addBurialSiteType.js'
 import { getBurialSiteTypes } from '../../helpers/functions.cache.js'
 
 export default function handler(
-  request: Request<unknown, unknown, AddBurialSiteTypeForm>,
+  request: Request<
+    unknown,
+    unknown,
+    { burialSiteType: string; orderNumber?: number | string }
+  >,
   response: Response
 ): void {
-  const burialSiteTypeId = addBurialSiteType(
+  const burialSiteTypeId = addRecord(
-    request.body,
+    'BurialSiteTypes',
+    request.body.burialSiteType,
+    request.body.orderNumber ?? -1,
     request.session.user as User
   )
 

handlers/admin-post/doBackupDatabase.js

@@ -1,4 +1,4 @@
-import { backupDatabase } from '../../database/backupDatabase.js';
+import { backupDatabase } from '../../helpers/database.helpers.js';
 export default async function handler(_request, response) {
     const backupDatabasePath = await backupDatabase();
     if (typeof backupDatabasePath === 'string') {

handlers/admin-post/doBackupDatabase.ts

@@ -1,6 +1,6 @@
 import type { Request, Response } from 'express'
 
-import { backupDatabase } from '../../database/backupDatabase.js'
+import { backupDatabase } from '../../helpers/database.helpers.js'
 
 export default async function handler(
   _request: Request,
@@ -15,13 +15,11 @@ export default async function handler(
 
     response.json({
       success: true,
-
       fileName
     })
   } else {
     response.json({
       success: false,
-
       errorMessage: 'Unable to write backup file.'
     })
   }