Merge branch 'hotfix/no_permissions_immediately_after_joining_project'

Author: Vikas Agarwal

src/projects/reducers/project.js

@@ -21,9 +21,12 @@ const initialState = {
   processingMembers: false,
   processingAttachments: false,
   error: false,
-  project: {}
+  project: {},
+  projectNonDirty: {}
 }
 
+// NOTE: We should always update projectNonDirty state whenever we update the project state
+
 const parseErrorObj = (action) => {
   const data = action.payload.response.data.result
   return {
@@ -40,7 +43,8 @@ export const projectState = function (state=initialState, action) {
   case LOAD_PROJECT_PENDING:
     return Object.assign({}, state, {
       isLoading: true,
-      project: null
+      project: null,
+      projectNonDirty: null
     })
 
   case LOAD_PROJECT_SUCCESS:
@@ -54,7 +58,8 @@ export const projectState = function (state=initialState, action) {
   case CLEAR_LOADED_PROJECT:
   case GET_PROJECTS_SUCCESS:
     return Object.assign({}, state, {
-      project: {}
+      project: {},
+      projectNonDirty: {}
     })
 
   case LOAD_DIRECT_PROJECT_SUCCESS:
@@ -70,6 +75,18 @@ export const projectState = function (state=initialState, action) {
           plannedDuration: action.payload.plannedDuration,
           projectedDuration: action.payload.projectedDuration
         }}
+      },
+      projectNonDirty: {
+        budget: { $set: {
+          actualCost: action.payload.actualCost,
+          projectedCost: action.payload.projectedCost,
+          totalBudget: action.payload.totalBudget
+        }},
+        duration: { $set: {
+          actualDuration: action.payload.actualDuration,
+          plannedDuration: action.payload.plannedDuration,
+          projectedDuration: action.payload.projectedDuration
+        }}
       }
     })
 
@@ -96,7 +113,8 @@ export const projectState = function (state=initialState, action) {
     return Object.assign({}, state, {
       processing: false,
       error: false,
-      project: {}
+      project: {},
+      projectNonDirty: {}
     })
 
   // Project attachments
@@ -110,15 +128,17 @@ export const projectState = function (state=initialState, action) {
   case ADD_PROJECT_ATTACHMENT_SUCCESS:
     return update(state, {
       processingAttachments: { $set : false },
-      project: { attachments: { $push: [action.payload] } }
+      project: { attachments: { $push: [action.payload] } },
+      projectNonDirty: { attachments: { $push: [action.payload] } }
     })
 
   case UPDATE_PROJECT_ATTACHMENT_SUCCESS: {
     // get index
     const idx = _.findIndex(state.project.attachments, a => a.id === action.payload.id)
     return update(state, {
       processingAttachments: { $set : false },
-      project: { attachments: { $splice : [[idx, 1, action.payload]] } }
+      project: { attachments: { $splice : [[idx, 1, action.payload]] } },
+      projectNonDirty: { attachments: { $splice : [[idx, 1, action.payload]] } }
     })
   }
 
@@ -128,7 +148,8 @@ export const projectState = function (state=initialState, action) {
     const idx = _.findIndex(state.project.attachments, a => a.id === action.payload)
     return update(state, {
       processing: { $set : false },
-      project: { attachments: { $splice: [[idx, 1]] } }
+      project: { attachments: { $splice: [[idx, 1]] } },
+      projectNonDirty: { attachments: { $splice: [[idx, 1]] } }
     })
   }
 
@@ -142,7 +163,8 @@ export const projectState = function (state=initialState, action) {
   case ADD_PROJECT_MEMBER_SUCCESS:
     return update (state, {
       processingMembers: { $set : false },
-      project: { members: { $push: [action.payload] } }
+      project: { members: { $push: [action.payload] } },
+      projectNonDirty: { members: { $push: [action.payload] } }
     })
 
   case UPDATE_PROJECT_MEMBER_SUCCESS: {
@@ -156,7 +178,8 @@ export const projectState = function (state=initialState, action) {
     updatedMembers.splice(idx, 1, action.payload)
     return update(state, {
       processingMembers: { $set : false },
-      project: { members: { $set: updatedMembers } }
+      project: { members: { $set: updatedMembers } },
+      projectNonDirty: { members: { $set: updatedMembers } }
     })
   }
 
@@ -165,7 +188,8 @@ export const projectState = function (state=initialState, action) {
     const idx = _.findIndex(state.project.members, a => a.id === action.payload)
     return update(state, {
       processingMembers: { $set : false },
-      project: { members: { $splice: [[idx, 1]] } }
+      project: { members: { $splice: [[idx, 1]] } },
+      projectNonDirty: { members: { $splice: [[idx, 1]] } }
     })
   }