This commit contains the removal of html from 'the important' models
 or at least contains replacements for model methods such as `print_as...()`,
 etc. etc. Except for `header_as_li`, methods are not completely deleted yet.

> Fixed/Improved the nested comments and comments in reports. They were
 not shown before.

> Improved DRY principle in several templates/codes.

> Created Managers in the `submissions` app to filter i.e. "pool" submissions
 (so exclude potential author from judging on his own publication) for the
 pool view in the query; not in the template!

> More...

Only works for comments on thesis links. All comment forms
should post to this link from now on! (So all logic pertaining
comment creation should disappear from the models' detail view.)

I also added a `new_comment.html` partial template and added tests.

Change all involved tests. Start moving email to templates.

Update all involved tests. Also extend model_form_data to
accept an argument `form_kwargs` to pass to form.

Creating a new ThesisLink didn't save the `requested_by`
 field, causing trouble in for example the vet view
 for these specific objects.

Test if thesis is vetted/deleted and mail is sent in
the accepted/refused case.

Theses that are accepted are vetted by the correct contributor.
Mails are not yet sent. Thesis is not yet deleted when refused.

Work in progress. All unvetted theses are now listed at
theses/unvetted_thesislinks. Vetting editors click on
a thesis link to vet, and are redirected to
theses/vet_thesislink/<id>, where they can change fields of a thesis
link.

theses/vet_thesislink/<id> is now a regular update view. It does not have
mail capacity, or refusal reasons, etc. I will work on that next.

I keep get weird bugs. Maybe merging in development will help.

This does not work yet. I wrote some failing tests, but while
I was doing that, I came to the conclusion that I want to make
the route of the get request different from the route of the post
request.

The acknowledgement message is no longer hardcoded in
theses/views.py, but resides in the top-level strings module.

Removes some 40 lines of boilerplate, since CreateView
handles the creation of a new ThesisLink.

A CreateView class based view should replace the current
request_thesislink function based view, since it saves
many lines of code and is less error prone and more easily readable.