From 29cce749fa9f6c044475f5b21e9b13e8f14ff74a Mon Sep 17 00:00:00 2001
From: Robert Wittams <robert@wittams.com>
Date: Tue, 8 Nov 2005 23:31:47 +0000
Subject: [PATCH] Templatised date_hierarchy.

git-svn-id: http://code.djangoproject.com/svn/django/branches/new-admin@1129 bcc190cf-cafb-0310-a4f2-bffc1f526a37
---
 .../admin/templates/admin/date_hierarchy.html |  8 +++
 .../contrib/admin/templatetags/admin_list.py  | 62 ++++++++++++-------
 django/contrib/admin/views/main.py            |  2 -
 3 files changed, 47 insertions(+), 25 deletions(-)
 create mode 100644 django/contrib/admin/templates/admin/date_hierarchy.html

diff --git a/django/contrib/admin/templates/admin/date_hierarchy.html b/django/contrib/admin/templates/admin/date_hierarchy.html
new file mode 100644
index 0000000000..7662ed18f9
--- /dev/null
+++ b/django/contrib/admin/templates/admin/date_hierarchy.html
@@ -0,0 +1,8 @@
+<div class="xfull">
+<ul class="toplinks">
+{% if back %}<li class="date-back"><a href="{{back.link}}">&lsaquo; {{back.title}}</a></li>{% endif %}
+{% for choice in choices %}
+<li> {% if choice.link%}<a href="{{choice.link}}">{%endif%}{{choice.title}}{%if choice.link%}</a>{%endif%}</li>
+{% endfor %}
+</ul><br class="clear" />
+</div>
diff --git a/django/contrib/admin/templatetags/admin_list.py b/django/contrib/admin/templatetags/admin_list.py
index 3f155564c0..ba9c7ec340 100644
--- a/django/contrib/admin/templatetags/admin_list.py
+++ b/django/contrib/admin/templatetags/admin_list.py
@@ -222,12 +222,11 @@ def result_list(cl):
 result_list = inclusion_tag("admin/change_list_results")(result_list)
 
 
-#@simple_tag
+#@inclusion_tag("admin/date_hierarchy")
 def date_hierarchy(cl):
     lookup_opts, params, lookup_params, lookup_mod = \
       cl.lookup_opts, cl.params, cl.lookup_params, cl.lookup_mod
     
-    raw_template = []
     if lookup_opts.admin.date_hierarchy:
         field_name = lookup_opts.admin.date_hierarchy
     
@@ -238,35 +237,52 @@ def date_hierarchy(cl):
         year_lookup = params.get(year_field)
         month_lookup = params.get(month_field)
         day_lookup = params.get(day_field)
-    
-        raw_template.append('<div class="xfull">\n<ul class="toplinks">\n')
+     
+        def link(d): 
+            return cl.get_query_string(d, [field_generic])
+     
+        def get_dates(unit, params):
+            return getattr(lookup_mod, 'get_%s_list' % field_name)(unit, **params)
+     
         if year_lookup and month_lookup and day_lookup:
-            raw_template.append('<li class="date-back"><a href="%s">&lsaquo; %s %s </a></li>' % \
-                (cl.get_query_string( {year_field: year_lookup, month_field: month_lookup}, [field_generic]), MONTHS[int(month_lookup)], year_lookup))
-            raw_template.append('<li>%s %s</li>' % (MONTHS[int(month_lookup)], day_lookup))
+            month_name = MONTHS[int(month_lookup)]
+            return { 'back': 
+                        { 'link' : link({year_field: year_lookup, month_field: month_lookup}), 
+                          'title': "%s %s" % ( month_name, year_lookup),
+                        },
+                      'choices': [ {'title': "%s %s" % ( month_name, day_lookup)} ]
+                  }
         elif year_lookup and month_lookup:
