checkpoint. Model changes done and UI tweaked to match.

 - Legacy-Id: 9588

Author: rjsparks

ietf/bin/create-break-sessions

@@ -22,8 +22,8 @@ scheduled = SessionStatusName.objects.get(slug='sched')
 
 for meeting in Meeting.objects.filter(type="ietf").order_by("date"):
     print "Checking %s schedules ..." % meeting
-    brk, __ = Session.objects.get_or_create(meeting=meeting, group=secretariat, requested_by=system, status=scheduled, name='Break', )
-    reg, __ = Session.objects.get_or_create(meeting=meeting, group=secretariat, requested_by=system, status=scheduled, name='Registration', )
+    brk, __ = Session.objects.get_or_create(meeting=meeting, group=secretariat, requested_by=system, status=scheduled, name='Break', type_id='break',)
+    reg, __ = Session.objects.get_or_create(meeting=meeting, group=secretariat, requested_by=system, status=scheduled, name='Registration', type_id='reg',)
 
     for schedule in meeting.schedule_set.all():
         print "  Checking for missing Break and Reg sessions in %s" % schedule

ietf/doc/tests.py

@@ -527,6 +527,7 @@ def test_document_material(self):
             status = SessionStatusName.objects.create(slug='scheduled', name='Scheduled'),
             modified = datetime.datetime.now(),
             requested_by = Person.objects.get(user__username="marschairman"),
+            type_id = "session",
             )
         SessionPresentation.objects.create(session=session, document=doc, rev=doc.rev)
 

ietf/doc/tests_material.py

@@ -148,6 +148,7 @@ def test_revise(self):
             status = SessionStatusName.objects.create(slug='scheduled', name='Scheduled'),
             modified = datetime.datetime.now(),
             requested_by = Person.objects.get(user__username="marschairman"),
+            type_id="session",
             )
         SessionPresentation.objects.create(session=session, document=doc, rev=doc.rev)
 

ietf/meeting/ajax.py

@@ -158,6 +158,8 @@ def timeslot_roomurl(request, num=None, roomid=None):
 # no authorization required to list.
 def timeslot_slotlist(request, mtg):
     slots = mtg.timeslot_set.all()
+    # Restrict graphical editing to slots of type 'session' for now
+    slots = slots.filter(type__slug='session')
     json_array=[]
     for slot in slots:
         json_array.append(slot.json_dict(request.build_absolute_uri('/')))

ietf/meeting/helpers.py

@@ -85,13 +85,16 @@ def build_all_agenda_slices(meeting):
     time_slices.sort()
     return time_slices,date_slices
 
-def get_scheduledsessions_from_schedule(schedule):
-   ss = schedule.scheduledsession_set.filter(timeslot__location__isnull = False).exclude(session__isnull = True).order_by('timeslot__time','timeslot__name','session__group__group')
-
-   return ss
+# Not Used
+#def get_scheduledsessions_from_schedule(schedule):
+#   ss = schedule.scheduledsession_set.filter(timeslot__location__isnull = False).exclude(session__isnull = True).order_by('timeslot__time','timeslot__name','session__group__group')
+#
+#   return ss
 
 def get_all_scheduledsessions_from_schedule(schedule):
-   ss = schedule.scheduledsession_set.filter(timeslot__location__isnull = False).order_by('timeslot__time','timeslot__name')
+   ss = schedule.scheduledsession_set.filter(timeslot__location__isnull = False)
+   ss = ss.filter(session__type__slug='session')
+   ss = ss.order_by('timeslot__time','timeslot__name')
 
    return ss
 

ietf/meeting/models.py

@@ -161,7 +161,10 @@ def get_schedule_by_name(self, name):
 
     @property
     def sessions_that_can_meet(self):
-        return self.session_set.exclude(status__slug='notmeet').exclude(status__slug='disappr').exclude(status__slug='deleted').exclude(status__slug='apprw')
+        qs = self.session_set.exclude(status__slug='notmeet').exclude(status__slug='disappr').exclude(status__slug='deleted').exclude(status__slug='apprw')
+        # Restrict graphical scheduling to meeting requests (Sessions) of type 'session' for now
+        qs = qs.filter(type__slug='session')
+        return qs
 
     def sessions_that_can_be_placed(self):
         from django.db.models import Q
@@ -286,6 +289,7 @@ class Room(models.Model):
     name = models.CharField(max_length=255)
     capacity = models.IntegerField(null=True, blank=True)
     resources = models.ManyToManyField(ResourceAssociation, blank = True)
+    session_types = models.ManyToManyField(TimeSlotTypeName, blank = True)
 
     def __unicode__(self):
         return "%s size: %s" % (self.name, self.capacity)
@@ -851,7 +855,8 @@ class Session(models.Model):
     meeting = models.ForeignKey(Meeting)
     name = models.CharField(blank=True, max_length=255, help_text="Name of session, in case the session has a purpose rather than just being a group meeting.")
     short = models.CharField(blank=True, max_length=32, help_text="Short version of 'name' above, for use in filenames.")
-    group = models.ForeignKey(Group)    # The group type determines the session type.  BOFs also need to be added as a group.
+    type = models.ForeignKey(TimeSlotTypeName)
+    group = models.ForeignKey(Group)    # The group type historically determined the session type.  BOFs also need to be added as a group. Note that not all meeting requests have a natural group to associate with.
     attendees = models.IntegerField(null=True, blank=True)
     agenda_note = models.CharField(blank=True, max_length=255)
     requested = models.DateTimeField(default=datetime.datetime.now)
@@ -1007,7 +1012,13 @@ def agenda_text(self):
         else:
             return "The agenda has not been uploaded yet."
 
