move URLItem save() override to just get_absolute_url(), since we won't have an item_id at save() time when adding
authorTero Marttila <terom@fixme.fi>
Thu, 15 Aug 2013 00:53:34 +0300
changeset 35 acc1a366fa7c
parent 34 4ebfe940acd1
child 36 95bb1d7463cc
move URLItem save() override to just get_absolute_url(), since we won't have an item_id at save() time when adding
qrurls/models.py
--- a/qrurls/models.py	Thu Aug 15 00:52:34 2013 +0300
+++ b/qrurls/models.py	Thu Aug 15 00:53:34 2013 +0300
@@ -92,7 +92,10 @@
         ordering = ['published']
 
     def get_absolute_url (self) :
-        return self.url
+        if self.image :
+            return reverse('shorturl_image', kwargs=dict(shorturl=self.shorturl, item_id=self.id))
+        else :
+            return self.url
     
     def published_age (self) :
         now = timezone.now()
@@ -127,9 +130,3 @@
                 url=self.get_absolute_url(),
         )
 
-    def save (self, *args, **kwargs) :
-        if self.image :
-            self.url = reverse('shorturl_image', kwargs=dict(shorturl=self.shorturl, item_id=self.id))
-
-        super(URLItem, self).save(*args, **kwargs)
-