commands, tpl: Get rid of repeating viper accesses

* all: Delete some blank lines
* commands, tpl: Get rid of repeating viper accesses
This commit is contained in:
Albert Nigmatzianov 2016-11-07 00:10:32 +01:00 committed by Bjørn Erik Pedersen
parent 4b4ab47553
commit 85a2d81e3c
9 changed files with 5 additions and 14 deletions

View file

@ -103,6 +103,7 @@ func convertContents(mark rune) (err error) {
return fmt.Errorf("No source files found") return fmt.Errorf("No source files found")
} }
contentDir := helpers.AbsPathify(viper.GetString("contentDir"))
jww.FEEDBACK.Println("processing", len(site.Source.Files()), "content files") jww.FEEDBACK.Println("processing", len(site.Source.Files()), "content files")
for _, file := range site.Source.Files() { for _, file := range site.Source.Files() {
jww.INFO.Println("Attempting to convert", file.LogicalName()) jww.INFO.Println("Attempting to convert", file.LogicalName())
@ -134,7 +135,7 @@ func convertContents(mark rune) (err error) {
metadata = newmetadata metadata = newmetadata
} }
page.SetDir(filepath.Join(helpers.AbsPathify(viper.GetString("contentDir")), file.Dir())) page.SetDir(filepath.Join(contentDir, file.Dir()))
page.SetSourceContent(psr.Content()) page.SetSourceContent(psr.Content())
if err = page.SetSourceMetaData(metadata, mark); err != nil { 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) 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)

View file

