Briq
/
django-admin-sortable-fixwin
6
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Eliminated setting of self.order to 1 if Max returns None, as order has a default value of 1 defined in the field.

master
Brandon Taylor 2011-11-16 20:50:12 -06:00
parent 7a242dd41c
commit fba568399c
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
adminsortable/models.py
View File

@ -36,9 +36,8 @@ class Sortable(models.Model):
def save(self, *args, **kwargs): def save(self, *args, **kwargs):
if not self.id: if not self.id:
try: try:
max_order = self.__class__.objects.aggregate(models.Max('order'))['order__max'] + 1 self.order = self.__class__.objects.aggregate(models.Max('order'))['order__max'] + 1
except TypeError, IndexError: except TypeError, IndexError:
max_order = 1 pass
self.order = max_order
super(Sortable, self).save(*args, **kwargs) super(Sortable, self).save(*args, **kwargs)