wrapping clients() response to handle both arrays and objects. Closes #…

…10.
feathersjs-ecosystem · Jan 6, 2016 · 858a264 · 858a264
1 parent d1e5d6f
commit 858a264
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/sockets/helpers.js b/src/sockets/helpers.js
@@ -34,7 +34,7 @@ export function setupEventHandlers(info, service, path) {
         service[ev] : defaultDispatcher;
       let eventName = `${path} ${ev}`;
 
-      info.clients().forEach(function (socket) {
+      Object.keys(info.clients()).forEach(function (socket) {
         dispatcher.call(service, data, info.params(socket), function (error, dispatchData) {
           if (error) {
             socket[info.method]('error', error);

diff --git a/src/sockets/index.js b/src/sockets/index.js
@@ -32,7 +32,7 @@ export function service(path, obj) {
     // Set up event handlers for this new service
     _setupEventHandlers(protoService, location);
     // For any existing connection add method handlers
-    info.clients().forEach(socket => _setupMethodHandlers(socket, location, protoService));
+    Object.keys(info.clients()).forEach(socket => _setupMethodHandlers(socket, location, protoService));
   }
 
   return protoService;