tpl: Fix panic in pairList.Less

While sorting on data sources with missing fields, a panic can occur in
pairList.Less if `Interface()` is called on a invalid `reflect.Value`.
This commit detects an invalid Value and replacing it with a zero value
for the comparison.

Make ByCount sort consistently

When two or more taxonomies have the same number of pages,
sort them by name to have consistent ByCount sorting of
taxonomies.

Fixes #1930

hugofs: Add missing not nil checks to tests

Use to most specific interface in WriteDestPage

docs: Add doc and a test for readDir

Fixes #2009

tpl: Make readDir use the WorkingDir fs

Fixes #2010

Add readFile template func

This also includes a refactor of the hugofs package and its usage.

The motivation for that is:

The Afero filesystems are brilliant. Hugo's way of adding a dozen of global variables for the different filesystems was a mistake. In readFile (and also in some other places in Hugo today) we need a way to restrict the access inside the working dir. We could use ioutil.ReadFile and implement the path checking, checking the base path and the dots ("..") etc. But it is obviously better to use an Afero BasePathFs combined witha ReadOnlyFs. We could create a use-once-filesystem and handle the initialization ourselves, but since this is also useful to others and the initialization depends on some other global state (which would mean to create a new file system on every invocation), we might as well do it properly and encapsulate the predefined set of filesystems. This change also leads the way, if needed, to encapsulate the file systems in a struct, making it possible to have several file system sets in action at once (parallel multilanguage site building? With Moore's law and all...)

Fixes #1551

docs: Mention CLA in the showcase contribution guidelines

docs: Remove incorrent reference to list.html

docs: Fix typo in templates/functions.md

docs: Add surge.sh to main page of static site hosters

See #1994

docs: Document GitHub flavoured code fences
Fixes #1259

docs: Document lastmod variable
Fixes #1351

docs: Document the split function
Fixes #1997

docs: Add silvergeko.it to the showcase

docs: Add robertbasic.com to the showcase

Fix error handling in SymbolicWalk

helpers: Fix SymbolicWalk for the root folder

handle the root folders themselves.

This commit fixes that.

Fixes #2018

docs: Adapt commit message for showcase additions

docs: Add search form powered by Algolia

hugolib: Some more unexports

commands: Apply Golint rules

helpers: Apply GoLint rules

tpl: Small Golint fixes

parser: Remove unused vars

Be explicit about the type in Scratch test

A puzzle, but it sometimes produces a int64 as it is now.

hugolib: Remove unused Tmpl on Page

Do not use defer for unlocking in Scratch

It is a fairly costly operation:

```
benchmark                 old ns/op     new ns/op     delta
BenchmarkScratchGet-4     109           31.6          -71.01%

benchmark                 old allocs     new allocs     delta
BenchmarkScratchGet-4     0              0              +0.00%

benchmark                 old bytes     new bytes     delta
BenchmarkScratchGet-4     0             0             +0.00%
´´´

hugolib: Taxonomy GoDoc cleanup

Add a simple benchmark for Scratch

hugolib: Even more Golint fixes

helpers: Fix GoDoc in Hugo version constants

commands: Apply Golint rules to hugo.go

hugolib: Apply Golint rules to rss_test.go

hugolib: Add missing GoDoc in pagination.go

hugolib: Some more GoLint fixes

livereload: Add missing GoDoc

hugoblib: Clean up the time tests

transform: Group vars and consts in chain_test

transform: Clean up the var naming mess in chain_test

hugolib: Some more Golint fixes

hugolib: Fix spelling mistake

hugolib: Add Godoc to pageSort

parser: Make the constant vars ... constants

parser: Fix ALL_CAPS var names in test

parser: Spring code cleaning

hugolib: Fix some more Golint warnings

helpers: Fix and add Godoc in path*

helpers: Remove the else in MakePathSanitized

Do some minor code cleaning of Site

* Unexport some methods/fields
* Add some godoc

Still plenty to do here.

transform: Golint fixes

target: Fix some Golint warnings

Spring cleaning of the menu code

heloers: Add a linefeed to make Golint happy

commands: Fix remaining Golint warning

utils: Add godoc

hugolib: Fix Java-styled all-caps consts

helpers: Silence some Golint warnings

commands: Do not use named return vals in Jekyll import

Check returned error in Scratch-test

Do make check in Travis

source: Consolidate if conditions

source: Remove unnecessary else condition

hugolib: Use field keys in composite literals

Small fix in shortcode tests to use field keys in a composite literal.

helpers: Apply gofmt

hugo: Add check tasks to Makefile

Add some helpful tasks to the Makefile to make it easier to test code quality
prior to committing.  Use `make check` to run the standard checks.  As of now,
the cyclo and lint tasks are not part of the standard checks since Hugo doesn't
pass those checks.

Add Scratch to shortcode

Fixes #2000

Re-render shortcode on template or data file change

Fixes  #1971

Add plainify template function

To strip away any HTML. May be useful for the .Title in head etc.

People may shoot themself in the foot with this, maybe ...

The replacement function is pretty fast.

Add support for symbolic links for content, layout, static, theme

Note: This is for the content roots only, but that should satisfy most needs.

Fixes #1855

Utils methods refactored for readability

Add '+' as one of the valid characters in urls specified in the front matter
Fixes #1290

Protect against concurrent Scratch read and write

Fixes #2005

docs: Draft of 0.16 release notes

Also included is an enhancement to the gh shortcode to support sending commit
hashes by prepending "0x" to the hash.

helpers: Rename getMmarkHtmlRenderer to getMmarkHTMLRenderer

To be consistent with the changes made in commit 67df33f, see #959.

livereload: Run go fmt on connection.go

See #1998 and commit bafb771

create: Refactor NewContent to be testable

NewContent is refactored to use the afero.Fs interface that should allow
full testing.  This commit also pulls the metadata creation logic out of
NewContent and into a separate function to decrease the cyclomatic
complexity of NewContent.

livereload: Simplify conditional

helpers: Support EXTENSION_BACKSLASH_LINE_BREAK for Blackfriday

Exposed as "backslashLineBreak" and enabled by default
as upstream have done.

Fixes #1935

helpers: Fix minor typo in content_test.go

Simplify GetDottedRelativePath

Return early from parseOptions

Remove duplicate doArithmetic

Revert "Add Codebeat badge"

Will have to review the rules on this code report  before people start sending PRs based on this.

This reverts commit c50c66879d7140311bcd6bfcb3fb5122d2b3f809.

Return early from appendThemeTemplates

Add Codebeat badge

Add Go Report Card badge to README

source: Consolidate filepath.Split calls

tpl: Send actual values to in from intersect

The `intersect` function uses `in` to avoid adding duplicates to the
resulting set.  We were passing `reflect.Value` items when we should
have been using `Value.Interface()` to send the actual data structure.
This fixes that.

See #1952

tpl: Add missing printf param in t.Errorf call

tpl: Treat booleans as set in default function

Booleans and `default` don't really make sense together, so we'll always treat
booleans as "set" and return the given value.

docs: Fix typos in functions.md and contributing.md

Remove CircleCI badge for now

No block support in shortcodes

So no need to check them.

tpl: Make the safeHTML and cousins accept more than strings

Remove unnecessary type conversions

parser: Unexport some internals

helpers: Unexport some internals

tpl: Fix race condition in regexp cache

Protect regular expression cache with a mutex.

Fixes #1973

transform: Remove unused code