Now the logic is correct and handles both cases:

- None values: Sets them to 0
- Negative values: Sets them to 0
- Positive values: Leaves them unchanged

This preserves the original intent of the code while being more readable than the redundant max() calls.

Author: karbassi

coderdojochi/models/session.py

@@ -272,10 +272,13 @@ def save(self, *args, **kwargs):
         if self.mentor_capacity is None:
             self.mentor_capacity = int(self.capacity / 2)
 
-        self.mentor_capacity = self.mentor_capacity or 0
+        # Ensure mentor_capacity is not negative
+        if self.mentor_capacity is None or self.mentor_capacity < 0:
+            self.mentor_capacity = 0
 
-        # Capacity check
-        self.capacity = self.capacity or 0
+        # Ensure capacity is not negative
+        if self.capacity is None or self.capacity < 0:
+            self.capacity = 0
 
         super(Session, self).save(*args, **kwargs)