s4-net: Simplify SamDB connect code.
authorJelmer Vernooij <jelmer@samba.org>
Thu, 8 Apr 2010 10:19:51 +0000 (12:19 +0200)
committerJelmer Vernooij <jelmer@samba.org>
Thu, 8 Apr 2010 10:21:35 +0000 (12:21 +0200)
source4/scripting/python/samba/netcmd/domainlevel.py
source4/scripting/python/samba/netcmd/dsacl.py
source4/scripting/python/samba/netcmd/enableaccount.py
source4/scripting/python/samba/netcmd/fsmo.py
source4/scripting/python/samba/netcmd/newuser.py
source4/scripting/python/samba/netcmd/pwsettings.py
source4/scripting/python/samba/netcmd/setexpiry.py
source4/scripting/python/samba/netcmd/setpassword.py

index fb26dd09deec2ebe5ca534bb8348f59851e4f0f1..c97ed7bad64ac95188a48a2919e27a8f219b0c68 100644 (file)
@@ -73,12 +73,7 @@ class cmd_domainlevel(Command):
         lp = sambaopts.get_loadparm()
         creds = credopts.get_credentials(lp)
 
-        if H is not None:
-            url = H
-        else:
-            url = lp.get("sam database")
-
-        samdb = SamDB(url=url, session_info=system_session(),
+        samdb = SamDB(url=H, session_info=system_session(),
             credentials=creds, lp=lp)
 
         domain_dn = SamDB.domain_dn(samdb)
index f846b26345a0365e99c47b01e4e08ff9cc218d2c..58fb9c611a75e1460e63cf2a259b38e2c340ad8c 100644 (file)
@@ -132,15 +132,11 @@ class cmd_ds_acl_set(Command):
         lp = sambaopts.get_loadparm()
         creds = credopts.get_credentials(lp)
 
-        if car == None or action == None or objectdn == None or trusteedn == None:
+        if (car is None or action is None or objectdn is None or 
+            trusteedn is None):
             return self.usage()
 
-        if host is not None:
-            url = host
-        else:
-            url = lp.get("sam database")
-
-        samdb = SamDB(url=url, session_info=system_session(),
+        samdb = SamDB(url=host, session_info=system_session(),
             credentials=creds, lp=lp)
         cars = {'change-rid' : GUID_DRS_CHANGE_RID_MASTER,
                 'change-pdc' : GUID_DRS_CHANGE_PDC,
index d4af0a84f1f26339ffed6e123500d50487589a2c..0e57b5714c451c8bcf032e18767bcd355148b60a 100644 (file)
@@ -55,11 +55,6 @@ class cmd_enableaccount(Command):
         lp = sambaopts.get_loadparm()
         creds = credopts.get_credentials(lp)
 
-        if H is not None:
-            url = H
-        else:
-            url = lp.get("sam database")
-
-        samdb = SamDB(url=url, session_info=system_session(),
+        samdb = SamDB(url=H, session_info=system_session(),
             credentials=creds, lp=lp)
         samdb.enable_account(filter)
index 171b89a2aab9bcd6a65b9e0810653a1dc9518062..bcf80ce2903e951783932bb85fd541fb17699575 100644 (file)
@@ -135,12 +135,7 @@ all=all of the above"""),
         lp = sambaopts.get_loadparm()
         creds = credopts.get_credentials(lp)
 
-        if host is not None:
-            url = host
-        else:
-            url = lp.get("sam database")
-
-        samdb = SamDB(url=url, session_info=system_session(),
+        samdb = SamDB(url=host, session_info=system_session(),
             credentials=creds, lp=lp)
 
         domain_dn = SamDB.domain_dn(samdb)
index 6f12de50e79c908736c3e1e301431efd26a1948f..3815219460f558cc4df8ad0ea71de5bd2bff45ff 100644 (file)
@@ -59,12 +59,7 @@ class cmd_newuser(Command):
         lp = sambaopts.get_loadparm()
         creds = credopts.get_credentials(lp)
 
-        if H is not None:
-            url = H
-        else:
-            url = lp.get("sam database")
-
-        samdb = SamDB(url=url, session_info=system_session(), credentials=creds,
+        samdb = SamDB(url=H, session_info=system_session(), credentials=creds,
             lp=lp)
         samdb.newuser(username, unixname, password,
             force_password_change_at_next_login_req=must_change_at_next_login)
index 50525cc5cee8d5b77a46e1a7b3888609a650ab32..dda009ae06ac3d8fca2cf5ca5fa63e9d0a69a0a6 100644 (file)
@@ -69,12 +69,7 @@ class cmd_pwsettings(Command):
         lp = sambaopts.get_loadparm()
         creds = credopts.get_credentials(lp)
 
-        if H is not None:
-            url = H
-        else:
-            url = lp.get("sam database")
-
-        samdb = SamDB(url=url, session_info=system_session(),
+        samdb = SamDB(url=H, session_info=system_session(),
             credentials=creds, lp=lp)
 
         domain_dn = SamDB.domain_dn(samdb)
index 0c5dc5afff9eb2255cf70b733f5207bcb1165e3b..b95192aa9cb2eb6ae25d75c4b283bfe8882493f2 100644 (file)
@@ -61,12 +61,7 @@ class cmd_setexpiry(Command):
         if days is None:
             days = 0
 
-        if H is not None:
-            url = H
-        else:
-            url = lp.get("sam database")
-
-        samdb = SamDB(url=url, session_info=system_session(),
+        samdb = SamDB(url=H, session_info=system_session(),
             credentials=creds, lp=lp)
 
         samdb.setexpiry(filter, days*24*3600, no_expiry_req=noexpiry)
index e86711d9ceee89a9441d3c40d4021152d52eb908..c4a9b00698cccb05d10f3a749ebbf61599c56ed1 100644 (file)
@@ -65,12 +65,7 @@ class cmd_setpassword(Command):
         lp = sambaopts.get_loadparm()
         creds = credopts.get_credentials(lp)
 
-        if H is not None:
-            url = H
-        else:
-            url = lp.get("sam database")
-
-        samdb = SamDB(url=url, session_info=system_session(),
+        samdb = SamDB(url=H, session_info=system_session(),
                       credentials=creds, lp=lp)
 
         samdb.setpassword(filter, password,