* #RI-3848 - Button is displayed with incorrect scanned parameters

* Upper case for all commands in workbench

Author: egor-zalenski

redisinsight/ui/src/components/query/QueryWrapper.tsx

@@ -80,7 +80,7 @@ const QueryWrapper = (props: Props) => {
       const command = [commandLine, multiCommands].join('') ? [commandLine, multiCommands].join(';') : null
 
       return {
-        command,
+        command: command?.toUpperCase(),
         databaseId: instanceId,
         multiple: multiCommands ? 'Multiple' : 'Single',
         pipeline: batchSize > 1,

redisinsight/ui/src/pages/browser/components/browser-left-panel/BrowserLeftPanel.tsx

@@ -54,13 +54,14 @@ const BrowserLeftPanel = (props: Props) => {
   const { loading: patternLoading, viewType, searchMode, isSearched: patternIsSearched } = useSelector(keysSelector)
   const { contextInstanceId } = useSelector(appContextSelector)
   const {
-    keyList: { isDataLoaded, scrollPatternTopPosition, scrollRedisearchTopPosition }
+    keyList: { isDataPatternLoaded, isDataRedisearchLoaded, scrollPatternTopPosition, scrollRedisearchTopPosition }
   } = useSelector(appContextBrowser)
 
   const keyListRef = useRef<any>()
 
   const dispatch = useDispatch()
 
+  const isDataLoaded = searchMode === SearchMode.Pattern ? isDataPatternLoaded : isDataRedisearchLoaded
   const keysState = searchMode === SearchMode.Pattern ? patternKeysState : redisearchKeysState
   const loading = searchMode === SearchMode.Pattern ? patternLoading : redisearchLoading || redisearchListLoading
   const isSearched = searchMode === SearchMode.Pattern ? patternIsSearched : redisearchIsSearched
@@ -79,8 +80,8 @@ const BrowserLeftPanel = (props: Props) => {
       searchMode,
       '0',
       keyViewType === KeyViewType.Browser ? SCAN_COUNT_DEFAULT : SCAN_TREE_COUNT_DEFAULT,
-      () => dispatch(setBrowserKeyListDataLoaded(true)),
-      () => dispatch(setBrowserKeyListDataLoaded(false))
+      () => dispatch(setBrowserKeyListDataLoaded(searchMode, true)),
+      () => dispatch(setBrowserKeyListDataLoaded(searchMode, false))
     ))
   }, [searchMode])
 

redisinsight/ui/src/pages/browser/components/key-list/KeyList.tsx

@@ -33,6 +33,7 @@ import {
   fetchKeysMetadata,
   keysDataSelector,
   keysSelector,
+  resetKeysData,
   selectedKeySelector,
   setLastBatchKeys,
   sourceKeysFetch,
@@ -117,6 +118,7 @@ const KeyList = forwardRef((props: Props, ref) => {
     if (itemsRef.current.length === 0) {
       cancelAllMetadataRequests()
       setFirstDataLoaded(true)
+      dispatch(resetKeysData(searchMode))
       rerender({})
       return
     }

redisinsight/ui/src/pages/browser/components/keys-header/KeysHeader.tsx

@@ -168,8 +168,8 @@ const KeysHeader = (props: Props) => {
       searchMode,
       '0',
       viewType === KeyViewType.Browser ? SCAN_COUNT_DEFAULT : SCAN_TREE_COUNT_DEFAULT,
-      () => dispatch(setBrowserKeyListDataLoaded(true)),
-      () => dispatch(setBrowserKeyListDataLoaded(false)),
+      () => dispatch(setBrowserKeyListDataLoaded(searchMode, true)),
+      () => dispatch(setBrowserKeyListDataLoaded(searchMode, false)),
     ))
   }
 
@@ -229,11 +229,14 @@ const KeysHeader = (props: Props) => {
         }
       })
     }
-    dispatch(resetKeysData(searchMode))
-    dispatch(changeKeyViewType(type))
     dispatch(resetBrowserTree())
+    dispatch(resetKeysData(searchMode))
     localStorageService.set(BrowserStorageItem.browserViewType, type)
     loadKeys(type)
+
+    setTimeout(() => {
+      dispatch(changeKeyViewType(type))
+    }, 0)
   }
 
   const handleSwitchSearchMode = (mode: SearchMode) => {

redisinsight/ui/src/pages/databaseAnalysis/components/top-keys/Table.tsx

@@ -59,8 +59,8 @@ const Table = (props: Props) => {
       SearchMode.Pattern,
       '0',
       viewType === KeyViewType.Browser ? SCAN_COUNT_DEFAULT : SCAN_TREE_COUNT_DEFAULT,
-      () => dispatch(setBrowserKeyListDataLoaded(true)),
-      () => dispatch(setBrowserKeyListDataLoaded(false)),
+      () => dispatch(setBrowserKeyListDataLoaded(SearchMode.Pattern, true)),
+      () => dispatch(setBrowserKeyListDataLoaded(SearchMode.Pattern, false)),
     ))
     dispatch(resetBrowserTree())
     dispatch(setBrowserSelectedKey(stringToBuffer(name)))

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

