Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

[1.0.X] Fixed #9053 -- Allowed for sorting of callable and ModelAdmin…

… methods specified in list_display (added in r8352). Previously attempting to sort on the former would raise an exception and the latter simply didn't sort. Also added tests for this function. Thanks rgl and jenan.

Backport of [9211], also updated svnmerge.py metatdata.


git-svn-id: http://code.djangoproject.com/svn/django/branches/releases/1.0.X@9212 bcc190cf-cafb-0310-a4f2-bffc1f526a37
  • Loading branch information...
commit 9810648befa7c4ebe0fbebd9e8965930647fce37 1 parent aea4265
@kmtracey kmtracey authored
View
7 django/contrib/admin/views/main.py
@@ -157,7 +157,12 @@ def get_ordering(self):
# See whether field_name is a name of a non-field
# that allows sorting.
try:
- attr = getattr(self.model, field_name)
+ if callable(field_name):
+ attr = field_name
+ elif hasattr(self.model_admin, field_name):
+ attr = getattr(self.model_admin, field_name)
+ else:
+ attr = getattr(self.model, field_name)
order_field = attr.admin_order_field
except AttributeError:
pass
View
14 tests/regressiontests/admin_views/fixtures/admin-views-users.xml
@@ -74,8 +74,20 @@
<field type="CharField" name="name">Test section</field>
</object>
<object pk="1" model="admin_views.article">
- <field type="TextField" name="content">&lt;p&gt;test content&lt;/p&gt;</field>
+ <field type="TextField" name="content">&lt;p&gt;Middle content&lt;/p&gt;</field>
<field type="DateTimeField" name="date">2008-03-18 11:54:58</field>
<field to="admin_views.section" name="section" rel="ManyToOneRel">1</field>
</object>
+ <object pk="2" model="admin_views.article">
+ <field type="TextField" name="content">&lt;p&gt;Oldest content&lt;/p&gt;</field>
+ <field type="DateTimeField" name="date">2000-03-18 11:54:58</field>
+ <field to="admin_views.section" name="section" rel="ManyToOneRel">1</field>
+ </object>
+ <object pk="3" model="admin_views.article">
+ <field type="TextField" name="content">&lt;p&gt;Newest content&lt;/p&gt;</field>
+ <field type="DateTimeField" name="date">2009-03-18 11:54:58</field>
+ <field to="admin_views.section" name="section" rel="ManyToOneRel">1</field>
+ </object>
+
+
</django-objects>
View
14 tests/regressiontests/admin_views/models.py
@@ -19,12 +19,20 @@ class Article(models.Model):
def __unicode__(self):
return self.title
+
+ def model_year(self):
+ return self.date.year
+ model_year.admin_order_field = 'date'
+
+def callable_year(dt_value):
+ return dt_value.year
+callable_year.admin_order_field = 'date'
class ArticleInline(admin.TabularInline):
model = Article
class ArticleAdmin(admin.ModelAdmin):
- list_display = ('content', 'date')
+ list_display = ('content', 'date', callable_year, 'model_year', 'modeladmin_year')
list_filter = ('date',)
def changelist_view(self, request):
@@ -34,6 +42,10 @@ def changelist_view(self, request):
'extra_var': 'Hello!'
}
)
+
+ def modeladmin_year(self, obj):
+ return obj.date.year
+ modeladmin_year.admin_order_field = 'date'
class CustomArticle(models.Model):
content = models.TextField()
View
85 tests/regressiontests/admin_views/tests.py
@@ -71,34 +71,83 @@ def testBasicEditPost(self):
post_data = {
"name": u"Test section",
# inline data
- "article_set-TOTAL_FORMS": u"4",
- "article_set-INITIAL_FORMS": u"1",
+ "article_set-TOTAL_FORMS": u"6",
+ "article_set-INITIAL_FORMS": u"3",
"article_set-0-id": u"1",
# there is no title in database, give one here or formset
# will fail.
"article_set-0-title": u"Need a title.",
- "article_set-0-content": u"&lt;p&gt;test content&lt;/p&gt;",
+ "article_set-0-content": u"&lt;p&gt;Middle content&lt;/p&gt;",
"article_set-0-date_0": u"2008-03-18",
"article_set-0-date_1": u"11:54:58",
- "article_set-1-id": u"",
- "article_set-1-title": u"",
- "article_set-1-content": u"",
- "article_set-1-date_0": u"",
- "article_set-1-date_1": u"",
- "article_set-2-id": u"",
- "article_set-2-title": u"",
- "article_set-2-content": u"",
- "article_set-2-date_0": u"",
- "article_set-2-date_1": u"",
+ "article_set-1-id": u"2",
+ "article_set-1-title": u"Need a title.",
+ "article_set-1-content": u"&lt;p&gt;Oldest content&lt;/p&gt;",
+ "article_set-1-date_0": u"2000-03-18",
+ "article_set-1-date_1": u"11:54:58",
+ "article_set-2-id": u"3",
+ "article_set-2-title": u"Need a title.",
+ "article_set-2-content": u"&lt;p&gt;Newest content&lt;/p&gt;",
+ "article_set-2-date_0": u"2009-03-18",
+ "article_set-2-date_1": u"11:54:58",
"article_set-3-id": u"",
"article_set-3-title": u"",
"article_set-3-content": u"",
"article_set-3-date_0": u"",
"article_set-3-date_1": u"",
+ "article_set-4-id": u"",
+ "article_set-4-title": u"",
+ "article_set-4-content": u"",
+ "article_set-4-date_0": u"",
+ "article_set-4-date_1": u"",
+ "article_set-5-id": u"",
+ "article_set-5-title": u"",
+ "article_set-5-content": u"",
+ "article_set-5-date_0": u"",
+ "article_set-5-date_1": u"",
}
response = self.client.post('/test_admin/admin/admin_views/section/1/', post_data)
self.failUnlessEqual(response.status_code, 302) # redirect somewhere
+ def testChangeListSortingCallable(self):
+ """
+ Ensure we can sort on a list_display field that is a callable
+ (column 2 is callable_year in ArticleAdmin)
+ """
+ response = self.client.get('/test_admin/admin/admin_views/article/', {'ot': 'asc', 'o': 2})
+ self.failUnlessEqual(response.status_code, 200)
+ self.failUnless(
+ response.content.index('Oldest content') < response.content.index('Middle content') and
+ response.content.index('Middle content') < response.content.index('Newest content'),
+ "Results of sorting on callable are out of order."
+ )
+
+ def testChangeListSortingModel(self):
+ """
+ Ensure we can sort on a list_display field that is a Model method
+ (colunn 3 is 'model_year' in ArticleAdmin)
+ """
+ response = self.client.get('/test_admin/admin/admin_views/article/', {'ot': 'dsc', 'o': 3})
+ self.failUnlessEqual(response.status_code, 200)
+ self.failUnless(
+ response.content.index('Newest content') < response.content.index('Middle content') and
+ response.content.index('Middle content') < response.content.index('Oldest content'),
+ "Results of sorting on Model method are out of order."
+ )
+
+ def testChangeListSortingModelAdmin(self):
+ """
+ Ensure we can sort on a list_display field that is a ModelAdmin method
+ (colunn 4 is 'modeladmin_year' in ArticleAdmin)
+ """
+ response = self.client.get('/test_admin/admin/admin_views/article/', {'ot': 'asc', 'o': 4})
+ self.failUnlessEqual(response.status_code, 200)
+ self.failUnless(
+ response.content.index('Oldest content') < response.content.index('Middle content') and
+ response.content.index('Middle content') < response.content.index('Newest content'),
+ "Results of sorting on ModelAdmin method are out of order."
+ )
+
def get_perm(Model, perm):
"""Return the permission object, for the Model"""
ct = ContentType.objects.get_for_model(Model)
@@ -252,7 +301,7 @@ def testAddView(self):
# Try POST just to make sure
post = self.client.post('/test_admin/admin/admin_views/article/add/', add_dict)
self.failUnlessEqual(post.status_code, 403)
- self.failUnlessEqual(Article.objects.all().count(), 1)
+ self.failUnlessEqual(Article.objects.all().count(), 3)
self.client.get('/test_admin/admin/logout/')
# Add user may login and POST to add view, then redirect to admin root
@@ -260,7 +309,7 @@ def testAddView(self):
self.client.post('/test_admin/admin/', self.adduser_login)
post = self.client.post('/test_admin/admin/admin_views/article/add/', add_dict)
self.assertRedirects(post, '/test_admin/admin/')
- self.failUnlessEqual(Article.objects.all().count(), 2)
+ self.failUnlessEqual(Article.objects.all().count(), 4)
self.client.get('/test_admin/admin/logout/')
# Super can add too, but is redirected to the change list view
@@ -268,7 +317,7 @@ def testAddView(self):
self.client.post('/test_admin/admin/', self.super_login)
post = self.client.post('/test_admin/admin/admin_views/article/add/', add_dict)
self.assertRedirects(post, '/test_admin/admin/admin_views/article/')
- self.failUnlessEqual(Article.objects.all().count(), 3)
+ self.failUnlessEqual(Article.objects.all().count(), 5)
self.client.get('/test_admin/admin/logout/')
# 8509 - if a normal user is already logged in, it is possible
@@ -392,7 +441,7 @@ def testDeleteView(self):
self.failUnlessEqual(request.status_code, 403)
post = self.client.post('/test_admin/admin/admin_views/article/1/delete/', delete_dict)
self.failUnlessEqual(post.status_code, 403)
- self.failUnlessEqual(Article.objects.all().count(), 1)
+ self.failUnlessEqual(Article.objects.all().count(), 3)
self.client.get('/test_admin/admin/logout/')
# Delete user can delete
@@ -406,7 +455,7 @@ def testDeleteView(self):
self.failUnlessEqual(response.status_code, 200)
post = self.client.post('/test_admin/admin/admin_views/article/1/delete/', delete_dict)
self.assertRedirects(post, '/test_admin/admin/')
- self.failUnlessEqual(Article.objects.all().count(), 0)
+ self.failUnlessEqual(Article.objects.all().count(), 2)
self.client.get('/test_admin/admin/logout/')
class AdminViewStringPrimaryKeyTest(TestCase):
Please sign in to comment.
Something went wrong with that request. Please try again.