Do not use .fetch*() in User.byUsername(...)

It's better to leave the fetch call for the caller. This way the caller
can:

  - Specify more queries by calling .query(...) again
  - Add arguments for the .fetch(...) call. For example
    withRelated("tickets").

Also .fetchOne() is not needed if we use the model directly without the
collection.

cc @jpkorhonen