diff options
author | bep <bjorn.erik.pedersen@gmail.com> | 2015-03-07 02:02:06 +0300 |
---|---|---|
committer | bep <bjorn.erik.pedersen@gmail.com> | 2015-03-07 02:02:06 +0300 |
commit | bd2b679eaf45669e23a724a5dacd5421c5ec61e1 (patch) | |
tree | b17d13faab28086195701c56e54fe34dd1a86f3c /helpers/path.go | |
parent | e685dfc0c607b3051dbac29e1209900e58de1605 (diff) |
helpers: apply some Golint rules
Diffstat (limited to 'helpers/path.go')
-rw-r--r-- | helpers/path.go | 27 |
1 files changed, 11 insertions, 16 deletions
diff --git a/helpers/path.go b/helpers/path.go index 20255a835..973554664 100644 --- a/helpers/path.go +++ b/helpers/path.go @@ -28,7 +28,7 @@ import ( "github.com/spf13/viper" ) -// Bridge for common functionality in filepath vs path +// FilepathPathBridge is a bridge for common functionality in filepath vs path type FilepathPathBridge interface { Base(in string) string Clean(in string) string @@ -149,9 +149,8 @@ func IsEmpty(path string, fs afero.Fs) (bool, error) { list, err := f.Readdir(-1) // f.Close() - see bug fix above return len(list) == 0, nil - } else { - return fi.Size() == 0, nil } + return fi.Size() == 0, nil } // Check if a file or directory exists. @@ -317,9 +316,8 @@ func GuessSection(in string) string { if parts[0] == "content" { if len(parts) > 2 { return parts[1] - } else { - return "" } + return "" } return parts[0] @@ -328,9 +326,8 @@ func GuessSection(in string) string { func PathPrep(ugly bool, in string) string { if ugly { return Uglify(in) - } else { - return PrettifyPath(in) } + return PrettifyPath(in) } // Same as PrettifyUrlPath() but for file paths. @@ -348,16 +345,14 @@ func PrettiyPath(in string, b FilepathPathBridge) string { return b.Separator() } return b.Join(b.Clean(in), "index.html") - } else { - name, ext := FileAndExt(in, b) - if name == "index" { - // /section/name/index.html -> /section/name/index.html - return b.Clean(in) - } else { - // /section/name.html -> /section/name/index.html - return b.Join(b.Dir(in), name, "index"+ext) - } } + name, ext := FileAndExt(in, b) + if name == "index" { + // /section/name/index.html -> /section/name/index.html + return b.Clean(in) + } + // /section/name.html -> /section/name/index.html + return b.Join(b.Dir(in), name, "index"+ext) } // FindCWD returns the current working directory from where the Hugo |