#RI-3558 - fix pr comments

Author: rsergeenko

redisinsight/ui/src/pages/databaseAnalysis/components/analysis-data-view/AnalysisDataView.tsx

@@ -4,7 +4,7 @@ import { isNull } from 'lodash'
 import { useParams } from 'react-router-dom'
 import { sendEventTelemetry, TelemetryEvent } from 'uiSrc/telemetry'
 import { Nullable } from 'uiSrc/utils'
-import { EmptyMessage } from 'uiSrc/pages/databaseAnalysis/constants'
+import { DEFAULT_EXTRAPOLATION, EmptyMessage } from 'uiSrc/pages/databaseAnalysis/constants'
 import {
   TopKeys,
   EmptyAnalysisMessage,
@@ -24,7 +24,7 @@ export interface Props {
 
 const AnalysisDataView = (props: Props) => {
   const { loading, reports = [], data } = props
-  const [extrapolation, setExtrapolation] = useState(1)
+  const [extrapolation, setExtrapolation] = useState(DEFAULT_EXTRAPOLATION)
 
   const { instanceId } = useParams<{ instanceId: string }>()
 

redisinsight/ui/src/pages/databaseAnalysis/components/analysis-ttl-view/ExpirationGroupsView.tsx

@@ -4,6 +4,7 @@ import AutoSizer from 'react-virtualized-auto-sizer'
 import { useDispatch, useSelector } from 'react-redux'
 import cx from 'classnames'
 
+import { DEFAULT_EXTRAPOLATION } from 'uiSrc/pages/databaseAnalysis'
 import { extrapolate, formatBytes, formatExtrapolation, Nullable } from 'uiSrc/utils'
 import { AreaChart } from 'uiSrc/components/charts'
 import { AreaChartData, AreaChartDataType, DEFAULT_MULTIPLIER_GRID } from 'uiSrc/components/charts/area-chart/AreaChart'
@@ -29,7 +30,7 @@ const ExpirationGroupsView = (props: Props) => {
   const dispatch = useDispatch()
 
   useEffect(() => {
-    setIsExtrapolated(extrapolation !== 1)
+    setIsExtrapolated(extrapolation !== DEFAULT_EXTRAPOLATION)
   }, [extrapolation])
 
   useEffect(() => {
@@ -80,7 +81,7 @@ const ExpirationGroupsView = (props: Props) => {
           <EuiTitle className="section-title">
             <h4>MEMORY LIKELY TO BE FREED OVER TIME</h4>
           </EuiTitle>
-          {extrapolation !== 1 && (
+          {extrapolation !== DEFAULT_EXTRAPOLATION && (
             <EuiSwitch
               compressed
               color="subdued"

redisinsight/ui/src/pages/databaseAnalysis/components/summary-per-data/SummaryPerData.tsx

@@ -6,6 +6,7 @@ import { DonutChart } from 'uiSrc/components/charts'
 import { ChartData } from 'uiSrc/components/charts/donut-chart/DonutChart'
 import { KeyIconSvg, MemoryIconSvg } from 'uiSrc/components/database-overview/components/icons'
 import { GROUP_TYPES_COLORS, GroupTypesColors } from 'uiSrc/constants'
+import { DEFAULT_EXTRAPOLATION } from 'uiSrc/pages/databaseAnalysis'
 import { extrapolate, formatBytes, getGroupTypeDisplay, Nullable } from 'uiSrc/utils'
 import { getPercentage, numberWithSpaces } from 'uiSrc/utils/numbers'
 
@@ -43,7 +44,7 @@ const SummaryPerData = ({ data, loading, extrapolation, onSwitchExtrapolation }:
   }
 
   useEffect(() => {
-    setIsExtrapolated(extrapolation !== 1)
+    setIsExtrapolated(extrapolation !== DEFAULT_EXTRAPOLATION)
   }, [data, extrapolation])
 
   useEffect(() => {
@@ -122,7 +123,7 @@ const SummaryPerData = ({ data, loading, extrapolation, onSwitchExtrapolation }:
         <EuiTitle className="section-title">
           <h4>SUMMARY PER DATA TYPE</h4>
         </EuiTitle>
-        {extrapolation !== 1 && (
+        {extrapolation !== DEFAULT_EXTRAPOLATION && (
           <EuiSwitch
             compressed
             color="subdued"

redisinsight/ui/src/pages/databaseAnalysis/components/top-namespace/TopNamespace.tsx

@@ -2,7 +2,7 @@ import React, { useEffect, useState } from 'react'
 import cx from 'classnames'
 import { EuiButton, EuiSwitch, EuiTitle } from '@elastic/eui'
 import { Nullable } from 'uiSrc/utils'
-import { TableView } from 'uiSrc/pages/databaseAnalysis'
+import { DEFAULT_EXTRAPOLATION, TableView } from 'uiSrc/pages/databaseAnalysis'
 import { TableLoader } from 'uiSrc/pages/databaseAnalysis/components'
 import { DatabaseAnalysis } from 'apiSrc/modules/database-analysis/models'
 
@@ -22,7 +22,7 @@ const TopNamespace = (props: Props) => {
   const [isExtrapolated, setIsExtrapolated] = useState<boolean>(true)
 
   useEffect(() => {
-    setIsExtrapolated(extrapolation !== 1)
+    setIsExtrapolated(extrapolation !== DEFAULT_EXTRAPOLATION)
   }, [data, extrapolation])
 
   if (loading) {
@@ -61,7 +61,7 @@ const TopNamespace = (props: Props) => {
         >
           by Number of Keys
         </EuiButton>
-        {extrapolation !== 1 && (
+        {extrapolation !== DEFAULT_EXTRAPOLATION && (
           <EuiSwitch
             compressed
             color="subdued"

redisinsight/ui/src/pages/databaseAnalysis/constants.ts

@@ -15,3 +15,5 @@ export type Content = {
   title: string
   text: (path: string) => ReactNode
 }
+
+export const DEFAULT_EXTRAPOLATION = 1