fix undefined transaction time

alter table BurialSiteTypes
add bodyCapacityMax smallint

alter table BurialSiteTypes
add crematedCapacityMax smallint

alter table BurialSites
add bodyCapacity smallint

alter table BurialSites
add crematedCapacity smallint

_config.yml

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

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
-await initializeDatabase();
+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
-await initializeDatabase()
+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
-    await initializeDatabase();
+    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
-  await initializeDatabase()
+  initializeDatabase()
 }
 
 interface ServerError extends Error {

data/base.config.js

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

data/base.config.ts

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

data/configDefaults.d.ts

@@ -33,6 +33,8 @@ 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,6 +32,10 @@ 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,5 @@
-import type { config as MSSQLConfig } from 'mssql'
-
 import { hoursToMillis } from '@cityssm/to-millis'
+import type { config as MSSQLConfig } from 'mssql'
 
 import type {
   ConfigActiveDirectory,
@@ -52,6 +51,12 @@ 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,6 +6,8 @@ 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,6 +10,7 @@ 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();
@@ -38,17 +39,22 @@ 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.cemeteryId === '' ? undefined : burialSiteForm.cemeteryId, burialSiteForm.cemeterySvgId, burialSiteForm.burialSiteImage ?? '', burialSiteForm.burialSiteLatitude === ''
+        : burialSiteForm.burialSiteStatusId, burialSiteForm.bodyCapacity === ''
+        ? 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,6 +16,9 @@ export interface AddBurialSiteForm {
   burialSiteStatusId: number | string
   burialSiteTypeId: number | string
 
+  bodyCapacity?: number | string
+  crematedCapacity?: number | string
+
   burialSiteImage?: string
   cemeteryId: number | string
   cemeterySvgId?: string
@@ -35,6 +38,7 @@ 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
@@ -80,12 +84,13 @@ 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 (?, ?, ?, ?, ?,
           ?, ?, ?, ?, ?, ?, ?, ?, ?, ?,
           ?, ?, ?, ?)`
     )
@@ -100,6 +105,15 @@ 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

@@ -0,0 +1,7 @@
+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

@@ -0,0 +1,20 @@
+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

@@ -0,0 +1,50 @@
+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,7 +14,9 @@ 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, addForm.maxLength ?? 100, addForm.orderNumber ?? -1, user.userName, rightNowMillis, user.userName, rightNowMillis);
+        .run(addForm.burialSiteTypeId, addForm.burialSiteTypeField, addForm.fieldType ?? 'text', addForm.fieldValues ?? '', 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, rightNowMillis, user.userName, rightNowMillis);
     database.close();
     clearCacheByTableName('BurialSiteTypeFields');
     return result.lastInsertRowid;

database/addBurialSiteTypeField.ts

@@ -47,6 +47,7 @@ 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,7 +1,8 @@
+import { type DateString, type TimeString } from '@cityssm/utils-datetime';
 export interface AddTransactionForm {
     contractId: number | string;
-    transactionDateString?: string;
+    transactionDateString?: '' | DateString;
-    transactionTimeString?: string;
+    transactionTimeString?: '' | TimeString;
     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 ?? '') === ''
-        ? dateStringToInteger(contractTransactionForm.transactionDateString)
+        ? dateToInteger(rightNow)
-        : dateToInteger(rightNow);
+        : dateStringToInteger(contractTransactionForm.transactionDateString);
-    const transactionTime = contractTransactionForm.transactionTimeString
+    const transactionTime = (contractTransactionForm.transactionTimeString ?? '') === ''
-        ? timeStringToInteger(contractTransactionForm.transactionTimeString)
+        ? dateToTimeInteger(rightNow)
-        : dateToTimeInteger(rightNow);
+        : timeStringToInteger(contractTransactionForm.transactionTimeString);
     database
         .prepare(`insert into ContractTransactions (
         contractId, transactionIndex,

database/addContractTransaction.ts

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

database/addContractTypeField.js

@@ -13,7 +13,9 @@ 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, addForm.maxLength ?? 100, addForm.orderNumber ?? -1, user.userName, rightNowMillis, user.userName, rightNowMillis);
+        .run(addForm.contractTypeId ?? undefined, addForm.contractTypeField, addForm.fieldType ?? 'text', addForm.fieldValues ?? '', 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, rightNowMillis, user.userName, rightNowMillis);
     database.close();
     clearCacheByTableName('ContractTypeFields');
     return result.lastInsertRowid;

database/addContractTypeField.ts

@@ -5,9 +5,11 @@ import { clearCacheByTableName } from '../helpers/functions.cache.js'
 
 export interface AddContractTypeFieldForm {
   contractTypeId?: number | string
+
   contractTypeField: string
-  fieldValues?: string
+
   fieldType?: string
+  fieldValues?: string
   isRequired?: string
   maxLength?: number | string
   minLength?: number | string
@@ -43,6 +45,7 @@ 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,6 +6,7 @@ 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 {
-    funeralHomeName: string;
     funeralHomeKey: string;
+    funeralHomeName: 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 {
-  funeralHomeName: string
   funeralHomeKey: string
+  funeralHomeName: string
 
   funeralHomeAddress1: string
   funeralHomeAddress2: string

database/addRecord.d.ts

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

database/addRecord.js

@@ -3,7 +3,6 @@ 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,13 +5,11 @@ 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 {
-    workOrderTypeId: number | string;
-    workOrderNumber?: string;
     workOrderDescription: string;
-    workOrderOpenDateString?: string;
+    workOrderNumber?: string;
+    workOrderTypeId: number | string;
     workOrderCloseDateString?: string;
+    workOrderOpenDateString?: 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,11 +11,14 @@ import addWorkOrderContract from './addWorkOrderContract.js'
 import getNextWorkOrderNumber from './getNextWorkOrderNumber.js'
 
 export interface AddWorkOrderForm {
-  workOrderTypeId: number | string
-  workOrderNumber?: string
   workOrderDescription: string
-  workOrderOpenDateString?: string
+  workOrderNumber?: string
+
+  workOrderTypeId: number | string
+
   workOrderCloseDateString?: string
+  workOrderOpenDateString?: string
+  
   contractId?: string
 }
 
@@ -67,8 +70,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 {
-    workOrderId: number | string;
     burialSiteId: number | string;
+    workOrderId: 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 {
-  workOrderId: number | string
   burialSiteId: number | string
+  workOrderId: number | string
 }
 
 export default function addWorkOrderBurialSite(

database/addWorkOrderComment.d.ts

@@ -1,5 +1,5 @@
 export interface AddWorkOrderCommentForm {
-    workOrderId: string;
     comment: string;
+    workOrderId: 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 {
-  workOrderId: string
   comment: string
+  workOrderId: string
 }
 
 export default function addWorkOrderComment(

database/addWorkOrderMilestone.d.ts

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

database/addWorkOrderMilestone.ts

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

database/backupDatabase.d.ts

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

database/backupDatabase.js

@@ -0,0 +1,28 @@
+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

@@ -0,0 +1,37 @@
+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 financials, do not delete related.
+     * - Maintain financial data, 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 financials, do not delete related.
+   * - Maintain financial data, do not delete related.
    */
 
   purgedRecordCount += database

database/closeWorkOrder.d.ts

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

database/closeWorkOrder.ts

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

database/completeWorkOrderMilestone.d.ts

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

database/completeWorkOrderMilestone.js

@@ -4,6 +4,12 @@ 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 = ?,
@@ -11,11 +17,7 @@ export default function completeWorkOrderMilestone(milestoneForm, user) {
         recordUpdate_userName = ?,
         recordUpdate_timeMillis = ?
         where workOrderMilestoneId = ?`)
-        .run((milestoneForm.workOrderMilestoneCompletionDateString ?? '') === ''
+        .run(completionDate, completionTime, user.userName, rightNow.getTime(), milestoneForm.workOrderMilestoneId);
-        ? 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,4 +1,6 @@
 import {
+  type DateString,
+  type TimeString,
   dateStringToInteger,
   dateToInteger,
   dateToTimeInteger,
@@ -11,8 +13,8 @@ import { sunriseDB } from '../helpers/database.helpers.js'
 export interface CompleteWorkOrderMilestoneForm {
   workOrderMilestoneId: number | string
 
-  workOrderMilestoneCompletionDateString?: string
+  workOrderMilestoneCompletionDateString?: '' | DateString
-  workOrderMilestoneCompletionTimeString?: string
+  workOrderMilestoneCompletionTimeString?: '' | TimeString
 }
 
 export default function completeWorkOrderMilestone(
@@ -23,6 +25,20 @@ 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
@@ -33,16 +49,8 @@ export default function completeWorkOrderMilestone(
         where workOrderMilestoneId = ?`
     )
     .run(
-      (milestoneForm.workOrderMilestoneCompletionDateString ?? '') === ''
+      completionDate,
-        ? dateToInteger(rightNow)
+      completionTime,
-        : dateStringToInteger(
-            milestoneForm.workOrderMilestoneCompletionDateString!
-          ),
-      (milestoneForm.workOrderMilestoneCompletionTimeString ?? '') === ''
-        ? dateToTimeInteger(rightNow)
-        : timeStringToInteger(
-            milestoneForm.workOrderMilestoneCompletionTimeString!
-          ),
       user.userName,
       rightNow.getTime(),
       milestoneForm.workOrderMilestoneId

database/copyContract.ts

@@ -29,6 +29,7 @@ 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,23 +30,6 @@ 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,30 +44,6 @@ 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 { clearCacheByTableName } from '../helpers/functions.cache.js';
+import { cacheTableNames, clearCacheByTableName } from '../helpers/functions.cache.js';
 const recordIdColumns = new Map([
     ['BurialSiteComments', 'burialSiteCommentId'],
     ['BurialSiteStatuses', 'burialSiteStatusId'],
@@ -55,6 +55,9 @@ 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,7 +1,11 @@
 import sqlite from 'better-sqlite3'
 
 import { sunriseDB } from '../helpers/database.helpers.js'
-import { clearCacheByTableName } from '../helpers/functions.cache.js'
+import {
+  CacheTableNames,
+  cacheTableNames,
+  clearCacheByTableName
+} from '../helpers/functions.cache.js'
 
 type RecordTable =
   | 'BurialSiteComments'
@@ -73,7 +77,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)
@@ -84,7 +88,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)
@@ -92,7 +96,10 @@ export function deleteRecord(
 
   database.close()
 
-  clearCacheByTableName(recordTable)
+  // Clear cache for tables that are cached
+  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): Promise<BurialSite | undefined>;
+export default function getBurialSite(burialSiteId: number | string, includeDeleted?: boolean): Promise<BurialSite | undefined>;
-export declare function getBurialSiteByBurialSiteName(burialSiteName: string): Promise<BurialSite | undefined>;
+export declare function getBurialSiteByBurialSiteName(burialSiteName: string, includeDeleted?: boolean): Promise<BurialSite | undefined>;

database/getBurialSite.js

@@ -13,25 +13,31 @@ 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, includeDeleted = false) {
-export default async function getBurialSite(burialSiteId) {
+    return await _getBurialSite(`l.burialSiteId = ?`, burialSiteId, includeDeleted);
-    return await _getBurialSite(`${baseSQL} and l.burialSiteId = ?`, burialSiteId);
 }
-export async function getBurialSiteByBurialSiteName(burialSiteName) {
+export async function getBurialSiteByBurialSiteName(burialSiteName, includeDeleted = false) {
-    return await _getBurialSite(`${baseSQL} and l.burialSiteName = ?`, burialSiteName);
+    return await _getBurialSite(`l.burialSiteName = ?`, burialSiteName, includeDeleted);
 }
-async function _getBurialSite(sql, burialSiteIdOrLotName) {
+async function _getBurialSite(whereClausePiece, burialSiteIdOrLotName, includeDeleted = false) {
     const database = sqlite(sunriseDB, { readonly: true });
-    const burialSite = database.prepare(sql).get(burialSiteIdOrLotName);
+    const burialSite = database
+        .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,41 +17,55 @@ 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(`${baseSQL} and l.burialSiteId = ?`, burialSiteId)
+  return await _getBurialSite(
+    `l.burialSiteId = ?`,
+    burialSiteId,
+    includeDeleted
+  )
 }
 
 export async function getBurialSiteByBurialSiteName(
-  burialSiteName: string
+  burialSiteName: string,
+  includeDeleted = false
 ): Promise<BurialSite | undefined> {
   return await _getBurialSite(
-    `${baseSQL} and l.burialSiteName = ?`,
+    `l.burialSiteName = ?`,
-    burialSiteName
+    burialSiteName,
+    includeDeleted
   )
 }
 
 async function _getBurialSite(
-  sql: string,
+  whereClausePiece: string,
-  burialSiteIdOrLotName: number | string
+  burialSiteIdOrLotName: number | string,
+  includeDeleted = false
 ): Promise<BurialSite | undefined> {
   const database = sqlite(sunriseDB, { readonly: true })
 
-  const burialSite = database.prepare(sql).get(burialSiteIdOrLotName) as
+  const burialSite = database
-    | BurialSite
+    .prepare(
-    | undefined
+      `${baseSQL} ${includeDeleted ? 'where' : 'where l.recordDelete_timeMillis is null and'} ${whereClausePiece}`
+    )
+    .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(): BurialSiteStatus[];
+export default function getBurialSiteStatuses(includeDeleted?: boolean): BurialSiteStatus[];

database/getBurialSiteStatuses.js

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

database/getBurialSiteStatuses.ts

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

database/getBurialSiteTypes.js

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

database/getBurialSiteTypes.ts

@@ -11,11 +11,13 @@ export default function getBurialSiteTypes(
 ): BurialSiteType[] {
   const database = sqlite(sunriseDB)
 
-  const updateOrderNumbers = !database.readonly && !includeDeleted
+  const updateOrderNumbers = !includeDeleted
 
   const burialSiteTypes = database
     .prepare(
-      `select burialSiteTypeId, burialSiteType, orderNumber
+      `select burialSiteTypeId, burialSiteType,
+        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;
-    burialSiteTypeId?: number | string;
     burialSiteStatusId?: number | string;
+    burialSiteTypeId?: 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): {
-    count: number;
     burialSites: BurialSite[];
+    count: number;
 };

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 { sqlWhereClause, sqlParameters } = buildWhereClause(filters);
+    const { sqlParameters, sqlWhereClause } = buildWhereClause(filters);
     const currentDate = dateToInteger(new Date());
     let count = 0;
     if (options.limit !== -1) {
@@ -20,7 +20,8 @@ export default function getBurialSites(filters, options, connectedDatabase) {
             group by burialSiteId
           ) o on l.burialSiteId = o.burialSiteId
           ${sqlWhereClause}`)
-            .get(sqlParameters).recordCount;
+            .pluck()
+            .get(sqlParameters);
     }
     let burialSites = [];
     if (options.limit === -1 || count > 0) {
@@ -37,6 +38,7 @@ 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
@@ -70,8 +72,8 @@ export default function getBurialSites(filters, options, connectedDatabase) {
         database.close();
     }
     return {
-        count,
+        burialSites,
-        burialSites
+        count
     };
 }
 function buildWhereClause(filters) {

database/getBurialSites.ts

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

database/getContract.js

@@ -16,6 +16,7 @@ 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,6 +33,7 @@ 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(): ContractType[];
+export default function getContractTypes(includeDeleted?: boolean): ContractType[];

database/getContractTypes.js

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

database/getContractTypes.ts

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

database/getContracts.d.ts

@@ -6,13 +6,14 @@ export interface GetContractsFilters {
     contractEffectiveDateString?: string;
     contractStartDateString?: DateString;
     contractTime?: '' | 'current' | 'future' | 'past';
-    deceasedName?: string;
-    contractTypeId?: number | string;
     cemeteryId?: number | string;
+    contractTypeId?: number | string;
+    deceasedName?: string;
     burialSiteName?: string;
     burialSiteNameSearchType?: '' | 'endsWith' | 'startsWith';
     burialSiteTypeId?: number | string;
     funeralHomeId?: number | string;
+    funeralTime?: '' | 'upcoming';
     notWorkOrderId?: number | string;
     workOrderId?: number | string;
 }
@@ -20,6 +21,7 @@ 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, timeIntegerToPeriodString, timeIntegerToString } from '@cityssm/utils-datetime';
+import { dateIntegerToString, dateStringToInteger, dateToInteger, 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,48 +20,55 @@ export default async function getContracts(filters, options, connectedDatabase)
     if (isLimited) {
         count = database
             .prepare(`select count(*) as recordCount
-            from Contracts o
+          from Contracts c
-            left join BurialSites l on o.burialSiteId = l.burialSiteId
+          left join BurialSites l on c.burialSiteId = l.burialSiteId
           left join Cemeteries m on l.cemeteryId = m.cemeteryId
           ${sqlWhereClause}`)
-            .get(sqlParameters).recordCount;
+            .pluck()
+            .get(sqlParameters);
     }
     let contracts = [];
     if (count !== 0) {
+        const sqlLimitClause = isLimited
+            ? ` limit ${options.limit} offset ${options.offset}`
+            : '';
         contracts = database
-            .prepare(`select o.contractId,
+            .prepare(`select c.contractId,
-          o.contractTypeId, t.contractType, t.isPreneed,
+            c.contractTypeId, t.contractType, t.isPreneed,
-          o.burialSiteId, lt.burialSiteType, l.burialSiteName,
+            c.burialSiteId, lt.burialSiteType, l.burialSiteName,
+            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,
+            c.contractStartDate, userFn_dateIntegerToString(c.contractStartDate) as contractStartDateString,
-          o.contractEndDate, userFn_dateIntegerToString(o.contractEndDate) as contractEndDateString,
+            c.contractEndDate, userFn_dateIntegerToString(c.contractEndDate) as contractEndDateString,
-          o.purchaserName, o.purchaserAddress1, o.purchaserAddress2,
+            c.purchaserName, c.purchaserAddress1, c.purchaserAddress2,
-          o.purchaserCity, o.purchaserProvince, o.purchaserPostalCode,
+            c.purchaserCity, c.purchaserProvince, c.purchaserPostalCode,
-          o.purchaserPhoneNumber, o.purchaserEmail, o.purchaserRelationship,
+            c.purchaserPhoneNumber, c.purchaserEmail, c.purchaserRelationship,
-          o.funeralHomeId, o.funeralDirectorName, f.funeralHomeName,
+            c.funeralHomeId, c.funeralDirectorName, f.funeralHomeName,
 
-          o.funeralDate, userFn_dateIntegerToString(o.funeralDate) as funeralDateString,
+            c.funeralDate, userFn_dateIntegerToString(c.funeralDate) as funeralDateString,
-          o.funeralTime,
+            c.funeralTime,
-          userFn_timeIntegerToString(o.funeralTime) as funeralTimeString,
+            userFn_timeIntegerToString(c.funeralTime) as funeralTimeString,
-          userFn_timeIntegerToPeriodString(o.funeralTime) as funeralTimePeriodString,
+            userFn_timeIntegerToPeriodString(c.funeralTime) as funeralTimePeriodString,
-          o.directionOfArrival,
+            c.directionOfArrival,
-          o.committalTypeId, c.committalType
+            c.committalTypeId, cm.committalType
-          from Contracts o
+          from Contracts c
-          left join ContractTypes t on o.contractTypeId = t.contractTypeId
+          left join ContractTypes t on c.contractTypeId = t.contractTypeId
-          left join CommittalTypes c on o.committalTypeId = c.committalTypeId
+          left join CommittalTypes cm on c.committalTypeId = cm.committalTypeId
-          left join BurialSites l on o.burialSiteId = l.burialSiteId
+          left join BurialSites l on c.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 o.funeralHomeId = f.funeralHomeId
+          left join FuneralHomes f on c.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,
-            o.burialSiteId, o.contractId desc
+                  c.burialSiteId, c.contractId desc`}
-          ${isLimited ? ` limit ${options.limit} offset ${options.offset}` : ''}`)
+          ${sqlLimitClause}`)
             .all(sqlParameters);
         if (!isLimited) {
             count = contracts.length;
@@ -98,38 +105,38 @@ async function addInclusions(contract, options, database) {
 }
 // eslint-disable-next-line complexity
 function buildWhereClause(filters) {
-    let sqlWhereClause = ' where o.recordDelete_timeMillis is null';
+    let sqlWhereClause = ' where c.recordDelete_timeMillis is null';
     const sqlParameters = [];
     if ((filters.burialSiteId ?? '') !== '') {
-        sqlWhereClause += ' and o.burialSiteId = ?';
+        sqlWhereClause += ' and c.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, 'o');
+    const deceasedNameFilters = getDeceasedNameWhereClause(filters.deceasedName, 'c');
     if (deceasedNameFilters.sqlParameters.length > 0) {
-        sqlWhereClause += ` and o.contractId in (
+        sqlWhereClause += ` and c.contractId in (
-        select contractId from ContractInterments o
+        select contractId from ContractInterments c
         where recordDelete_timeMillis is null
         ${deceasedNameFilters.sqlWhereClause})`;
         sqlParameters.push(...deceasedNameFilters.sqlParameters);
     }
     if ((filters.contractTypeId ?? '') !== '') {
-        sqlWhereClause += ' and o.contractTypeId = ?';
+        sqlWhereClause += ' and c.contractTypeId = ?';
         sqlParameters.push(filters.contractTypeId);
     }
-    const contractTimeFilters = getContractTimeWhereClause(filters.contractTime ?? '', 'o');
+    const contractTimeFilters = getContractTimeWhereClause(filters.contractTime ?? '', 'c');
     sqlWhereClause += contractTimeFilters.sqlWhereClause;
     sqlParameters.push(...contractTimeFilters.sqlParameters);
     if ((filters.contractStartDateString ?? '') !== '') {
-        sqlWhereClause += ' and o.contractStartDate = ?';
+        sqlWhereClause += ' and c.contractStartDate = ?';
         sqlParameters.push(dateStringToInteger(filters.contractStartDateString));
     }
     if ((filters.contractEffectiveDateString ?? '') !== '') {
         sqlWhereClause += ` and (
-        o.contractEndDate is null
+        c.contractEndDate is null
-        or (o.contractStartDate <= ? and o.contractEndDate >= ?)
+        or (c.contractStartDate <= ? and c.contractEndDate >= ?)
       )`;
         sqlParameters.push(dateStringToInteger(filters.contractEffectiveDateString), dateStringToInteger(filters.contractEffectiveDateString));
     }
@@ -142,17 +149,21 @@ function buildWhereClause(filters) {
         sqlParameters.push(filters.burialSiteTypeId);
     }
     if ((filters.funeralHomeId ?? '') !== '') {
-        sqlWhereClause += ' and o.funeralHomeId = ?';
+        sqlWhereClause += ' and c.funeralHomeId = ?';
         sqlParameters.push(filters.funeralHomeId);
     }
+    if ((filters.funeralTime ?? '') === 'upcoming') {
+        sqlWhereClause += ' and c.funeralDate >= ?';
+        sqlParameters.push(dateToInteger(new Date()));
+    }
     if ((filters.workOrderId ?? '') !== '') {
         sqlWhereClause +=
-            ' and o.contractId in (select contractId from WorkOrderContracts where recordDelete_timeMillis is null and workOrderId = ?)';
+            ' and c.contractId in (select contractId from WorkOrderContracts where recordDelete_timeMillis is null and workOrderId = ?)';
         sqlParameters.push(filters.workOrderId);
     }
     if ((filters.notWorkOrderId ?? '') !== '') {
         sqlWhereClause +=
-            ' and o.contractId not in (select contractId from WorkOrderContracts where recordDelete_timeMillis is null and workOrderId = ?)';
+            ' and c.contractId not in (select contractId from WorkOrderContracts where recordDelete_timeMillis is null and workOrderId = ?)';
         sqlParameters.push(filters.notWorkOrderId);
     }
     return {

database/getContracts.ts

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

database/getNextWorkOrderNumber.js

@@ -5,6 +5,7 @@ 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,6 +13,7 @@ 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(): WorkOrderMilestoneType[];
+export default function getWorkOrderMilestoneTypes(includeDeleted?: boolean): WorkOrderMilestoneType[];

database/getWorkOrderMilestoneTypes.js

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

database/getWorkOrderMilestoneTypes.ts

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

database/getWorkOrders.js

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

database/getWorkOrders.ts

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

database/initializeDatabase.js

@@ -1,9 +1,10 @@
 // eslint-disable-next-line @eslint-community/eslint-comments/disable-enable-pair
-/* eslint-disable max-lines, no-secrets/no-secrets */
+/* eslint-disable @typescript-eslint/no-magic-numbers, 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';
@@ -23,6 +24,8 @@ 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
@@ -94,6 +97,9 @@ 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 '',
@@ -412,12 +418,42 @@ export function initializeDatabase() {
 }
 function initializeData() {
     debug('Initializing data...');
-    addRecord('BurialSiteTypes', 'Casket Grave', 1, initializingUser);
+    addBurialSiteType({
-    addRecord('BurialSiteTypes', 'Columbarium', 2, initializingUser);
+        burialSiteType: 'In-Ground Grave',
-    addRecord('BurialSiteTypes', 'Mausoleum', 2, initializingUser);
+        bodyCapacityMax: 2,
-    addRecord('BurialSiteTypes', 'Niche Wall', 2, initializingUser);
+        crematedCapacityMax: 6,
-    addRecord('BurialSiteTypes', 'Urn Garden', 2, initializingUser);
+        orderNumber: 1
-    addRecord('BurialSiteTypes', 'Crematorium', 2, initializingUser);
+    }, 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 max-lines, no-secrets/no-secrets */
+/* eslint-disable @typescript-eslint/no-magic-numbers, max-lines, no-secrets/no-secrets */
 
 import sqlite from 'better-sqlite3'
 import Debug from 'debug'
@@ -7,6 +7,7 @@ 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'
@@ -30,6 +31,8 @@ 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})`,
 
@@ -112,6 +115,9 @@ 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 '',
@@ -482,12 +488,71 @@ export function initializeDatabase(): boolean {
 function initializeData(): void {
   debug('Initializing data...')
 
-  addRecord('BurialSiteTypes', 'Casket Grave', 1, initializingUser)
+  addBurialSiteType(
-  addRecord('BurialSiteTypes', 'Columbarium', 2, initializingUser)
+    {
-  addRecord('BurialSiteTypes', 'Mausoleum', 2, initializingUser)
+      burialSiteType: 'In-Ground Grave',
-  addRecord('BurialSiteTypes', 'Niche Wall', 2, initializingUser)
+
-  addRecord('BurialSiteTypes', 'Urn Garden', 2, initializingUser)
+      bodyCapacityMax: 2,
-  addRecord('BurialSiteTypes', 'Crematorium', 2, initializingUser)
+      crematedCapacityMax: 6,
+      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

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

database/restoreBurialSite.js

@@ -0,0 +1,17 @@
+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

@@ -0,0 +1,25 @@
+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,6 +7,8 @@ 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,6 +11,7 @@ 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 === ''
@@ -40,6 +41,8 @@ export default function updateBurialSite(updateForm, user) {
           burialSiteName = ?,
           burialSiteTypeId = ?,
           burialSiteStatusId = ?,
+          bodyCapacity = ?,
+          crematedCapacity = ?,
           cemeteryId = ?,
           cemeterySvgId = ?,
           burialSiteImage = ?,
@@ -51,7 +54,9 @@ 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.cemeteryId === '' ? undefined : updateForm.cemeteryId, updateForm.cemeterySvgId, updateForm.burialSiteImage, updateForm.burialSiteLatitude === ''
+        : updateForm.burialSiteStatusId, updateForm.bodyCapacity === '' ? undefined : updateForm.bodyCapacity, updateForm.crematedCapacity === ''
+        ? undefined
+        : updateForm.crematedCapacity, updateForm.cemeteryId === '' ? undefined : updateForm.cemeteryId, updateForm.cemeterySvgId, updateForm.burialSiteImage, updateForm.burialSiteLatitude === ''
         ? undefined
         : updateForm.burialSiteLatitude, updateForm.burialSiteLongitude === ''
         ? undefined

database/updateBurialSite.ts

@@ -19,6 +19,9 @@ export interface UpdateBurialSiteForm {
   burialSiteStatusId: number | string
   burialSiteTypeId: number | string
 
+  bodyCapacity?: number | string
+  crematedCapacity?: number | string
+
   burialSiteImage: string
   cemeteryId: number | string
   cemeterySvgId: string
@@ -37,6 +40,7 @@ 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
@@ -79,6 +83,8 @@ export default function updateBurialSite(
           burialSiteName = ?,
           burialSiteTypeId = ?,
           burialSiteStatusId = ?,
+          bodyCapacity = ?,
+          crematedCapacity = ?,
           cemeteryId = ?,
           cemeterySvgId = ?,
           burialSiteImage = ?,
@@ -100,6 +106,13 @@ 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

@@ -0,0 +1,7 @@
+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

@@ -0,0 +1,23 @@
+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

@@ -0,0 +1,52 @@
+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,6 +5,7 @@ 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' | 'BurialSiteTypes' | 'CommittalTypes' | 'WorkOrderMilestoneTypes' | 'WorkOrderTypes';
+type RecordTable = 'BurialSiteStatuses' | 'CommittalTypes' | 'WorkOrderMilestoneTypes' | 'WorkOrderTypes';
 export declare function updateRecord(recordTable: RecordTable, recordId: number | string, recordName: string, user: User): boolean;
 export {};

database/updateRecord.js

@@ -3,7 +3,6 @@ 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',
@@ -13,13 +12,17 @@ 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 ${recordNameIdColumns.get(recordTable)[0]} = ?,
+        set ${columnNames[0]} = ?,
         recordUpdate_userName = ?,
         recordUpdate_timeMillis = ?
         where recordDelete_timeMillis is null
-        and ${recordNameIdColumns.get(recordTable)[1]} = ?`)
+        and ${columnNames[1]} = ?`)
         .run(recordName, user.userName, Date.now(), recordId);
     database.close();
     clearCacheByTableName(recordTable);

database/updateRecord.ts

@@ -5,14 +5,12 @@ 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',
@@ -29,14 +27,20 @@ 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 ${recordNameIdColumns.get(recordTable)![0]} = ?,
+        set ${columnNames[0]} = ?,
         recordUpdate_userName = ?,
         recordUpdate_timeMillis = ?
         where recordDelete_timeMillis is null
-        and ${recordNameIdColumns.get(recordTable)![1]} = ?`
+        and ${columnNames[1]} = ?`
     )
     .run(recordName, user.userName, Date.now(), recordId)
 

handlers/admin-post/doAddBurialSiteType.d.ts

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

handlers/admin-post/doAddBurialSiteType.js

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

handlers/admin-post/doAddBurialSiteType.ts

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

handlers/admin-post/doBackupDatabase.js

@@ -1,4 +1,4 @@
-import { backupDatabase } from '../../helpers/database.helpers.js';
+import { backupDatabase } from '../../database/backupDatabase.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 '../../helpers/database.helpers.js'
+import { backupDatabase } from '../../database/backupDatabase.js'
 
 export default async function handler(
   _request: Request,
@@ -15,11 +15,13 @@ export default async function handler(
 
     response.json({
       success: true,
+
       fileName
     })
   } else {
     response.json({
       success: false,
+
       errorMessage: 'Unable to write backup file.'
     })
   }