- client: fix logic of top clients normalization

client/src/components/Settings/Clients/AutoClients.js

@@ -38,7 +38,7 @@ class AutoClients extends Component {
         },
         {
             Header: this.props.t('requests_count'),
-            accessor: row => this.props.normalizedTopClients[row.ip] || 0,
+            accessor: row => this.props.normalizedTopClients.auto[row.ip] || 0,
             sortMethod: (a, b) => b - a,
             id: 'statistics',
             minWidth: COLUMN_MIN_WIDTH,

client/src/components/Settings/Clients/ClientsTable.js

@@ -40,6 +40,7 @@ class ClientsTable extends Component {
         } else {
             this.handleFormAdd(config);
         }
+        this.props.getStats();
     };
 
     getClient = (name, clients) => {
@@ -64,6 +65,7 @@ class ClientsTable extends Component {
         // eslint-disable-next-line no-alert
         if (window.confirm(this.props.t('client_confirm_delete', { key: data.name }))) {
             this.props.deleteClient(data);
+            this.props.getStats();
         }
     };
 
@@ -161,7 +163,7 @@ class ClientsTable extends Component {
         {
             Header: this.props.t('requests_count'),
             id: 'statistics',
-            accessor: row => this.props.normalizedTopClients[row.name] || 0,
+            accessor: row => this.props.normalizedTopClients.configured[row.name] || 0,
             sortMethod: (a, b) => b - a,
             minWidth: 120,
             Cell: (row) => {
@@ -305,6 +307,7 @@ ClientsTable.propTypes = {
     processingAdding: PropTypes.bool.isRequired,
     processingDeleting: PropTypes.bool.isRequired,
     processingUpdating: PropTypes.bool.isRequired,
+    getStats: PropTypes.func.isRequired,
 };
 
 export default withNamespaces()(ClientsTable);

client/src/components/Settings/Clients/index.js

@@ -23,6 +23,7 @@ class Clients extends Component {
             updateClient,
             deleteClient,
             toggleClientModal,
+            getStats,
         } = this.props;
 
         return (
@@ -44,6 +45,7 @@ class Clients extends Component {
                             processingAdding={clients.processingAdding}
                             processingDeleting={clients.processingDeleting}
                             processingUpdating={clients.processingUpdating}
+                            getStats={getStats}
                         />
                         <AutoClients
                             autoClients={dashboard.autoClients}

client/src/helpers/helpers.js

@@ -261,11 +261,26 @@ export const redirectToCurrentProtocol = (values, httpPort = 80) => {
 
 export const normalizeTextarea = text => text && text.replace(/[;, ]/g, '\n').split('\n').filter(n => n);
 
-export const normalizeTopClients = clients => clients.reduce((accumulator, clientObj) => {
-    const { name, count } = clientObj;
+/**
+ * Normalizes the topClients array
+ *
+ * @param {Object[]} topClients
+ * @param {string} topClients.name
+ * @param {number} topClients.count
+ * @param {Object} topClients.info
+ * @param {string} topClients.info.name
+ * @returns {Object} normalizedTopClients
+ * @returns {Object.<string, number>} normalizedTopClients.auto - auto clients
+ * @returns {Object.<string, number>} normalizedTopClients.configured - configured clients
+ */
+
+export const normalizeTopClients = topClients => topClients.reduce((nameToCountMap, clientObj) => {
+    const { name, count, info: { name: infoName } } = clientObj;
     // eslint-disable-next-line no-param-reassign
-    accumulator[name] = count;
-    return accumulator;
+    nameToCountMap.auto = { ...nameToCountMap.auto, [name]: count };
+    // eslint-disable-next-line no-param-reassign
+    nameToCountMap.configured = { ...nameToCountMap.configured, [infoName]: count };
+    return nameToCountMap;
 }, {});
 
 export const getClientInfo = (clients, ip) => {