WL-3576 : Adding logic for email sender/recipient in Contact Us Tool
WL-3576 : Indentation fix
WL-3576 : Indentation fix
WL-3576 : No need to pass in userId in securitycheck.
WL-3576 : Fix indentation (for my line and a few other lines around it)
WL-3576 : Fix code style change
WL-3576 : Acc to Jira we weren't going to do checks if the user was logged in

I've moved the security stuff out of the userId==null check.
But we still need to/can do this check as we put different attributes in the request depending on whether the user is logged in or not
WL-3576 : Remove irrelevant comment
WL-3576 : Fix loggedin, no perms user will always go to technical email

If the user is logged in and doesn't have permission to see the site members then the email will always go to the central technical address and never the site contact email.
Merge remote-tracking branch 'origin/weblearn-10.x' into WL-3576

* origin/weblearn-10.x:
  WL-3566 : Add if-else logic to which email recipient to show
  WL-3566 : move click attaching to on page load rather than every method call
  WL-3566 : Should be doing the fit frame after changing the html
  WL-3592 : contact list is empty
  WL-3566 : Indentation
  WL-3567 : Use dbWrite method as we don't care about return value
  WL-3576 : Add email sent confirmation message to feedback tool
  WL-3567 : Add autoincrement column to avoid db insert exeption

Conflicts:
	src/java/org/sakaiproject/feedback/tool/FeedbackTool.java
	src/webapp/js/feedback.js
Merge pull request #26 from ox-it/WL-3576

WL-3576 : Adding logic for email sender/recipient in Contact Us Tool