Closed (fixed)
Project:
Node JS Chat
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
24 Jun 2013 at 12:24 UTC
Updated:
28 Jul 2013 at 10:50 UTC
Hi,
As i was getting this error who does cause to not display the user list who are already in the chat when a new user connect.
i did changed line 231 of Chat.inc
$uids_on_channel = $users_on_channel['uids'];
Comments
Comment #1
slv_ commentedHi Julien,
That's happening because of the version of the nodejs module that you use. It's probably the stable one, where the 'uids' have to be accessed as an array. In the dev version, that is an object, and the dev version of nodejs_chat is based on that one, so there's a bit of chance that you might indeed come across this.
I will be fixing this today, and pushing a fix shortly (going offline for some time, so as soon as go online again it'll be pushed!), so that this works with both the stable version of nodejs and the dev version.
Thanks for reporting it!
Comment #2
slv_ commentedThis is now fixed and push to 7.x-1.x branch. Now the class checks for the type of structure returned by nodejs, so it's compatible with both objects and arrays.
Marking as fixed and closing.
Comment #2.0
slv_ commentedremove white space