Cleaned up javascript code
minor #4376 Cleaned up javascript code (flip111)

This PR was squashed before being merged into the 2.3 branch (closes #4376).

Discussion
----------

Cleaned up javascript code

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | 2.3
| Fixed tickets | n/a

1. Reused variable `emailList` which is best practice and easier to read
2. Removed `return false`. There is no use for it in this context and many consider it to be bad use as well. For example: http://fuelyourcoding.com/jquery-events-stop-misusing-return-false/

Commits
-------

8931249 Cleaned up javascript code