diff --git a/bookmarks/queries.py b/bookmarks/queries.py index 6235e2f..112f8df 100644 --- a/bookmarks/queries.py +++ b/bookmarks/queries.py @@ -87,13 +87,7 @@ def _base_bookmarks_query( elif search.shared == BookmarkSearch.FILTER_SHARED_UNSHARED: query_set = query_set.filter(shared=False) - # Sort by date added - if search.sort == BookmarkSearch.SORT_ADDED_ASC: - query_set = query_set.order_by("date_added") - elif search.sort == BookmarkSearch.SORT_ADDED_DESC: - query_set = query_set.order_by("-date_added") - - # Sort by title + # Sort if ( search.sort == BookmarkSearch.SORT_TITLE_ASC or search.sort == BookmarkSearch.SORT_TITLE_DESC @@ -124,6 +118,11 @@ def _base_bookmarks_query( query_set = query_set.order_by(order_field) elif search.sort == BookmarkSearch.SORT_TITLE_DESC: query_set = query_set.order_by(order_field).reverse() + elif search.sort == BookmarkSearch.SORT_ADDED_ASC: + query_set = query_set.order_by("date_added") + else: + # Sort by date added, descending by default + query_set = query_set.order_by("-date_added") return query_set diff --git a/bookmarks/tests/test_bookmarks_list_template.py b/bookmarks/tests/test_bookmarks_list_template.py index 4cc6bdc..8115bcc 100644 --- a/bookmarks/tests/test_bookmarks_list_template.py +++ b/bookmarks/tests/test_bookmarks_list_template.py @@ -210,7 +210,7 @@ class BookmarkListTemplateTest(TestCase, BookmarkFactoryMixin, HtmlTestMixin): def assertNotesToggle(self, html: str, count=1): self.assertInHTML( - f""" + """