OPSK-81 - fix to send notication to project managers

when somebody signs up and indicates they are a member of the project
OPSK-81 project and institution selection during registration should pass id's

no need to pass the title as well, and then parse the string to split it
fix
OPSK-82 - Prevent an project manager from removing themself from changing their own projects
removed old people_i_may_know method
get Person#institutions through work-groups

rather than through a manual method, use has_many=>through
Merge branch 'master' into project-manager-fixes
project admin button changed to take straight to admin members

the old admin page didn't haven't anything else of much use, and just required another click to get to the relevant page.
changed the button path and text to reflect this.
the old admin page remains in place, should we need it to reintroduce the updating of JERM settings in the future.
updated method name for new user email notification for project manage and admin

had a no_profile at the end, which is confusing because they do have a profile. Dropped the no_profile and updated associated files and calls.
updated the text of the email sent to the project manager

for when a new user signs up and indicates that they are the member of a project.
made the text more relevant to the latest version, fixed some english, and added some helpful links
removed Multipart class which is no longer needed

now handled with the restclient libraries
refactored out the preview from controllers

made more general as a shared method, and removed the duplication from many controllers
a general controller destory method for AssetsCommon

removed duplication and moved to AssetsCommon for those controllers that include it
unless the current destroy method is very specific
more generalisation and removing duplication of destroy method

currently the destroy is in it's own modules, but could eventually form a general
set of CRUD actions
shared some duplicated common behaviour in events_controller
some refactoring and tidying of avatar helper
fixes for tests
a little refactoring of content-type tidying up some complex methods
fixed tests that weren't web-mocked out properly

overzealous exception handling in content-blob was convering this up. Since being fixed a couple of tests were failind due to http calls not being mocked out with webmock (since the exception was previously being caught).
Now changed to mock out the http calls with a 404 error, which will produce the same error expected in the tests
re-enable facetted search and browsing by default

following recent fixes