1) Restructured the position editing page to better integrate the defer button.

   Fixes bug ietf-tools#659

2) Copied the jquery-1.5.1.min.js library into /static/js/lib (from /static/js)
    because that's where the main document template was looking for it. There have
    probably been several problems since the change that left this dangling that
    showed up as things not behaving, but not as errors. The one that clued me in
    was the discuss text-box not being hidden when the position wasn't DISCUSS.
    They should all behave better now.

3) Added a django form validator to the discuss text field to not allow actions
   when the position is Discuss, but the discuss-text is empty.
   2 and 3 together fix bug ietf-tools#660 

4) Added logic to the undefer fuction to put the document back on the previous
   telechat (if that hasn't already passed). That is, date1 from telechat dates,
   and not date2.
   Fixes bug ietf-tools#661

5) Changed the last_call_announcement generation function's template to include
   the abstracts from the document sets (which with modern usage means the abstract
   of the document) reflecting the IESGs decision of a few months back.
   Fixes bug ietf-tools#662

6) Removed the old tracker link from the document's main page
   Fixes bug ietf-tools#663 

_M   .
M    ietf/idrfc/views_ballot.py
M    ietf/templates/idrfc/last_call_announcement.txt
M    ietf/templates/idrfc/undefer_ballot.html
M    ietf/templates/idrfc/doc_tab_document.html
M    ietf/templates/idrfc/edit_position.html
_M   ietf/utils/draft.py
_M   static/media
A  + static/js/lib/jquery-1.5.1.min.js
 - Legacy-Id: 3107

Author: rjsparks

ietf/idrfc/views_ballot.py

@@ -59,6 +59,13 @@ class EditPositionForm(forms.Form):
     discuss_text = forms.CharField(required=False, widget=forms.Textarea)
     comment_text = forms.CharField(required=False, widget=forms.Textarea)
     return_to_url = forms.CharField(required=False, widget=forms.HiddenInput)
+    def clean_discuss_text(self):
+       entered_discuss = self.cleaned_data["discuss_text"]
+       entered_pos = self.cleaned_data["position"]
+       if entered_pos == "discuss" and not entered_discuss:
+         print "Raising discuss ",entered_pos," ",entered_discuss
+         raise forms.ValidationError("You must enter a non-empty discuss")
+       return entered_discuss
 
 @group_required('Area_Director','Secretariat')
 def edit_position(request, name):
@@ -164,7 +171,13 @@ def edit_position(request, name):
                     qstr += "&ad=%s" % request.GET.get('ad')
                 return HttpResponseRedirect(urlreverse("doc_send_ballot_comment", kwargs=dict(name=doc.filename)) + qstr)
             else:
-                return HttpResponseRedirect(return_to_url)
+              if request.POST.get("Defer"):
+                  return HttpResponseRedirect(urlreverse("doc_defer_ballot", kwargs=dict(name=doc)))
+              else:
+                  if request.POST.get("Undefer"):
+                      return HttpResponseRedirect(urlreverse("doc_undefer_ballot", kwargs=dict(name=doc)))
+                  else:
+                      return HttpResponseRedirect(return_to_url)
     else:
         initial = {}
         if pos:
@@ -307,12 +320,14 @@ def undefer_ballot(request, name):
         raise Http404()
 
     login = IESGLogin.objects.get(login_name=request.user.username)
+    telechat_date = TelechatDates.objects.all()[0].date1
 
     if request.method == 'POST':
         doc.idinternal.ballot.defer = False
         doc.idinternal.ballot.save()
 
         doc.idinternal.change_state(IDState.objects.get(document_state_id=IDState.IESG_EVALUATION), None)
+        doc.idinternal.telechat_date = telechat_date
         doc.idinternal.event_date = date.today()
         doc.idinternal.save()
 
@@ -321,7 +336,7 @@ def undefer_ballot(request, name):
         return HttpResponseRedirect(doc.idinternal.get_absolute_url())
 
     return render_to_response('idrfc/undefer_ballot.html',
-                              dict(doc=doc),
+                              dict(doc=doc,telechat_date=telechat_date),
                               context_instance=RequestContext(request))
 
 class LastCallTextForm(forms.ModelForm):

ietf/templates/idrfc/doc_tab_document.html

@@ -65,16 +65,6 @@
 {% endif %}
 {% endifequal %}
 
