diff --git a/alfa-client/libs/admin-settings/src/lib/organisationseinheit/organisationseinheit-container/organisationseinheit-form/organisationseinheit.formservice.spec.ts b/alfa-client/libs/admin-settings/src/lib/organisationseinheit/organisationseinheit-container/organisationseinheit-form/organisationseinheit.formservice.spec.ts index d7d9d2d97ec2b926c0e6086bdfa57f08524d87fb..d7cd5319e0b30b59e98615dfc82cc913d467f2ac 100644 --- a/alfa-client/libs/admin-settings/src/lib/organisationseinheit/organisationseinheit-container/organisationseinheit-form/organisationseinheit.formservice.spec.ts +++ b/alfa-client/libs/admin-settings/src/lib/organisationseinheit/organisationseinheit-container/organisationseinheit-form/organisationseinheit.formservice.spec.ts @@ -125,7 +125,7 @@ describe('OrganisationseinheitFormService', () => { }); formService.create(); - expect(userService.createOrganisationseinheit).toHaveBeenCalledWith(null, []); + expect(userService.createOrganisationseinheit).toHaveBeenCalledWith('', []); }); it('should return progress observable', () => { @@ -160,7 +160,7 @@ describe('OrganisationseinheitFormService', () => { expect(userService.saveOrganisationseinheit).toHaveBeenCalledWith({ id: formService.source.id, - name: null, + name: '', organisationseinheitIds: [], }); }); diff --git a/alfa-client/libs/admin-settings/src/lib/organisationseinheit/organisationseinheit-container/organisationseinheit-form/organisationseinheit.formservice.ts b/alfa-client/libs/admin-settings/src/lib/organisationseinheit/organisationseinheit-container/organisationseinheit-form/organisationseinheit.formservice.ts index 73ff18ade3415b5bc09ab1576ffd2cba62ebed50..296b62503083f33b271b1b9ec0a61143c1a7a59f 100644 --- a/alfa-client/libs/admin-settings/src/lib/organisationseinheit/organisationseinheit-container/organisationseinheit-form/organisationseinheit.formservice.ts +++ b/alfa-client/libs/admin-settings/src/lib/organisationseinheit/organisationseinheit-container/organisationseinheit-form/organisationseinheit.formservice.ts @@ -58,15 +58,23 @@ export class OrganisationseinheitFormservice { } private getName(): string { - return this.form.get(OrganisationseinheitFormservice.ORGANISATIONSEINHEIT_NAME_FIELD).value; + return this.getStringFromPotentiallyEmptyField( + OrganisationseinheitFormservice.ORGANISATIONSEINHEIT_NAME_FIELD, + ); } private getOrganisationseinheitIds(): string[] { return this.splitOrganisationseinheitIds( - this.form.get(OrganisationseinheitFormservice.ORGANISATIONSEINHEIT_IDS_FIELD).value ?? '', + this.getStringFromPotentiallyEmptyField( + OrganisationseinheitFormservice.ORGANISATIONSEINHEIT_IDS_FIELD, + ), ); } + private getStringFromPotentiallyEmptyField(fieldName: string): string { + return this.form.get(fieldName).value ?? ''; + } + public isPatch(): boolean { return isNotNil(this.source); }