Merged in [16583] from rjsparks@nostrum.com:

    Restore ability to remove documents while managing a personal list. Required updating code to match the recent change to Document's primary key. Fixes ietf-tools#2757.
 - Legacy-Id: 16608
Note: SVN reference [16583] has been migrated to Git commit 8216098

Author: levkowetz

ietf/community/tests.py

@@ -123,7 +123,7 @@ def test_manage_personal_list(self):
         self.assertContains(r, draft.name)
 
         # remove document
-        r = self.client.post(url, { "action": "remove_document", "document": draft.name })
+        r = self.client.post(url, { "action": "remove_document", "document": draft.pk })
         self.assertEqual(r.status_code, 302)
         clist = CommunityList.objects.get(user__username="plain")
         self.assertTrue(not clist.added_docs.filter(pk=draft.pk))

ietf/community/views.py

@@ -67,9 +67,9 @@ def manage_list(request, username=None, acronym=None, group_type=None):
         add_doc_form = AddDocumentsForm()
 
     if request.method == 'POST' and action == 'remove_document':
-        document_name = request.POST.get('document')
-        if clist.pk is not None and document_name:
-            document = get_object_or_404(clist.added_docs, name=document_name)
+        document_id = request.POST.get('document')
+        if clist.pk is not None and document_id:
+            document = get_object_or_404(clist.added_docs, id=document_id)
             clist.added_docs.remove(document)
 
             return HttpResponseRedirect("")