fix(deps): use named parameters

Author: pspi

socket.js

@@ -6,93 +6,94 @@
  */
 
 angular.module('btford.socket-io', []).
-  provider('socketFactory', function () {
+provider('socketFactory', function() {
 
     'use strict';
 
     // when forwarding events, prefix the event name
     var defaultPrefix = 'socket:',
-      ioSocket;
+        ioSocket;
 
     // expose to provider
-    this.$get = function ($rootScope, $timeout) {
+    this.$get = ['$rootScope', '$timeout',
+        function($rootScope, $timeout) {
 
-      var asyncAngularify = function (socket, callback) {
-        return callback ? function () {
-          var args = arguments;
-          $timeout(function () {
-            callback.apply(socket, args);
-          }, 0);
-        } : angular.noop;
-      };
+            var asyncAngularify = function(socket, callback) {
+                return callback ? function() {
+                    var args = arguments;
+                    $timeout(function() {
+                        callback.apply(socket, args);
+                    }, 0);
+                } : angular.noop;
+            };
 
-      return function socketFactory (options) {
-        options = options || {};
-        var socket = options.ioSocket || io.connect();
-        var prefix = options.prefix || defaultPrefix;
-        var defaultScope = options.scope || $rootScope;
+            return function socketFactory(options) {
+                options = options || {};
+                var socket = options.ioSocket || io.connect();
+                var prefix = options.prefix || defaultPrefix;
+                var defaultScope = options.scope || $rootScope;
 
-        var addListener = function (eventName, callback) {
-          socket.on(eventName, callback.__ng = asyncAngularify(socket, callback));
-        };
+                var addListener = function(eventName, callback) {
+                    socket.on(eventName, callback.__ng = asyncAngularify(socket, callback));
+                };
 
-        var addOnceListener = function (eventName, callback) {
-          socket.once(eventName, callback.__ng = asyncAngularify(socket, callback));
-        };
+                var addOnceListener = function(eventName, callback) {
+                    socket.once(eventName, callback.__ng = asyncAngularify(socket, callback));
+                };
 
-        var wrappedSocket = {
-          on: addListener,
-          addListener: addListener,
-          once: addOnceListener,
+                var wrappedSocket = {
+                    on: addListener,
+                    addListener: addListener,
+                    once: addOnceListener,
 
-          emit: function (eventName, data, callback) {
-            var lastIndex = arguments.length - 1;
-            var callback = arguments[lastIndex];
-            if(typeof callback == 'function') {
-              callback = asyncAngularify(socket, callback);
-              arguments[lastIndex] = callback;
-            }
-            return socket.emit.apply(socket, arguments);
-          },
+                    emit: function(eventName, data, callback) {
+                        var lastIndex = arguments.length - 1;
+                        var callback = arguments[lastIndex];
+                        if (typeof callback == 'function') {
+                            callback = asyncAngularify(socket, callback);
+                            arguments[lastIndex] = callback;
+                        }
+                        return socket.emit.apply(socket, arguments);
+                    },
 
-          removeListener: function (ev, fn) {
-            if (fn && fn.__ng) {
-              arguments[1] = fn.__ng;
-            }
-            return socket.removeListener.apply(socket, arguments);
-          },
+                    removeListener: function(ev, fn) {
+                        if (fn && fn.__ng) {
+                            arguments[1] = fn.__ng;
+                        }
+                        return socket.removeListener.apply(socket, arguments);
+                    },
 
-          removeAllListeners: function() {
-            return socket.removeAllListeners.apply(socket, arguments);
-          },
+                    removeAllListeners: function() {
+                        return socket.removeAllListeners.apply(socket, arguments);
+                    },
 
-          disconnect: function (close) {
-            return socket.disconnect(close);
-          },
+                    disconnect: function(close) {
+                        return socket.disconnect(close);
+                    },
 
-          // when socket.on('someEvent', fn (data) { ... }),
-          // call scope.$broadcast('someEvent', data)
-          forward: function (events, scope) {
-            if (events instanceof Array === false) {
-              events = [events];
-            }
-            if (!scope) {
-              scope = defaultScope;
-            }
-            events.forEach(function (eventName) {
-              var prefixedEvent = prefix + eventName;
-              var forwardBroadcast = asyncAngularify(socket, function (data) {
-                scope.$broadcast(prefixedEvent, data);
-              });
-              scope.$on('$destroy', function () {
-                socket.removeListener(eventName, forwardBroadcast);
-              });
-              socket.on(eventName, forwardBroadcast);
-            });
-          }
-        };
+                    // when socket.on('someEvent', fn (data) { ... }),
+                    // call scope.$broadcast('someEvent', data)
+                    forward: function(events, scope) {
+                        if (events instanceof Array === false) {
+                            events = [events];
+                        }
+                        if (!scope) {
+                            scope = defaultScope;
+                        }
+                        events.forEach(function(eventName) {
+                            var prefixedEvent = prefix + eventName;
+                            var forwardBroadcast = asyncAngularify(socket, function(data) {
+                                scope.$broadcast(prefixedEvent, data);
+                            });
+                            scope.$on('$destroy', function() {
+                                socket.removeListener(eventName, forwardBroadcast);
+                            });
+                            socket.on(eventName, forwardBroadcast);
+                        });
+                    }
+                };
 
-        return wrappedSocket;
-      };
-    };
-  });
+                return wrappedSocket;
+            };
+        };
+    }]);