From: Mark Huang <mlhuang@cs.princeton.edu>
Date: Thu, 7 Sep 2006 23:49:15 +0000 (+0000)
Subject: - use Person.set_primary_site
X-Git-Tag: pycurl-7_13_1~778
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=59c02e0f444347d2ca3fbee0bac41c34f6986d43;p=plcapi.git

- use Person.set_primary_site
---

diff --git a/PLC/Methods/AdmSetPersonPrimarySite.py b/PLC/Methods/AdmSetPersonPrimarySite.py
index 32463336..463afa57 100644
--- a/PLC/Methods/AdmSetPersonPrimarySite.py
+++ b/PLC/Methods/AdmSetPersonPrimarySite.py
@@ -51,14 +51,6 @@ class AdmSetPersonPrimarySite(Method):
         if site['site_id'] not in person['site_ids']:
             raise PLCInvalidArgument, "Not a member of the specified site"
 
-        person_id = person['person_id']
-        site_id = site['site_id']
-        self.api.db.do("UPDATE person_site SET is_primary = False" \
-                       " WHERE person_id = %(person_id)d",
-                       locals())
-        self.api.db.do("UPDATE person_site SET is_primary = True" \
-                       " WHERE person_id = %(person_id)d" \
-                       " AND site_id = %(site_id)d",
-                       locals())
+        person.set_primary_site(site)
 
         return 1