Refactor breadcrumb class
Refactor links array and remove last link option

The option to render the last crumb as a link was functionally deprecated.  It can now be removed.
Apply PSR formatting
Remove reset method and add test
Update main page template
Update defined_page.php for secure images
Some servers returning references to an invalid index value if the SQL query returned is empty
Shipping module unable to register an error
Correct issues on Gift Certificate defines
Page not found when incorrect EZPage link remove status
Merge pull request #151 from ajeh/gift-certificates

Correct issues on Gift Certificate defines
Update docblocks
Merge pull request #152 from ajeh/EZPages-header

Page not found when incorrect EZPage link remove status
Merge pull request #150 from ajeh/shipping-module-error-reference

Shipping module unable to register an error
Merge pull request #147 from texdc/160_refactor_breadcrumb

Refactor Breadcrumb class
Merge pull request #146 from ajeh/definedpage

Update defined_page.php for secure images
Merge pull request #149 from ajeh/zen_has_product_attributes

Some servers returning references to an invalid index
Request Class Issue #65

See also Pull requests #119

I've made the request class a first class object rather than use the static/facade interface used previously
That only really made sense if we had some DI container to manage it. Although given the use of some 'global' classes
in the code, the use of a DI container is looking more necessary. Something for the future.
I've also added codeception to composer and committed some basic start up stuff for that.
I think codeception looks better in terms of writing acceptance tests, over a selenium/sourcelabs framework.
doing some minor reformatting while we touch these files
pr #65 - Request Class - Inject Aura Web Request in constructor to aid unit testing.