-
Notifications
You must be signed in to change notification settings - Fork 9
Open
Labels
🔨 web/uiThings relating to Flask routes and Jinja templatesThings relating to Flask routes and Jinja templatesbugeasy task
Description
I've stumbled across this piece of code, which most likely is a bug:
pycroft/web/blueprints/finance/__init__.py
Lines 788 to 795 in 865ebcf
| account = session.get(Account, account_id) | |
| if not account: | |
| abort(404) | |
| account.legacy = not account.legacy | |
| session.commit() |
it's missing a session.add(account).
- Add a test witnessing this bug
- make it pass :)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
🔨 web/uiThings relating to Flask routes and Jinja templatesThings relating to Flask routes and Jinja templatesbugeasy task