From 8147bd2bc8336b4ab46febc0e04ae9f1dd87df5a Mon Sep 17 00:00:00 2001
From: Calvin <naruse_shinji@163.com>
Date: Sat, 8 Jun 2024 11:04:38 +0800
Subject: [PATCH] fixed: required field check for tag settings

---
 .../src/app/routes/setting/tags/tags.component.ts  | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/web-app/src/app/routes/setting/tags/tags.component.ts b/web-app/src/app/routes/setting/tags/tags.component.ts
index f9dfb8ede05..ca1f0abebbb 100644
--- a/web-app/src/app/routes/setting/tags/tags.component.ts
+++ b/web-app/src/app/routes/setting/tags/tags.component.ts
@@ -17,7 +17,8 @@
  * under the License.
  */
 
-import { Component, Inject, OnInit } from '@angular/core';
+import {Component, Inject, OnInit, ViewChild} from '@angular/core';
+import { NgForm } from '@angular/forms';
 import { I18NService } from '@core';
 import { ALAIN_I18N_TOKEN } from '@delon/theme';
 import { NzModalService } from 'ng-zorro-antd/modal';
@@ -34,6 +35,8 @@ import { formatTagName } from '../../../shared/utils/common-util';
   templateUrl: './tags.component.html'
 })
 export class SettingTagsComponent implements OnInit {
+  @ViewChild('tagForm', { static: false }) tagForm: NgForm | undefined;
+
   constructor(
     private notifySvc: NzNotificationService,
     private modal: NzModalService,
@@ -183,6 +186,15 @@ export class SettingTagsComponent implements OnInit {
     this.isManageModalVisible = false;
   }
   onManageModalOk() {
+    if (this.tagForm?.invalid) {
+      Object.values(this.tagForm.controls).forEach(control => {
+        if (control.invalid) {
+          control.markAsDirty();
+          control.updateValueAndValidity({ onlySelf: true });
+        }
+      });
+      return;
+    }
     this.isManageModalOkLoading = true;
     this.tag.name = this.tag.name.trim();
     if (this.tag.tagValue != undefined) {