I'm always surprised to see methods declared in the constructor.

I get that you prefer to do that for a controller exposed in the scope but why not use the prototype for a service ?