From c057c4a9cc773fe6af6b15981a8f65ff5032b7ba Mon Sep 17 00:00:00 2001
From: Alex Windett <windettalex@gmail.com>
Date: Wed, 25 Jan 2017 11:14:49 +0000
Subject: [PATCH] reconstructing dispatch payload to seperate person object and
 person index; also only sending uuid,status,role as props

---
 .../components/people-list/table/person-role.js  | 10 +++++++---
 .../src/components/people-list/table/tooltip.js  | 16 ++++++++++++----
 frontend/src/state/reducers/orgs.js              |  3 ++-
 3 files changed, 21 insertions(+), 8 deletions(-)

diff --git a/frontend/src/components/people-list/table/person-role.js b/frontend/src/components/people-list/table/person-role.js
index 9fb2c543..da48ed25 100644
--- a/frontend/src/components/people-list/table/person-role.js
+++ b/frontend/src/components/people-list/table/person-role.js
@@ -60,9 +60,12 @@ const PersonRole = (props) => {
   const toggled = toggledID;
   const handleClick = () => handleRoleTooltip(person.uuid);
   const handleOptionSelect = (updatedMember) => handleRoleUpdate(updatedMember);
-  const _person = {
-    ...person,
-    personIndex
+
+  // Only send relevent info as props
+  const _person =  {
+    uuid: person.uuid,
+    status: person.status,
+    role: person.role
   };
 
   return (
@@ -76,6 +79,7 @@ const PersonRole = (props) => {
           handleSelect={handleOptionSelect}
           options={membersRolesOptions}
           person={_person}
+          personIndex={personIndex}
           />
         : null }
     </StyledWrapper>
diff --git a/frontend/src/components/people-list/table/tooltip.js b/frontend/src/components/people-list/table/tooltip.js
index 69a5681b..80cc6294 100644
--- a/frontend/src/components/people-list/table/tooltip.js
+++ b/frontend/src/components/people-list/table/tooltip.js
@@ -16,15 +16,22 @@ const arrowPosition = {
 module.exports = ({
   handleSelect,
   person = {},
+  personIndex,
   options = [],
 }) => {
 
   const _options = options.map( (option, i) => {
 
-    const _onClick = () => handleSelect({
-      ...person,
-      role: option
-    });
+    const payload = {
+      person: {
+        uuid: person.uuid,
+        status: person.status,
+        role: option
+      },
+      personIndex
+    };
+
+    const _onClick = () => handleSelect(payload);
 
     return (
       <li
@@ -53,4 +60,5 @@ module.exports.propTypes = {
   handleSelect: React.PropTypes.func,
   options: React.PropTypes.array,
   person: React.PropTypes.object,
+  personIndex: React.PropTypes.number,
 };
\ No newline at end of file
diff --git a/frontend/src/state/reducers/orgs.js b/frontend/src/state/reducers/orgs.js
index e7ce272b..896ff027 100644
--- a/frontend/src/state/reducers/orgs.js
+++ b/frontend/src/state/reducers/orgs.js
@@ -51,6 +51,7 @@ module.exports = handleActions({
     // TODO:
     // Change "1" to org index. At the moment only updates
     // "biz-tech"
+
     return {
       ...state,
       ui: {
@@ -64,7 +65,7 @@ module.exports = handleActions({
           members: [
             ...state.data[1].members.slice(0, action.payload.personIndex),
             {
-              ...action.payload
+              ...action.payload.person
             },
             ...state.data[1].members.slice(action.payload.personIndex + 1)
           ]