Ticket #728: show-collection-in-reverse-created-order.patch

mediagoblin/db/models.py

@@ -429 +429 @@
         UniqueConstraint('creator', 'slug'),
         {})
 
-    def get_collection_items(self, ascending=False):
+    def get_collection_items(self, order_col=None, ascending=False):
         #TODO, is this still needed with self.collection_items being available?
-        order_col = CollectionItem.position
+        if order_col is None:
+            order_col = CollectionItem.position
         if not ascending:
             order_col = desc(order_col)
-        return CollectionItem.query.filter_by(
-            collection=self.id).order_by(order_col)
+        return CollectionItem.query.join(MediaEntry).filter(
+            CollectionItem.collection==self.id).order_by(order_col)
 
 
 class CollectionItem(Base, CollectionItemMixin):

mediagoblin/user_pages/views.py

@@ -314 +314 @@
     if not collection:
         return render_404(request)
 
-    cursor = collection.get_collection_items()
+    cursor = collection.get_collection_items(order_col=MediaEntry.created)
 
     pagination = Pagination(page, cursor)
     collection_items = pagination()