git://git.onelab.eu
/
plcapi.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
allow tech to Get node and interface tags
[plcapi.git]
/
PLC
/
Methods
/
DeletePersonFromSite.py
diff --git
a/PLC/Methods/DeletePersonFromSite.py
b/PLC/Methods/DeletePersonFromSite.py
index
4479183
..
8b9ed0c
100644
(file)
--- a/
PLC/Methods/DeletePersonFromSite.py
+++ b/
PLC/Methods/DeletePersonFromSite.py
@@
-1,3
+1,5
@@
+# $Id$
+# $URL$
from PLC.Faults import *
from PLC.Method import Method
from PLC.Parameter import Parameter, Mixed
from PLC.Faults import *
from PLC.Method import Method
from PLC.Parameter import Parameter, Mixed
@@
-31,17
+33,27
@@
class DeletePersonFromSite(Method):
persons = Persons(self.api, [person_id_or_email])
if not persons:
raise PLCInvalidArgument, "No such account"
persons = Persons(self.api, [person_id_or_email])
if not persons:
raise PLCInvalidArgument, "No such account"
+ person = persons[0]
- person = persons.values()[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"
# Get site information
sites = Sites(self.api, [site_id_or_login_base])
if not sites:
raise PLCInvalidArgument, "No such site"
+ site = sites[0]
- site = sites.values()[0]
+ if site['peer_id'] is not None:
+ raise PLCInvalidArgument, "Not a local site"
if site['site_id'] in person['site_ids']:
site.remove_person(person)
if site['site_id'] in person['site_ids']:
site.remove_person(person)
+ # Logging variables
+ self.event_objects = {'Site': [site['site_id']],
+ 'Person': [person['person_id']]}
+ self.message = 'Person %d deleted from site %d ' % \
+ (person['person_id'], site['site_id'])
+
return 1
return 1