Revert unnecessary `$default_ssl_vhost` validation change.

The `validate_bool()` call was moved in
112437730f70e18daf3aed9cedffd0c46348204d making it FreeBSD centric, but
it should be validated outside of any other condition.
Fix spec test for `default_ssl_vhost  => true`

The spec test for `default_ssl_vhost  => true` was retesting the test
for `default_vhost  => true` instead.
Merge pull request #453 from dbeckham/fix_broken_validate_ssl_vhost

Revert unnecessary `$default_ssl_vhost` validation change.
Merge pull request #454 from dbeckham/fix_apache_vhost_ssl_spec_test

Fix spec test for `default_ssl_vhost  => true`
need to use the provider variable not the provider key value from the directory hash for matches
Add ability to include additional external configurations in vhost
Add directives: IndexOptions, IndexOrderDefault
Update spec tests for index_options and index_order_default parameters
Merge pull request #455 from nesi/directory

Add IndexOptions parameter, fix directory fragment not setting AllowOverride
update contributing to reflect modules workflow

CONTRIBUTING has been dropped in from puppetlabs/puppet, with
good intention I assume, however it does in no way reflect how
we work on modules.

* change all references from Redmine to refer to GitHub
* Fix CLA links
* remove the entire section for committers, we assume they know what
they are doing...

Further, we remove all contractions. Why? Because it reads easier.

Finally we will need a section on how to write tests (and run them
locally) because I'm getting sick of repeating myself.
Ignore vim swap files
Merge pull request #462 from igalic/contrib

update contributing to reflect modules workflow
Merge pull request #457 from alnewkirk/master

Add ability to include additional external configurations in vhost
*really* ignore vim swap files.
tiny style fixes to CONTRIBUTING

* we have no release branches, remove reference
* remove contractions
Merge branch 'master' into production