IniParser: write 'func' options with "ini-name"

If a 'ini-name' tag is set on the option, write it to file even if the option is a function.
jessevdk · Dec 12, 2019 · e9ecd9d · e9ecd9d
1 parent c0795c8
commit e9ecd9d
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/ini.go b/ini.go
@@ -216,7 +216,7 @@ func writeGroupIni(cmd *Command, group *Group, namespace string, writer io.Write
 	comments := (options & IniIncludeComments) != IniNone
 
 	for _, option := range group.options {
-		if option.isFunc() || option.Hidden {
+		if option.Hidden || option.isFunc() && len(option.tag.Get("ini-name")) == 0 {
 			continue
 		}
 

diff --git a/ini_test.go b/ini_test.go
@@ -49,6 +49,9 @@ int-map = b:3
 verbose = true
 verbose = true
 
+; Call phone number
+call =
+
 ; A slice of pointers to string
 ; PtrSlice =
 
@@ -113,6 +116,9 @@ Opt =
 ; Show verbose debug information
 ; verbose =
 
+; Call phone number
+; call =
+
 ; A slice of pointers to string
 ; PtrSlice =
 
@@ -176,6 +182,9 @@ EnvDefault2 = env-def
 ; Show verbose debug information
 ; verbose =
 
+; Call phone number
+; call =
+
 ; A slice of pointers to string
 ; PtrSlice =