From: Thierry Parmentelat Date: Fri, 5 Jun 2015 07:11:27 +0000 (+0200) Subject: no change - spaces X-Git-Tag: sfa-3.1-17~14 X-Git-Url: http://git.onelab.eu/?p=sfa.git;a=commitdiff_plain;h=45497b8b240a6347ed340f2219996afccc558a29 no change - spaces --- diff --git a/sfa/managers/registry_manager.py b/sfa/managers/registry_manager.py index f17b3161..52911256 100644 --- a/sfa/managers/registry_manager.py +++ b/sfa/managers/registry_manager.py @@ -211,7 +211,7 @@ class RegistryManager: local_records=local_records.all() for local_record in local_records: - augment_with_sfa_builtins (local_record) + augment_with_sfa_builtins(local_record) logger.info("Resolve, (details=%s,type=%s) local_records=%s "%(details,type,local_records)) local_dicts = [ record.__dict__ for record in local_records ] @@ -235,7 +235,7 @@ class RegistryManager: # xxx somehow here calling dict(record) issues a weird error # however record.todict() seems to work fine # records.extend( [ dict(record) for record in local_records ] ) - records.extend( [ record.todict(exclude_types=[InstrumentedList]) for record in local_records ] ) + records.extend( [ record.todict(exclude_types=(InstrumentedList,)) for record in local_records ] ) if not records: raise RecordNotFound(str(hrns)) diff --git a/sfa/storage/model.py b/sfa/storage/model.py index c040852f..7ed955f4 100644 --- a/sfa/storage/model.py +++ b/sfa/storage/model.py @@ -210,7 +210,7 @@ class RegAuthority (RegRecord): def update_pis (self, pi_hrns, dbsession): # strip that in case we have words pi_hrns = [ x.strip() for x in pi_hrns ] - request = dbsession.query (RegUser).filter(RegUser.hrn.in_(pi_hrns)) + request = dbsession.query(RegUser).filter(RegUser.hrn.in_(pi_hrns)) logger.info("RegAuthority.update_pis: %d incoming pis, %d matches found"\ % (len(pi_hrns), request.count())) pis = dbsession.query(RegUser).filter(RegUser.hrn.in_(pi_hrns)).all() @@ -475,23 +475,24 @@ def make_record_xml (xml): # were the relationships data came from the testbed side # for each type, a dict of the form {:} # so after that, an 'authority' record will e.g. have a 'reg-pis' field with the hrns of its pi-users -augment_map={'authority': {'reg-pis' : 'reg_pis',}, - 'slice': {'reg-researchers' : 'reg_researchers',}, - 'user': {'reg-pi-authorities' : 'reg_authorities_as_pi', - 'reg-slices' : 'reg_slices_as_researcher',}, - } +augment_map = {'authority': {'reg-pis' : 'reg_pis',}, + 'slice': {'reg-researchers' : 'reg_researchers',}, + 'user': {'reg-pi-authorities' : 'reg_authorities_as_pi', + 'reg-slices' : 'reg_slices_as_researcher',}, + } + def augment_with_sfa_builtins(local_record): # don't ruin the import of that file in a client world from sfa.util.xrn import Xrn # add a 'urn' field - setattr(local_record,'reg-urn',Xrn(xrn=local_record.hrn, type=local_record.type).urn) + setattr(local_record, 'reg-urn', Xrn(xrn=local_record.hrn, type=local_record.type).urn) # users have keys and this is needed to synthesize 'users' sent over to CreateSliver if local_record.type == 'user': user_keys = [ key.key for key in local_record.reg_keys ] setattr(local_record, 'reg-keys', user_keys) # search in map according to record type - type_map=augment_map.get(local_record.type, {}) + type_map = augment_map.get(local_record.type, {}) # use type-dep. map to do the job for (field_name, attribute) in type_map.items(): # get related objects