From f4c379ea25ef0957c81ee2f319e6ad37c2588c43 Mon Sep 17 00:00:00 2001
From: Montana <montana-ctr@friends.dds.mil>
Date: Mon, 17 Sep 2018 13:39:35 -0400
Subject: [PATCH] Use modal inputs to pass environment id and env role to route

---
 atst/routes/workspaces.py              |  2 ++
 templates/components/modal.html        |  2 +-
 templates/workspaces/members/edit.html | 32 ++++++++++++++------------
 3 files changed, 20 insertions(+), 16 deletions(-)

diff --git a/atst/routes/workspaces.py b/atst/routes/workspaces.py
index b3ac5b1b..0c42c15e 100644
--- a/atst/routes/workspaces.py
+++ b/atst/routes/workspaces.py
@@ -15,6 +15,7 @@ from atst.domain.reports import Reports
 from atst.domain.workspaces import Workspaces
 from atst.domain.workspace_users import WorkspaceUsers
 from atst.domain.environments import Environments
+from atst.models.environment_role import EnvironmentRole
 from atst.forms.new_project import NewProjectForm
 from atst.forms.new_member import NewMemberForm
 from atst.forms.edit_member import EditMemberForm
@@ -222,6 +223,7 @@ def view_member(workspace_id, member_id):
         member=member,
         projects=projects,
         form=form,
+        EnvironmentRole=EnvironmentRole,
     )
 
 
diff --git a/templates/components/modal.html b/templates/components/modal.html
index 288ec667..8897238d 100644
--- a/templates/components/modal.html
+++ b/templates/components/modal.html
@@ -1,7 +1,7 @@
 {% from "components/icon.html" import Icon %}
 
 {% macro Modal(name, dismissable=False) -%}
-  <div v-if="activeModal === '{{name}}'" v-cloak>
+  <div v-show="activeModal === '{{name}}'" v-cloak>
     <div class='modal {% if dismissable %}modal--dismissable{% endif%}'>
       <div class='modal__container'>
         <div class='modal__dialog' role='dialog' aria-modal='true'>
diff --git a/templates/workspaces/members/edit.html b/templates/workspaces/members/edit.html
index 4c4e0dd7..71cfe8b4 100644
--- a/templates/workspaces/members/edit.html
+++ b/templates/workspaces/members/edit.html
@@ -64,20 +64,28 @@
         </header>
         <ul v-show='isVisible'>
           {% for env in project.environments %}
+
+            {% set role = EnvironmentRole.get(member.user_id, env.id).role or 'no access' %}
+            {% set label_class = 'label' %}
+            {% if role != 'no access' %}
+              {% set label_class = 'label label--success' %}
+            {% endif %}
+
             {% call Modal(name=env.name + 'RolesModal', dismissable=False) %}
                 <div class='block-list'>
                   <ul>
                     {% for choice in form.environment_role.choices %}
                     <li class='block-list__item block-list__item--selectable'>
+
                       {% if choice[0] != ""  %}
                         <input
-                          name='environment_role'
+                          name='env_{{ env.name }}'
                           type='radio'
-                          id="environment_role_{{ choice[0] }}"
-                          value='{{ choice[0] }}'
-                          checked='{{ value == choice[0] }}'
+                          id="env_{{ env.name }}"
+                          value='{ "id": "{{ env.id }}", "role": "{{ choice[0] }}" }'
+                          checked='{{ role  == choice[0] }}'
                         />
-                        <label for="environment_role_{{ choice[0] }}">
+                        <label for="env_{{ env.name }}">
                           {% if choice[1].description %}
                             <dl>
                               <dt>{{ choice[1].name }}</dt>
@@ -104,17 +112,11 @@
             <span class='project-list-item__environment'>
               {{ env.name }}
             </span>
+
             <div class='project-list-item__environment__actions'>
-              <input type="hidden" name="{{ env.id }}">
-              {% set role = 'no access' %}
-              {% set label = 'label' %}
-              {% for er in member.user.environment_roles %}
-                {% if er.get(member.user_id, env.id) %}
-                  {% set role = er.get(member.user_id, env.id).role %}
-                  {% set label = 'label label--success' %}
-                {% endif %}
-              {% endfor %}
-              <span class="label {{ label }}">{{ role }}</span>
+              <div>{{ form.data["environment_role"] }}</div>
+
+              <span class="{{ label_class }}">{{ role }}</span>
               <button v-on:click="openModal('{{ env.name }}RolesModal')" type="button" class="icon-link">set role</button>
             </div>