# HG changeset patch
# User Edmund Wong <ewong@crazy-cat.org>
# Date 1550480451 -28800
# Node ID b06e1141f20fbc52c6dd2a45329de49c917f14f4
# Parent  346b97cf1aaad8a23573453401a4cbc47763730f
model: rename confusing kwargs argument for repo_group update method (Issue #331)

diff -r 346b97cf1aaa -r b06e1141f20f kallithea/model/repo_group.py
--- a/kallithea/model/repo_group.py	Tue Feb 19 00:02:33 2019 +0100
+++ b/kallithea/model/repo_group.py	Mon Feb 18 17:00:51 2019 +0800
@@ -277,24 +277,24 @@
 
         return updates
 
-    def update(self, repo_group, kwargs):
+    def update(self, repo_group, repo_group_args):
         try:
             repo_group = RepoGroup.guess_instance(repo_group)
             old_path = repo_group.full_path
 
             # change properties
-            if 'group_description' in kwargs:
-                repo_group.group_description = kwargs['group_description']
-            if 'parent_group_id' in kwargs:
-                repo_group.parent_group_id = kwargs['parent_group_id']
-            if 'enable_locking' in kwargs:
-                repo_group.enable_locking = kwargs['enable_locking']
+            if 'group_description' in repo_group_args:
+                repo_group.group_description = repo_group_args['group_description']
+            if 'parent_group_id' in repo_group_args:
+                repo_group.parent_group_id = repo_group_args['parent_group_id']
+            if 'enable_locking' in repo_group_args:
+                repo_group.enable_locking = repo_group_args['enable_locking']
 
-            if 'parent_group_id' in kwargs:
-                assert kwargs['parent_group_id'] != u'-1', kwargs # RepoGroupForm should have converted to None
-                repo_group.parent_group = RepoGroup.get(kwargs['parent_group_id'])
-            if 'group_name' in kwargs:
-                group_name = kwargs['group_name']
+            if 'parent_group_id' in repo_group_args:
+                assert repo_group_args['parent_group_id'] != u'-1', repo_group_args  # RepoGroupForm should have converted to None
+                repo_group.parent_group = RepoGroup.get(repo_group_args['parent_group_id'])
+            if 'group_name' in repo_group_args:
+                group_name = repo_group_args['group_name']
                 if kallithea.lib.utils2.repo_name_slug(group_name) != group_name:
                     raise Exception('invalid repo group name %s' % group_name)
                 repo_group.group_name = repo_group.get_new_name(group_name)