Validate max player numbers

Fixes #78

Author: FlorianSW

models/match.py

@@ -60,13 +60,20 @@ class Match(db.Document):
         "queryset_class": CustomQuerySet
     }
 
+    def clean(self):
+        if self.player_dist1 is not None or self.player_dist2 is not None:
+            if sum(self.player_dist1) > 50 or sum(self.player_dist2) > 50:
+                raise ValueError("player distributions cannot exceed 50 players")
+        if self.players > 100:
+            raise ValueError("players cannot exceed 100 players")
+        if (self.player_dist1 is not None or self.player_dist2 is not None) and self.players != 0:
+            raise ValueError("players and player distributions cannot both be set")
 
     def needs_confirmations(self):
         if (self.conf1 != "" and self.conf1 is not None) and (self.conf2 != "" and self.conf2 is not None):
             # do the calcs then
             return False
         return True
 
-
     def to_dict(self):
         return json.loads(self.to_json())

rest/matches.py

@@ -234,6 +234,8 @@ def post(self):
             return handle_error(f"match already exists in database", 400)
         except ValidationError as e:
             return handle_error(f"required field is empty: {e}")
+        except ValueError as e:
+            return handle_error(f"request is invalid: {e}")
         except Exception as e:
             return handle_error(f"error creating match in database, terminated with error: {e}", 500)
         else: