From: Bjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>
Date: Sat, 30 Jul 2016 14:21:57 +0000 (+0200)
Subject: Do not return empty theme dirs
X-Git-Tag: v0.17~206
X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=5f5fccbc36c1072150d1b665d9d9ef93a5401b82;p=brevno-suite%2Fhugo

Do not return empty theme dirs

This prevents reading data etc. from the root.

Fixes #2320
---

diff --git a/helpers/path.go b/helpers/path.go
index a31f3394..1d3419fb 100644
--- a/helpers/path.go
+++ b/helpers/path.go
@@ -183,13 +183,15 @@ func GetThemeDataDirPath() (string, error) {
 }
 
 func getThemeDirPath(path string) (string, error) {
-	var themeDir string
-	if ThemeSet() {
-		themeDir = filepath.Join(GetThemeDir(), path)
-		if _, err := os.Stat(themeDir); os.IsNotExist(err) {
-			return "", fmt.Errorf("Unable to find %s directory for theme %s in %s", path, viper.GetString("theme"), themeDir)
-		}
+	if !ThemeSet() {
+		return "", errors.New("No theme set")
 	}
+
+	themeDir := filepath.Join(GetThemeDir(), path)
+	if _, err := os.Stat(themeDir); os.IsNotExist(err) {
+		return "", fmt.Errorf("Unable to find %s directory for theme %s in %s", path, viper.GetString("theme"), themeDir)
+	}
+
 	return themeDir, nil
 }