refactor(basis): 优化时间工具类

Author: conifercone

mumu-basis/src/main/kotlin/baby/mumu/basis/kotlin/tools/TimeUtils.kt

@@ -16,8 +16,6 @@
 package baby.mumu.basis.kotlin.tools
 
 import baby.mumu.basis.dto.BaseDataTransferObject
-import baby.mumu.basis.exception.MuMuException
-import baby.mumu.basis.response.ResponseCode
 import org.apiguardian.api.API
 import java.time.LocalDateTime
 import java.time.OffsetDateTime
@@ -173,13 +171,12 @@ object TimeUtils {
      */
     @API(status = API.Status.INTERNAL, since = "2.7.0")
     @JvmStatic
-    fun validateTimezone(timezone: String) {
-        if (timezone.isNotBlank()) {
-            try {
-                ZoneId.of(timezone)
-            } catch (e: Exception) {
-                throw MuMuException(ResponseCode.TIME_ZONE_IS_NOT_AVAILABLE, e)
-            }
+    fun isValidTimeZone(timezone: String): Boolean {
+        try {
+            ZoneId.of(timezone)
+            return true
+        } catch (_: Exception) {
+            return false
         }
     }
 }

mumu-basis/src/test/java/baby/mumu/basis/tools/TimeUtilsTest.java

@@ -0,0 +1,37 @@
+/*
+ * Copyright (c) 2024-2025, the original author or authors.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package baby.mumu.basis.tools;
+
+import baby.mumu.basis.kotlin.tools.TimeUtils;
+import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.Test;
+
+/**
+ * 日期工具类测试
+ *
+ * @author <a href="mailto:[email protected]">kaiyu.shan</a>
+ * @since 2.7.0
+ */
+public class TimeUtilsTest {
+
+  @Test
+  public void isValidTimeZone() {
+    boolean test = TimeUtils.isValidTimeZone("test");
+    Assertions.assertFalse(test);
+    boolean test1 = TimeUtils.isValidTimeZone("Asia/Shanghai");
+    Assertions.assertTrue(test1);
+  }
+}

mumu-services/mumu-authentication/authentication-infrastructure/src/main/java/baby/mumu/authentication/infrastructure/account/gatewayimpl/AccountGatewayImpl.java

@@ -140,7 +140,11 @@ public void register(Account account) {
         account.getUsername(), account.getEmail())
         && !accountArchivedRepository.existsByIdOrUsernameOrEmail(account.getId(),
         account.getUsername(), account.getEmail())).ifPresentOrElse(accountPO -> {
-      TimeUtils.validateTimezone(accountPO.getTimezone());
+      // 验证时区是否为有效时区类型
+      if (StringUtils.isNotBlank(accountPO.getTimezone()) && !TimeUtils.isValidTimeZone(
+        accountPO.getTimezone())) {
+        throw new MuMuException(ResponseCode.TIME_ZONE_IS_NOT_AVAILABLE);
+      }
       accountPO.setPassword(passwordEncoder.encode(accountPO.getPassword()));
       accountRepository.persist(accountPO);
       account.setId(accountPO.getId());