remove unnecessary return statement inside Discourse.Router.map

Ember's Router.map(callback) method doesn't expect any value to be
returned by callback function. Therefore, return statements present
inside Discourse.Router.map are unnecessary (and are silently ignored by
Ember).
Merge pull request #1451 from velesin/remove_return_from_router_map

remove unnecessary return statement inside Discourse.Router.map