X-Git-Url: https://git.openstreetmap.org./osqa.git/blobdiff_plain/fac2bf37e4b1032efe670d5f63d126ff9faa6f88..981f2713a5caa510f596a089f78579f5ed317bd5:/forum/models/node.py diff --git a/forum/models/node.py b/forum/models/node.py index 6d51793..e8d6334 100644 --- a/forum/models/node.py +++ b/forum/models/node.py @@ -1,8 +1,12 @@ +# -*- coding: utf-8 -*- + from base import * +import logging import re from tag import Tag import markdown +from django.utils.encoding import smart_unicode from django.utils.translation import ugettext as _ from django.utils.safestring import mark_safe from django.utils.html import strip_tags @@ -29,7 +33,7 @@ class NodeContent(models.Model): return auto_user_link(self, self._as_markdown(content, *['auto_linker'])) @classmethod - def _as_markdown_raw(cls, content, *extensions): + def _as_markdown(cls, content, *extensions): try: return mark_safe(sanitize_html(markdown.markdown(content, extensions=extensions))) except Exception, e: @@ -38,13 +42,8 @@ class NodeContent(models.Model): str(e), cls.__name__, str(e), traceback.format_exc())) return '' - # Replace \ with \\ to preserve backslashes during markdown processing - @classmethod - def _as_markdown(cls, content, *extensions): - return cls._as_markdown_raw(content.replace('\\','\\\\'), *extensions) - def as_markdown(self, *extensions): - return self._as_markdown(self.body, *extensions) + return self._as_markdown(smart_unicode(self.body), *extensions) @property def headline(self): @@ -247,7 +246,7 @@ class Node(BaseModel, NodeContent): marked = models.BooleanField(default=False) comment_count = DenormalizedField("children", node_type="comment", canceled=False) - flag_count = DenormalizedField("flags") + flag_count = DenormalizedField("actions", action_type="flag", canceled=False) friendly_name = _("post") @@ -319,16 +318,27 @@ class Node(BaseModel, NodeContent): @property def summary(self): - content = strip_tags(self.html)[:SUMMARY_LENGTH] + content = strip_tags(self.html) # Remove multiple spaces. content = re.sub(' +',' ', content) - # Remove line breaks. We don't need them at all. - content = content.replace("\n", '') + # Replace line breaks with a space, we don't need them at all. + content = content.replace("\n", ' ') + + # Truncate and all an ellipsis if length greater than summary length. + content = (content[:SUMMARY_LENGTH] + '...') if len(content) > SUMMARY_LENGTH else content return content + # Can be used to block subscription notifications for a specific node from a module + def _is_notifiable(self): + return True + + @property + def is_notifiable(self): + return self._is_notifiable() + @models.permalink def get_revisions_url(self): return ('revisions', (), {'id': self.id}) @@ -440,15 +450,13 @@ class Node(BaseModel, NodeContent): if not self.nis.deleted: tag.add_to_usage_count(1) - tag.save() if not self.nis.deleted: for name in tag_changes['removed']: try: tag = Tag.objects.get(name=name) tag.add_to_usage_count(-1) - tag.save() - except: + except Tag.DoesNotExist: pass return True @@ -462,16 +470,13 @@ class Node(BaseModel, NodeContent): if action: for tag in self.tags.all(): tag.add_to_usage_count(-1) - tag.save() else: for tag in Tag.objects.filter(name__in=self.tagname_list()): tag.add_to_usage_count(1) - tag.save() def delete(self, *args, **kwargs): for tag in self.tags.all(): tag.add_to_usage_count(-1) - tag.save() self.active_revision = None self.save()