Use tuples instead of lists for class variables

3 files changed, 5 insertions, 4 deletions

diff --git a/ipalib/plugins/basegroup.py b/ipalib/plugins/basegroup.py
index 54913566c..7cba75a47 100644
--- a/ipalib/plugins/basegroup.py
+++ b/ipalib/plugins/basegroup.py
@@ -27,7 +27,7 @@ from ipalib import Str, Int, Flag, List  # Parameter types
 from ldap.dn import escape_dn_chars
 
 
-default_attributes = ['cn','description','member','memberof']
+default_attributes = ('cn','description','member','memberof')
 default_class = "groupofnames"
 
 
@@ -88,7 +88,7 @@ class BaseGroup(Object):
 class basegroup_add(crud.Add):
     'Add a new group.'
 
-    base_classes = ["top", default_class]
+    base_classes = ("top", default_class)
 
     def execute(self, cn, **kw):
         """
@@ -214,7 +214,7 @@ class basegroup_find(crud.Find):
 
         return ldap.search(**search_kw)
 
-    def output_for_cli(self, textui, result, uid, **options):
+    def output_for_cli(self, textui, result, criteria, **options):
         counter = result[0]
         groups = result[1:]
         if counter == 0 or len(groups) == 0:
diff --git a/ipalib/plugins/hostgroup.py b/ipalib/plugins/hostgroup.py
index 71b14993e..8afc563fc 100644
--- a/ipalib/plugins/hostgroup.py
+++ b/ipalib/plugins/hostgroup.py
@@ -38,7 +38,7 @@ api.register(hostgroup)
 
 class hostgroup_add(basegroup_add):
     'Add a new hostgroup.'
-    base_classes = ["top", "groupofnames", "ipaHostGroup"]
+    base_classes = ("top", "groupofnames", "ipaHostGroup")
 
 api.register(hostgroup_add)
 
diff --git a/ipalib/plugins/rolegroup.py b/ipalib/plugins/rolegroup.py
index c843c0988..092b9551f 100644
--- a/ipalib/plugins/rolegroup.py
+++ b/ipalib/plugins/rolegroup.py
@@ -38,6 +38,7 @@ api.register(rolegroup)
 
 class rolegroup_add(basegroup_add):
     'Add a new rolegroup.'
+    base_classes = ("top", "groupofnames", "nestedgroup")
 
 api.register(rolegroup_add)