management/commands/anonymize_data.py

@@ -10,7 +10,16 @@ class Command(BaseCommand):
     Command to protect sensitive data during the beta phase, to prevent a right escalation.
     Phone number, email address, postal address, first and last name are removed.
     """
+    def add_arguments(self, parser):
+        parser.add_argument('--force', '-f', action='store_true', help="Actually anonymize data.")
+
     def handle(self, *args, **kwargs):
+        if not kwargs['force']:
+            self.stderr.write("CAUTION: This is a dangerous script. This will reset all personal data with "
+                              "sample data. Don't use this in production! If you know what you are doing, "
+                              "please add --force option.")
+            exit(1)
+
         cur = connection.cursor()
         cur.execute("UPDATE member_profile SET "
                     "phone_number = '0123456789', "

management/commands/check_consistency.py

 # Copyright (C) 2018-2020 by BDE ENS Paris-Saclay
 # SPDX-License-Identifier: GPL-3.0-or-later
-from django.core.mail import send_mail
+
 from django.core.management import BaseCommand
 from django.db.models import Sum, F
 
@@ -14,7 +14,6 @@ class Command(BaseCommand):
         parser.add_argument('--check-all', '-a', action='store_true', help='Check all notes')
         parser.add_argument('--check', '-c', type=int, nargs='+', help='Select note ids')
         parser.add_argument('--fix', '-f', action='store_true', help='Fix note balances')
-        parser.add_argument('--mail', '-m', action='store_true', help='Send mail to admins if there is an error')
 
     def handle(self, *args, **options):
         error = False
@@ -23,10 +22,11 @@ class Command(BaseCommand):
         if options["sum_all"]:
             s = Note.objects.aggregate(Sum("balance"))["balance__sum"]
             if s:
-                err_log += self.style.NOTICE("LA SOMME DES NOTES NE VAUT PAS ZÉRO : " + pretty_money(s)) + "\n"
+                self.stderr.write(self.style.NOTICE("LA SOMME DES NOTES NE VAUT PAS ZÉRO : " + pretty_money(s)))
                 error = True
             else:
-                self.stdout.write(self.style.SUCCESS("La somme des notes vaut bien zéro."))
+                if options["verbosity"] > 0:
+                    self.stdout.write(self.style.SUCCESS("La somme des notes vaut bien zéro."))
 
         notes = Note.objects.none()
         if options["check_all"]:
@@ -42,19 +42,13 @@ class Command(BaseCommand):
                 .annotate(total=F("quantity") * F("amount")).aggregate(Sum("total"))["total__sum"] or 0
             calculated_balance = incoming - outcoming
             if calculated_balance != balance:
-                err_log += self.style.NOTICE("LA SOMME DES TRANSACTIONS DE LA NOTE {} NE CORRESPOND PAS "
-                                             "AVEC LE MONTANT RÉEL".format(str(note))) + "\n"
-                err_log += self.style.NOTICE("Attendu : {}, calculé : {}"
-                                             .format(pretty_money(balance), pretty_money(calculated_balance))) + "\n"
+                self.stderr.write(self.style.NOTICE(f"LA SOMME DES TRANSACTIONS DE LA NOTE {note} NE CORRESPOND PAS "
+                                                    "AVEC LE MONTANT RÉEL"))
+                self.stderr.write(self.style.NOTICE(f"Attendu : {pretty_money(balance)}, "
+                                                    f"calculé : {pretty_money(calculated_balance)}"))
                 if options["fix"]:
                     note.balance = calculated_balance
                     note.save()
                 error = True
 
-        if error:
-            self.stderr.write(err_log)
-            if options["mail"]:
-                send_mail("[Note Kfet] La base de données n'est pas consistante", err_log,
-                          "NoteKfet2020 <notekfet2020@crans.org>", ["respo-info.bde@lists.crans.org"])
-
         exit(1 if error else 0)

management/commands/compilejsmessages.py

 # Copyright (C) 2018-2020 by BDE ENS Paris-Saclay
 # SPDX-License-Identifier: GPL-3.0-or-later
+
 import os
 
 from django.conf import settings
@@ -12,17 +13,15 @@ class Command(BaseCommand):
     """
     Generate Javascript translation files
     """
-    def add_arguments(self, parser):
-        parser.add_argument('--out', '-o', type=str, default='static', help='Output directory, where static files are.')
-
     def handle(self, *args, **kwargs):
         for code, _ in settings.LANGUAGES:
             if code == settings.LANGUAGE_CODE:
                 continue
-            self.stdout.write(f"Generate {code} javascript localization file")
+            if kwargs["verbosity"] > 0:
+                self.stdout.write(f"Generate {code} javascript localization file")
             with translation.override(code):
                 resp = JavaScriptCatalog().get(None, packages="member+note")
-                if not os.path.isdir(kwargs["out"] + "/js/jsi18n"):
-                    os.makedirs(kwargs["out"] + "/js/jsi18n")
-                with open(kwargs["out"] + f"/js/jsi18n/{code}.js", "wb") as f:
+                if not os.path.isdir(settings.STATIC_ROOT + "/js/jsi18n"):
+                    os.makedirs(settings.STATIC_ROOT + "/js/jsi18n")
+                with open(settings.STATIC_ROOT + f"/js/jsi18n/{code}.js", "wb") as f:
                     f.write(resp.content)

management/commands/extract_ml_registrations.py

@@ -25,6 +25,10 @@ class Command(BaseCommand):
     def handle(self, *args, **options):
         # TODO: Improve the mailing list extraction system, and link it automatically with Mailman.
 
+        if options['verbosity'] == 0:
+            # This is useless, but this what the user asked.
+            return
+
         if options["type"] == "members":
             for membership in Membership.objects.filter(
                 club__name="BDE",

management/commands/force_delete_user.py

@@ -78,11 +78,11 @@ class Command(BaseCommand):
                     # Deleting transactions
                     transactions = Transaction.objects.filter(Q(source=user.note) | Q(destination=user.note)).all()
                     local_deleted += list(transactions)
-                    if kwargs['verbosity'] >= 1:
-                        for tr in transactions:
+                    for tr in transactions:
+                        if kwargs['verbosity'] >= 1:
                             self.stdout.write(f"Removing {tr}...")
-                    if force:
-                        transactions.delete()
+                        if force:
+                            tr.delete()
 
                     # Deleting memberships
                     memberships = user.memberships.all()
@@ -173,4 +173,5 @@ class Command(BaseCommand):
             message += "Ont été supprimés en conséquence les objets suivants :\n\n"
             for obj in deleted:
                 message += f"{repr(obj)} (pk: {obj.pk})\n"
-            mail_admins("Utilisateurs supprimés", message)
+            if force and kwargs['doit']:
+                mail_admins("Utilisateurs supprimés", message)

management/commands/make_su.py

@@ -16,7 +16,11 @@ class Command(BaseCommand):
             user = User.objects.get(username=uname)
             user.is_active = True
             if kwargs['STAFF']:
+                if kwargs['verbosity'] > 0:
+                    self.stdout.write(f"Add {user} to staff users...")
                 user.is_staff = True
             if kwargs['SUPER']:
+                if kwargs['verbosity'] > 0:
+                    self.stdout.write(f"Add {user} to superusers...")
                 user.is_superuser = True
             user.save()

management/commands/refresh_highlighted_buttons.py

@@ -23,7 +23,8 @@ class Command(BaseCommand):
         for d in queryset.all():
             button_id = d["template"]
             button = TransactionTemplate.objects.get(pk=button_id)
-            self.stdout.write(self.style.WARNING("Highlight button {name} ({count:d} transactions)..."
-                                                 .format(name=button.name, count=d["transaction_count"])))
+            if kwargs['verbosity'] > 0:
+                self.stdout.write(self.style.WARNING("Highlight button {name} ({count:d} transactions)..."
+                                                     .format(name=button.name, count=d["transaction_count"])))
             button.highlighted = True
             button.save()

management/commands/send_mail_to_negative_balances.py

 # Copyright (C) 2018-2020 by BDE ENS Paris-Saclay
 # SPDX-License-Identifier: GPL-3.0-or-later
 
-from datetime import date
+from datetime import date, timedelta
 
 from django.core.mail import send_mail
 from django.core.management import BaseCommand
@@ -17,12 +17,22 @@ class Command(BaseCommand):
         parser.add_argument("--spam", "-s", action='store_true', help="Spam negative users")
         parser.add_argument("--report", "-r", action='store_true', help="Report the list of negative users to admins")
         parser.add_argument("--negative-amount", "-n", action='store', type=int, default=1000,
-                            help="Maximum amount to be considered as very negative")
+                            help="Maximum amount to be considered as very negative (inclusive)")
+        parser.add_argument("--add-years", "-y", action='store', type=int, default=0,
+                            help="Add also people that have a negative balance since N years "
+                                 "(default to only active members)")
 
     def handle(self, *args, **options):
         activate('fr')
+
+        if options['negative_amount'] == 0:
+            # Don't log empty notes
+            options['negative_amount'] = 1
+
         notes = Note.objects.filter(
-            Q(noteuser__user__memberships__date_end__gte=date.today()) | Q(noteclub__isnull=False),
+            Q(noteuser__user__memberships__date_end__gte=
+              date.today() - timedelta(days=int(365.25 * options['add_years'])))
+            | Q(noteclub__isnull=False),
             balance__lte=-options["negative_amount"],
             is_active=True,
         ).order_by('balance').distinct().all()

management/commands/send_reports.py

@@ -20,8 +20,8 @@ class Command(BaseCommand):
 
     def handle(self, *args, **options):
         activate('fr')
-        if "notes" in options:
-            notes = NoteUser.objects.filter(pk__in=options["notes"]).all()
+        if 'notes' in options and options['notes']:
+            notes = NoteUser.objects.filter(pk__in=options['notes']).all()
         else:
             notes = NoteUser.objects.filter(
                 user__memberships__date_end__gte=timezone.now(),
@@ -33,8 +33,9 @@ class Command(BaseCommand):
             delta = now.date() - last_report.date()
             if delta.days < note.user.profile.report_frequency:
                 continue
-            note.user.profile.last_report = now
-            note.user.profile.save()
+            if not options["debug"]:
+                note.user.profile.last_report = now
+                note.user.profile.save()
             last_transactions = Transaction.objects.filter(
                 Q(source=note) | Q(destination=note),
                 created_at__gte=last_report,