issue gugemichael#5

yxl1014 · Jun 12, 2023 · 5c163de · 5c163de
1 parent b4a414e
commit 5c163de
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/core/src/main/java/org/nesty/core/server/rest/controller/ControllerMethodDescriptor.java b/core/src/main/java/org/nesty/core/server/rest/controller/ControllerMethodDescriptor.java
@@ -133,7 +133,7 @@ private Object[] resolveParams(HttpContext context) throws ControllerParamsNotMa
                 RequestParam reqParam = (RequestParam) params[i].annotation;
                 value = context.getHttpParams().get(reqParam.value());
                 // only if required is fase
-                if (value == null && !reqParam.required())
+                if ((value == null || value.isEmpty()) && !reqParam.required())
                     required = false;
                 break;
             case REQUEST_BODY:
@@ -149,7 +149,7 @@ private Object[] resolveParams(HttpContext context) throws ControllerParamsNotMa
                 break;
             }
 
-            if ((value == null || value.isEmpty()) && required && !auto)
+            if (required && !auto && (value == null || value.isEmpty()))
                 throw new ControllerParamsNotMatchException(String.format("resolve %s failed", params[i].annotation.annotationType().getName()));
 
             try {