From 35e1e41d3a1c60c613ad4716ca6c796680a797bc Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?David=20Barrag=C3=A1n=20Merino?=
 <david.barragan@kaleidos.net>
Date: Fri, 6 Nov 2015 12:05:24 +0100
Subject: [PATCH] Minor code style fixes

---
 .gitignore          |  1 +
 client.coffee       | 34 ++++++++++++++++++----------------
 subscription.coffee | 22 ++++++++++++----------
 3 files changed, 31 insertions(+), 26 deletions(-)

diff --git a/.gitignore b/.gitignore
index 769fd20..9d140f3 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,3 +1,4 @@
+*.swp
 node_modules/
 npm-debug.log
 config.json
diff --git a/client.coffee b/client.coffee
index 6601a6b..e2374de 100644
--- a/client.coffee
+++ b/client.coffee
@@ -4,47 +4,49 @@ SubscriptionManager = require('./subscription').SubscriptionManager
 
 clients = {}
 
+
 class Client
     constructor: (@ws) ->
-        @id = uuid.v4()
+        @.id = uuid.v4()
 
-        @handleEvents()
+        @.handleEvents()
 
     handleEvents: () ->
-        @ws.on 'message', @handleMessage.bind(@)
+        @ws.on 'message', @.handleMessage.bind(@)
 
     handleMessage: (message) ->
         msg = JSON.parse(message)
 
         if msg.cmd == 'auth'
-            @auth(msg.data)
+            @.auth(msg.data)
         else if msg.cmd == 'subscribe'
-            @addSubscription(msg.routing_key)
+            @.addSubscription(msg.routing_key)
         else if msg.cmd == 'unsubscribe'
-            @removeSubscription(msg.routing_key)
+            @.removeSubscription(msg.routing_key)
 
     auth: (auth) ->
         if auth.token and auth.sessionId and signing.verify(auth.token)
-            @auth = auth
+            @.auth = auth
 
     addSubscription: (routing_key) ->
-        if @auth
-            if !@subscriptionManager
-                @subscriptionManager = new SubscriptionManager(@id, @auth, @ws)
-            @subscriptionManager.add(routing_key)
+        if @.auth
+            if !@.subscriptionManager
+                @.subscriptionManager = new SubscriptionManager(@.id, @.auth, @ws)
+            @.subscriptionManager.add(routing_key)
 
     close: () ->
-        if @subscriptionManager
-            @subscriptionManager.destroy()
+        if @.subscriptionManager
+            @.subscriptionManager.destroy()
 
     removeSubscription: (routing_key) ->
-        if @subscriptionManager
-            @subscriptionManager.remove(routing_key)
+        if @.subscriptionManager
+            @.subscriptionManager.remove(routing_key)
+
 
 exports.createClient = (ws) ->
     client = new Client(ws)
     clients[client.id] = client
     client.ws.on 'close', (() ->
         @.close()
-        delete clients[@id]
+        delete clients[@.id]
     ).bind(client)
diff --git a/subscription.coffee b/subscription.coffee
index 7453579..2c26cf7 100644
--- a/subscription.coffee
+++ b/subscription.coffee
@@ -1,5 +1,6 @@
 queue = require('./rabbit')
 
+
 class Subscription
     constructor: (@client_id, @auth, @ws, @routing_key) ->
 
@@ -16,31 +17,32 @@ class Subscription
         @ws.send clientMsgStr
 
     start: () ->
-        queue.subscribe(@client_id, @routing_key, @handleMessage.bind(@))
+        queue.subscribe(@client_id, @routing_key, @.handleMessage.bind(@))
 
     stop: () ->
         queue.unsubscribe(@client_id, @routing_key)
 
+
 class SubscriptionManager
     constructor: (@client_id, @auth, @ws) ->
-        @subscriptions = {}
+        @.subscriptions = {}
 
     add: (routing_key) ->
-        if !@subscriptions[routing_key]
-            @subscriptions[routing_key] = {}
+        if !@.subscriptions[routing_key]
+            @.subscriptions[routing_key] = {}
         else
-            @subscriptions[routing_key].stop()
+            @.subscriptions[routing_key].stop()
 
-        @subscriptions[routing_key] = new Subscription(@client_id, @auth, @ws, routing_key)
-        @subscriptions[routing_key].start()
+        @.subscriptions[routing_key] = new Subscription(@client_id, @auth, @ws, routing_key)
+        @.subscriptions[routing_key].start()
 
     remove: (routing_key) ->
-        @subscriptions[routing_key].stop()
+        @.subscriptions[routing_key].stop()
 
-        delete @subscriptions[routing_key]
+        delete @.subscriptions[routing_key]
 
     destroy: () ->
-        @subscriptions = {}
+        @.subscriptions = {}
         queue.destroy(@client_id)
 
 exports.SubscriptionManager = SubscriptionManager