Merge pull request #495 from tracim/fix/126/drops_duplicated_revisions_workaround

tracim/tracim/model/data.py

@@ -1111,16 +1111,6 @@ class Content(DeclarativeBase):
         self._properties = json.dumps(properties_struct)
         ContentChecker.check_properties(self)
 
-    @property
-    def clean_revisions(self):
-        """
-        This property return revisions with really only one of each revisions:
-        Actually, .revisions list give duplicated last revision,
-        see https://github.com/tracim/tracim/issues/126
-        :return: list of revisions
-        """
-        return list(set(self.revisions))
-
     def created_as_delta(self, delta_from_datetime:datetime=None):
         if not delta_from_datetime:
             delta_from_datetime = datetime.utcnow()
@@ -1213,13 +1203,13 @@ class Content(DeclarativeBase):
         return last_comment
 
     def get_previous_revision(self) -> 'ContentRevisionRO':
-        rev_ids = [revision.revision_id for revision in self.clean_revisions]
+        rev_ids = [revision.revision_id for revision in self.revisions]
         rev_ids.sort()
 
         if len(rev_ids)>=2:
             revision_rev_id = rev_ids[-2]
 
-            for revision in self.clean_revisions:
+            for revision in self.revisions:
                 if revision.revision_id == revision_rev_id:
                     return revision
 
@@ -1248,7 +1238,7 @@ class Content(DeclarativeBase):
         events = []
         for comment in self.get_comments():
             events.append(VirtualEvent.create_from_content(comment))
-        for revision in self.clean_revisions:
+        for revision in self.revisions:
             events.append(VirtualEvent.create_from_content_revision(revision))
 
         sorted_events = sorted(events,