Create a redirect for the document-ID numeric URL to the filename one.

 - Legacy-Id: 814

Author: fenner

ietf/idtracker/testurls.list

@@ -8,7 +8,7 @@
 200		/idtracker/status/
 skipdiff,200	/idtracker/status/last-call/	https://datatracker.ietf.org/public/lastcall.cgi
 200		/idtracker/rfc3847/		https://datatracker.ietf.org/public/pidtracker.cgi?command=view_id&dTag=3847&rfc_flag=1
-200		/idtracker/12689/		https://datatracker.ietf.org/public/pidtracker.cgi?command=view_id&dTag=12689&rfc_flag=0
+301,skipdiff	/idtracker/12689/		https://datatracker.ietf.org/public/pidtracker.cgi?command=view_id&dTag=12689&rfc_flag=0
 skipredirect,200 /idtracker/draft-ietf-isis-link-attr/	https://datatracker.ietf.org/public/pidtracker.cgi?command=view_id&dTag=12689&rfc_flag=0
 skipredirect,200 /idtracker/draft-ietf-isis-link-attr/comment/65232/	https://datatracker.ietf.org/public/pidtracker.cgi?command=view_comment&id=65232
 200		/idtracker/comment/65232/	https://datatracker.ietf.org/public/pidtracker.cgi?command=view_comment&id=65232

ietf/idtracker/urls.py

@@ -30,9 +30,9 @@
 )
 urlpatterns += patterns('django.views.generic.list_detail',
      (r'^rfc(?P<object_id>\d+)/$', 'object_detail', rfc_dict),
-     (r'^(?P<object_id>\d+)/$', 'object_detail', id_dict),
 )
 urlpatterns += patterns('',
+     (r'^(?P<object_id>\d+)/$', views.redirect_id),
      (r'^(?P<slug>[^/]+)/$', views.view_id, dict(id_dict, slug_field='draft__filename')),
      (r'^comment/(?P<object_id>\d+)/$', views.view_comment, comment_dict),
      (r'^ballot/(?P<object_id>\d+)/$', views.view_ballot, ballot_dict),

ietf/idtracker/views.py

@@ -1,11 +1,12 @@
 # Copyright The IETF Trust 2007, All Rights Reserved
 
 # Create your views here.
-from django.http import HttpResponseRedirect
+from django.http import HttpResponseRedirect, HttpResponsePermanentRedirect
 from django import newforms as forms
 from django.template import RequestContext
 from django.shortcuts import get_object_or_404, render_to_response
 from django.db.models import Q
+from django.core.urlresolvers import reverse
 from django.views.generic.list_detail import object_detail, object_list
 from ietf.idtracker.models import InternetDraft, IDInternal, IDState, IDSubState, Rfc, DocumentWrapper
 from ietf.idtracker.forms import IDSearch, EmailFeedback
@@ -187,6 +188,11 @@ def last_call(request):
     queryset = IDInternal.objects.filter(primary_flag=1).filter(cur_state__state__in=('In Last Call', 'Waiting for Writeup', 'Waiting for AD Go-Ahead')).order_by('cur_state', 'status_date', 'ballot_id')
     return object_list(request, template_name="idtracker/status_of_items.html", queryset=queryset, extra_context={'title': 'Documents in Last Call'})
 
+def redirect_id(request, object_id):
+    '''Redirect from historical document ID to preferred filename url.'''
+    doc = get_object_or_404(InternetDraft, id_document_tag=object_id)
+    return HttpResponsePermanentRedirect(reverse(view_id, args=[doc.filename]))
+
 # Wrappers around object_detail to give permalink a handle.
 # The named-URLs feature in django 0.97 will eliminate the
 # need for these.