-{# links to old system #}
-<span id="doc_edit_button" class="yui-button yui-link-button" style="margin-left:2px;"><span class="first-child">
-{% if doc.in_ietf_process %}
-<a href="https://datatracker.ietf.org/cgi-bin/idtracker.cgi?command=view_id&amp;{% if info.is_rfc %}dTag={{doc.rfc_number}}&amp;rfc_flag=1{% else %}dTag={{doc.tracker_id}}{% endif %}" rel="nofollow" target="_blank">Edit (old IESG Tracker)</a>
-{% else %}
-<a href="https://datatracker.ietf.org/cgi-bin/idtracker.cgi?command=add_id_confirm&amp;{% if info.is_rfc %}dTag={{doc.rfc_number}}&amp;rfc_flag=1{% else %}dTag={{doc.tracker_id}}&amp;rfc_flag=0{% endif %}&amp;ballot_id=0" rel="nofollow" target="_blank">Add (old IESG Tracker)</a>
-{% endif %}
-</span></span>
-{# end links to old #}
-
 </div>
 {% endif %}{# if user in group #}
 {% endblock doc_metabuttons%}

ietf/templates/idrfc/edit_position.html

@@ -31,26 +31,29 @@
 {% block content %}
 <h1>Change position for {{ ad }} on {{ doc }}</h1>
 
-<form class="position-form" action="" method="POST">
-
-<div>
-  <div class="position">{{ form.position }}</div>
-
     {% if ballot.was_deferred %}
-      <div style="margin-top:8px; margin-bottom:8px;"><span id="doc_undefer_ballot_button" class="yui-button yui-link-button"><span class="first-child"><a href="{% url doc_undefer_ballot name=doc %}">Undefer ballot</a></span></span></div>
       <div style="margin-top:8px; margin-bottom:8px;">Ballot deferred by {{ ballot.deferred_by }} on {{ ballot.deferred_date }}.</div>
-    {% else %}
-      <div style="margin-top:8px; margin-bottom:8px;"><span id="doc_defer_ballot_button" class="yui-button yui-link-button"><span class="first-child"><a href="{% url doc_defer_ballot name=doc %}">Defer ballot</a></span></span></div>
     {% endif %}
+
+<form class="position-form" action="" method="POST">
+
+<div>
+  <span class="position">{{ form.position }}</span>
+  <span class="actions">
+    <input type="submit" name="send_mail" value="Save and send email"/>
+    <input type="submit" value="Save"/>
+    {% if ballot.was_deferred %}<input type="submit" name="Undefer" value="Undefer"/>{% else %}<input type="submit" name="Defer" value="Defer"/>{% endif %}
+  </span>
 </div>
 
   <div style="clear:left"></div>
-  
+
   <div class="discuss-widgets">
   <div class="discuss-text">
     {{ form.discuss_text.label_tag }}:
     {% if discuss %}<span class="last-edited">(last edited {{ discuss.date }})</span>{% endif %}
   </div>
+  {{ form.discuss_text.errors }}
   {{ form.discuss_text }}
   </div>
 
@@ -62,12 +65,10 @@ <h1>Change position for {{ ad }} on {{ doc }}</h1>
 
   <div class="actions">
     <a href="{{ return_to_url }}">Back</a>
-    <input type="submit" name="send_mail" value="Save and send email"/>
-    <input type="submit" value="Save"/>
   </div>
-  
-  {{ form.return_to_url }}
 
+   {{ form.return_to_url }}
+  
 </form>
 {% endblock %}
 

ietf/templates/idrfc/last_call_announcement.txt

@@ -11,6 +11,13 @@ The IESG has received a request from {{ group }} to consider the following docum
 
 The IESG plans to make a decision in the next few weeks, and solicits final comments on this action. Please send substantive comments to the ietf@ietf.org mailing lists by {{ expiration_date }}. Exceptionally, comments may be sent to iesg@ietf.org instead. In either case, please retain the beginning of the Subject line to allow automated sorting.{% endfilter %}
 
+Abstract{{ docs|pluralize }}
+
+{% for d in docs %}
+{{ d.abstract}}
+
+{% endfor %}
+
 The file{{ urls|pluralize }} can be obtained via
 {% for u in urls %}{{ u }}
 {% endfor %}{% if impl_report %}

ietf/templates/idrfc/undefer_ballot.html

@@ -8,6 +8,8 @@ <h1>Undefer ballot for {{ doc }}</h1>
 <form action="" method="POST">
   <p>Undefer the ballot for {{ doc.file_tag }}?</p>
 
+  <p>The ballot will then be on the IESG agenda of {{ telechat_date }}.</p>
+
   <div class="actions">
     <a href="{{ doc.idinternal.get_absolute_url }}">Back</a>
     <input type="submit" value="Undefer ballot"/>