@ -426,7 +426,6 @@ func watchConfig() {
} }
func build(watches ...bool) error { func build(watches ...bool) error {
// Hugo writes the output to memory instead of the disk // Hugo writes the output to memory instead of the disk
// This is only used for benchmark testing. Cause the content is only visible // This is only used for benchmark testing. Cause the content is only visible
// in memory // in memory
@ -786,7 +785,7 @@ func NewWatcher(port int) error {
jww.FEEDBACK.Printf("Syncing all static files\n") jww.FEEDBACK.Printf("Syncing all static files\n")
err := copyStatic() err := copyStatic()
if err != nil { 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 { } else {
staticSourceFs := getStaticSourceFs() staticSourceFs := getStaticSourceFs()

View file

@ -42,7 +42,6 @@ func HasPygments() bool {
// Highlight takes some code and returns highlighted code. // Highlight takes some code and returns highlighted code.
func Highlight(code, lang, optsStr string) string { func Highlight(code, lang, optsStr string) string {
if !HasPygments() { if !HasPygments() {
jww.WARN.Println("Highlighting requires Pygments to be installed and in the path") jww.WARN.Println("Highlighting requires Pygments to be installed and in the path")
return code return code
@ -197,7 +196,6 @@ func createOptionsString(options map[string]string) string {
} }
func parseDefaultPygmentsOpts() (map[string]string, error) { func parseDefaultPygmentsOpts() (map[string]string, error) {
options := make(map[string]string) options := make(map[string]string)
err := parseOptions(options, viper.GetString("pygmentsOptions")) err := parseOptions(options, viper.GetString("pygmentsOptions"))
if err != nil { if err != nil {
@ -225,7 +223,6 @@ func parseDefaultPygmentsOpts() (map[string]string, error) {
} }
func parsePygmentsOpts(in string) (string, error) { func parsePygmentsOpts(in string) (string, error) {
options, err := parseDefaultPygmentsOpts() options, err := parseDefaultPygmentsOpts()
if err != nil { if err != nil {
return "", err return "", err

View file

@ -22,7 +22,6 @@ import (
// LoadGlobalConfig loads Hugo configuration into the global Viper. // LoadGlobalConfig loads Hugo configuration into the global Viper.
func LoadGlobalConfig(relativeSourcePath, configFilename string) error { func LoadGlobalConfig(relativeSourcePath, configFilename string) error {
if relativeSourcePath == "" { if relativeSourcePath == "" {
relativeSourcePath = "." relativeSourcePath = "."
} }

View file

@ -105,7 +105,6 @@ func (h mmarkHandler) PageConvert(p *Page, t tpl.Template) HandledResult {
} }
func commonConvert(p *Page, t tpl.Template) HandledResult { func commonConvert(p *Page, t tpl.Template) HandledResult {
if p.rendered { if p.rendered {
panic(fmt.Sprintf("Page %q already rendered, does not need conversion", p.BaseFileName())) panic(fmt.Sprintf("Page %q already rendered, does not need conversion", p.BaseFileName()))
} }

View file

@ -584,7 +584,6 @@ func (p *Page) permalink() (*url.URL, error) {
if err != nil { if err != nil {
return nil, err return nil, err
} }
// fmt.Printf("have a section override for %q in section %s → %s\n", p.Title, p.Section, permalink)
} else { } else {
if len(pSlug) > 0 { if len(pSlug) > 0 {
permalink = helpers.URLPrep(viper.GetBool("uglyURLs"), path.Join(dir, p.Slug+"."+p.Extension())) permalink = helpers.URLPrep(viper.GetBool("uglyURLs"), path.Join(dir, p.Slug+"."+p.Extension()))

View file

@ -917,7 +917,6 @@ func (s *SiteInfo) SitemapAbsURL() string {
} }
func (s *Site) initializeSiteInfo() { func (s *Site) initializeSiteInfo() {
var ( var (
lang *helpers.Language = s.Language lang *helpers.Language = s.Language
languages helpers.Languages languages helpers.Languages
@ -1980,7 +1979,6 @@ func (s *Site) renderSectionLists(prepare bool) error {
} }
if n.paginator != nil { if n.paginator != nil {
paginatePath := helpers.Config().GetString("paginatePath") paginatePath := helpers.Config().GetString("paginatePath")
// write alias for page 1 // write alias for page 1
@ -2029,7 +2027,6 @@ func (s *Site) renderSectionLists(prepare bool) error {
} }
func (s *Site) renderHomePage(prepare bool) error { func (s *Site) renderHomePage(prepare bool) error {
n := s.newHomeNode(prepare, 0) n := s.newHomeNode(prepare, 0)
if prepare { if prepare {
return nil return nil

View file

@ -65,6 +65,7 @@ func SetI18nTfuncs(bndl *bundle.Bundle) {
jww.WARN.Printf("No translation bundle found for default language %q", defaultContentLanguage) jww.WARN.Printf("No translation bundle found for default language %q", defaultContentLanguage)
} }
enableMissingTranslationPlaceholders := viper.GetBool("enableMissingTranslationPlaceholders")
for _, lang := range bndl.LanguageTags() { for _, lang := range bndl.LanguageTags() {
currentLang := lang currentLang := lang
@ -78,7 +79,7 @@ func SetI18nTfuncs(bndl *bundle.Bundle) {
if Logi18nWarnings { if Logi18nWarnings {
i18nWarningLogger.Printf("i18n|MISSING_TRANSLATION|%s|%s", currentLang, translationID) i18nWarningLogger.Printf("i18n|MISSING_TRANSLATION|%s|%s", currentLang, translationID)
} }
if viper.GetBool("enableMissingTranslationPlaceholders") { if enableMissingTranslationPlaceholders {
return fmt.Sprintf("[i18n] %s", translationID) return fmt.Sprintf("[i18n] %s", translationID)
} }
if defaultT != nil { if defaultT != nil {

View file

@ -114,7 +114,6 @@ func resDeleteCache(id string, fs afero.Fs) error {
// resGetRemote loads the content of a remote file. This method is thread safe. // 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) { func resGetRemote(url string, fs afero.Fs, hc *http.Client) ([]byte, error) {
c, err := resGetCache(url, fs, viper.GetBool("ignoreCache")) c, err := resGetCache(url, fs, viper.GetBool("ignoreCache"))
if c != nil && err == nil { if c != nil && err == nil {
return c, nil return c, nil