Mercurial > libervia-backend
changeset 1491:704ca56f5ca9
core (disco): added checkFeatures to check several features at once + identities are now managed with a tuple in findFeaturesSet and checkFeatures
author | Goffi <goffi@goffi.org> |
---|---|
date | Tue, 25 Aug 2015 16:05:01 +0200 |
parents | 55cff13b1f10 |
children | 6ec2741b7268 |
files | src/core/sat_main.py src/memory/disco.py |
diffstat | 2 files changed, 25 insertions(+), 16 deletions(-) [+] |
line wrap: on
line diff
--- a/src/core/sat_main.py Tue Aug 25 15:39:16 2015 +0200 +++ b/src/core/sat_main.py Tue Aug 25 16:05:01 2015 +0200 @@ -703,6 +703,9 @@ def checkFeature(self, *args, **kwargs): return self.memory.disco.checkFeature(*args, **kwargs) + def checkFeatures(self, *args, **kwargs): + return self.memory.disco.checkFeatures(*args, **kwargs) + def getDiscoInfos(self, *args, **kwargs): return self.memory.disco.getInfos(*args, **kwargs)
--- a/src/memory/disco.py Tue Aug 25 15:39:16 2015 +0200 +++ b/src/memory/disco.py Tue Aug 25 16:05:01 2015 +0200 @@ -81,14 +81,30 @@ @param feature: feature namespace @param jid_: jid of the target, or None for profile's server @param profile_key: %(doc_profile_key)s - @return: None if feature is found @raise: exceptions.FeatureNotFound """ disco_infos = yield self.getInfos(jid_, profile_key) if not feature in disco_infos.features: raise exceptions.FeatureNotFound - defer.returnValue(feature in disco_infos) + + @defer.inlineCallbacks + def checkFeatures(self, features, jid_=None, identity=None, profile_key=C.PROF_KEY_NONE): + """Like checkFeature, but check several features at once, and check also identity + + @param features(iterable[unicode]): features to check + @param jid_(jid.JID): jid of the target, or None for profile's server + @param identity(None, tuple(unicode, unicode): if not None, the entity must have an identity with this (category, type) tuple + @param profile_key: %(doc_profile_key)s + + @raise: exceptions.FeatureNotFound + """ + disco_infos = yield self.getInfos(jid_, profile_key) + if not set(features).issubset(disco_infos.features): + raise exceptions.FeatureNotFound + + if identity is not None and identity not in disco_infos.identities: + raise exceptions.FeatureNotFound def getInfos(self, jid_=None, profile_key=C.PROF_KEY_NONE): """get disco infos from jid_, filling capability hash if needed @@ -177,12 +193,11 @@ reactor.callLater(TIMEOUT, d.cancel) # FIXME: one bad service make a general timeout return d - def findFeaturesSet(self, features, category=None, type_=None, jid_=None, profile_key=C.PROF_KEY_NONE): + def findFeaturesSet(self, features, identity=None, jid_=None, profile_key=C.PROF_KEY_NONE): """Return entities (including jid_ and its items) offering features @param features: iterable of features which must be present - @param category: if not None, accept only this category - @param type_: if not None, accept only this type + @param identity(None, tuple(unicode, unicode)): if not None, accept only this (category/type) identity @param jid_: the jid of the target server (None for profile's server) @param profile_key: %(doc_profile_key)s @return: a set of found entities @@ -194,17 +209,8 @@ found_entities = set() def infosCb(infos, entity): - if category is not None or type_ is not None: - categories = set() - types = set() - for identity in infos.identities: - id_cat, id_type = identity - categories.add(id_cat) - types.add(id_type) - if category is not None and category not in categories: - return - if type_ is not None and type_ not in types: - return + if identity is not None and identity not in infos.identities: + return if features.issubset(infos.features): found_entities.add(entity)