From 27a524b0905ec73c1eef233f94700feb9f465011 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Bj=C3=B8rn=20Erik=20Pedersen?=
 <bjorn.erik.pedersen@gmail.com>
Date: Sat, 14 Apr 2018 10:34:02 +0200
Subject: [PATCH] commands: Properly handle CLI slice arguments

Like `--disableKinds` -- this handling was kind of broken when we recently moved this from global vars

See #4607
---
 commands/commands_test.go |  5 +++++
 commands/hugo.go          | 15 ++++++++++++++-
 2 files changed, 19 insertions(+), 1 deletion(-)

diff --git a/commands/commands_test.go b/commands/commands_test.go
index 5173e740..2eefaa2e 100644
--- a/commands/commands_test.go
+++ b/commands/commands_test.go
@@ -57,8 +57,10 @@ func TestCommandsPersistentFlags(t *testing.T) {
 	}{{[]string{"server",
 		"--config=myconfig.toml",
 		"--contentDir=mycontent",
+		"--disableKinds=page,home",
 		"--layoutDir=mylayouts",
 		"--theme=mytheme",
+		"--gc",
 		"--themesDir=mythemes",
 		"--cleanDestinationDir",
 		"--navigateToChanged",
@@ -100,7 +102,10 @@ func TestCommandsPersistentFlags(t *testing.T) {
 		assert.Equal("mytheme", cfg.GetString("theme"))
 		assert.Equal("mythemes", cfg.GetString("themesDir"))
 
+		assert.Equal([]string{"page", "home"}, cfg.Get("disableKinds"))
+
 		assert.True(cfg.GetBool("uglyURLs"))
+		assert.True(cfg.GetBool("gc"))
 
 		// The flag is named i18n-warnings
 		assert.True(cfg.GetBool("logI18nWarnings"))
diff --git a/commands/hugo.go b/commands/hugo.go
index 5c301c7f..41c06c02 100644
--- a/commands/hugo.go
+++ b/commands/hugo.go
@@ -243,7 +243,20 @@ If you need to set this configuration value from the command line, set it via an
 		if targetKey != "" {
 			configKey = targetKey
 		}
-		cfg.Set(configKey, f.Value.String())
+		// Gotta love this API.
+		switch f.Value.Type() {
+		case "bool":
+			bv, _ := flags.GetBool(key)
+			cfg.Set(configKey, bv)
+		case "string":
+			cfg.Set(configKey, f.Value.String())
+		case "stringSlice":
+			bv, _ := flags.GetStringSlice(key)
+			cfg.Set(configKey, bv)
+		default:
+			panic(fmt.Sprintf("update switch with %s", f.Value.Type()))
+		}
+
 	}
 }
 
-- 
2.30.2