@stamhoofd/models 2.120.6 → 2.122.0

package/src/models/CachedBalance.test.ts

@@ -1,5 +1,5 @@
-import { BalanceItemStatus } from '../../../../../shared/structures/dist/BalanceItem.js';
-import { ReceivableBalanceType } from '../../../../../shared/structures/dist/ReceivableBalance.js';
+import { BalanceItemStatus } from '@stamhoofd/structures/BalanceItem.js';
+import { ReceivableBalanceType } from '@stamhoofd/structures/ReceivableBalance.js';
 import { MemberFactory } from '../factories/MemberFactory.js';
 import { OrganizationFactory } from '../factories/OrganizationFactory.js';
 import { UserFactory } from '../factories/UserFactory.js';
@@ -21,19 +21,19 @@ describe('CachedBalance', () => {
 
     test('Balances for members are summed', async () => {
         const organization = await new OrganizationFactory({}).create();
-        const member = await new MemberFactory({organization}).create();
+        const member = await new MemberFactory({ organization }).create();
 
         const balanceA = new BalanceItem();
-        balanceA.dueAt = null
+        balanceA.dueAt = null;
         balanceA.quantity = 2;
         balanceA.unitPrice = 1_00;
         balanceA.memberId = member.id;
         balanceA.organizationId = organization.id;
-        balanceA.status = BalanceItemStatus.Due
+        balanceA.status = BalanceItemStatus.Due;
         await balanceA.save();
 
         // Update cached balance for user.
-        await CachedBalance.updateForMembers(organization.id, [member.id])
+        await CachedBalance.updateForMembers(organization.id, [member.id]);
         const cached = await CachedBalance.getForObjects([member.id], organization.id, ReceivableBalanceType.member);
         expect(cached).toHaveLength(1);
         expect(cached[0].amountOpen).toBe(2_00);
@@ -44,7 +44,7 @@ describe('CachedBalance', () => {
 
     test('Balances less than 7 days in the future are summed', async () => {
         const organization = await new OrganizationFactory({}).create();
-        const member = await new MemberFactory({organization}).create();
+        const member = await new MemberFactory({ organization }).create();
 
         const balanceA = new BalanceItem();
         balanceA.dueAt = new Date(now.getTime() + 1000 * 60 * 60 * 24 * 6); // 6 days later
@@ -52,11 +52,11 @@ describe('CachedBalance', () => {
         balanceA.unitPrice = 1_00;
         balanceA.memberId = member.id;
         balanceA.organizationId = organization.id;
-        balanceA.status = BalanceItemStatus.Due
+        balanceA.status = BalanceItemStatus.Due;
         await balanceA.save();
 
         // Update cached balance for user.
-        await CachedBalance.updateForMembers(organization.id, [member.id])
+        await CachedBalance.updateForMembers(organization.id, [member.id]);
         const cached = await CachedBalance.getForObjects([member.id], organization.id, ReceivableBalanceType.member);
         expect(cached).toHaveLength(1);
         expect(cached[0].amountOpen).toBe(1_00);
@@ -67,7 +67,7 @@ describe('CachedBalance', () => {
 
     test('Balances more than 7 days in the future are not summed', async () => {
         const organization = await new OrganizationFactory({}).create();
-        const member = await new MemberFactory({organization}).create();
+        const member = await new MemberFactory({ organization }).create();
 
         const balanceA = new BalanceItem();
         balanceA.dueAt = new Date(now.getTime() + 1000 * 60 * 60 * 24 * 8); // 8 days later
@@ -75,11 +75,11 @@ describe('CachedBalance', () => {
         balanceA.unitPrice = 1_00;
         balanceA.memberId = member.id;
         balanceA.organizationId = organization.id;
-        balanceA.status = BalanceItemStatus.Due
+        balanceA.status = BalanceItemStatus.Due;
         await balanceA.save();
 
         // Update cached balance for user.
-        await CachedBalance.updateForMembers(organization.id, [member.id])
+        await CachedBalance.updateForMembers(organization.id, [member.id]);
         const cached = await CachedBalance.getForObjects([member.id], organization.id, ReceivableBalanceType.member);
         expect(cached).toHaveLength(1);
         expect(cached[0].amountOpen).toBe(0);
@@ -90,7 +90,7 @@ describe('CachedBalance', () => {
 
     test('Paid balances more than 7 days in the future are summed', async () => {
         const organization = await new OrganizationFactory({}).create();
-        const member = await new MemberFactory({organization}).create();
+        const member = await new MemberFactory({ organization }).create();
 
         const balanceA = new BalanceItem();
         balanceA.dueAt = new Date(now.getTime() + 1000 * 60 * 60 * 24 * 8); // 8 days later
@@ -98,12 +98,12 @@ describe('CachedBalance', () => {
         balanceA.unitPrice = 1_00;
         balanceA.memberId = member.id;
         balanceA.organizationId = organization.id;
-        balanceA.status = BalanceItemStatus.Due
+        balanceA.status = BalanceItemStatus.Due;
         balanceA.pricePaid = 2_00;
         await balanceA.save();
 
         // Update cached balance for user.
-        await CachedBalance.updateForMembers(organization.id, [member.id])
+        await CachedBalance.updateForMembers(organization.id, [member.id]);
         const cached = await CachedBalance.getForObjects([member.id], organization.id, ReceivableBalanceType.member);
         expect(cached).toHaveLength(1);
         expect(cached[0].amountOpen).toBe(0);
@@ -112,9 +112,9 @@ describe('CachedBalance', () => {
         expect(cached[0].nextDueAt).toEqual(null);
     });
 
-     test('Paid balances more than 7 days in the future are summed and due at is still set if only partially paid', async () => {
+    test('Paid balances more than 7 days in the future are summed and due at is still set if only partially paid', async () => {
         const organization = await new OrganizationFactory({}).create();
-        const member = await new MemberFactory({organization}).create();
+        const member = await new MemberFactory({ organization }).create();
 
         const balanceA = new BalanceItem();
         balanceA.dueAt = new Date(now.getTime() + 1000 * 60 * 60 * 24 * 8); // 8 days later
@@ -122,12 +122,12 @@ describe('CachedBalance', () => {
         balanceA.unitPrice = 1_00;
         balanceA.memberId = member.id;
         balanceA.organizationId = organization.id;
-        balanceA.status = BalanceItemStatus.Due
+        balanceA.status = BalanceItemStatus.Due;
         balanceA.pricePaid = 1_00;
         await balanceA.save();
 
         // Update cached balance for user.
-        await CachedBalance.updateForMembers(organization.id, [member.id])
+        await CachedBalance.updateForMembers(organization.id, [member.id]);
         const cached = await CachedBalance.getForObjects([member.id], organization.id, ReceivableBalanceType.member);
         expect(cached).toHaveLength(1);
         expect(cached[0].amountOpen).toBe(0);
@@ -138,7 +138,7 @@ describe('CachedBalance', () => {
 
     test('Canceled items are not summed', async () => {
         const organization = await new OrganizationFactory({}).create();
-        const member = await new MemberFactory({organization}).create();
+        const member = await new MemberFactory({ organization }).create();
 
         const balanceA = new BalanceItem();
         balanceA.dueAt = null;
@@ -146,11 +146,11 @@ describe('CachedBalance', () => {
         balanceA.unitPrice = 1_00;
         balanceA.memberId = member.id;
         balanceA.organizationId = organization.id;
-        balanceA.status = BalanceItemStatus.Canceled
+        balanceA.status = BalanceItemStatus.Canceled;
         await balanceA.save();
 
         // Update cached balance for user.
-        await CachedBalance.updateForMembers(organization.id, [member.id])
+        await CachedBalance.updateForMembers(organization.id, [member.id]);
         const cached = await CachedBalance.getForObjects([member.id], organization.id, ReceivableBalanceType.member);
         expect(cached).toHaveLength(1);
         expect(cached[0].amountOpen).toBe(0);
@@ -160,7 +160,7 @@ describe('CachedBalance', () => {
 
     test('Hidden items are not summed', async () => {
         const organization = await new OrganizationFactory({}).create();
-        const member = await new MemberFactory({organization}).create();
+        const member = await new MemberFactory({ organization }).create();
 
         const balanceA = new BalanceItem();
         balanceA.dueAt = null;
@@ -168,11 +168,11 @@ describe('CachedBalance', () => {
         balanceA.unitPrice = 1_00;
         balanceA.memberId = member.id;
         balanceA.organizationId = organization.id;
-        balanceA.status = BalanceItemStatus.Hidden
+        balanceA.status = BalanceItemStatus.Hidden;
         await balanceA.save();
 
         // Update cached balance for user.
-        await CachedBalance.updateForMembers(organization.id, [member.id])
+        await CachedBalance.updateForMembers(organization.id, [member.id]);
         const cached = await CachedBalance.getForObjects([member.id], organization.id, ReceivableBalanceType.member);
         expect(cached).toHaveLength(1);
         expect(cached[0].amountOpen).toBe(0);
@@ -183,8 +183,8 @@ describe('CachedBalance', () => {
     describe('nextDueAt for users with members', () => {
         test('Two balance items in the future are merged if less than 7 days in the future', async () => {
             const organization = await new OrganizationFactory({}).create();
-            const member = await new MemberFactory({organization}).create();
-            const user = await new UserFactory({organization}).create();
+            const member = await new MemberFactory({ organization }).create();
+            const user = await new UserFactory({ organization }).create();
 
             // Link member with user
             await Member.users.reverse('members').link(user, [member]);
@@ -195,7 +195,7 @@ describe('CachedBalance', () => {
             balanceA.unitPrice = 1_00;
             balanceA.memberId = member.id;
             balanceA.organizationId = organization.id;
-            balanceA.status = BalanceItemStatus.Due
+            balanceA.status = BalanceItemStatus.Due;
             await balanceA.save();
 
             const balanceB = new BalanceItem();
@@ -204,12 +204,12 @@ describe('CachedBalance', () => {
             balanceB.unitPrice = 1_00;
             balanceB.userId = user.id;
             balanceB.organizationId = organization.id;
-            balanceB.status = BalanceItemStatus.Due
+            balanceB.status = BalanceItemStatus.Due;
             await balanceB.save();
 
             // Update cached balance for user.
-            await CachedBalance.updateForMembers(organization.id, [member.id])
-            await CachedBalance.updateForUsers(organization.id, [user.id])
+            await CachedBalance.updateForMembers(organization.id, [member.id]);
+            await CachedBalance.updateForUsers(organization.id, [user.id]);
             {
                 const cached = await CachedBalance.getForObjects([user.id], organization.id, ReceivableBalanceType.user);
                 expect(cached).toHaveLength(1);
@@ -231,8 +231,8 @@ describe('CachedBalance', () => {
 
         test('[Regression] Two balance items more than 7 days in the future set correct nextDueAt', async () => {
             const organization = await new OrganizationFactory({}).create();
-            const member = await new MemberFactory({organization}).create();
-            const user = await new UserFactory({organization}).create();
+            const member = await new MemberFactory({ organization }).create();
+            const user = await new UserFactory({ organization }).create();
 
             // Link member with user
             await Member.users.reverse('members').link(user, [member]);
@@ -243,7 +243,7 @@ describe('CachedBalance', () => {
             balanceA.unitPrice = 1_00;
             balanceA.memberId = member.id;
             balanceA.organizationId = organization.id;
-            balanceA.status = BalanceItemStatus.Due
+            balanceA.status = BalanceItemStatus.Due;
             await balanceA.save();
 
             const balanceB = new BalanceItem();
@@ -252,12 +252,12 @@ describe('CachedBalance', () => {
             balanceB.unitPrice = 1_00;
             balanceB.userId = user.id;
             balanceB.organizationId = organization.id;
-            balanceB.status = BalanceItemStatus.Due
+            balanceB.status = BalanceItemStatus.Due;
             await balanceB.save();
 
             // Update cached balance for user.
-            await CachedBalance.updateForMembers(organization.id, [member.id])
-            await CachedBalance.updateForUsers(organization.id, [user.id])
+            await CachedBalance.updateForMembers(organization.id, [member.id]);
+            await CachedBalance.updateForUsers(organization.id, [user.id]);
             {
                 const cached = await CachedBalance.getForObjects([user.id], organization.id, ReceivableBalanceType.user);
                 expect(cached).toHaveLength(1);
@@ -278,9 +278,9 @@ describe('CachedBalance', () => {
 
             // Advance time with one day, balanceA is now less than 7 days in the future and should be included in the cached balance.
             // nextDueAt should be different and set to balanceB.dueAt
-            vitest.setSystemTime(new Date(now.getTime() + 1000 * 60 * 60 * 24 * 1))
-            await CachedBalance.updateForMembers(organization.id, [member.id])
-            await CachedBalance.updateForUsers(organization.id, [user.id])
+            vitest.setSystemTime(new Date(now.getTime() + 1000 * 60 * 60 * 24 * 1));
+            await CachedBalance.updateForMembers(organization.id, [member.id]);
+            await CachedBalance.updateForUsers(organization.id, [user.id]);
             const cachedAfter = await CachedBalance.getForObjects([user.id], organization.id, ReceivableBalanceType.user);
             expect(cachedAfter).toHaveLength(1);
             expect(cachedAfter[0].amountOpen).toBe(1_00);
@@ -291,8 +291,8 @@ describe('CachedBalance', () => {
 
         test('Two balance items more than 7 days in the future that are partially paid are summed correctly', async () => {
             const organization = await new OrganizationFactory({}).create();
-            const member = await new MemberFactory({organization}).create();
-            const user = await new UserFactory({organization}).create();
+            const member = await new MemberFactory({ organization }).create();
+            const user = await new UserFactory({ organization }).create();
 
             // Link member with user
             await Member.users.reverse('members').link(user, [member]);
@@ -304,7 +304,7 @@ describe('CachedBalance', () => {
             balanceA.memberId = member.id;
             balanceA.organizationId = organization.id;
             balanceA.pricePaid = 50;
-            balanceA.status = BalanceItemStatus.Due
+            balanceA.status = BalanceItemStatus.Due;
             await balanceA.save();
 
             const balanceB = new BalanceItem();
@@ -314,12 +314,12 @@ describe('CachedBalance', () => {
             balanceB.userId = user.id;
             balanceB.organizationId = organization.id;
             balanceB.pricePaid = 50;
-            balanceB.status = BalanceItemStatus.Due
+            balanceB.status = BalanceItemStatus.Due;
             await balanceB.save();
 
             // Update cached balance for user.
-            await CachedBalance.updateForMembers(organization.id, [member.id])
-            await CachedBalance.updateForUsers(organization.id, [user.id])
+            await CachedBalance.updateForMembers(organization.id, [member.id]);
+            await CachedBalance.updateForUsers(organization.id, [user.id]);
             const cached = await CachedBalance.getForObjects([user.id], organization.id, ReceivableBalanceType.user);
             expect(cached).toHaveLength(1);
             expect(cached[0].amountOpen).toBe(0);

package/src/models/CachedBalance.ts

@@ -1,11 +1,11 @@
 import { column } from '@simonbackx/simple-database';
-import type { SQLWhere} from '@stamhoofd/sql';
+import type { SQLWhere } from '@stamhoofd/sql';
 import { QueryableModel, SQL, SQLAlias, SQLMin, SQLSelectAs, SQLSum, SQLWhereSign } from '@stamhoofd/sql';
 import { BalanceItemStatus, BalanceItem as BalanceItemStruct, ReceivableBalanceType } from '@stamhoofd/structures';
+import { Formatter } from '@stamhoofd/utility';
 import { v4 as uuidv4 } from 'uuid';
 import { BalanceItem } from './BalanceItem.js';
 import { MemberUser } from './MemberUser.js';
-import { Formatter } from '@stamhoofd/utility';
 
 /**
  * Keeps track of how much a member/user owes or needs to be reimbursed.
@@ -328,8 +328,7 @@ export class CachedBalance extends QueryableModel {
 
                 result[1].amountPending += amountPending;
                 result[1].amountPaid += amountPaid;
-            }
-            else {
+            } else {
                 results.push([objectId, { amountPaid, amountOpen: 0, amountPending, nextDueAt: amountOpen !== 0 ? dueAt : null }]);
             }
         }
@@ -407,10 +406,12 @@ export class CachedBalance extends QueryableModel {
 
     static async updateForOrganizations(organizationId: string, organizationIds: string[]) {
         if (organizationIds.length === 0) {
-            return;
+            return [];
         }
         const results = await this.fetchForObjects(organizationId, organizationIds, 'payingOrganizationId');
         await this.setForResults(organizationId, results, ReceivableBalanceType.organization);
+
+        return results;
     }
 
     static async updateForMembers(organizationId: string, memberIds: string[]) {
@@ -462,8 +463,7 @@ export class CachedBalance extends QueryableModel {
                     if (memberCachedBalance.nextDueAt && (!result[1].nextDueAt || memberCachedBalance.nextDueAt < result[1].nextDueAt)) {
                         result[1].nextDueAt = memberCachedBalance.nextDueAt;
                     }
-                }
-                else {
+                } else {
                     // Not possible
                     throw new Error('User not found in results');
                 }

package/src/models/Document.ts

@@ -3,6 +3,7 @@ import { DocumentData, DocumentStatus, Document as DocumentStruct, Platform, Ver
 import { Formatter } from '@stamhoofd/utility';
 import { v4 as uuidv4 } from 'uuid';
 
+import { SimpleError } from '@simonbackx/simple-errors';
 import { QueryableModel } from '@stamhoofd/sql';
 import { render } from '../helpers/Handlebars.js';
 import type { Member, MemberWithRegistrations, RegistrationWithMember } from './Member.js';
@@ -38,6 +39,9 @@ export class Document extends QueryableModel {
     @column({ type: 'string' })
     status = DocumentStatus.Draft;
 
+    @column({ type: 'boolean' })
+    isLocked = false;
+
     /**
      * Assigned when exporting the document
      */
@@ -63,6 +67,18 @@ export class Document extends QueryableModel {
     })
     updatedAt: Date;
 
+    override save(options?: Parameters<QueryableModel['save']>[0] & { forceSave?: boolean }): Promise<boolean> {
+        if (!options?.forceSave && this.isLocked) {
+            throw new SimpleError({
+                code: 'locked',
+                message: 'Document is locked',
+                human: $t(`%1Uc`),
+            });
+        }
+
+        return super.save(options);
+    }
+
     getStructure() {
         return DocumentStruct.create(this);
     }
@@ -84,6 +100,7 @@ export class Document extends QueryableModel {
         };
         const platformLogo = Platform.shared.config.logoDocuments ?? Platform.shared.config.horizontalLogo ?? Platform.shared.config.squareLogo;
         const organizationLogo = organization.meta.horizontalLogo ?? organization.meta.squareLogo;
+        const logo = organizationLogo || platformLogo;
 
         if (organizationLogo) {
             data['organization'] = {
@@ -98,6 +115,10 @@ export class Document extends QueryableModel {
             };
         }
 
+        if (logo) {
+            data['logo'] = logo.encode({ version: Version }) ?? null;
+        }
+
         for (const field of this.data.fieldAnswers.values()) {
             const keys = field.settings.id.split('.');
             let current = data;
@@ -121,6 +142,11 @@ export class Document extends QueryableModel {
     }
 
     async updateData(): Promise<void> {
+        if (this.isLocked) {
+            console.log('Document is locked, skipping update');
+            return;
+        }
+
         const DocumentTemplate = (await import('./DocumentTemplate.js')).DocumentTemplate;
         const template = await DocumentTemplate.getByID(this.templateId);
         if (!template) {
@@ -165,8 +191,7 @@ export class Document extends QueryableModel {
                     await this.updateForRegistrations(loadedMember.registrations.filter(r => r.registeredAt && r.deactivatedAt === null && r.organizationId === organizationId).map(r => r.id), organizationId);
                 }
             }
-        }
-        catch (e) {
+        } catch (e) {
             console.error(e);
         }
     }
@@ -176,13 +201,12 @@ export class Document extends QueryableModel {
             console.log('Updating documents for registration', registration.id);
 
             const DocumentTemplate = (await import('./DocumentTemplate.js')).DocumentTemplate;
-            const templates = await DocumentTemplate.where({ updatesEnabled: 1, organizationId: registration.organizationId });
+            const templates = await DocumentTemplate.where({ updatesEnabled: 1, isLocked: 0, organizationId: registration.organizationId });
 
             for (const template of templates) {
                 await template.updateForRegistration(registration);
             }
-        }
-        catch (e) {
+        } catch (e) {
             console.error(e);
         }
     }
@@ -208,7 +232,7 @@ export class Document extends QueryableModel {
             console.log('Updating documents for updateForRegistrations', registrationIds, organizationId);
 
             const DocumentTemplate = (await import('./DocumentTemplate.js')).DocumentTemplate;
-            const templates = await DocumentTemplate.where({ updatesEnabled: 1, organizationId });
+            const templates = await DocumentTemplate.where({ updatesEnabled: 1, isLocked: 0, organizationId });
 
             if (templates.length) {
                 const Member = (await import('./Member.js')).Member;
@@ -219,8 +243,7 @@ export class Document extends QueryableModel {
                     await template.updateForRegistrations(loadedRegistrations);
                 }
             }
-        }
-        catch (e) {
+        } catch (e) {
             console.error(e);
         }
     }
@@ -230,7 +253,7 @@ export class Document extends QueryableModel {
             console.log('Updating documents for group', groupId);
 
             const DocumentTemplate = (await import('./DocumentTemplate.js')).DocumentTemplate;
-            const templates = await DocumentTemplate.where({ updatesEnabled: 1, organizationId });
+            const templates = await DocumentTemplate.where({ updatesEnabled: 1, isLocked: 0, organizationId });
 
             if (templates.length) {
                 const Member = (await import('./Member.js')).Member;
@@ -240,8 +263,7 @@ export class Document extends QueryableModel {
                     await template.updateForRegistrations(registrations);
                 }
             }
-        }
-        catch (e) {
+        } catch (e) {
             console.error(e);
         }
     }
@@ -263,8 +285,7 @@ export class Document extends QueryableModel {
             const context = this.buildContext(organization);
             const renderedHtml = await render(htmlTemplate, context);
             return renderedHtml;
-        }
-        catch (e) {
+        } catch (e) {
             console.error('Failed to render document html', e);
             return null;
         }

package/src/models/DocumentTemplate.ts

@@ -1,7 +1,7 @@
 import { column } from '@simonbackx/simple-database';
 import { isSimpleError, isSimpleErrors, SimpleError } from '@simonbackx/simple-errors';
 import { QueueHandler } from '@stamhoofd/queues';
-import type { Parent, RecordAnswer} from '@stamhoofd/structures';
+import type { Parent, RecordAnswer } from '@stamhoofd/structures';
 import { BalanceItemStatus, DocumentData, DocumentPrivateSettings, DocumentSettings, DocumentStatus, DocumentTemplatePrivate, GroupType, NationalRegisterNumberOptOut, RecordAddressAnswer, RecordAnswerDecoder, RecordDateAnswer, RecordPriceAnswer, RecordSettings, RecordTextAnswer, RecordType } from '@stamhoofd/structures';
 import { Formatter, Sorter } from '@stamhoofd/utility';
 import { v4 as uuidv4 } from 'uuid';
@@ -37,6 +37,9 @@ export class DocumentTemplate extends QueryableModel {
     @column({ type: 'string' })
     status = DocumentStatus.Draft;
 
+    @column({ type: 'boolean' })
+    isLocked = false;
+
     @column({ type: 'boolean' })
     updatesEnabled = true;
 
@@ -87,6 +90,18 @@ export class DocumentTemplate extends QueryableModel {
     })
     updatedAt: Date;
 
+    override save(options?: Parameters<QueryableModel['save']>[0] & { forceSave?: boolean }): Promise<boolean> {
+        if (!options?.forceSave && this.isLocked) {
+            throw new SimpleError({
+                code: 'locked',
+                message: 'Document template is locked',
+                human: $t(`%1Uc`),
+            });
+        }
+
+        return super.save(options);
+    }
+
     getPrivateStructure() {
         return DocumentTemplatePrivate.create(this);
     }
@@ -363,8 +378,7 @@ export class DocumentTemplate extends QueryableModel {
                                 found = true;
                                 fieldAnswers.set(field.id, clone);
                                 break;
-                            }
-                            else {
+                            } else {
                                 console.warn('Found type mismatch for default data: ' + linkedToMemberAnswerSettingsId + ' - ' + field.id);
                             }
                         }
@@ -412,8 +426,7 @@ export class DocumentTemplate extends QueryableModel {
             for (const answer of fieldAnswers.values()) {
                 try {
                     answer.validate();
-                }
-                catch (e) {
+                } catch (e) {
                     missingData = true;
 
                     console.log('Missing data because of validation error', e, answer, answer.settings);
@@ -453,6 +466,14 @@ export class DocumentTemplate extends QueryableModel {
     }
 
     async updateForRegistration(registration: RegistrationWithMember, existingDocuments?: Document[]): Promise<Document[]> {
+        if (this.isLocked) {
+            throw new SimpleError({
+                code: 'locked',
+                message: 'Document template is locked',
+                human: $t(`%1Uc`),
+            });
+        }
+
         existingDocuments = existingDocuments !== undefined ? existingDocuments : await Document.where({ templateId: this.id, registrationId: registration.id }, { limit: 5 });
 
         if (!this.checkRegistrationIncluded(registration)) {
@@ -549,8 +570,7 @@ export class DocumentTemplate extends QueryableModel {
                 if (age > this.settings.maxAge) {
                     return false;
                 }
-            }
-            else {
+            } else {
                 console.warn('Missing registration.startDate in fieldAnswers when checking maxAge');
             }
         }
@@ -581,11 +601,20 @@ export class DocumentTemplate extends QueryableModel {
     async buildAll({ generateNumbers = false } = {}) {
         QueueHandler.abort('documents-build-all/' + this.id);
         return await QueueHandler.schedule('documents-build-all/' + this.id, async ({ abort }) => {
+            if (this.isLocked) {
+                return await Document.where({ templateId: this.id });
+            }
+
             if (!this.updatesEnabled) {
                 // Check status
                 const documents = await Document.where({ templateId: this.id });
                 for (const document of documents) {
                     abort.throwIfAborted();
+
+                    if (document.isLocked) {
+                        continue;
+                    }
+
                     await this.updateAnswers(document); // Only update global data
                     if (document.status === DocumentStatus.Draft || document.status === DocumentStatus.Published) {
                         document.status = this.status;
@@ -597,6 +626,11 @@ export class DocumentTemplate extends QueryableModel {
                 if (generateNumbers) {
                     for (const document of documents) {
                         abort.throwIfAborted();
+
+                        if (document.isLocked) {
+                            continue;
+                        }
+
                         if (document.number === null && document.status === DocumentStatus.Published) {
                             document.number = this.nextNumberForDocuments(documents);
                             await document.save();
@@ -627,11 +661,15 @@ export class DocumentTemplate extends QueryableModel {
             const documents = await Document.where({ templateId: this.id });
             for (const document of documents) {
                 abort.throwIfAborted();
+
+                if (document.isLocked) {
+                    continue;
+                }
+
                 if (!documentSet.has(document.id)) {
                     if (document.number === null) {
                         await document.delete();
-                    }
-                    else {
+                    } else {
                         document.status = DocumentStatus.Deleted;
                         await document.save();
                     }
@@ -644,6 +682,11 @@ export class DocumentTemplate extends QueryableModel {
             if (generateNumbers) {
                 for (const document of allDocuments) {
                     abort.throwIfAborted();
+
+                    if (document.isLocked) {
+                        continue;
+                    }
+
                     if (document.number === null && document.status === DocumentStatus.Published) {
                         document.number = this.nextNumberForDocuments(allDocuments);
                         await document.save();
@@ -733,8 +776,7 @@ export class DocumentTemplate extends QueryableModel {
             const context = await this.buildContext(organization);
             const renderedHtml = await render(this.privateSettings.templateDefinition.xmlExport, context);
             return renderedHtml;
-        }
-        catch (e) {
+        } catch (e) {
             if (isSimpleError(e) || isSimpleErrors(e)) {
                 throw e;
             }
@@ -753,8 +795,7 @@ export class DocumentTemplate extends QueryableModel {
             answer.settings = field;
             try {
                 answer.validate();
-            }
-            catch (e) {
+            } catch (e) {
                 // Invalid
                 return false;
             }
@@ -774,8 +815,7 @@ export class DocumentTemplate extends QueryableModel {
                 if (!existing.isReviewedAfter(addAnswer)) {
                     newAnswers.set(addAnswer.settings.id, addAnswer);
                 }
-            }
-            else {
+            } else {
                 newAnswers.set(addAnswer.settings.id, addAnswer);
             }
         }
@@ -786,8 +826,7 @@ export class DocumentTemplate extends QueryableModel {
         if (document.status !== DocumentStatus.Deleted) {
             if (!complete) {
                 document.status = DocumentStatus.MissingData;
-            }
-            else {
+            } else {
                 if (document.status === DocumentStatus.MissingData) {
                     document.status = this.status;
                 }