Fixes the new line problems in the OSQA exporter.

[osqa.git] / forum / models / node.py

diff --git a/forum/models/node.py b/forum/models/node.py
index f62a8e078b9c2fc17b8a80d6a39cc9617df6c9ff..43d12b323edb5d643f87ca1f790e5d236ecb92da 100644 (file)
--- a/forum/models/node.py
+++ b/forum/models/node.py
@@ -6,6 +6,7 @@ import markdown
 from django.utils.translation import ugettext as _
 from django.utils.safestring import mark_safe
 from django.utils.html import strip_tags
 from django.utils.translation import ugettext as _
 from django.utils.safestring import mark_safe
 from django.utils.html import strip_tags

+from forum.utils.userlinking import auto_user_link

 from forum.utils.html import sanitize_html
 from utils import PickledObjectField
 
 from forum.utils.html import sanitize_html
 from utils import PickledObjectField
 


@@ -21,7 +22,7 @@ class NodeContent(models.Model):
 
     @property
     def html(self):
 
     @property
     def html(self):

-        return self.as_markdown()
+        return self.body

 
     @classmethod
     def _as_markdown(cls, content, *extensions):
 
     @classmethod
     def _as_markdown(cls, content, *extensions):


@@ -78,13 +79,10 @@ class NodeMetaClass(BaseMetaClass):
             return node_cls.objects.filter(parent=self)
 
         def parent(self):
             return node_cls.objects.filter(parent=self)
 
         def parent(self):

-            p = self.__dict__.get('_%s_cache' % name, None)
+            if (self.parent is not None) and self.parent.node_type == name:
+                return self.parent.leaf

 
 

-            if p is None and (self.parent is not None) and self.parent.node_type == name:
-                p = self.parent.leaf
-                self.__dict__['_%s_cache' % name] = p
-
-            return p
+            return None

 
         Node.add_to_class(name + 's', property(children))
         Node.add_to_class(name, property(parent))
 
         Node.add_to_class(name + 's', property(children))
         Node.add_to_class(name, property(parent))


@@ -247,7 +245,7 @@ class Node(BaseModel, NodeContent):
     @classmethod
     def _generate_cache_key(cls, key, group="node"):
         return super(Node, cls)._generate_cache_key(key, group)
     @classmethod
     def _generate_cache_key(cls, key, group="node"):
         return super(Node, cls)._generate_cache_key(key, group)

-
+        

     @classmethod
     def get_type(cls):
         return cls.__name__.lower()
     @classmethod
     def get_type(cls):
         return cls.__name__.lower()


@@ -283,6 +281,13 @@ class Node(BaseModel, NodeContent):
 
         return nis
 
 
         return nis
 

+    @property
+    def last_activity(self):
+        try:
+            return self.actions.order_by('-action_date')[0].action_date
+        except:
+            return self.last_seen
+

     @property
     def state_list(self):
         return [s.state_type for s in self.states.all()]
     @property
     def state_list(self):
         return [s.state_type for s in self.states.all()]


@@ -327,13 +332,13 @@ class Node(BaseModel, NodeContent):
     def create_revision(self, user, **kwargs):
         number = self.revisions.aggregate(last=models.Max('revision'))['last'] + 1
         revision = self._create_revision(user, number, **kwargs)
     def create_revision(self, user, **kwargs):
         number = self.revisions.aggregate(last=models.Max('revision'))['last'] + 1
         revision = self._create_revision(user, number, **kwargs)

-        self.activate_revision(user, revision)
+        self.activate_revision(user, revision, extensions=['urlize'])

         return revision
 
         return revision
 

-    def activate_revision(self, user, revision):
+    def activate_revision(self, user, revision, extensions=['urlize']):

         self.title = revision.title
         self.tagnames = revision.tagnames
         self.title = revision.title
         self.tagnames = revision.tagnames

-        self.body = revision.body
+        self.body = auto_user_link(self, self._as_markdown(revision.body, *extensions))

 
         self.active_revision = revision
         self.update_last_activity(user)
 
         self.active_revision = revision
         self.update_last_activity(user)


@@ -414,19 +419,21 @@ class Node(BaseModel, NodeContent):
         super(Node, self).delete(*args, **kwargs)
 
     def save(self, *args, **kwargs):
         super(Node, self).delete(*args, **kwargs)
 
     def save(self, *args, **kwargs):

-        tags_changed = self._process_changes_in_tags()
-

         if not self.id:
             self.node_type = self.get_type()
             super(BaseModel, self).save(*args, **kwargs)
             self.active_revision = self._create_revision(self.author, 1, title=self.title, tagnames=self.tagnames,
                                                          body=self.body)
         if not self.id:
             self.node_type = self.get_type()
             super(BaseModel, self).save(*args, **kwargs)
             self.active_revision = self._create_revision(self.author, 1, title=self.title, tagnames=self.tagnames,
                                                          body=self.body)

+            self.activate_revision(self.author, self.active_revision)

             self.update_last_activity(self.author, time=self.added_at)
 
         if self.parent_id and not self.abs_parent_id:
             self.abs_parent = self.parent.absolute_parent
             self.update_last_activity(self.author, time=self.added_at)
 
         if self.parent_id and not self.abs_parent_id:
             self.abs_parent = self.parent.absolute_parent

-
+        
+        tags_changed = self._process_changes_in_tags()
+        

         super(Node, self).save(*args, **kwargs)
         super(Node, self).save(*args, **kwargs)

+        

         if tags_changed: self.tags = list(Tag.objects.filter(name__in=self.tagname_list()))
 
     class Meta:
         if tags_changed: self.tags = list(Tag.objects.filter(name__in=self.tagname_list()))
 
     class Meta: