Fix adding document to agenda not being saved correctly (with regression test)

 - Legacy-Id: 2498

Author: OleLaursen

branch/iesg-tracker/ietf/idrfc/tests.py

@@ -188,6 +188,51 @@ def test_edit_info(self):
         self.assertEquals(len(mail_outbox), mailbox_before + 1)
         self.assertTrue(draft.filename in mail_outbox[-1]['Subject'])
 
+    def test_edit_telechat_date(self):
+        draft = InternetDraft.objects.get(filename="draft-ietf-mipshop-pfmipv6")
+        draft.idinternal.agenda = False
+        draft.idinternal.save()
+        url = urlreverse('doc_edit_info', kwargs=dict(name=draft.filename))
+        login_testing_unauthorized(self, "klm", url)
+
+        data = dict(intended_status=str(draft.intended_status_id),
+                    status_date=str(date.today() + timedelta(2)),
+                    area_acronym=str(draft.idinternal.area_acronym_id),
+                    via_rfc_editor="1",
+                    job_owner=str(draft.idinternal.job_owner_id),
+                    state_change_notice_to="test@example.com",
+                    note="",
+                    )
+
+        from ietf.iesg.models import TelechatDates
+
+        # add to telechat
+        data["telechat_date"] = TelechatDates.objects.all()[0].date1.isoformat()
+        r = self.client.post(url, data)
+        self.assertEquals(r.status_code, 302)
+
+        draft = InternetDraft.objects.get(filename="draft-ietf-mipshop-pfmipv6")
+        self.assertTrue(draft.idinternal.agenda)
+        self.assertEquals(draft.idinternal.telechat_date, TelechatDates.objects.all()[0].date1)
+
+        # change telechat
+        data["telechat_date"] = TelechatDates.objects.all()[0].date2.isoformat()
+        r = self.client.post(url, data)
+        self.assertEquals(r.status_code, 302)
+
+        draft = InternetDraft.objects.get(filename="draft-ietf-mipshop-pfmipv6")
+        self.assertTrue(draft.idinternal.agenda)
+        self.assertEquals(draft.idinternal.telechat_date, TelechatDates.objects.all()[0].date2)
+
+        # remove from agenda
+        data["telechat_date"] = ""
+        r = self.client.post(url, data)
+        self.assertEquals(r.status_code, 302)
+
+        draft = InternetDraft.objects.get(filename="draft-ietf-mipshop-pfmipv6")
+        self.assertTrue(not draft.idinternal.agenda)
+        
+
     def test_add_draft(self):
         draft = InternetDraft.objects.get(filename="draft-ah-rfc2141bis-urn")
         url = urlreverse('doc_edit_info', kwargs=dict(name=draft.filename))
@@ -375,6 +420,7 @@ def test_edit_position(self):
         self.assertEquals(len(pos), 0)
         self.assertEquals(draft.idinternal.comments().count(), comments_before + 1)
         self.assertTrue("Position" in draft.idinternal.comments()[0].comment_text)
+
     def test_edit_position_as_secretary(self):
         draft = InternetDraft.objects.get(filename="draft-ietf-mipshop-pfmipv6")
         url = urlreverse('doc_edit_position', kwargs=dict(name=draft.filename))

branch/iesg-tracker/ietf/idrfc/utils.py

@@ -83,6 +83,7 @@ def update_telechat(request, idinternal, new_telechat_date, new_returning_item=N
         if on_agenda:
             add_document_comment(request, doc,
                                  "Placed on agenda for telechat - %s" % new_telechat_date)
+            idinternal.telechat_date = new_telechat_date
         else:
             add_document_comment(request, doc,
                                  "Removed from agenda for telechat")