shithub: hugo

Download patch

ref: 85a2d81e3c1f62b561e6c2715cc406446879d09e
parent: 4b4ab4755339a8c1d034be1f75b323d585dac1df
author: Albert Nigmatzianov <[email protected]>
date: Sun Nov 6 19:10:32 EST 2016 

commands, tpl: Get rid of repeating viper accesses

* all: Delete some blank lines
* commands, tpl: Get rid of repeating viper accesses


--- a/commands/convert.go
+++ b/commands/convert.go
@@ -103,6 +103,7 @@
 		return fmt.Errorf("No source files found")
 	}
 
+	contentDir := helpers.AbsPathify(viper.GetString("contentDir"))
 	jww.FEEDBACK.Println("processing", len(site.Source.Files()), "content files")
 	for _, file := range site.Source.Files() {
 		jww.INFO.Println("Attempting to convert", file.LogicalName())
@@ -134,7 +135,7 @@
 			metadata = newmetadata
 		}
 
-		page.SetDir(filepath.Join(helpers.AbsPathify(viper.GetString("contentDir")), file.Dir()))
+		page.SetDir(filepath.Join(contentDir, file.Dir()))
 		page.SetSourceContent(psr.Content())
 		if err = page.SetSourceMetaData(metadata, mark); err != nil {
 			jww.ERROR.Printf("Failed to set source metadata for file %q: %s. For more info see For more info see https://github.com/spf13/hugo/issues/2458", page.FullFilePath(), err)
--- a/commands/hugo.go
+++ b/commands/hugo.go
@@ -426,7 +426,6 @@
 }
 
 func build(watches ...bool) error {
-
 	// Hugo writes the output to memory instead of the disk
 	// This is only used for benchmark testing. Cause the content is only visible
 	// in memory
@@ -786,7 +785,7 @@
 						jww.FEEDBACK.Printf("Syncing all static files\n")
 						err := copyStatic()
 						if err != nil {
-							utils.StopOnErr(err, fmt.Sprintf("Error copying static files to %s", helpers.AbsPathify(viper.GetString("publishDir"))))
+							utils.StopOnErr(err, fmt.Sprintf("Error copying static files to %s", publishDir))
 						}
 					} else {
 						staticSourceFs := getStaticSourceFs()
--- a/helpers/pygments.go
+++ b/helpers/pygments.go
@@ -42,7 +42,6 @@
 
 // Highlight takes some code and returns highlighted code.
 func Highlight(code, lang, optsStr string) string {
-
 	if !HasPygments() {
 		jww.WARN.Println("Highlighting requires Pygments to be installed and in the path")
 		return code
@@ -197,7 +196,6 @@
 }
 
 func parseDefaultPygmentsOpts() (map[string]string, error) {
-
 	options := make(map[string]string)
 	err := parseOptions(options, viper.GetString("pygmentsOptions"))
 	if err != nil {
@@ -225,7 +223,6 @@
 }
 
 func parsePygmentsOpts(in string) (string, error) {
-
 	options, err := parseDefaultPygmentsOpts()
 	if err != nil {
 		return "", err
--- a/hugolib/config.go
+++ b/hugolib/config.go
@@ -22,7 +22,6 @@
 
 // LoadGlobalConfig loads Hugo configuration into the global Viper.
 func LoadGlobalConfig(relativeSourcePath, configFilename string) error {
-
 	if relativeSourcePath == "" {
 		relativeSourcePath = "."
 	}
--- a/hugolib/handler_page.go
+++ b/hugolib/handler_page.go
@@ -105,7 +105,6 @@
 }
 
 func commonConvert(p *Page, t tpl.Template) HandledResult {
-
 	if p.rendered {
 		panic(fmt.Sprintf("Page %q already rendered, does not need conversion", p.BaseFileName()))
 	}
--- a/hugolib/page.go
+++ b/hugolib/page.go
@@ -584,7 +584,6 @@
 		if err != nil {
 			return nil, err
 		}
-		// fmt.Printf("have a section override for %q in section %s → %s\n", p.Title, p.Section, permalink)
 	} else {
 		if len(pSlug) > 0 {
 			permalink = helpers.URLPrep(viper.GetBool("uglyURLs"), path.Join(dir, p.Slug+"."+p.Extension()))
--- a/hugolib/site.go
+++ b/hugolib/site.go
@@ -917,7 +917,6 @@
 }
 
 func (s *Site) initializeSiteInfo() {
-
 	var (
 		lang      *helpers.Language = s.Language
 		languages helpers.Languages
@@ -1980,7 +1979,6 @@
 		}
 
 		if n.paginator != nil {
-
 			paginatePath := helpers.Config().GetString("paginatePath")
 
 			// write alias for page 1
@@ -2029,7 +2027,6 @@
 }
 
 func (s *Site) renderHomePage(prepare bool) error {
-
 	n := s.newHomeNode(prepare, 0)
 	if prepare {
 		return nil
--- a/tpl/template_i18n.go
+++ b/tpl/template_i18n.go
@@ -65,6 +65,7 @@
 		jww.WARN.Printf("No translation bundle found for default language %q", defaultContentLanguage)
 	}
 
+	enableMissingTranslationPlaceholders := viper.GetBool("enableMissingTranslationPlaceholders")
 	for _, lang := range bndl.LanguageTags() {
 		currentLang := lang
 
@@ -78,7 +79,7 @@
 			if Logi18nWarnings {
 				i18nWarningLogger.Printf("i18n|MISSING_TRANSLATION|%s|%s", currentLang, translationID)
 			}
-			if viper.GetBool("enableMissingTranslationPlaceholders") {
+			if enableMissingTranslationPlaceholders {
 				return fmt.Sprintf("[i18n] %s", translationID)
 			}
 			if defaultT != nil {
--- a/tpl/template_resources.go
+++ b/tpl/template_resources.go
@@ -114,7 +114,6 @@
 
 // resGetRemote loads the content of a remote file. This method is thread safe.
 func resGetRemote(url string, fs afero.Fs, hc *http.Client) ([]byte, error) {
-
 	c, err := resGetCache(url, fs, viper.GetBool("ignoreCache"))
 	if c != nil && err == nil {
 		return c, nil