From 18b8a795adaf97c9d58320a96051e7c4cea5b668 Mon Sep 17 00:00:00 2001
From: Robert Sparks <rjsparks@nostrum.com>
Date: Thu, 12 Dec 2019 17:17:51 +0000
Subject: [PATCH] Show diff controls for a bis -00 document. Fixes #2846.
 Commit ready for merge.  - Legacy-Id: 17147

---
 ietf/doc/tests.py                        | 15 ++++++++++++++-
 ietf/templates/doc/document_history.html |  6 +++---
 2 files changed, 17 insertions(+), 4 deletions(-)

diff --git a/ietf/doc/tests.py b/ietf/doc/tests.py
index 08dea2e9b..3c0562e9e 100644
--- a/ietf/doc/tests.py
+++ b/ietf/doc/tests.py
@@ -855,7 +855,20 @@ class DocTestCase(TestCase):
         r = self.client.get(url)
         self.assertEqual(r.status_code, 200)
         self.assertContains(r, e.desc)
-        
+
+    def test_history_bis_00(self):
+        rfcname='rfc9090'
+        rfc = WgRfcFactory(alias2=rfcname)
+        bis_draft = WgDraftFactory(name='draft-ietf-{}-{}bis'.format(rfc.group.acronym,rfcname))
+
+        url = urlreverse('ietf.doc.views_doc.document_history', kwargs=dict(name=bis_draft.name))
+        r = self.client.get(url)
+        self.assertEqual(r.status_code, 200) 
+        q = PyQuery(unicontent(r))
+        attr1='value="{}"'.format(rfcname)
+        self.assertEqual(len(q('option['+attr1+'][selected="selected"]')), 1)
+
+
     def test_document_feed(self):
         doc = IndividualDraftFactory()
 
diff --git a/ietf/templates/doc/document_history.html b/ietf/templates/doc/document_history.html
index 02c01dded..42c6c0538 100644
--- a/ietf/templates/doc/document_history.html
+++ b/ietf/templates/doc/document_history.html
@@ -19,7 +19,7 @@
   {% origin %}
   {{ top|safe }}
 
-  {% if diff_revisions and diff_revisions|length > 1 %}
+  {% if diff_revisions and diff_revisions|length > 1 or doc.name|rfcbis %}
     <h2>Revision differences</h2>
 
     <form class="form-horizontal diff-form" action="{{rfcdiff_base_url}}" method="get" target="_blank">
@@ -31,13 +31,13 @@
         <div class="col-sm-10">
           <select class="form-control" id="url1" name="url1">
 	    {% for name, rev, time, url in diff_revisions %}
-	      <option value="{{ url }}" {% if forloop.counter == 2 %} selected="selected" {% endif %}>
+	      <option value="{{ url }}" {% if diff_revisions|length > 1 and forloop.counter == 2 %} selected="selected" {% endif %}>
 	        {{ name }}{% if rev %}-{{ rev }}{% endif %}
 	        ({{ time|date:"Y-m-d" }})
 	      </option>
 	    {% endfor %}
 	    {% if doc.name|rfcbis %}
-	      <option value="{{ doc.name|rfcbis }}">
+	      <option value="{{ doc.name|rfcbis }}"{% if diff_revisions and diff_revisions|length == 1 %} selected="selected"{% endif %}>
 	        {{ doc.name|rfcbis }}
 	      </option>
 	    {% endif %}