-    def type(self):
+    # FIXME - This used to be called 'type'. It is only used in agenda.csv and agenda.txt.
+    # It will return the _wrong thing_ if you look back at an agenda of an earlier meeting
+    # where group X was a BOF at the time, but is now a WG.
+    # It also doesn't return anything useful for RG, area sessions, or anything that's not group type 'wg'.
+    # A better thing to do is find a way to note when a meeting was a BoF meeting and use that, removing this
+    # function altogether.
+    def lame_description(self):
         if self.group.type.slug in [ "wg" ]:
             return "BOF" if self.group.state.slug in ["bof", "bof-conc"] else "WG"
         else:

ietf/meeting/test_data.py

@@ -21,6 +21,8 @@ def make_meeting_test_data():
     projector = ResourceAssociation.objects.create(name=pname,icon="notfound.png",desc="Basic projector")
     room = Room.objects.create(meeting=meeting, name="Test Room", capacity=123)
     breakfast_room = Room.objects.create(meeting=meeting, name="Breakfast Room", capacity=40)
+    room.session_types.add("session")
+    breakfast_room.session_types.add("lead")
     room.resources = [projector]
 
     # mars WG
@@ -30,7 +32,7 @@ def make_meeting_test_data():
     mars_session = Session.objects.create(meeting=meeting, group=mars,
                                           attendees=10, requested_by=system_person,
                                           requested_duration=20, status_id="schedw",
-                                          scheduled=datetime.datetime.now())
+                                          scheduled=datetime.datetime.now(),type_id="session")
     mars_session.resources = [projector]
     ScheduledSession.objects.create(timeslot=slot, session=mars_session, schedule=schedule)
 
@@ -40,7 +42,7 @@ def make_meeting_test_data():
     ames_session = Session.objects.create(meeting=meeting, group=Group.objects.get(acronym="ames"),
                                           attendees=10, requested_by=system_person,
                                           requested_duration=20, status_id="schedw",
-                                          scheduled=datetime.datetime.now())
+                                          scheduled=datetime.datetime.now(),type_id="session")
     ScheduledSession.objects.create(timeslot=slot, session=ames_session, schedule=schedule)
 
     # IESG breakfast
@@ -51,7 +53,7 @@ def make_meeting_test_data():
                                           name="IESG Breakfast",
                                           attendees=25, requested_by=system_person,
                                           requested_duration=20, status_id="schedw",
-                                          scheduled=datetime.datetime.now())
+                                          scheduled=datetime.datetime.now(),type_id="lead")
     ScheduledSession.objects.create(timeslot=breakfast_slot, session=iesg_session, schedule=schedule)
 
     meeting.agenda = schedule

ietf/meeting/tests_api.py

@@ -204,7 +204,7 @@ def test_sessions_json(self):
         r = self.client.get(url)
         self.assertEqual(r.status_code, 200)
         info = json.loads(r.content)
-        self.assertEqual(set([x['short_name'] for x in info]),set([s.session.short_name for s in meeting.agenda.scheduledsession_set.all()]))
+        self.assertEqual(set([x['short_name'] for x in info]),set([s.session.short_name for s in meeting.agenda.scheduledsession_set.filter(session__type_id='session')]))
 
         schedule = meeting.agenda
         url = urlreverse("ietf.meeting.ajax.scheduledsessions_json",

ietf/meeting/views.py

@@ -261,7 +261,7 @@ def edit_agenda(request, num=None, owner=None, name=None):
 
     #rooms = meeting.room_set.order_by("capacity")
     #rooms = rooms.all()
-    rooms = meeting.room_set.filter(timeslot__type__slug='session').distinct().order_by("capacity")
+    rooms = meeting.room_set.filter(session_types__slug='session').distinct().order_by("capacity")
     saveas = SaveAsForm()
     saveasurl=reverse(edit_agenda,
                       args=[meeting.number, schedule.owner_email(), schedule.name])
@@ -644,7 +644,7 @@ def ical_agenda(request, num=None, name=None, ext=None):
 
 def meeting_requests(request, num=None) :
     meeting = get_meeting(num)
-    sessions = Session.objects.filter(meeting__number=meeting.number,group__parent__isnull = False).exclude(requested_by=0).order_by("group__parent__acronym","status__slug","group__acronym")
+    sessions = Session.objects.filter(meeting__number=meeting.number, type__slug='session', group__parent__isnull = False).exclude(requested_by=0).order_by("group__parent__acronym","status__slug","group__acronym")
 
     groups_not_meeting = Group.objects.filter(state='Active',type__in=['WG','RG','BOF']).exclude(acronym__in = [session.group.acronym for session in sessions]).order_by("parent__acronym","acronym")
 

ietf/secr/meetings/views.py

@@ -105,13 +105,17 @@ def build_nonsession(meeting,schedule):
                               short=get_session(slot).short,
                               group=get_session(slot).group,
                               requested_by=system,
-                              status_id='sched')
+                              status_id='sched',
+                              type=slot.type,
+                             )
         else:
             session, __ = Session.objects.get_or_create(meeting=meeting,
                               name=slot.name,
                               group=secretariat,
                               requested_by=system,
-                              status_id='sched')
+                              status_id='sched',
+                              type=slot.type,
+                             )
         session.save()
 
         ts = TimeSlot.objects.create(type=slot.type,
@@ -469,7 +473,9 @@ def non_session(request, meeting_id, schedule_name):
                                   short=short,
                                   group=group,
                                   requested_by=Person.objects.get(name='(system)'),
-                                  status_id='sched')
+                                  status_id='sched',
+                                  type=type,
+                             )
             session.save()
 
             # create association