We discussed this during our code review session on 3rd August 2017.
Currently by using mw.msg, mw.user and mw.template we make it difficult to easily see usages (as we don't use dependency injection)
- Where mw.user is used we will add a variable to the top of the file to make it clearer that these are dependencies and being used
- Do same for mw.msg
- Do same for mw.template
var msg = mw.msg, user = mw.user, template = mw.template,