From e1c4eea3c706bb45cd73416e2e8626bd50a97cfb Mon Sep 17 00:00:00 2001
From: Alban Gruin
Date: Mon, 27 Nov 2017 16:04:27 +0100
Subject: Correction des interfaces utilisateur

---
 feeds.py | 53 ++++++++++++++++++++++++-----------------------------
 1 file changed, 24 insertions(+), 29 deletions(-)

(limited to 'feeds.py')

diff --git a/feeds.py b/feeds.py
index aced5e6..95ecaf8 100644
--- a/feeds.py
+++ b/feeds.py
@@ -24,7 +24,7 @@ from django.utils.feedgenerator import Atom1Feed, SyndicationFeed
 
 from icalendar import Calendar, Event
 
-from .models import Course, Group
+from .models import Course, Group, Timetable
 from .templatetags.rooms import format_rooms
 from .utils import get_current_or_next_week, get_week, group_courses
 
@@ -58,8 +58,9 @@ class IcalFeed(Feed):
 
     def get_object(self, request, year_slug, timetable_slug, group_slug):
         try:
-            group = Group.objects.get(timetable__year__slug=year_slug,
-                                      timetable__slug=timetable_slug,
+            timetable = Timetable.objects.get(year__slug=year_slug,
+                                              slug=timetable_slug)
+            group = Group.objects.get(timetable=timetable.source,
                                       slug=group_slug)
         except:
             raise ObjectDoesNotExist
@@ -81,7 +82,7 @@ class IcalFeed(Feed):
         return item.name
 
     def items(self, obj):
-        return Course.objects.get_courses_for_group(obj)
+        return Course.objects.get_courses(obj)
 
     def item_extra_kwargs(self, item):
         return {"uid": "{0}@celcatsanitizer".format(item.id),
@@ -103,39 +104,37 @@ class RSSFeed(Feed):
         _, end = get_week(year, week)
 
         try:
-            group = Group.objects.get(timetable__year__slug=year_slug,
-                                      timetable__slug=timetable_slug,
-                                      slug=group_slug)
+            self.timetable = Timetable.objects.get(year__slug=year_slug,
+                                                   slug=timetable_slug)
+            self.group = Group.objects.get(timetable=self.timetable.source,
+                                           slug=group_slug)
         except:
             raise ObjectDoesNotExist
         else:
-            updates = Course.objects.get_courses_for_group(group,
-                                                           begin__lt=end) \
+            updates = Course.objects.get_courses(self.group, begin__lt=end) \
                                     .annotate(year=ExtractYear("begin"),
                                               week=ExtractWeek("begin")) \
                                     .values("year", "week") \
                                     .annotate(Count("year", distinct=True),
                                               Max("last_update")) \
                                     .order_by("-year", "-week")[:5]
-            return group, updates
+            return updates
 
     def link(self, obj):
-        group = obj[0]
         link = reverse("timetable",
-                       kwargs={"year_slug": group.timetable.year.slug,
-                               "timetable_slug": group.timetable.slug,
-                               "group_slug": group.slug})
+                       kwargs={"year_slug": self.timetable.year.slug,
+                               "timetable_slug": self.timetable.slug,
+                               "group_slug": self.group.slug})
         return link
 
     def title(self, obj):
-        return "Emploi du temps du groupe {0}".format(obj[0])
+        return "Emploi du temps du groupe {0}".format(self.group)
 
     def item_link(self, item):
-        group = item["group"]
         return reverse("timetable",
-                       kwargs={"year_slug": group.timetable.year.slug,
-                               "timetable_slug": group.timetable.slug,
-                               "group_slug": group.slug,
+                       kwargs={"year_slug": self.timetable.year.slug,
+                               "timetable_slug": self.timetable.slug,
+                               "group_slug": self.group.slug,
                                "year": item["year"],
                                "week": item["week"]})
 
@@ -143,7 +142,7 @@ class RSSFeed(Feed):
         return item["description"]
 
     def item_title(self, item):
-        return "{0}, semaine {1} de {2}".format(item["group"],
+        return "{0}, semaine {1} de {2}".format(self.group,
                                                 item["week"],
                                                 item["year"])
 
@@ -152,23 +151,19 @@ class RSSFeed(Feed):
 
     def items(self, obj):
         template = loader.get_template("timetable_common.html")
-        group = obj[0]
 
-        for update in obj[1]:
+        for update in obj:
             start, end = get_week(update["year"], update["week"])
-            courses = Course.objects.get_courses_for_group(group,
-                                                           begin__gte=start,
-                                                           begin__lt=end)
-            context = {"group": group,
-                       "courses": group_courses(courses),
+            courses = Course.objects.get_courses(self.group, begin__gte=start,
+                                                 begin__lt=end)
+            context = {"courses": group_courses(courses),
                        "last_update": update["last_update__max"],
                        "year": update["year"],
                        "week": update["week"]}
 
-            update["group"] = group
             update["description"] = template.render(context)
 
-        return obj[1]
+        return obj
 
 
 class AtomFeed(RSSFeed):
-- 
cgit v1.2.1