From 6b32f2b933557cd04be4cdc32b2d60d1f5a54894 Mon Sep 17 00:00:00 2001
From: Pavel Petrovich <gaara33377@gmail.com>
Date: Tue, 30 Jun 2020 01:03:35 +0300
Subject: [PATCH] save entity fix

---
 client/src/Components/Comments/Comments.jsx   | 26 +++++++--
 .../Components/ModalWindow/ModalWindow.jsx    | 31 ++++++++---
 .../TaskModule/TaskView/DescriptionTask.jsx   |  2 +
 .../Modules/TaskModule/TaskView/TaskView.jsx  |  9 ++--
 .../actions/publicActions/middleware/index.js |  4 ++
 .../reducers/routerStateReducer/index.js      | 53 +++++++++++++------
 client/src/Utils/Hooks/coreHook/coreHook.js   |  3 +-
 logger.error-audit.json                       | 17 +++---
 8 files changed, 102 insertions(+), 43 deletions(-)

diff --git a/client/src/Components/Comments/Comments.jsx b/client/src/Components/Comments/Comments.jsx
index 19654c99..9dfde8b7 100644
--- a/client/src/Components/Comments/Comments.jsx
+++ b/client/src/Components/Comments/Comments.jsx
@@ -8,6 +8,8 @@ import { Button, Empty, message, notification } from 'antd';
 
 import Textarea from 'Components/Textarea';
 import Comment from './Comment';
