Fix complete crash when refreshing a public instance

[astorije]

This fixes #124. The Client constructor was changed in #57, but was used twice and updated only in one place.
Removed global reference to sockets as it's not needed anymore. I tested and did not find any new stuff breaking, but reviews will confirm.

Marking as priority as releases are on hold until #124 is fixed in master.

[maxpoulin64]

Ouch. 👍

[xPaw]

Are you sure this doesn't need changing? https://github.com/thelounge/lounge/blob/5f11c262236078a8d671b5fbdb515de02a0671e5/src/client.js#L336-L337

[astorije]

@xPaw, yes, because Client still has a reference to the sockets as part of the constructor: https://github.com/thelounge/lounge/blob/5f11c262236078a8d671b5fbdb515de02a0671e5/src/client.js#L60
Now we can wonder if it's useful to have both around, but this is a quick bug fix to fix the master branch, I don't think I should refactor other things in this PR.

[xPaw]

I was asking in case that needed to be changed. 👍

[astorije]

Yeah, that was the starting point of my debugging, but quickly found the actual culprit later :-)
That being said, do you want me to get rid of the sockets in the Client model in a further PR? I'd say we'll clean this up over time anyway, but you pick.

[xPaw]

Either is fine. Merging this.