X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=PLC%2FMethods%2FAddSite.py;h=10d6585998b5d248e65eefdd148b9d446e9f178c;hb=ed7fa1ebf97ec2f88f18f8fa538e46c6ae9525c4;hp=8f77d865929342b385ebc362e697631b0815152a;hpb=7e36d319170a9ec7c4f1f31c76087c2b671dd7a7;p=plcapi.git

diff --git a/PLC/Methods/AddSite.py b/PLC/Methods/AddSite.py
index 8f77d86..10d6585 100644
--- a/PLC/Methods/AddSite.py
+++ b/PLC/Methods/AddSite.py
@@ -5,7 +5,8 @@ from PLC.Sites import Site, Sites
 from PLC.Auth import PasswordAuth
 
 can_update = lambda (field, value): field in \
-             ['is_public', 'latitude', 'longitude', 'url']
+             ['name', 'abbreviated_name', 'login_base',
+              'is_public', 'latitude', 'longitude', 'url']
 
 class AddSite(Method):
     """
@@ -18,14 +19,11 @@ class AddSite(Method):
 
     roles = ['admin']
 
-    update_fields = dict(filter(can_update, Site.fields.items()))
+    site_fields = dict(filter(can_update, Site.fields.items()))
 
     accepts = [
         PasswordAuth(),
-        Site.fields['name'],
-        Site.fields['abbreviated_name'],
-        Site.fields['login_base'],
-        update_fields
+        site_fields
         ]
 
     returns = Parameter(int, 'New site_id (> 0) if successful')
@@ -34,13 +32,11 @@ class AddSite(Method):
     object_type = 'Site'
     object_ids = []
 
-    def call(self, auth, name, abbreviated_name, login_base, site_fields = {}):
+    def call(self, auth, site_fields):
         site_fields = dict(filter(can_update, site_fields.items()))
         site = Site(self.api, site_fields)
-        site['name'] = name
-        site['abbreviated_name'] = abbreviated_name
-        site['login_base'] = login_base
         site.sync()
+
 	self.object_ids = [site['site_id']]
         
 	return site['site_id']