diff options
author | Étienne Loks <etienne.loks@peacefrogs.net> | 2013-02-19 11:08:10 +0100 |
---|---|---|
committer | Étienne Loks <etienne.loks@peacefrogs.net> | 2013-02-19 11:08:10 +0100 |
commit | dce6d0499efe505096495b36728eeb8c9fb8ced1 (patch) | |
tree | c14401c54908114c3a807172f2e449c9719a3055 | |
parent | 7db8e005f4204ab5dabf117b0bf91a12c70fd74c (diff) | |
download | Chimère-dce6d0499efe505096495b36728eeb8c9fb8ced1.tar.bz2 Chimère-dce6d0499efe505096495b36728eeb8c9fb8ced1.zip |
Be tolerent to markers with not category associated
-rw-r--r-- | chimere/models.py | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/chimere/models.py b/chimere/models.py index c3d4733..0fc22ee 100644 --- a/chimere/models.py +++ b/chimere/models.py @@ -521,11 +521,14 @@ class Marker(GeographicItem): def default_category(self): # Should we select only available ones ? # Should we catch if not exists ? - return self.categories.all()[0] + cats = self.categories + if cats.count(): + return cats.all()[0] def get_absolute_url(self, area_name=''): - parameters = 'current_feature=%d&checked_categories=%s' % (self.id, - self.default_category.pk) + parameters = 'current_feature=%d' % self.id + if self.default_category: + parameters += '&checked_categories=%s' % self.default_category.pk urn = TinyUrl.getUrnByParameters(parameters) area_name = area_name + '/' if area_name else '' url = reverse('chimere:tiny', args=[area_name, urn]) |