From 87b173b035fefa157f5c9cf0500a994ac6296aa5 Mon Sep 17 00:00:00 2001 From: leigh-mil Date: Tue, 13 Aug 2019 14:09:04 -0400 Subject: [PATCH] Use tuple instead of a dict for the user data and form --- atst/routes/ccpo.py | 4 +--- templates/ccpo/users.html | 8 ++++++-- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/atst/routes/ccpo.py b/atst/routes/ccpo.py index 384a6b50..553e2a00 100644 --- a/atst/routes/ccpo.py +++ b/atst/routes/ccpo.py @@ -26,9 +26,7 @@ def activity_history(): @user_can(Permissions.VIEW_CCPO_USER, message="view ccpo users") def users(): users = Users.get_ccpo_users() - data = {} - for user in users: - data[user] = CCPOUserForm(obj=user) + data = [(user, CCPOUserForm(obj=user)) for user in users] return render_template("ccpo/users.html", data=data) diff --git a/templates/ccpo/users.html b/templates/ccpo/users.html index 584f2bf8..7e0d2ff8 100644 --- a/templates/ccpo/users.html +++ b/templates/ccpo/users.html @@ -25,7 +25,9 @@ - {% for user, form in data.items() %} + {% for user_data in data %} + {% set user = user_data[0] %} + {% set form = user_data[1] %} {% set modal_id = "disable_ccpo_user_{}".format(user.dod_id) %} {% set disable_button_class = 'button-danger-outline' %} {% if user == g.current_user %} @@ -56,7 +58,9 @@ {% endif %} {% if user_can(permissions.DELETE_CCPO_USER) %} - {% for user, form in data.items() %} + {% for user_data in data %} + {% set user = user_data[0] %} + {% set form = user_data[1] %} {% set modal_id = "disable_ccpo_user_{}".format(user.dod_id) %} {% call Modal(name=modal_id) %} {{