Provide previous ballot positions for both current and former ADs; add test to cover this situation. Commit ready for merge.

 - Legacy-Id: 19208

Author: russhousley

ietf/doc/tests.py

@@ -1521,7 +1521,24 @@ def test_document_ballot(self):
         r = self.client.get(urlreverse("ietf.doc.views_doc.document_ballot", kwargs=dict(name=doc.name)))
         self.assertEqual(r.status_code, 200)
         self.assertContains(r,  '(%s for -%s)' % (pos.comment_time.strftime('%Y-%m-%d'), oldrev))
-        
+
+        # Now simulate a new ballot against the new revision and make sure the "was" position is included
+        pos2 = BallotPositionDocEvent.objects.create(
+            doc=doc,
+            rev=doc.rev,
+            ballot=ballot,
+            type="changed_ballot_position",
+            pos_id="noobj",
+            comment="Still looks okay to me",
+            comment_time=datetime.datetime.now(),
+            balloter=Person.objects.get(user__username="ad"),
+            by=Person.objects.get(name="(System)"))
+
+        r = self.client.get(urlreverse("ietf.doc.views_doc.document_ballot", kwargs=dict(name=doc.name)))
+        self.assertEqual(r.status_code, 200)
+        self.assertContains(r, pos2.comment)
+        self.assertContains(r,  '(was %s)' % pos.pos)
+
     def test_document_ballot_needed_positions(self):
         # draft
         doc = IndividualDraftFactory(intended_std_level_id='ps')

ietf/templates/doc/document_ballot_content.html

@@ -74,7 +74,9 @@ <h4><span class="label label-{{ n|pos_to_label }}"> {{ n.name }}</span></h4>
       {% for p in positions %}
         {% if not p.is_old_pos %}
           <h4 class="anchor-target" id="{{ p.balloter.plain_name|slugify }}">{{ p.balloter.plain_name }}
-            <span class="pull-right"><span class="label label-{{ p.pos|pos_to_label }}">{{p.pos}}</span>
+            <span class="pull-right">
+            {% if p.old_positions %}<span class="text-muted small">(was {{ p.old_positions|join:", " }})</span>{% endif %}
+            <span class="label label-{{ p.pos|pos_to_label }}">{{p.pos}}</span>
             {% if user|has_role:"Secretariat" %}
               <a href="{% url "ietf.doc.views_ballot.edit_position" name=doc.name ballot_id=ballot.pk %}?balloter={{ p.balloter.pk }}" title="Click to edit the position of {{ p.balloter.plain_name }}" class="btn btn-default btn-xs">Edit</a>
             {% endif %}