Move dotenv.IsComplexValue to stores.

Signed-off-by: Felix Fontein <[email protected]>

Author: felixfontein

cmd/sops/main.go

@@ -241,7 +241,7 @@ func main() {
 
 				var env []string
 				for _, item := range tree.Branches[0] {
-					if dotenv.IsComplexValue(item.Value) {
+					if stores.IsComplexValue(item.Value) {
 						return cli.NewExitError(fmt.Errorf("cannot use complex value in environment; key is %s", item.Key), codes.ErrorGeneric)
 					}
 					if _, ok := item.Key.(sops.Comment); ok {

stores/dotenv/store.go

@@ -138,7 +138,7 @@ func (store *Store) EmitEncryptedFile(in sops.Tree) ([]byte, error) {
 func (store *Store) EmitPlainFile(in sops.TreeBranches) ([]byte, error) {
 	buffer := bytes.Buffer{}
 	for _, item := range in[0] {
-		if IsComplexValue(item.Value) {
+		if stores.IsComplexValue(item.Value) {
 			return nil, fmt.Errorf("cannot use complex value in dotenv file; key is %s", item.Key)
 		}
 		var line string
@@ -176,14 +176,9 @@ func (store *Store) EmitExample() []byte {
 	return bytes
 }
 
+// DEPRECATED, use stores.IsComplexValue() instead!
 func IsComplexValue(v interface{}) bool {
-	switch v.(type) {
-	case []interface{}:
-		return true
-	case sops.TreeBranch:
-		return true
-	}
-	return false
+	return stores.IsComplexValue(v)
 }
 
 // HasSopsTopLevelKey checks whether a top-level "sops" key exists.

stores/stores.go

@@ -535,6 +535,17 @@ func HasSopsTopLevelKey(branch sops.TreeBranch) bool {
 	return false
 }
 
+// IsComplexValue returns true if the given value is an array or dictionary/hash.
+func IsComplexValue(v interface{}) bool {
+	switch v.(type) {
+	case []interface{}:
+		return true
+	case sops.TreeBranch:
+		return true
+	}
+	return false
+}
+
 // ValToString converts a simple value to a string.
 // It does not handle complex values (arrays and mappings).
 func ValToString(v interface{}) string {