From 0533690d2423214c3cd1c013cab6242c10f9d094 Mon Sep 17 00:00:00 2001
From: Alban Gruin
Date: Sat, 29 Sep 2018 16:45:53 +0200
Subject: ups2018: pas de création de cours si l’événement est global

Pour l’instant, on ne souhaite pas enregistrer les événement globaux.
On vérifie donc si l’événement est global, et, si c’est le cas, on ne
traite pas l’événement. Or, on insérait un cours dans la base de
données avant de faire ce test. Un cours avec aucun attribut était
donc laissé dans la base de données.

Ce commit déplace donc la création du cours après avoir vérifié si un
événement est global.

Signed-off-by: Alban Gruin <alban@pa1ch.fr>
---
 management/parsers/ups2018.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

(limited to 'management')

diff --git a/management/parsers/ups2018.py b/management/parsers/ups2018.py
index 805c761..522a26a 100644
--- a/management/parsers/ups2018.py
+++ b/management/parsers/ups2018.py
@@ -87,15 +87,15 @@ class Parser(AbstractParser):
             if event_year != year or event_week != week:
                 return
 
-        course = Course.objects.create(
-            source=self.source, begin=begin, end=end
-        )
-
         data = event["text"].split("<br>")
         rooms = None
         if data[0] == "Global Event":
             return
 
+        course = Course.objects.create(
+            source=self.source, begin=begin, end=end
+        )
+
         min_i = 0
         if len(data) > 0 and re.match("^\(\d+:\d+-\d+:\d+\)$", data[0]):
             min_i = 1
-- 
cgit v1.2.1