From 5b0f29dfa433bbcd3e354f680f22266b2104fbe1 Mon Sep 17 00:00:00 2001 From: Thomas David Baker Date: Sat, 14 Dec 2019 14:38:28 +0000 Subject: [PATCH] Remove fill_form from /admin/people/notes as it is not working. I had a go at understanding this and failed. This at least gets things working again :) --- decksite/controllers/admin.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/decksite/controllers/admin.py b/decksite/controllers/admin.py index 411f6f66c..f93cc5a2a 100644 --- a/decksite/controllers/admin.py +++ b/decksite/controllers/admin.py @@ -173,11 +173,12 @@ def player_notes() -> str: return view.page() @auth.admin_required -@fill_form('person_id', 'note') @APP.route('/admin/people/notes/', methods=['POST']) -def post_player_note(person_id: int, note: str) -> wrappers.Response: +def post_player_note() -> wrappers.Response: + if not request.form.get('person_id') or not request.form.get('note'): + raise InvalidArgumentException(f'Did not find any of the expected keys in POST to /admin/people/notes: {request.form}') creator = ps.load_person_by_discord_id(session['id']) - ps.add_note(creator.id, person_id, note) + ps.add_note(creator.id, request.form['person_id'], request.form['note']) return redirect(url_for('player_notes')) @auth.admin_required