diff --git a/synapse/api/filtering.py b/synapse/api/filtering.py
index 014e2e1fc9..20b6951d47 100644
--- a/synapse/api/filtering.py
+++ b/synapse/api/filtering.py
@@ -16,37 +16,18 @@
from twisted.internet import defer
-# TODO(paul)
-_filters_for_user = {}
-
-
class Filtering(object):
def __init__(self, hs):
super(Filtering, self).__init__()
- self.hs = hs
+ self.store = hs.get_datastore()
- @defer.inlineCallbacks
def get_user_filter(self, user_localpart, filter_id):
- filters = _filters_for_user.get(user_localpart, None)
-
- if not filters or filter_id >= len(filters):
- raise KeyError()
+ return self.store.get_user_filter(user_localpart, filter_id)
- # trivial yield to make it a generator so d.iC works
- yield
- defer.returnValue(filters[filter_id])
-
- @defer.inlineCallbacks
def add_user_filter(self, user_localpart, definition):
- filters = _filters_for_user.setdefault(user_localpart, [])
-
- filter_id = len(filters)
- filters.append(definition)
-
- # trivial yield, see above
- yield
- defer.returnValue(filter_id)
+ # TODO(paul): implement sanity checking of the definition
+ return self.store.add_user_filter(user_localpart, definition)
# TODO(paul): surely we should probably add a delete_user_filter or
# replace_user_filter at some point? There's no REST API specified for
diff --git a/synapse/storage/__init__.py b/synapse/storage/__init__.py
index 4beb951b9f..efa63031bd 100644
--- a/synapse/storage/__init__.py
+++ b/synapse/storage/__init__.py
@@ -30,9 +30,9 @@ from .transactions import TransactionStore
from .keys import KeyStore
from .event_federation import EventFederationStore
from .media_repository import MediaRepositoryStore
-
from .state import StateStore
from .signatures import SignatureStore
+from .filtering import FilteringStore
from syutil.base64util import decode_base64
from syutil.jsonutil import encode_canonical_json
@@ -82,6 +82,7 @@ class DataStore(RoomMemberStore, RoomStore,
DirectoryStore, KeyStore, StateStore, SignatureStore,
EventFederationStore,
MediaRepositoryStore,
+ FilteringStore,
):
def __init__(self, hs):
diff --git a/synapse/storage/filtering.py b/synapse/storage/filtering.py
new file mode 100644
index 0000000000..18e0e7c298
--- /dev/null
+++ b/synapse/storage/filtering.py
@@ -0,0 +1,46 @@
+# -*- coding: utf-8 -*-
+# Copyright 2015 OpenMarket Ltd
+#
+# 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.
+
+from twisted.internet import defer
+
+from ._base import SQLBaseStore
+
+
+# TODO(paul)
+_filters_for_user = {}
+
+
+class FilteringStore(SQLBaseStore):
+ @defer.inlineCallbacks
+ def get_user_filter(self, user_localpart, filter_id):
+ filters = _filters_for_user.get(user_localpart, None)
+
+ if not filters or filter_id >= len(filters):
+ raise KeyError()
+
+ # trivial yield to make it a generator so d.iC works
+ yield
+ defer.returnValue(filters[filter_id])
+
+ @defer.inlineCallbacks
+ def add_user_filter(self, user_localpart, definition):
+ filters = _filters_for_user.setdefault(user_localpart, [])
+
+ filter_id = len(filters)
+ filters.append(definition)
+
+ # trivial yield, see above
+ yield
+ defer.returnValue(filter_id)
|
