Fix:(issue_1860) Remove hidden flags from flag categories

urfave · Feb 17, 2024 · 7d74b97 · 7d74b97
1 parent 094a9bc
commit 7d74b97
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/category.go b/category.go
@@ -104,7 +104,7 @@ func newFlagCategoriesFromFlags(fs []Flag) FlagCategories {
 	var categorized bool
 	for _, fl := range fs {
 		if cf, ok := fl.(CategorizableFlag); ok {
-			if cat := cf.GetCategory(); cat != "" {
+			if cat := cf.GetCategory(); cat != "" && cf.IsVisible() {
 				fc.AddFlag(cat, cf)
 				categorized = true
 			}
@@ -114,7 +114,7 @@ func newFlagCategoriesFromFlags(fs []Flag) FlagCategories {
 	if categorized {
 		for _, fl := range fs {
 			if cf, ok := fl.(CategorizableFlag); ok {
-				if cf.GetCategory() == "" {
+				if cf.GetCategory() == "" && cf.IsVisible() {
 					fc.AddFlag("", fl)
 				}
 			}

diff --git a/command_test.go b/command_test.go
@@ -466,6 +466,11 @@ func TestCommand_VisibleFlagCategories(t *testing.T) {
 				Aliases:  []string{"altd1", "altd2"},
 				Category: "cat1",
 			},
+			&StringFlag{
+				Name:     "sfd",
+				Category: "cat2",
+				Hidden:   true,
+			},
 		},
 	}