fixup! Extract payment information to configuration

agdsn · Jan 4, 2024 · ba93873 · ba93873
1 parent e9cf171
commit ba93873
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/sipa/model/pycroft/user.py b/sipa/model/pycroft/user.py
@@ -299,7 +299,7 @@ def finance_information(self) -> FinanceInformation:
     def payment_details(self) -> PaymentDetails:
         return PaymentDetails(
             recipient=current_app.config["PAYMENT_BENEFICIARY"],
-            iban=IBAN(current_app.config["PAYMENT_IBAN"]),
+            iban=IBAN(current_app.config["PAYMENT_IBAN"], validate_bban=True),
             purpose=f"{self.user_data.user_id}, {self.user_data.name}, {self.user_data.room}",
         )
 

diff --git a/sipa/model/sample/user.py b/sipa/model/sample/user.py
@@ -4,6 +4,7 @@
 
 from flask import current_app
 from flask_login import AnonymousUserMixin
+from schwifty import IBAN
 from werkzeug.local import LocalProxy
 
 from sipa.model.exceptions import PasswordInvalid, UserNotFound
@@ -232,7 +233,7 @@ def birthdate(self):
     def payment_details(self) -> PaymentDetails:
         return PaymentDetails(
             recipient="Donald Duck",
-            iban="EH12432543209523",
+            iban=IBAN("EH12432543209523", allow_invalid=True),
             purpose=self.id.value,
         )