From 422021e5ce7c3f94a0e0c2f498220639ac81bb75 Mon Sep 17 00:00:00 2001 From: Montana Date: Mon, 22 Apr 2019 10:41:44 -0400 Subject: [PATCH] Only show active members for ppoc update dropdown --- atst/routes/portfolios/admin.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/atst/routes/portfolios/admin.py b/atst/routes/portfolios/admin.py index 5ab4c1d3..460451d8 100644 --- a/atst/routes/portfolios/admin.py +++ b/atst/routes/portfolios/admin.py @@ -70,9 +70,11 @@ def render_admin_page(portfolio, form=None): ) assign_ppoc_form = member_forms.AssignPPOCForm() - assign_ppoc_form.user_id.choices += [ - (user.id, user.full_name) for user in portfolio.users if user != portfolio.owner - ] + for pf_role in portfolio.roles: + if pf_role.user != portfolio.owner and pf_role.is_active: + assign_ppoc_form.user_id.choices += [ + (pf_role.user.id, pf_role.user.full_name) + ] return render_template( "portfolios/admin.html",