[Fix] closing user being transmitted in close by visitor

[mrsimpson]

@RocketChat/core

Closes #9558

This fixes a "syntax error" violating the closeRoom method signature.

@sampaiodiego: is it semantically correct to pass Meteor.user() as closing use or should this also be the visitor

[sampaiodiego]

is it semantically correct to pass Meteor.user() as closing use or should this also be the visitor

nope. unfortunately there is no Meteor.user() when visitor calls it.. this should not fix the issue

[renatobecker-zz]

We are closing this PR because we are going to fix this issue unifying some methods that are still using the user property internally.
As a visitor is not a logged user, the Meteor.user() can't be passed to internal method.
After this fix, the user property will no longer be used.

Anyway, thanks for your effort and help.

[ruKurz]

As far I can see this PR is closed by #9714