Merge pull request #4515 from unicef/generateroles_handle_2

[225556] generateroles in upgrade but skip updating existing roles

src/hct_mis_api/apps/core/management/commands/generateroles.py

@@ -292,27 +292,20 @@ def handle(self, *args: Any, **options: Any) -> None:
             print("Old incompatible roles pairs were deleted.")
 
         roles_created = []
-        roles_updated = []
         for default_role in default_roles_matrix:
-            role, created = Role.objects.update_or_create(
+            role, created = Role.objects.get_or_create(
                 subsystem=Role.HOPE,
                 name=default_role["name"],
                 defaults={"permissions": [permission.value for permission in default_role["permissions"]]},
             )
 
             if created:
                 roles_created.append(role.name)
-            else:
-                roles_updated.append(role.name)
 
         if roles_created:
             print(f"New roles were created: {', '.join(roles_created)}")
         else:
             print("No new roles were created.")
-        if roles_updated:
-            print(f"These roles were updated: {', '.join(roles_updated)}")
-        else:
-            print("No roles were updated")
 
         incompatible_roles_created = []
         for role_pair in default_incompatible_roles:

src/hct_mis_api/apps/core/management/commands/upgrade.py

@@ -7,6 +7,7 @@ class Command(BaseCommand):
     def handle(self, *args: Any, **options: Any) -> None:
         call_command("migratealldb")
         call_command("collectstatic", "--no-default-ignore", interactive=False)
+        call_command("generateroles")
         from adminactions.perms import create_extra_permissions
 
         create_extra_permissions()