Refactor of volunteer routes, simplifying, making identifiers consistent

 - Moving the merging of the new document resulting from the database update to the callback in the exported route functions
 - Making parameter identifiers to the callback to the common update function consistent
Added _layoutFile to app.locals call to remove it from all res.render calls

Added the entry `_layoutFile: 'default'` too the call to `app.locals` in the
file `app.js` so that it could be removed from all the calls to `res.render`
in each of the routes. This means that whenever `res.render` is called it will
always use `default` for `_layoutFile`. This is simpler and it will be more
efficient because `_layoutFile` does not have to be set for each
request/response. We have the design choice of whether or not to create multiple
layouts for different user types or to use logic within our templates in order
to change the single layout based on user type. If you want to use a different
layout then you can override `default` by passing in a new `_layoutFile` to
`res.render` as we've been doing.
Remove unneeded type field from anchor tags in teams markup
Adding delete modal to volunteer account page

**Changelog:**

 - Added modal for deleting the volunteer account to the volunteer account view at `server/views/volunteer-account.html`. This modal does not require any JavaScript to function at all. It is activated by any link that has a `href` attribute consisting of a hash-tag and the modal element's `id`. The confirm delete link within the modal sends a GET request to either `/volunteer/account/delete` or `staff/volunteer/account/delete/:id` depending on whether the user is a staff member.

 - Changed the HTTP request type for `/staff/volunteer/account/delete/:id` from `POST` to `GET` in the router at `server/router/index.js`.

 - Made the jQuery calls more efficient by, after getting the form element, only only searching the `form` elements descendents for the remaining DOM queries.
Simplifying jobsite routes by using the Array indexOf method
Fixing small bugs with jobsite requests

**Changelog:**

 - Added the paths `/jobsite/request/sucess` and `/jobsite/request/failure` to the open paths list in the middleware at `server/middleware/wlauth`.

 - Fixing popover message in `public/js/jobsite-request-controller.js`.
Fixing bug where last two chars cut off of zip code error in JSRF
Merge pull request #22 from CoryG89/master

Various simplifications