Added event handler service which.. handles events. More specifically, it $broadcasts events depending on their type, and does processing on events (shuffling keys, adding events to $rootScope so displays will automatically update, sending delivery receipts, and so on). Some of this logic was previously contained in the RoomController, which fails the moment you add >1 room into the mix, hence requiring a Service to handle events, rather than having each individual controller maintain their part of the world.

2 files changed, 100 insertions, 4 deletions

diff --git a/webclient/components/matrix/event-handler-service.js b/webclient/components/matrix/event-handler-service.js
new file mode 100644
index 0000000000..30d7ab35ca
--- /dev/null
+++ b/webclient/components/matrix/event-handler-service.js
@@ -0,0 +1,87 @@
+/*
+Copyright 2014 matrix.org
+
+Licensed under the Apache License, Version 2.0 (the "License");
+you may not use this file except in compliance with the License.
+You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/
+
+'use strict';
+
+/*
+This service handles what should happen when you get an event. This service does
+not care where the event came from, it only needs enough context to be able to 
+process them. Events may be coming from the event stream, the REST API (via 
+direct GETs or via a pagination stream API), etc.
+
+Typically, this service will store events or broadcast them to any listeners
+(e.g. controllers) via $broadcast. Alternatively, it may update the $rootScope
+if typically all the $on method would do is update its own $scope.
+*/
+angular.module('eventHandlerService', [])
+.factory('eventHandlerService', ['matrixService', '$rootScope', function(matrixService, $rootScope) {
+    var MSG_EVENT = "MSG_EVENT";
+    var MEMBER_EVENT = "MEMBER_EVENT";
+    var PRESENCE_EVENT = "PRESENCE_EVENT";
+    
+    var handleMessage = function(event, isLiveEvent) {
+        if ("membership_target" in event.content) {
+            // event.user_id = event.content.membership_target;
+        }
+        
+        // $broadcast this, as controllers may want to do funky things such as
+        // scroll to the bottom, etc which cannot be expressed via simple $scope
+        // updates.
+        console.log("Bcast " + JSON.stringify(event));
+        $rootScope.$broadcast(MSG_EVENT, event, isLiveEvent);
+    };
+    
+    var handleRoomMember = function(event, isLiveEvent) {
+        $rootScope.$broadcast(MEMBER_EVENT, event, isLiveEvent);
+    };
+    
+    var handlePresence = function(event, isLiveEvent) {
+        $rootScope.$broadcast(PRESENCE_EVENT, event, isLiveEvent);
+    };
+    
+    
+    return {
+        MSG_EVENT: MSG_EVENT,
+        MEMBER_EVENT: MEMBER_EVENT,
+        PRESENCE_EVENT: PRESENCE_EVENT,
+        
+    
+        handleEvent: function(event, isLiveEvent) {
+            switch(event.type) {
+                case "m.room.message":
+                    handleMessage(event, isLiveEvent);
+                    break;
+                case "m.room.member":
+                    handleRoomMember(event, isLiveEvent);
+                    break;
+                case "m.presence":
+                    handlePresence(event, isLiveEvent);
+                    break;
+                default:
+                    console.log("Unable to handle event type " + event.type);
+                    break;
+            }
+        },
+        
+        // isLiveEvents determines whether notifications should be shown, whether
+        // messages get appended to the start/end of lists, etc.
+        handleEvents: function(events, isLiveEvents) {
+            for (var i=0; i<events.length; i++) {
+                this.handleEvent(events[i], isLiveEvents);
+            }
+        }
+    };
+}]);
diff --git a/webclient/components/matrix/event-stream-service.js b/webclient/components/matrix/event-stream-service.js
index 0a3a12192b..1cb9960b9a 100644
--- a/webclient/components/matrix/event-stream-service.js
+++ b/webclient/components/matrix/event-stream-service.js
@@ -16,8 +16,17 @@ limitations under the License.
 
 'use strict';
 
+/*
+This service manages where in the event stream the web client currently is and 
+provides methods to resume/pause/stop the event stream. This service is not
+responsible for parsing event data. For that, see the eventDataHandler.
+*/
 angular.module('eventStreamService', [])
 .factory('eventStreamService', ['matrixService', function(matrixService) {
+    var END = "END";
+    var START = "START";
+    var TIMEOUT_MS = 5000;
+    
     var settings = {
         from: "END",
         to: undefined,
@@ -28,7 +37,7 @@ angular.module('eventStreamService', [])
     // interrupts the stream. Only valid if there is a stream conneciton 
     // open.
     var interrupt = function(shouldPoll) {
-        console.log("[EventStream] interrupt("+shouldPoll+") "+
+        console.log("p[EventStream] interrupt("+shouldPoll+") "+
                     JSON.stringify(settings));
     };
     
@@ -42,7 +51,7 @@ angular.module('eventStreamService', [])
         resume: function() {
             console.log("[EventStream] resume "+JSON.stringify(settings));
             // run the stream from the latest token
-            return matrixService.getEventStream(settings.from, 5000);
+            return matrixService.getEventStream(settings.from, TIMEOUT_MS);
         },
         
         // pause the stream. Resuming it will continue from the current position
@@ -55,13 +64,13 @@ angular.module('eventStreamService', [])
         },
         
         // stop the stream and wipe the position in the stream. Typically used
-        // when logging out.
+        // when logging out / logged out.
         stop: function() {
             console.log("[EventStream] stop "+JSON.stringify(settings));
             // kill any running stream
             interrupt(false);
             // clear the latest token
-            settings.from = "END";
+            settings.from = END;
             saveStreamSettings();
         }
     };