improve consistency in order checking

once consistent, may be able to clean up
deepsource-autofix-76c6eb20
Dan Gowans 2023-01-06 12:51:21 -05:00
parent 3af27e0739
commit 45cddb2264
11 changed files with 50 additions and 47 deletions

View File

@ -32,13 +32,13 @@ export function getFeeCategories(filters, options) {
" order by orderNumber, feeCategory")
.all(sqlParameters);
if (options.includeFees) {
let expectedFeeCategoryOrderNumber = -1;
let expectedOrderNumber = 0;
for (const feeCategory of feeCategories) {
expectedFeeCategoryOrderNumber += 1;
if (updateOrderNumbers && feeCategory.orderNumber !== expectedFeeCategoryOrderNumber) {
updateRecordOrderNumber("FeeCategories", feeCategory.feeCategoryId, expectedFeeCategoryOrderNumber, database);
feeCategory.orderNumber = expectedFeeCategoryOrderNumber;
if (updateOrderNumbers && feeCategory.orderNumber !== expectedOrderNumber) {
updateRecordOrderNumber("FeeCategories", feeCategory.feeCategoryId, expectedOrderNumber, database);
feeCategory.orderNumber = expectedOrderNumber;
}
expectedOrderNumber += 1;
feeCategory.fees = getFees(feeCategory.feeCategoryId, filters, database);
}
}

View File

@ -60,22 +60,22 @@ export function getFeeCategories(
.all(sqlParameters);
if (options.includeFees) {
let expectedFeeCategoryOrderNumber = -1;
let expectedOrderNumber = 0;
for (const feeCategory of feeCategories) {
expectedFeeCategoryOrderNumber += 1;
if (updateOrderNumbers && feeCategory.orderNumber !== expectedFeeCategoryOrderNumber) {
if (updateOrderNumbers && feeCategory.orderNumber !== expectedOrderNumber) {
updateRecordOrderNumber(
"FeeCategories",
feeCategory.feeCategoryId,
expectedFeeCategoryOrderNumber,
expectedOrderNumber,
database
);
feeCategory.orderNumber = expectedFeeCategoryOrderNumber;
feeCategory.orderNumber = expectedOrderNumber;
}
expectedOrderNumber += 1;
feeCategory.fees = getFees(feeCategory.feeCategoryId, filters, database);
}
}

View File

@ -32,13 +32,13 @@ export function getFees(feeCategoryId, additionalFilters, connectedDatabase) {
" order by f.orderNumber, f.feeName")
.all(sqlParameters);
if (updateOrderNumbers) {
let expectedFeeOrderNumber = -1;
let expectedOrderNumber = 0;
for (const fee of fees) {
expectedFeeOrderNumber += 1;
if (fee.orderNumber !== expectedFeeOrderNumber) {
updateRecordOrderNumber("Fees", fee.feeId, expectedFeeOrderNumber, database);
fee.orderNumber = expectedFeeOrderNumber;
if (fee.orderNumber !== expectedOrderNumber) {
updateRecordOrderNumber("Fees", fee.feeId, expectedOrderNumber, database);
fee.orderNumber = expectedOrderNumber;
}
expectedOrderNumber += 1;
}
}
if (!connectedDatabase) {

View File

@ -58,15 +58,15 @@ export function getFees(
.all(sqlParameters);
if (updateOrderNumbers) {
let expectedFeeOrderNumber = -1;
let expectedOrderNumber = 0;
for (const fee of fees) {
expectedFeeOrderNumber += 1;
if (fee.orderNumber !== expectedFeeOrderNumber) {
updateRecordOrderNumber("Fees", fee.feeId, expectedFeeOrderNumber, database);
fee.orderNumber = expectedFeeOrderNumber;
if (fee.orderNumber !== expectedOrderNumber) {
updateRecordOrderNumber("Fees", fee.feeId, expectedOrderNumber, database);
fee.orderNumber = expectedOrderNumber;
}
expectedOrderNumber += 1;
}
}

View File

@ -2,9 +2,10 @@ import sqlite from "better-sqlite3";
import { lotOccupancyDB as databasePath } from "../../data/databasePaths.js";
import type * as recordTypes from "../../types/recordTypes";
import { updateRecordOrderNumber } from "./updateRecordOrderNumber.js";
import type * as recordTypes from "../../types/recordTypes";
export function getLotOccupantTypes(): recordTypes.LotOccupantType[] {
const database = sqlite(databasePath);

View File

@ -2,9 +2,10 @@ import sqlite from "better-sqlite3";
import { lotOccupancyDB as databasePath } from "../../data/databasePaths.js";
import type * as recordTypes from "../../types/recordTypes";
import { updateRecordOrderNumber } from "./updateRecordOrderNumber.js";
import type * as recordTypes from "../../types/recordTypes";
export function getLotStatuses(): recordTypes.LotStatus[] {
const database = sqlite(databasePath);

View File

@ -12,13 +12,13 @@ export function getLotTypeFields(lotTypeId, connectedDatabase) {
and lotTypeId = ?
order by orderNumber, lotTypeField`)
.all(lotTypeId);
let expectedFieldOrderNumber = -1;
let expectedOrderNumber = 0;
for (const lotTypeField of lotTypeFields) {
expectedFieldOrderNumber += 1;
if (lotTypeField.orderNumber !== expectedFieldOrderNumber) {
updateRecordOrderNumber("LotTypeFields", lotTypeField.lotTypeFieldId, expectedFieldOrderNumber, database);
lotTypeField.orderNumber = expectedFieldOrderNumber;
if (lotTypeField.orderNumber !== expectedOrderNumber) {
updateRecordOrderNumber("LotTypeFields", lotTypeField.lotTypeFieldId, expectedOrderNumber, database);
lotTypeField.orderNumber = expectedOrderNumber;
}
expectedOrderNumber += 1;
}
if (!connectedDatabase) {
database.close();

View File

@ -23,21 +23,21 @@ export function getLotTypeFields(
)
.all(lotTypeId);
let expectedFieldOrderNumber = -1;
let expectedOrderNumber = 0;
for (const lotTypeField of lotTypeFields) {
expectedFieldOrderNumber += 1;
if (lotTypeField.orderNumber !== expectedFieldOrderNumber) {
if (lotTypeField.orderNumber !== expectedOrderNumber) {
updateRecordOrderNumber(
"LotTypeFields",
lotTypeField.lotTypeFieldId,
expectedFieldOrderNumber,
expectedOrderNumber,
database
);
lotTypeField.orderNumber = expectedFieldOrderNumber;
lotTypeField.orderNumber = expectedOrderNumber;
}
expectedOrderNumber += 1;
}
if (!connectedDatabase) {

View File

@ -17,13 +17,13 @@ export function getOccupancyTypeFields(occupancyTypeId, connectedDatabase) {
(occupancyTypeId ? " and occupancyTypeId = ?" : " and occupancyTypeId is null") +
" order by orderNumber, occupancyTypeField")
.all(sqlParameters);
let expectedFieldOrderNumber = -1;
let expectedOrderNumber = 0;
for (const occupancyTypeField of occupancyTypeFields) {
expectedFieldOrderNumber += 1;
if (occupancyTypeField.orderNumber !== expectedFieldOrderNumber) {
updateRecordOrderNumber("OccupancyTypeFields", occupancyTypeField.occupancyTypeFieldId, expectedFieldOrderNumber, database);
occupancyTypeField.orderNumber = expectedFieldOrderNumber;
if (occupancyTypeField.orderNumber !== expectedOrderNumber) {
updateRecordOrderNumber("OccupancyTypeFields", occupancyTypeField.occupancyTypeFieldId, expectedOrderNumber, database);
occupancyTypeField.orderNumber = expectedOrderNumber;
}
expectedOrderNumber += 1;
}
if (!connectedDatabase) {
database.close();

View File

@ -30,21 +30,21 @@ export function getOccupancyTypeFields(
)
.all(sqlParameters);
let expectedFieldOrderNumber = -1;
let expectedOrderNumber = 0;
for (const occupancyTypeField of occupancyTypeFields) {
expectedFieldOrderNumber += 1;
if (occupancyTypeField.orderNumber !== expectedFieldOrderNumber) {
if (occupancyTypeField.orderNumber !== expectedOrderNumber) {
updateRecordOrderNumber(
"OccupancyTypeFields",
occupancyTypeField.occupancyTypeFieldId,
expectedFieldOrderNumber,
expectedOrderNumber,
database
);
occupancyTypeField.orderNumber = expectedFieldOrderNumber;
occupancyTypeField.orderNumber = expectedOrderNumber;
}
expectedOrderNumber += 1;
}
if (!connectedDatabase) {

View File

@ -2,9 +2,10 @@ import sqlite from "better-sqlite3";
import { lotOccupancyDB as databasePath } from "../../data/databasePaths.js";
import type * as recordTypes from "../../types/recordTypes";
import { updateRecordOrderNumber } from "./updateRecordOrderNumber.js";
import type * as recordTypes from "../../types/recordTypes";
export function getWorkOrderTypes(): recordTypes.WorkOrderType[] {
const database = sqlite(databasePath);