repo group: fix API for updating parent

With other bugs out of the way, the API implementation just has to pass the new
group id correctly.

Note that - as in many other places - moving things to the top level doesn't
work.

Original patch has been modified by Mads Kiilerich.

CONTRIBUTORS

@@ -2,6 +2,7 b' List of contributors to Kallithea projec'
 
     Mads Kiilerich <[e-mail]> 2016-2022
     Manuel Jacob <[e-mail]> 2019-2020 2022
+    toras9000 <[e-mail]> 2022
     Thomas De Schampheleire <[e-mail]> 2014-2021
     ssantos <[e-mail]> 2018-2021
     Private <[e-mail]> 2019-2021

kallithea/controllers/api/api.py

@@ -1791,13 +1791,14 b' class ApiController(JSONRPCController):'
                           owner=None,
                           parent=None):
         repo_group = get_repo_group_or_error(repogroupid)
+        parent_repo_group_id = None if parent is None else get_repo_group_or_error(parent).group_id
 
         updates = {}
         try:
             store_update(updates, group_name, 'group_name')
             store_update(updates, description, 'group_description')
             store_update(updates, owner, 'owner')
-            store_update(updates, parent, 'parent_group')
+            store_update(updates, parent_repo_group_id, 'parent_group_id')
             repo_group = RepoGroupModel().update(repo_group, updates)
             meta.Session().commit()
             return dict(

kallithea/templates/about.html

@@ -26,6 +26,7 b''
 
   <li>Copyright &copy; 2012&ndash;2022, Mads Kiilerich</li>
   <li>Copyright &copy; 2019&ndash;2020, 2022, Manuel Jacob</li>
+  <li>Copyright &copy; 2022, toras9000</li>
   <li>Copyright &copy; 2014&ndash;2021, Thomas De Schampheleire</li>
   <li>Copyright &copy; 2015&ndash;2017, 2019&ndash;2021, Étienne Gilli</li>
   <li>Copyright &copy; 2018&ndash;2021, ssantos</li>

kallithea/tests/api/api_base.py

@@ -1854,7 +1854,7 b' class _BaseTestApi(object):'
         #('owner', {'owner': base.TEST_USER_REGULAR_LOGIN}),  # currently broken
         ('description', {'description': 'new description'}),
         ('group_name', {'group_name': 'new_repo_name'}),
-        #('parent', {'parent': 'test_group_for_update'}),  # currently broken
+        ('parent', {'parent': 'test_group_for_update'}),
     ])
     def test_api_update_repo_group(self, changing_attr, updates):
         group_name = 'lololo'