[polari/wip/fmuellner/combined-gsoc: 70/103] userTracker: Split out some code

[Florian Müllner <fmuellner src gnome org>]

commit b5cf12cd35875df31e6dbd6ab8001d96893a238b
Author: Florian Müllner <fmuellner gnome org>
Date:   Thu Jul 14 02:21:22 2016 +0200

    userTracker: Split out some code

 src/userTracker.js |   14 ++++++++------
 1 files changed, 8 insertions(+), 6 deletions(-)
---
diff --git a/src/userTracker.js b/src/userTracker.js
index 4e4ef51..7ef53ae 100644
--- a/src/userTracker.js
+++ b/src/userTracker.js
@@ -255,15 +255,17 @@ const UserTracker = new Lang.Class({
         this._untrackMember(this._globalContactMapping, member, room);
     },
 
+    _pushMember: function(map, baseNick, member) {
+        if (!map.has(baseNick))
+            map.set(baseNick, []);
+        let contacts = map.get(baseNick);
+        return contacts.push(member);
+    },
+
     _trackMember: function(map, member, room) {
         let baseNick = Polari.util_get_basenick(member.alias);
 
-        if (map.has(baseNick))
-            map.get(baseNick).push(member);
-        else
-            map.set(baseNick, [member]);
-
-        if (map.get(baseNick).length == 1) {
+        if (this._pushMember(map, baseNick, member) == 1) {
             if (map == this._globalContactMapping) {
                 this.emit("status-changed::" + baseNick, member.alias, Tp.ConnectionPresenceType.AVAILABLE);
                 //log("[global status] user " + member.alias + " is globally online");