From f87e7efbef3f79a96821bdd6307d264567153635 Mon Sep 17 00:00:00 2001
From: olivier <>
Date: Fri, 7 Mar 2008 10:44:01 +0000
Subject: [PATCH] sort related items by id on collection detail

---
 telemeta/models/media.py                     | 3 +++
 telemeta/models/query.py                     | 1 +
 telemeta/templates/collection_detail.html    | 2 +-
 telemeta/templates/collection_detail_dc.html | 2 +-
 telemeta/templates/collection_xspf.xml       | 2 +-
 telemeta/templatetags/telemeta_utils.py      | 6 ------
 telemeta/web/base.py                         | 7 -------
 7 files changed, 7 insertions(+), 16 deletions(-)

diff --git a/telemeta/models/media.py b/telemeta/models/media.py
index 1970bcb1..aa321495 100644
--- a/telemeta/models/media.py
+++ b/telemeta/models/media.py
@@ -152,6 +152,9 @@ class MediaCollection(Model, MediaCore):
           return False
         return True
 
+    def ordered_items(self):
+        return self.items.order_by('id', '_title')
+
     def __str__(self):
         #return self.title
         return self.id
diff --git a/telemeta/models/query.py b/telemeta/models/query.py
index 915b5165..184bdd26 100644
--- a/telemeta/models/query.py
+++ b/telemeta/models/query.py
@@ -140,6 +140,7 @@ class MediaItemQuerySet(CoreQuerySet):
             where = ["collection_id NOT IN (SELECT id FROM telemeta_collection)"]);
 
     def by_recording_date(self, pattern):
+        "Find items by recording date"
         return self.filter(Q(dates_enregistr__icontains=pattern) 
             | Q(annee_enreg__icontains=pattern))
 
diff --git a/telemeta/templates/collection_detail.html b/telemeta/templates/collection_detail.html
index 36957ad5..2b5d2253 100644
--- a/telemeta/templates/collection_detail.html
+++ b/telemeta/templates/collection_detail.html
@@ -62,7 +62,7 @@
 
     <hr>
     <h4>Items</h4>
-    {% with object.items.all as items %}
+    {% with object.ordered_items as items %}
         {% include "inc/mediaitem_list.html" %}
     {% endwith %}
 {% else %}
diff --git a/telemeta/templates/collection_detail_dc.html b/telemeta/templates/collection_detail_dc.html
index a13db9dd..82f5b9d4 100644
--- a/telemeta/templates/collection_detail_dc.html
+++ b/telemeta/templates/collection_detail_dc.html
@@ -22,7 +22,7 @@
             <td>{{ element.value }}</td></tr>
             {% endif %}
         {% endfor %}
-        {% for item in object.items.all %}
+        {% for item in object.ordered_items %}
             <tr><th>relation</th><td>hasPart</td><td>
             <a href="{% url telemeta-item-dublincore item.id|urlencode %}">
                 {{ item.id }}</a>
diff --git a/telemeta/templates/collection_xspf.xml b/telemeta/templates/collection_xspf.xml
index 41a0c190..acc8a3cb 100644
--- a/telemeta/templates/collection_xspf.xml
+++ b/telemeta/templates/collection_xspf.xml
@@ -10,7 +10,7 @@
 {% endblock %}
 
 {% block tracklist %}
-{% for item in collection.items.all %}
+{% for item in collection.ordered_items %}
     <track>
       <title>{{ item.title }}</title>
       <meta rel="type">mp3</meta>
diff --git a/telemeta/templatetags/telemeta_utils.py b/telemeta/templatetags/telemeta_utils.py
index f20186b9..95a4a78d 100644
--- a/telemeta/templatetags/telemeta_utils.py
+++ b/telemeta/templatetags/telemeta_utils.py
@@ -2,12 +2,6 @@ from django import template
 
 register = template.Library()
 
-
-#@register.filter
-#def cleanid(value):
-#    "Escapes a value for use in a URL (converts slashes)"
-#    return value.replace('/', '--')
-
 @register.filter
 def tolist(dict):
     "Converts a simple dict into a list"
diff --git a/telemeta/web/base.py b/telemeta/web/base.py
index eb358d82..bef9c344 100644
--- a/telemeta/web/base.py
+++ b/telemeta/web/base.py
@@ -165,13 +165,6 @@ class WebView(Component):
     def __get_admin_context_vars(self):
         return {"enumerations": self.__get_enumerations_list()}
 
-    def __continents_to_countries(self, tree):
-        countries = []
-        for continent in tree:
-            countries += continent['countries']
-        #countries.sort()
-        return countries
-
     def admin_index(self, request):
         return render_to_response('admin.html', self. __get_admin_context_vars())
 
-- 
2.39.5