From fba568399c3d11d71569ace58b7fd56aa0eb285d Mon Sep 17 00:00:00 2001 From: Brandon Taylor Date: Wed, 16 Nov 2011 20:50:12 -0600 Subject: [PATCH] Eliminated setting of self.order to 1 if Max returns None, as order has a default value of 1 defined in the field. --- adminsortable/models.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/adminsortable/models.py b/adminsortable/models.py index f3d58e8..0d74262 100755 --- a/adminsortable/models.py +++ b/adminsortable/models.py @@ -36,9 +36,8 @@ class Sortable(models.Model): def save(self, *args, **kwargs): if not self.id: 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: - max_order = 1 - self.order = max_order + pass super(Sortable, self).save(*args, **kwargs)