Refactor the registration number cleanup

Author: tudoramariei

backend/donations/management/commands/registration_numbers_cleanup.py

@@ -18,11 +18,12 @@ class Command(BaseCommand):
     def handle(self, *args, **options):
         errors: List[str] = []
         target_ngos = Ngo.objects.filter(registration_number_valid=None)
+
         if target_ngos.count() == 0:
             target_ngos = Ngo.objects.filter(registration_number_valid=False)
+
         if target_ngos.count() == 0:
             self.stdout.write(self.style.SUCCESS("No NGOs to clean registration numbers for."))
-
             return
 
         for ngo_id in target_ngos.values_list("pk", flat=True):
@@ -54,7 +55,7 @@ def clean_ngo(self, ngo_id: int) -> Dict[str, str]:
 
     def clean_ngo_registration_number(self, ngo: Ngo) -> Dict[str, str]:
         initial_registration_number = ngo.registration_number
-        cleaned_registration_number = self._clean_registration_number(initial_registration_number)
+        cleaned_registration_number = self._clean_up_registration_number(initial_registration_number)
 
         if not re.match(REGISTRATION_NUMBER_REGEX, cleaned_registration_number):
             self.stdout.write(
@@ -107,7 +108,7 @@ def clean_ngo_registration_number(self, ngo: Ngo) -> Dict[str, str]:
         }
 
     @staticmethod
-    def _clean_registration_number(reg_num: str) -> Optional[str]:
+    def _clean_up_registration_number(reg_num: str) -> Optional[str]:
         if re.match(REGISTRATION_NUMBER_REGEX, reg_num):
             return reg_num