From e21b87dfb62f5e925c71592440d4f113710234ad Mon Sep 17 00:00:00 2001
From: "Serge A. Zaitsev" <zaitsev.serge@gmail.com>
Date: Fri, 4 Sep 2015 14:49:56 +0200
Subject: [PATCH] more obvious rules of variable overriding

---
 zs.go | 23 +++++++++++++----------
 1 file changed, 13 insertions(+), 10 deletions(-)

diff --git a/zs.go b/zs.go
index 6b497a7..05eaae6 100644
--- a/zs.go
+++ b/zs.go
@@ -104,21 +104,23 @@ func getVars(path string, globals Vars) (Vars, string, error) {
 	title := strings.Replace(strings.Replace(path, "_", " ", -1), "-", " ", -1)
 	v["title"] = strings.ToTitle(title)
 	v["description"] = ""
+	v["file"] = path
+	v["url"] = path[:len(path)-len(filepath.Ext(path))] + ".html"
+	v["output"] = filepath.Join(PUBDIR, v["url"])
 
-	// Copy globals (will override title and description for markdown layouts
+	// Override default values with globals
 	for name, value := range globals {
 		v[name] = value
 	}
 
-	// Add default values extracted from file name/path
-	if _, err := os.Stat(filepath.Join(ZSDIR, "layout.amber")); err == nil {
-		v["layout"] = "layout.amber"
-	} else {
-		v["layout"] = "layout.html"
+	// Add layout if none is specified
+	if _, ok := v["layout"]; !ok {
+		if _, err := os.Stat(filepath.Join(ZSDIR, "layout.amber")); err == nil {
+			v["layout"] = "layout.amber"
+		} else {
+			v["layout"] = "layout.html"
+		}
 	}
-	v["file"] = path
-	v["url"] = path[:len(path)-len(filepath.Ext(path))] + ".html"
-	v["output"] = filepath.Join(PUBDIR, v["url"])
 
 	delim := "\n---\n"
 	if sep := strings.Index(s, delim); sep == -1 {
@@ -132,6 +134,7 @@ func getVars(path string, globals Vars) (Vars, string, error) {
 			fmt.Println("ERROR: failed to parse header", err)
 			return nil, "", err
 		} else {
+			// Override default values + globals with the ones defines in the file
 			for key, value := range vars {
 				v[key] = value
 			}
@@ -398,7 +401,7 @@ func main() {
 			fmt.Println("var: filename expected")
 		} else {
 			s := ""
-			if vars, _, err := getVars(args[0], globals()); err != nil {
+			if vars, _, err := getVars(args[0], Vars{}); err != nil {
 				fmt.Println("var: " + err.Error())
 			} else {
 				if len(args) > 1 {