From 4d927e3871997d75e7a78b212b63b50fe64091dd Mon Sep 17 00:00:00 2001 From: infacc Date: Wed, 25 Oct 2023 12:31:04 +0200 Subject: [PATCH 1/2] fix(template tab): disallow empty tab names --- .../template-details/template-details.component.html | 3 ++- .../template-details/template-details.component.ts | 6 +++++- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/src/app/components-small/template-details/template-details.component.html b/src/app/components-small/template-details/template-details.component.html index c3ea833..5b2777b 100644 --- a/src/app/components-small/template-details/template-details.component.html +++ b/src/app/components-small/template-details/template-details.component.html @@ -10,7 +10,8 @@

Template Tab

Name: - + + Invalid tab name! Description: diff --git a/src/app/components-small/template-details/template-details.component.ts b/src/app/components-small/template-details/template-details.component.ts index 932622b..a12755a 100644 --- a/src/app/components-small/template-details/template-details.component.ts +++ b/src/app/components-small/template-details/template-details.component.ts @@ -35,7 +35,7 @@ export class TemplateDetailsComponent implements OnInit { }; templateForm: FormGroup = this.fb.group({ - name: [this.initialValues.name, Validators.required], + name: [this.initialValues.name, [Validators.required, Validators.minLength(1)]], description: this.initialValues.description, sortKey: this.initialValues.sortKey, location: [this.initialValues.location, [Validators.required, isInSetValidator(Object.keys(TAB_GROUP_NAME_OVERRIDES))]] @@ -59,6 +59,10 @@ export class TemplateDetailsComponent implements OnInit { async onSubmit() { let findString: string; let response: ApiResponse | null; + if (this.templateForm.invalid) { + console.log("TemplateDetailsComponent: form invalid"); + return; + } if (this.templateLink != null) { findString = "create"; response = await this.registry.getByApiLink(this.templateLink); From 93ed15a2f1cc69d0b4a03cad07d021ab34a4b24b Mon Sep 17 00:00:00 2001 From: infacc Date: Wed, 25 Oct 2023 12:34:47 +0200 Subject: [PATCH 2/2] remove debug output --- .../template-details/template-details.component.ts | 1 - 1 file changed, 1 deletion(-) diff --git a/src/app/components-small/template-details/template-details.component.ts b/src/app/components-small/template-details/template-details.component.ts index a12755a..35c6251 100644 --- a/src/app/components-small/template-details/template-details.component.ts +++ b/src/app/components-small/template-details/template-details.component.ts @@ -60,7 +60,6 @@ export class TemplateDetailsComponent implements OnInit { let findString: string; let response: ApiResponse | null; if (this.templateForm.invalid) { - console.log("TemplateDetailsComponent: form invalid"); return; } if (this.templateLink != null) {