@@ -71,8 +71,8 @@ const NameSpacesTable = (props: Props) => {
       SearchMode.Pattern,
       '0',
       viewType === KeyViewType.Browser ? SCAN_COUNT_DEFAULT : SCAN_TREE_COUNT_DEFAULT,
-      () => dispatch(setBrowserKeyListDataLoaded(true)),
-      () => dispatch(setBrowserKeyListDataLoaded(false)),
+      () => dispatch(setBrowserKeyListDataLoaded(SearchMode.Pattern, true)),
+      () => dispatch(setBrowserKeyListDataLoaded(SearchMode.Pattern, false)),
     ))
     dispatch(resetBrowserTree())
     history.push(Pages.browser(instanceId))

redisinsight/ui/src/pages/workbench/WorkbenchPage.spec.tsx

@@ -167,7 +167,7 @@ describe('Telemetry', () => {
     expect(sendEventTelemetry).toBeCalledWith({
       event: TelemetryEvent.WORKBENCH_COMMAND_SUBMITTED,
       eventData: {
-        command: 'info;',
+        command: 'info;'.toUpperCase(),
         databaseId: INSTANCE_ID_MOCK,
         group: false,
         multiple: 'Single',
@@ -205,7 +205,7 @@ describe('Telemetry', () => {
     expect(sendEventTelemetry).toBeCalledWith({
       event: TelemetryEvent.WORKBENCH_COMMAND_SUBMITTED,
       eventData: {
-        command: 'info;',
+        command: 'info;'.toUpperCase(),
         databaseId: INSTANCE_ID_MOCK,
         group: false,
         multiple: 'Single',

redisinsight/ui/src/slices/app/context.ts

@@ -5,13 +5,15 @@ import { BrowserStorageItem, DEFAULT_DELIMITER, KeyTypes } from 'uiSrc/constants
 import { localStorageService } from 'uiSrc/services'
 import { RootState } from '../store'
 import { RedisResponseBuffer, StateAppContext } from '../interfaces'
+import { SearchMode } from '../interfaces/keys'
 
 export const initialState: StateAppContext = {
   contextInstanceId: '',
   lastPage: '',
   browser: {
     keyList: {
-      isDataLoaded: false,
+      isDataPatternLoaded: false,
+      isDataRedisearchLoaded: false,
       scrollPatternTopPosition: 0,
       scrollRedisearchTopPosition: 0,
       isNotRendered: true,
@@ -73,8 +75,11 @@ const appContextSlice = createSlice({
     setBrowserSelectedKey: (state, { payload }: { payload: Nullable<RedisResponseBuffer> }) => {
       state.browser.keyList.selectedKey = payload
     },
-    setBrowserKeyListDataLoaded: (state, { payload }: { payload: boolean }) => {
-      state.browser.keyList.isDataLoaded = payload
+    setBrowserPatternKeyListDataLoaded: (state, { payload }: { payload: boolean }) => {
+      state.browser.keyList.isDataPatternLoaded = payload
+    },
+    setBrowserRedisearchKeyListDataLoaded: (state, { payload }: { payload: boolean }) => {
+      state.browser.keyList.isDataRedisearchLoaded = payload
     },
     setBrowserPatternScrollPosition: (state, { payload }: { payload: number }) => {
       state.browser.keyList.scrollPatternTopPosition = payload
@@ -176,7 +181,8 @@ const appContextSlice = createSlice({
 export const {
   setAppContextInitialState,
   setAppContextConnectedInstanceId,
-  setBrowserKeyListDataLoaded,
+  setBrowserPatternKeyListDataLoaded,
+  setBrowserRedisearchKeyListDataLoaded,
   setBrowserSelectedKey,
   setBrowserPatternScrollPosition,
   setBrowserRedisearchScrollPosition,
@@ -222,3 +228,13 @@ export const appContextAnalytics = (state: RootState) =>
 
 // The reducer
 export default appContextSlice.reducer
+
+// Asynchronous thunk action
+export function setBrowserKeyListDataLoaded(
+  searchMode: SearchMode,
+  value: boolean,
+) {
+  return searchMode === SearchMode.Pattern
+    ? setBrowserPatternKeyListDataLoaded(value)
+    : setBrowserRedisearchKeyListDataLoaded(value)
+}

redisinsight/ui/src/slices/browser/keys.ts

@@ -321,6 +321,8 @@ const keysSlice = createSlice({
 
     resetPatternKeysData: (state) => {
       // state.data.keys = []
+      state.data.total = 0
+      state.data.scanned = 0
       state.data.keys.length = 0
     },
 

redisinsight/ui/src/slices/browser/redisearch.ts

@@ -150,6 +150,8 @@ const redisearchSlice = createSlice({
     },
 
     resetRedisearchKeysData: (state) => {
+      state.data.total = 0
+      state.data.scanned = 0
       state.data.keys.length = 0
     },