Merge pull request #89 from NatLibFi/Revert-loan-and-hold-limit

Change operators > back to >=
NatLibFi · Aug 9, 2024 · 8872714 · 8872714
2 parents cb5f7a3 + 165f73c
commit 8872714
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/api/odl2.py b/api/odl2.py
@@ -112,7 +112,7 @@ def _checkout(
                     patron.loans,
                 )
             )
-            if len(loans) > self.loan_limit:  # Changed operator >= to >
+            if len(loans) >= self.loan_limit:  # Changed back operator > to >=
                 raise PatronLoanLimitReached(limit=self.loan_limit)
         return super()._checkout(patron, licensepool, hold)
 
@@ -125,7 +125,7 @@ def _place_hold(self, patron: Patron, licensepool: LicensePool) -> HoldInfo:
                     patron.holds,
                 )
             )
-            if len(holds) > self.hold_limit:  # Changed operator >= to >
+            if len(holds) >= self.hold_limit:  # Changed back operator > to >=
                 raise PatronHoldLimitReached(limit=self.hold_limit)
         return super()._place_hold(patron, licensepool)