Fix validate

reorganizes code so validate with database access is not being called inside a transaction

Likely fixes the bug which causes the database connection to block:
https://github.com/typeorm/typeorm/issues/6779

server/src/models/BaseModel.ts

@@ -2,8 +2,6 @@ import {
   BaseEntity,
   CreateDateColumn,
   UpdateDateColumn,
-  BeforeInsert,
-  BeforeUpdate,
   SaveOptions,
 } from "typeorm";
 import { validateOrReject, IsDate, IsOptional } from "class-validator";
@@ -36,14 +34,18 @@ export default abstract class BaseModel extends BaseEntity {
   }
 
   // validateOrReject to be run before saving/updating by TypeORM
-  @BeforeInsert()
-  @BeforeUpdate()
+  // @BeforeInsert and @BeforeUpdate have a bug and lock up the database connection
+  // see also: https://github.com/typeorm/typeorm/issues/6779
+  // @BeforeInsert()
+  // @BeforeUpdate()
   validateOrReject(): Promise<void> {
     return validateOrReject(this);
   }
 
   // reloads the entry after saving to the database
   async save(options?: SaveOptions): Promise<this> {
+    // run validateOrReject here as it blocks when run with @BeforeInsert and @BeforeUpdate
+    await this.validateOrReject();
     await super.save(options);
     await this.reload();
     return this;