-            raw_template.append('<li class="date-back"><a href="%s">&lsaquo; %s</a></li>' % \
-                (cl.get_query_string( {year_field: year_lookup}, [field_generic]), year_lookup))
             date_lookup_params = lookup_params.copy()
             date_lookup_params.update({year_field: year_lookup, month_field: month_lookup})
-            for day in getattr(lookup_mod, 'get_%s_list' % field_name)('day', **date_lookup_params):
-                raw_template.append('<li><a href="%s">%s</a></li>' % \
-                    (cl.get_query_string({year_field: year_lookup, month_field: month_lookup, day_field: day.day}, [field_generic]), day.strftime('%B %d')))
+            days = get_dates('day', date_lookup_params)
+            return { 'back': 
+                        { 'link' : link({year_field: year_lookup}), 
+                          'title' : year_lookup 
+                        },
+                    'choices': 
+                        [ { 'link' : link({year_field: year_lookup, month_field: month_lookup, day_field: day.day}), 
+                            'title': day.strftime('%B %d') } for day in days ]
+                    }
         elif year_lookup:
-            raw_template.append('<li class="date-back"><a href="%s">&lsaquo; %s</a></li>' % \
-                cl.get_query_string( {}, [year_field]), _('All dates'))
             date_lookup_params = lookup_params.copy()
             date_lookup_params.update({year_field: year_lookup})
-            for month in getattr(lookup_mod, 'get_%s_list' % field_name)('month', **date_lookup_params):
-                raw_template.append('<li><a href="%s">%s %s</a></li>' % \
-                    (cl.get_query_string( {year_field: year_lookup, month_field: month.month}, [field_generic]), month.strftime('%B'), month.year))
+            months = get_dates('month', date_lookup_params)
+            return { 'back':
+                       { 'link' : link({}),
+                         'title': _('All dates')
+                       },
+                      'choices':
+                      [ { 'link': link( {year_field: year_lookup, month_field: month.month}), 
+                          'title': "%s %s" % (month.strftime('%B') ,  month.year) } for month in months ]
+                  }
         else:
-            for year in getattr(lookup_mod, 'get_%s_list' % field_name)('year', **lookup_params):
-                raw_template.append('<li><a href="%s">%s</a></li>\n' % \
-                    (cl.get_query_string( {year_field: year.year}, [field_generic]), year.year))
-        raw_template.append('</ul><br class="clear" />\n</div>\n')
-    return ''.join(raw_template)
-date_hierarchy = simple_tag(date_hierarchy)
+            years = get_dates('year', lookup_params)
+            return { 'choices':
+                        [ { 'link': link( {year_field: year.year}),
+                            'title': year.year  } for year in years ]
+                   }
+date_hierarchy = inclusion_tag('admin/date_hierarchy')(date_hierarchy)
 
 #@inclusion_tag('admin/search_form')
 def search_form(cl):
diff --git a/django/contrib/admin/views/main.py b/django/contrib/admin/views/main.py
index 7f0c5c5906..99be09578b 100644
--- a/django/contrib/admin/views/main.py
+++ b/django/contrib/admin/views/main.py
@@ -120,7 +120,6 @@ class RelatedFilterSpec(FilterSpec):
 FilterSpec.register(lambda f: bool(f.rel), RelatedFilterSpec)
 
 class ChoicesFilterSpec(FilterSpec):
-   
     def __init__(self, f, request, params):
         super(ChoicesFilterSpec, self).__init__(f, request, params)
         self.lookup_kwarg = '%s__exact' % f.name
@@ -173,7 +172,6 @@ class DateFieldFilterSpec(FilterSpec):
 FilterSpec.register(lambda f: isinstance(f, meta.DateField), DateFieldFilterSpec)
 
 class BooleanFieldFilterSpec(FilterSpec):
-    
     def __init__(self, f, request, params):
         super(BooleanFieldFilterSpec, self).__init__(f, request, params)
         self.lookup_kwarg = '%s__exact' % f.name