Avoid n+1 queries when downloading all submissions

Lots of queries are redundant in the logic of downloading all
submissions and can be avoided by eagerly loading the associations.

This deals with #1484.

Tested:
- rspec
- rake test:units
- rake test:functionals
Indent the right brace the same as the first position after the preceding left parenthesis.