Cannot redefine admin ordering dynamically

[Manuel Saelices]

Sometimes you need redefine ordering dinamically, based in some logic at runtime

A example of that needed was this code:

class MyBaseAdmin(admin.ModelAdmin):

   def get_ordering(self, request):
      # I'd like i.e. a different order based on user or whatever
      if request.user.is_superuser():
         ...
      return super(MyBaseAdmin, self).get_ordering(request)

[Manuel Saelices]

Patch with tests

[Manuel Saelices]

The patch is not working in last Django trunk [16339]. I will work on it.

[Manuel Saelices]

A new patch with tests for the [16341] version

[Jannis Leidel]

Looks good, except that the default get_ordering method should return self.ordering or () instead of checking all places.

[Jannis Leidel]

In [16383]:

Fixed #12875 -- Added get_ordering to ModelAdmin. Many thanks to Manuel Saelices.