+import actionsTypes from 'actions.types';
+import { routeParser } from 'Utils';
 
 class Comments extends React.PureComponent {
   state = {
@@ -29,6 +31,7 @@ class Comments extends React.PureComponent {
         onUpdate,
         data: { key = '', comments = [], _id: id = '' } = {},
         data = {},
+        path,
         udata: { displayName = '', _id: uId = '' } = {},
       } = this.props;
 
@@ -50,10 +53,17 @@ class Comments extends React.PureComponent {
 
         this.setState({ ...this.state, onUpdateDisabled: true, value: '' });
 
+        const parsedRoutePath = routeParser({
+          pageType: 'moduleItem',
+          path,
+        });
+
         await onUpdate({
+          actionType: actionsTypes.$UPDATE_SINGLE,
+          parsedRoutePath,
           key,
           id,
-          updateBy: 'key',
+          updateBy: '_id',
           updateItem: [...comments, comment],
           updateField: 'comments',
           store: 'tasks',
@@ -84,15 +94,22 @@ class Comments extends React.PureComponent {
   };
 
   onDelete = async (event, idComment) => {
-    const { onUpdate, data: { _id: id = '', key = '', comments = [] } = {}, data = {} } = this.props;
+    const { path, onUpdate, data: { _id: id = '', key = '', comments = [] } = {}, data = {} } = this.props;
     const filterComments = comments.filter((it) => it.id !== idComment);
     try {
+      const parsedRoutePath = routeParser({
+        pageType: 'moduleItem',
+        path,
+      });
+
       await onUpdate({
+        actionType: actionsTypes.$UPDATE_SINGLE,
+        parsedRoutePath,
         id,
         key,
         item: data,
         store: 'tasks',
-        updateBy: 'key',
+        updateBy: '_id',
         updateItem: filterComments,
         updateField: 'comments',
       });
@@ -122,11 +139,12 @@ class Comments extends React.PureComponent {
 
     try {
       await onUpdate({
+        actionType: actionsTypes.$UPDATE_SINGLE,
         id,
         key,
         item: data,
         store: 'tasks',
-        updateBy: 'key',
+        updateBy: '_id',
         updateItem: newCommentsArray,
         updateField: 'comments',
       });
diff --git a/client/src/Components/ModalWindow/ModalWindow.jsx b/client/src/Components/ModalWindow/ModalWindow.jsx
index 7bc15e8a..1a68a0c1 100644
--- a/client/src/Components/ModalWindow/ModalWindow.jsx
+++ b/client/src/Components/ModalWindow/ModalWindow.jsx
@@ -4,7 +4,7 @@ import _ from 'lodash';
 import moment from 'moment';
 import { Modal, Button, message, Select } from 'antd';
 import { TASK_CONTROLL_JURNAL_SCHEMA } from 'Models/Schema/const';
-import { createNotification, isTimeLostValue } from 'Utils';
+import { createNotification, isTimeLostValue, routeParser } from 'Utils';
 import SimpleEditableModal from './SimpleEditableModal';
 import RegistrationModal from './RegistrationModal';
 import ActionList from 'Components/ActionList';
@@ -154,10 +154,19 @@ class ModalWindow extends React.PureComponent {
       routeDataActive = {},
       routeDataActive: { key = null } = {},
       onCancelEditModeContent,
+      path,
     } = this.props;
+
+    const parsedRoutePath = routeParser({
+      pageType: 'moduleItem',
+      path,
+    });
+
     onUpdate({
+      actionType: actionsTypes.$UPDATE_MANY,
+      parsedRoutePath,
       key,
-      updateBy: 'key',
+      updateBy: '_id',
       id: routeDataActive?._id,
       updateItem: valueDescription,
       updateField: 'description',
@@ -248,11 +257,18 @@ class ModalWindow extends React.PureComponent {
       });
 
     try {
+      const parsedRoutePath = routeParser({
+        pageType: 'moduleItem',
+        path,
+      });
+
       await onUpdate({
+        actionType: actionsTypes.$UPDATE_SINGLE,
+        parsedRoutePath,
         path,
         id: routeDataActive?._id,
         key,
-        updateBy: 'key',
+        updateBy: '_id',
         updateItem: _.isString(customStatus) ? customStatus : taskStatus,
         updateField: 'status',
         item: { ...routeDataActive },
@@ -416,6 +432,7 @@ class ModalWindow extends React.PureComponent {
     const { statusTaskValue, accessStatus } = this.props;
     const { type = '' } = this.state;
     const { [type]: visible = false } = this.state;
+
     return (
       <div className="dropDownWrapper">
         <ActionList {...actionProps} />
@@ -427,13 +444,13 @@ class ModalWindow extends React.PureComponent {
           title="Смена статуса"
         >
           <Select onChange={this.onChangeSelect} defaultValue={statusTaskValue}>
-            {accessStatus.map((status, i) =>
-              i === 0 ? (
-                <Option key={i + status} value={statusTaskValue}>
+            {accessStatus.map((status, index) =>
+              index === 0 ? (
+                <Option key={index + status} value={statusTaskValue}>
                   {statusTaskValue}
                 </Option>
               ) : (
-                <Option key={i + status} value={status}>
+                <Option key={status} value={status}>
                   {status}
                 </Option>
               ),
diff --git a/client/src/Modules/TaskModule/TaskView/DescriptionTask.jsx b/client/src/Modules/TaskModule/TaskView/DescriptionTask.jsx
index 47558654..fb8e84e7 100644
--- a/client/src/Modules/TaskModule/TaskView/DescriptionTask.jsx
+++ b/client/src/Modules/TaskModule/TaskView/DescriptionTask.jsx
@@ -19,6 +19,7 @@ const DescriptionTask = forwardRef(
       onUpdate,
       udata,
       rest,
+      path,
     },
     ref,
   ) => {
@@ -55,6 +56,7 @@ const DescriptionTask = forwardRef(
         <Comments
           commentProps={commentProps}
           udata={udata}
+          path={path}
           rules={true}
           onUpdate={onUpdate}
           data={routeDataActive}
diff --git a/client/src/Modules/TaskModule/TaskView/TaskView.jsx b/client/src/Modules/TaskModule/TaskView/TaskView.jsx
index 0b347626..1828817c 100644
--- a/client/src/Modules/TaskModule/TaskView/TaskView.jsx
+++ b/client/src/Modules/TaskModule/TaskView/TaskView.jsx
@@ -18,6 +18,7 @@ import TitleModule from 'Components/TitleModule';
 import modelContext from 'Models/context';
 import DescriptionTask from './DescriptionTask';
 import renderDescription from './renderDescription';
+import actionsTypes from 'actions.types';
 
 class TaskView extends React.PureComponent {
   state = {
@@ -416,11 +417,11 @@ class TaskView extends React.PureComponent {
     const { _id: id = '', key = '' } = modeControllEdit || {};
     try {
       await onUpdate({
+        actionType: actionsTypes.$UPDATE_MANY,
         id,
         key,
         path,
         updateBy: '_id',
-        actionType: 'update_many',
         updateItem: { ...validHash },
         item: { ...routeDataActive },
         store: 'tasks',
@@ -619,6 +620,7 @@ class TaskView extends React.PureComponent {
       setCurrentTab,
       router = {},
       columnStyleConfig = {},
+      currentActionTab: path = '',
     } = this.props;
 
     const {
@@ -685,7 +687,7 @@ class TaskView extends React.PureComponent {
       router,
       udata,
     };
-
+    console.log(path);
     const descriptionTaskProps = {
       ...commonProps,
       routeDataActive,
@@ -694,6 +696,7 @@ class TaskView extends React.PureComponent {
       filesArray,
       onUpdate,
       udata,
+      path,
       rest,
     };
 
@@ -763,7 +766,7 @@ const mapDispatchToProps = (dispatch) => {
   return {
     onCaching: async (props) => await dispatch(middlewareCaching(props)),
     onSaveCache: (props) => dispatch(сachingAction(props)),
-    onUpdate: (props) => dispatch(middlewareUpdate({ ...props })),
+    onUpdate: (props) => dispatch(middlewareUpdate(props)),
   };
 };
 
diff --git a/client/src/Redux/actions/publicActions/middleware/index.js b/client/src/Redux/actions/publicActions/middleware/index.js
index d1d75a2c..0f517282 100644
--- a/client/src/Redux/actions/publicActions/middleware/index.js
+++ b/client/src/Redux/actions/publicActions/middleware/index.js
@@ -178,11 +178,13 @@ const middlewareUpdate = (props = {}) => async (dispatch, getState, { schema, Re
     store = {},
     actionType = 'default',
     updateBy = '_id',
+    parsedRoutePath,
   } = props;
   const rest = new Request();
   /**
    * update by @property {string} id more prioritized than @property {string} key
    */
+
   try {
     const path =
       actionType === actionsTypes.$UPDATE_MANY
@@ -201,6 +203,7 @@ const middlewareUpdate = (props = {}) => async (dispatch, getState, { schema, Re
 
     if (error) throw new Error(error);
     const dep = {
+      parsedRoutePath,
       updateBy,
       store,
       schema,
@@ -213,6 +216,7 @@ const middlewareUpdate = (props = {}) => async (dispatch, getState, { schema, Re
     await updateEntityHook(dispatch, dep);
   } catch (error) {
     const depError = {
+      parsedRoutePath,
       store,
       actionType,
       clientDB,
diff --git a/client/src/Redux/reducers/routerStateReducer/index.js b/client/src/Redux/reducers/routerStateReducer/index.js
index ba003bbe..2d4dd57f 100644
--- a/client/src/Redux/reducers/routerStateReducer/index.js
+++ b/client/src/Redux/reducers/routerStateReducer/index.js
@@ -377,17 +377,32 @@ export default handleActions(
       } else return state;
     },
     [UPDATE_ITEM]: (state, { payload }) => {
-      const { routeDataActive, currentActionTab = '', routeData = {}, path = '' } = state;
-      const { id, updateBy = '_id', updaterItem = {}, store = path.split('__')[0] } = payload;
+      const { routeDataActive, routeData = {}, path: currentPath = '' } = state;
+      const {
+        id,
+        updateBy = '_id',
+        updaterItem = {},
+        store = currentPath.split('__')[0],
+        parsedRoutePath = {},
+      } = payload;
+      const { itemId = '', path = '' } = parsedRoutePath || {};
 
       const isExist = routeDataActive && routeDataActive[updateBy];
-      const currentModule = currentActionTab.split('__')[0];
+      const currentModule = path.split('__');
+
+      const [moduleName = '', entityId = ''] = currentModule || [];
+      const itemName = moduleName && entityId ? entityId : moduleName;
 
-      const isExistModule = !!routeData[currentModule];
-      const isExistStore = isExistModule ? !!routeData[currentModule][store] : null;
+      const { [store]: dataArray = [], ...data } =
+        !itemId && routeData[currentModule[0]]
+          ? routeData[itemName]
+          : currentModule[1] && itemId && routeData[itemId]
+          ? routeData[itemId]
+          : {};
+
+      const dataList = Array.isArray(dataArray) && dataArray?.length ? dataArray : data;
 
       const updateCurrent = isExist && routeDataActive[updateBy] === id ? true : false;
-      const dataList = isExistStore && isExistModule ? routeData[currentModule][store] : [];
 
       const newStore =
         Array.isArray(dataList) && dataList?.length
@@ -400,17 +415,21 @@ export default handleActions(
                   }
                 : updaterItem;
             })
-          : null;
-
-      const updateCurrentModule =
-        newStore && newStore.length
-          ? {
-              [currentModule]: {
-                ...routeData[currentModule],
-                [store]: newStore,
-              },
-            }
-          : {};
+          : dataList;
+
+      const updateCurrentModule = dataList
+        ? {
+            [itemName]: {
+              ...routeData[itemName],
+              [store]:
+                Array.isArray(newStore) && newStore?.length
+                  ? newStore
+                  : !!routeData[itemName][store]
+                  ? [...Array.from(routeData[currentModule][store]), newStore]
+                  : [newStore],
+            },
+          }
+        : {};
 
       return {
         ...state,
diff --git a/client/src/Utils/Hooks/coreHook/coreHook.js b/client/src/Utils/Hooks/coreHook/coreHook.js
index 96cfaf8a..76e74a41 100644
--- a/client/src/Utils/Hooks/coreHook/coreHook.js
+++ b/client/src/Utils/Hooks/coreHook/coreHook.js
@@ -67,7 +67,7 @@ const coreUpdaterDataHook = async (dispatch, dep = {}, multiple = false, badNetw
 };
 
 const updateEntityHook = async (dispatch, dep = {}) => {
-  const { store, schema, dataItems, id, updateItemStateAction, updateBy = '' } = dep;
+  const { parsedRoutePath = null, store, schema, dataItems, id, updateItemStateAction, updateBy = '' } = dep;
 
   const schemaTemplate = getStoreSchema(store);
 
@@ -81,6 +81,7 @@ const updateEntityHook = async (dispatch, dep = {}) => {
       updaterItem: dataItems,
       type: 'UPDATE',
       updateBy,
+      parsedRoutePath,
       store,
       id,
     }),
diff --git a/logger.error-audit.json b/logger.error-audit.json
index aecd0d96..f13b23d7 100644
--- a/logger.error-audit.json
+++ b/logger.error-audit.json
@@ -6,19 +6,14 @@
   "auditLog": "logger.error-audit.json",
   "files": [
     {
-      "date": 1593258193859,
-      "name": "logs\\logs-error-2020-06-27-14.log",
-      "hash": "e13e7a5f93099201e4885c5218632dbc"
+      "date": 1593466609490,
+      "name": "logs\\logs-error-2020-06-30-00.log",
+      "hash": "08f41f0f2f00f31bb65b58f6081244f4"
     },
     {
-      "date": 1593304175219,
-      "name": "logs\\logs-error-2020-06-28-03.log",
-      "hash": "61fed90d49ca704c4e56d01130e1206a"
-    },
-    {
-      "date": 1593306800794,
-      "name": "logs\\logs-error-2020-06-28-04.log",
-      "hash": "3ae1dc0d79fcdc43628ee8b606b5e764"
+      "date": 1593468118442,
+      "name": "logs\\logs-error-2020-06-30-01.log",
+      "hash": "11e91c590cba98806569d8c70518370d"
     }
   ]
 }