fix: trim collection name on save rather than on change (#2962)

* fix: trim collection name on save rather than on change

* move trim to server

server/routes/api/collections.test.ts

@@ -1253,14 +1253,14 @@ describe("#collections.update", () => {
     expect(body.data.name).toBe(collection.name);
   });
 
-  it("allows editing from non-private to private collection", async () => {
+  it("allows editing from non-private to private collection, and trims whitespace", async () => {
     const { user, collection } = await seed();
     const res = await server.post("/api/collections.update", {
       body: {
         token: user.getJwtToken(),
         id: collection.id,
         permission: null,
-        name: "Test",
+        name: "  Test  ",
       },
     });
     const body = await res.json();

server/routes/api/collections.ts

@@ -536,7 +536,7 @@ router.post("collections.update", auth(), async (ctx) => {
   let sharingChanged = false;
 
   if (name !== undefined) {
-    collection.name = name;
+    collection.name = name.trim();
   }
 
   if (description !== undefined) {