fix: handle existing arrays

unjs · Dec 13, 2022 · e09702a · e09702a
1 parent 94c2830
commit e09702a
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/src/utils/body.ts b/src/utils/body.ts
@@ -67,12 +67,14 @@ export async function readBody<T=any> (event: H3Event): Promise<T> {
     const form = new URLSearchParams(body);
     const parsedForm: Record<string, any> = Object.create(null);
     for (const [key, value] of form.entries()) {
-      if (key in parsedForm && !Array.isArray(parsedForm[key])) {
-        parsedForm[key] = [parsedForm[key]];
-        parsedForm[key].push(value);
-      } else {
+      if (!(key in parsedForm)) {
         parsedForm[key] = value;
+        continue;
+      }
+      if (!Array.isArray(parsedForm[key])) {
+        parsedForm[key] = [parsedForm[key]];
       }
+      parsedForm[key].push(value);
     }
     return parsedForm as unknown as T;
   }

diff --git a/test/body.test.ts b/test/body.test.ts
@@ -118,12 +118,12 @@ describe("", () => {
         expect(body).toMatchObject({
           field: "value",
           another: "true",
-          number: ["20", "30"]
+          number: ["20", "30", "40"]
         });
         return "200";
       }));
       const result = await request.post("/api/test")
-        .send("field=value&another=true&number=20&number=30");
+        .send("field=value&another=true&number=20&number=30&number=40");
 
       expect(result.text).toBe("200");
     });