PEP8

1 files changed, 40 insertions, 25 deletions

diff --git a/management/commands/timetables.py b/management/commands/timetables.py
index 86f389e..cf48af6 100644
--- a/management/commands/timetables.py
+++ b/management/commands/timetables.py
@@ -22,10 +22,13 @@ from django.db.models import Min
 
 from edt.models import Course, Source
 from edt.utils import get_week, tz_now
-from ._private import delete_courses_in_week, get_events, get_update_date, get_weeks, get_xml
+from ._private import delete_courses_in_week, get_events, get_update_date, \
+    get_weeks, get_xml
+
 
 @transaction.atomic
-def process_timetable_week(source, soup, weeks_in_soup, force, year=None, week=None):
+def process_timetable_week(source, soup, weeks_in_soup, force,
+                           year=None, week=None):
     if year is not None and week is not None:
         begin, end = get_week(year, week)
 
@@ -40,32 +43,34 @@ def process_timetable_week(source, soup, weeks_in_soup, force, year=None, week=N
     else:
         today = tz_now()
 
-    # On récupère la mise à jour la plus ancienne dans les cours de l’emploi du temps
+    # On récupère la mise à jour la plus ancienne dans les cours de
+    # l’emploi du temps
     last_update_date = Course.objects.filter(source=source)
 
     if today is not None:
-        # Cette date concerne les éléments commençant à partir d’aujourd’hui si la valeur
-        # n’est pas nulle.
+        # Cette date concerne les éléments commençant à partir
+        # d’aujourd’hui si la valeur n’est pas nulle.
         last_update_date = last_update_date.filter(begin__gte=today)
 
     if year is not None and week is not None:
-        # Si jamais on traite une semaine spécifique, on limite les cours sélectionnés
-        # à ceux qui commencent entre le début du traitement et la fin de la semaine
+        # Si jamais on traite une semaine spécifique, on limite les
+        # cours sélectionnés à ceux qui commencent entre le début du
+        # traitement et la fin de la semaine
         last_update_date = last_update_date.filter(begin__lt=end)
 
-    last_update_date = last_update_date.aggregate(Min("last_update")) \
-                       ["last_update__min"]
+    last_update_date = last_update_date \
+        .aggregate(Min("last_update"))["last_update__min"]
 
     # Date de mise à jour de Celcat, utilisée à des fins de statistiques
     new_update_date = get_update_date(soup)
 
-    # On ne fait pas la mise à jour si jamais la dernière date de MàJ est plus récente
-    # que celle indiquée par Celcat.
-    # Attention, le champ last_update de la classe Course représente l’heure à laquelle
-    # le cours a été inséré dans la base de données, et non pas la date indiquée par
-    # Celcat.
-    if not force and last_update_date is not None and new_update_date is not None and \
-       last_update_date >= new_update_date:
+    # On ne fait pas la mise à jour si jamais la dernière date de MàJ
+    # est plus récente que celle indiquée par Celcat.  Attention, le
+    # champ last_update de la classe Course représente l’heure à
+    # laquelle le cours a été inséré dans la base de données, et non
+    # pas la date indiquée par Celcat.
+    if not force and last_update_date is not None and \
+       new_update_date is not None and last_update_date >= new_update_date:
         return
 
     if year is not None and week is not None:
@@ -78,7 +83,8 @@ def process_timetable_week(source, soup, weeks_in_soup, force, year=None, week=N
         # présente dans Celcat et maintenant.
         delete_from = min(weeks_in_soup.values())
         if not force:
-            # Si jamais on force la MàJ, on efface tout à partir de la première semaine
+            # Si jamais on force la MàJ, on efface tout à partir de la
+            # première semaine
             delete_from = max(delete_from, today)
         Course.objects.filter(source=source, begin__gte=delete_from).delete()
 
@@ -91,13 +97,15 @@ def process_timetable_week(source, soup, weeks_in_soup, force, year=None, week=N
     source.last_update_date = new_update_date
     source.save()
 
+
 def process_timetable(source, force, year=None, weeks=None):
     soup = get_xml(source.url)
     weeks_in_soup = get_weeks(soup)
 
     if year is not None and weeks is not None:
         for week in weeks:
-            process_timetable_week(source, soup, weeks_in_soup, force, year, week)
+            process_timetable_week(source, soup, weeks_in_soup, force,
+                                   year, week)
     else:
         process_timetable_week(source, soup, weeks_in_soup, force)
 
@@ -106,9 +114,12 @@ class Command(BaseCommand):
     help = "Fetches registered celcat timetables"
 
     def add_arguments(self, parser):
-        parser.add_argument("--all", const=True, default=False, action="store_const")
-        parser.add_argument("--force", const=True, default=False, action="store_const")
-        parser.add_argument("--week", type=int, choices=range(1, 54), nargs="+")
+        parser.add_argument("--all", const=True, default=False,
+                            action="store_const")
+        parser.add_argument("--force", const=True, default=False,
+                            action="store_const")
+        parser.add_argument("--week", type=int, choices=range(1, 54),
+                            nargs="+")
         parser.add_argument("--year", type=int, nargs=1)
 
     def handle(self, *args, **options):
@@ -120,7 +131,8 @@ class Command(BaseCommand):
         elif options["week"] is None:
             _, week, day = tz_now().isocalendar()
             if day >= 6:
-                year, week, _ = (tz_now() + datetime.timedelta(weeks=1)).isocalendar()
+                year, week, _ = (tz_now() + datetime.timedelta(weeks=1)) \
+                                                            .isocalendar()
             weeks = [week]
         else:
             weeks = options["week"]
@@ -132,7 +144,8 @@ class Command(BaseCommand):
                 year = options["year"][0]
 
         for source in Source.objects.all():
-            self.stdout.write("Processing {0}".format(source.formatted_timetables))
+            self.stdout.write("Processing {0}".format(
+                source.formatted_timetables))
 
             try:
                 process_timetable(source, options["force"], year, weeks)
@@ -140,7 +153,8 @@ class Command(BaseCommand):
                 break
             except Exception:
                 self.stderr.write(
-                    self.style.ERROR("Failed to process {0}:".format(source.formatted_timetables))
+                    self.style.ERROR("Failed to process {0}:".format(
+                        source.formatted_timetables))
                 )
                 self.stderr.write(self.style.ERROR(traceback.format_exc()))
                 errcount += 1
@@ -148,4 +162,5 @@ class Command(BaseCommand):
         if errcount == 0:
             self.stdout.write(self.style.SUCCESS("Done."))
         else:
-            self.stdout.write(self.style.ERROR("Done with {0} errors.".format(errcount)))
+            self.stdout.write(self.style.ERROR("Done with {0} errors.".format(
+                errcount)))