altf4arnold/datatracker
1
0
Fork 0
Code Issues Pull requests Actions 3 Packages Projects Releases Wiki Activity

Handle num=None case in proceedings() and specify meeting number in test_proceedings_no_agenda. Fixes #3385. Commit ready for merge.

Browse source 
- Legacy-Id: 19300
This commit is contained in:
Jennifer Richards 2021-08-20 16:16:00 +00:00
parent b88a695ad1
commit ca78da6ff4
2 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
ietf/meeting/tests_views.py
View file

@ -615,7 +615,8 @@ class MeetingTests(TestCase):
self.assertEqual(r.status_code, 200)
def test_proceedings_no_agenda(self):
meeting = MeetingFactory(type_id='ietf',populate_schedule=False,date=datetime.date.today())
# Meeting number must be larger than the last special-cased proceedings (currently 96)
meeting = MeetingFactory(type_id='ietf',populate_schedule=False,date=datetime.date.today(), number='100')
url = urlreverse('ietf.meeting.views.proceedings')
r = self.client.get(url)
self.assertRedirects(r, urlreverse('ietf.meeting.views.materials'))

2
ietf/meeting/views.py
View file

@ -3791,7 +3791,7 @@ def proceedings(request, num=None):
meeting = get_meeting(num)
if (meeting.number.isdigit() and int(meeting.number) <= 96):
return HttpResponseRedirect( 'https://www.ietf.org/proceedings/%s' % num )
return HttpResponseRedirect('https://www.ietf.org/proceedings/%s' % meeting.number)
if not meeting.schedule or not meeting.schedule.assignments.exists():
kwargs = dict()