Merge pull request #147 from RedisInsight/bugfix/RI-1878

rsergeenko · web-flow · commit ca00d2d91484 · 2021-12-23T16:10:04.000+03:00
#RI-1878 - fix inserting optional arguments for auto-suggestions
diff --git a/redisinsight/ui/src/utils/commands.ts b/redisinsight/ui/src/utils/commands.ts
@@ -30,7 +30,7 @@ const generateArgName = (
   const multipleName = optional ? `[${multipleNameTemp.join(' ')}]` : multipleNameTemp
 
   if (type === CommandArgsType.Block && isArray(block)) {
-    const blocks = flatten(block?.map?.((block) => generateArgName(block, pureName)))
+    const blocks = flatten(block?.map?.((block) => generateArgName(block, pureName, onlyMandatory)))
     return optional ? `[${blocks?.join?.(' ')}]` : blocks
   }
 

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ const generateArgName = (`
`30`	`30`	const multipleName = optional ? `[${multipleNameTemp.join(' ')}]` : multipleNameTemp
`31`	`31`
`32`	`32`	`if (type === CommandArgsType.Block && isArray(block)) {`
`33`		`- const blocks = flatten(block?.map?.((block) => generateArgName(block, pureName)))`
	`33`	`+ const blocks = flatten(block?.map?.((block) => generateArgName(block, pureName, onlyMandatory)))`
`34`	`34`	return optional ? `[${blocks?.join?.(' ')}]` : blocks
`35`	`35`	`}`
`36`	`36`