Remove extra logging in command

backend/clubs/management/commands/merge_duplicate_committees.py

@@ -1,6 +1,4 @@
-import json
 import logging
-from datetime import datetime
 
 from django.core.management.base import BaseCommand
 from django.db import transaction
@@ -24,7 +22,6 @@ def add_arguments(self, parser):
 
     def handle(self, *args, **options):
         dry_run = options["dry_run"]
-        audit_log = []
 
         try:
             with transaction.atomic():
@@ -47,33 +44,14 @@ def handle(self, *args, **options):
                     if not primary_committee:
                         continue
 
-                    self.stdout.write(
+                    logger.info(
                         f"Processing duplicate committees with name '{name}' "
                         f"in application {application_id}"
                     )
 
-                    # Record the merge operation
-                    merge_record = {
-                        "timestamp": datetime.now().isoformat(),
-                        "primary_committee_id": primary_committee.id,
-                        "name": name,
-                        "application_id": application_id,
-                        "merged_committees": [],
-                    }
-
                     # Merge all other committees into the primary one
                     for committee in committees[1:]:
                         try:
-                            # Record the committee being merged
-                            committee_record = {
-                                "committee_id": committee.id,
-                                "questions": list(
-                                    committee.applicationquestion_set.values_list(
-                                        "id", flat=True
-                                    )
-                                ),
-                            }
-
                             if not dry_run:
                                 # Reassign questions to primary committee
                                 questions_moved = (
@@ -83,47 +61,30 @@ def handle(self, *args, **options):
                                 )
                                 committee.delete()
 
-                                self.stdout.write(
+                                logger.info(
                                     f"Moved {questions_moved} questions from committee "
                                     f"{committee.id} to {primary_committee.id}"
                                 )
                             else:
-                                self.stdout.write(
+                                logger.info(
                                     f"[DRY RUN] Would move questions from committee "
                                     f"{committee.id} to {primary_committee.id}"
                                 )
 
-                            merge_record["merged_committees"].append(committee_record)
-
                         except Exception as e:
-                            self.stdout.write(
-                                self.style.ERROR(
-                                    f"Failed to merge committee {committee.id} into "
-                                    f"{primary_committee.id}: {str(e)}"
-                                )
+                            logger.error(
+                                f"Failed to merge committee {committee.id} into "
+                                f"{primary_committee.id}: {str(e)}"
                             )
                             raise
 
-                    audit_log.append(merge_record)
-
                 if dry_run:
-                    self.stdout.write(
-                        self.style.SUCCESS("Dry run completed - rolling back")
-                    )
+                    logger.info("Dry run completed - rolling back")
                     raise Exception("Dry run completed - rolling back")
                 else:
-                    # Save audit log
-                    timestamp = datetime.now().strftime("%Y%m%d_%H%M%S")
-                    filename = f"committee_merge_audit_{timestamp}.json"
-                    with open(filename, "w") as f:
-                        json.dump(audit_log, f, indent=2)
-                    self.stdout.write(
-                        self.style.SUCCESS(
-                            f"Merge completed. Audit log saved to {filename}"
-                        )
-                    )
+                    logger.info("Successfully merged duplicate committees")
 
         except Exception as e:
             if not dry_run:
-                self.stdout.write(self.style.ERROR(f"Failed: {str(e)}"))
+                logger.error(f"Failed: {str(e)}")
             raise