Merge users instead of deleting

hipchat · Nov 28, 2014 · 598bace · 598bace
1 parent 66a5da3
commit 598bace
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/src/hipchat.coffee b/src/hipchat.coffee
@@ -104,7 +104,7 @@ class HipChat extends Adapter
 
       if @options.reconnect
         @waitAndReconnect()
-    
+
     firstTime = true
     connector.onConnect =>
       @logger.info "Connected to #{host} as @#{connector.mention_name}"
@@ -121,8 +121,12 @@ class HipChat extends Adapter
         # Save users to brain
         for user in users
           user.id = @userIdFromJid user.jid
-          # userForId will not overwrite an existing user
+          # userForId will not merge to an existing user
           if user.id of @robot.brain.data.users
+            oldUser = @robot.brain.data.users[user.id]
+            for key, value of oldUser
+              unless key of user
+                user[key] = value
             delete @robot.brain.data.users[user.id]
           @robot.brain.userForId user.id, user
 

diff --git a/src/test.coffee b/src/test.coffee
@@ -0,0 +1,7 @@
+
+user = id: 1, name: "yannick"
+oldUser = role: "lol", name: "bob"
+for key, value of oldUser
+  unless key of user
+    user[key] = value
+console.log user, oldUser