X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=PLC%2FMethods%2FAddPersonToSite.py;h=4cb786f74b5ba65fe1997b454ecd53d6c16adc99;hb=bfbd7ab0a157ca63f286836252d143341ca677ec;hp=5ada73135328d2b186d5c57e864aff89fa83e6ae;hpb=d4c363a3ee55334eacd91f303adf6ecc20055d1e;p=plcapi.git diff --git a/PLC/Methods/AddPersonToSite.py b/PLC/Methods/AddPersonToSite.py index 5ada731..4cb786f 100644 --- a/PLC/Methods/AddPersonToSite.py +++ b/PLC/Methods/AddPersonToSite.py @@ -31,18 +31,25 @@ class AddPersonToSite(Method): persons = Persons(self.api, [person_id_or_email]) if not persons: raise PLCInvalidArgument, "No such account" - person = persons[0] + if person['peer_id'] is not None: + raise PLCInvalidArgument, "Not a local account" + # Get site information sites = Sites(self.api, [site_id_or_login_base]) if not sites: raise PLCInvalidArgument, "No such site" - site = sites[0] + if site['peer_id'] is not None: + raise PLCInvalidArgument, "Not a local site" + if site['site_id'] not in person['site_ids']: site.add_person(person) + + # Logging variables self.object_ids = [site['site_id']] - + self.message = 'Person %d added to site %d' % \ + (person['person_id'], site['site_id']) return 1