fix(Network): update data on path change

Author: artemmufazalov

src/containers/Tenant/Diagnostics/Network/Network.js

@@ -12,7 +12,7 @@ import Icon from '../../../../components/Icon/Icon';
 import ProblemFilter, {problemFilterType} from '../../../../components/ProblemFilter/ProblemFilter';
 import {Illustration} from '../../../../components/Illustration';
 
-import {getNetworkInfo} from '../../../../store/reducers/network';
+import {getNetworkInfo, setDataWasNotLoaded} from '../../../../store/reducers/network';
 import {hideTooltip, showTooltip} from '../../../../store/reducers/tooltip';
 import {ALL, PROBLEMS} from '../../../../utils/constants';
 import {changeFilter} from '../../../../store/reducers/settings';
@@ -26,6 +26,7 @@ const b = cn('network');
 class Network extends React.Component {
     static propTypes = {
         getNetworkInfo: PropTypes.func,
+        setDataWasNotLoaded: PropTypes.func,
         netWorkInfo: PropTypes.object,
         hideTooltip: PropTypes.func,
         showTooltip: PropTypes.func,
@@ -75,16 +76,27 @@ class Network extends React.Component {
     }
 
     componentDidUpdate(prevProps) {
-        const {autorefresh, path} = this.props;
+        const {autorefresh, path, setDataWasNotLoaded, getNetworkInfo} = this.props;
 
-        if (autorefresh && !prevProps.autorefresh) {
-            getNetworkInfo(path);
+        const restartAutorefresh = () => {
+            this.autofetcher.stop();
             this.autofetcher.start();
             this.autofetcher.fetch(() => getNetworkInfo(path));
+        };
+
+        if (autorefresh && !prevProps.autorefresh) {
+            getNetworkInfo(path);
+            restartAutorefresh();
         }
         if (!autorefresh && prevProps.autorefresh) {
             this.autofetcher.stop();
         }
+
+        if (path !== prevProps.path) {
+            setDataWasNotLoaded();
+            getNetworkInfo(path);
+            restartAutorefresh();
+        }
     }
 
     componentWillUnmount() {
@@ -364,6 +376,7 @@ const mapDispatchToProps = {
     hideTooltip,
     showTooltip,
     changeFilter,
+    setDataWasNotLoaded,
 };
 
 export default connect(mapStateToProps, mapDispatchToProps)(Network);

src/store/reducers/network.js

@@ -6,7 +6,15 @@ const FETCH_ALL_NODES_NETWORK = createRequestActionTypes(
     'FETCH_ALL_NODES_NETWORK',
 );
 
-const network = (state = {data: {}, loading: true, wasLoaded: false}, action) => {
+const SET_DATA_WAS_NOT_LOADED = 'network/SET_DATA_WAS_NOT_LOADED';
+
+const initialState = {
+    data: {},
+    loading: false,
+    wasLoaded: false,
+};
+
+const network = (state = initialState, action) => {
     switch (action.type) {
         case FETCH_ALL_NODES_NETWORK.REQUEST: {
             return {
@@ -30,11 +38,24 @@ const network = (state = {data: {}, loading: true, wasLoaded: false}, action) =>
                 loading: false,
             };
         }
+
+        case SET_DATA_WAS_NOT_LOADED: {
+            return {
+                ...state,
+                wasLoaded: false,
+            };
+        }
         default:
             return state;
     }
 };
 
+export const setDataWasNotLoaded = () => {
+    return {
+        type: SET_DATA_WAS_NOT_LOADED,
+    };
+};
+
 export const getNetworkInfo = (tenant) => {
     return createApiRequest({
         request: window.api.getNetwork(tenant),