Merge pull request #61 from tracim/fix/52_fix_sqlalchemy_issue_for_webdav

tracim/lib/webdav/__init__.py

@@ -75,9 +75,9 @@ class WebdavAppFactory(object):
             WsgiDavDirBrowser,
             TracimUserSession,
             HTTPAuthenticator,
-            TracimEnv,
             ErrorPrinter,
             TracimWsgiDavDebugFilter,
+            TracimEnv,
 
         ]
         config['provider_mapping'] = {

tracim/lib/webdav/dav_provider.py

@@ -68,19 +68,19 @@ class Provider(DAVProvider):
         path = normpath(path)
         root_path = environ['http_authenticator.realm']
 
-        # If the requested path is the root, then we return a Root resource
+        # If the requested path is the root, then we return a RootResource resource
         if path == root_path:
-            return resources.Root(path, environ, user=user, session=session)
+            return resources.RootResource(path, environ, user=user, session=session)
 
         workspace_api = WorkspaceApi(current_user=user, session=session)
         workspace = self.get_workspace_from_path(path, workspace_api)
 
-        # If the request path is in the form root/name, then we return a Workspace resource
+        # If the request path is in the form root/name, then we return a WorkspaceResource resource
         parent_path = dirname(path)
         if parent_path == root_path:
             if not workspace:
                 return None
-            return resources.Workspace(
+            return resources.WorkspaceResource(
                 path=path,
                 environ=environ,
                 workspace=workspace,
@@ -107,10 +107,10 @@ class Provider(DAVProvider):
 
         # Easy cases : path either end with /.deleted, /.archived or /.history, then we return corresponding resources
         if path.endswith(SpecialFolderExtension.Archived) and self._show_archive:
-            return resources.ArchivedFolder(path, environ, workspace, content)
+            return resources.ArchivedFolderResource(path, environ, workspace, content)
 
         if path.endswith(SpecialFolderExtension.Deleted) and self._show_delete:
-            return resources.DeletedFolder(path, environ, workspace, content)
+            return resources.DeletedFolderResource(path, environ, workspace, content)
 
         if path.endswith(SpecialFolderExtension.History) and self._show_history:
             is_deleted_folder = re.search(r'/\.deleted/\.history$', path) is not None
@@ -120,13 +120,13 @@ class Provider(DAVProvider):
                 else HistoryType.Archived if is_archived_folder \
                 else HistoryType.Standard
 
-            return resources.HistoryFolder(path, environ, workspace, content, type)
+            return resources.HistoryFolderResource(path, environ, workspace, content, type)
 
         # Now that's more complicated, we're trying to find out if the path end with /.history/file_name
         is_history_file_folder = re.search(r'/\.history/([^/]+)$', path) is not None
 
         if is_history_file_folder and self._show_history:
-            return resources.HistoryFileFolder(
+            return resources.HistoryFileFolderResource(
                 path=path,
                 environ=environ,
                 content=content
@@ -143,7 +143,7 @@ class Provider(DAVProvider):
             content = self.get_content_from_revision(content_revision, content_api)
 
             if content.type == ContentType.File:
-                return resources.HistoryFile(path, environ, content, content_revision)
+                return resources.HistoryFileResource(path, environ, content, content_revision)
             else:
                 return resources.HistoryOtherFile(path, environ, content, content_revision)
 
@@ -153,7 +153,7 @@ class Provider(DAVProvider):
         if content is None:
             return None
         if content.type == ContentType.Folder:
-            return resources.Folder(
+            return resources.FolderResource(
                 path=path,
                 environ=environ,
                 workspace=content.workspace,
@@ -162,7 +162,7 @@ class Provider(DAVProvider):
                 user=user,
             )
         elif content.type == ContentType.File:
-            return resources.File(
+            return resources.FileResource(
                 path=path,
                 environ=environ,
                 content=content,
@@ -170,7 +170,7 @@ class Provider(DAVProvider):
                 user=user
             )
         else:
-            return resources.OtherFile(
+            return resources.OtherFileResource(
                 path=path,
                 environ=environ,
                 content=content,

tracim/lib/webdav/resources.py

@@ -74,13 +74,13 @@ class ManageActions(object):
         transaction.commit()
 
 
-class Root(DAVCollection):
+class RootResource(DAVCollection):
     """
-    Root ressource that represents tracim's home, which contains all workspaces
+    RootResource ressource that represents tracim's home, which contains all workspaces
     """
 
     def __init__(self, path: str, environ: dict, user: User, session: Session):
-        super(Root, self).__init__(path, environ)
+        super(RootResource, self).__init__(path, environ)
 
         self.user = user
         self.session = session
@@ -94,7 +94,7 @@ class Root(DAVCollection):
         )
 
     def __repr__(self) -> str:
-        return '<DAVCollection: Root>'
+        return '<DAVCollection: RootResource>'
 
     def getMemberNames(self) -> [str]:
         """
@@ -114,7 +114,13 @@ class Root(DAVCollection):
             workspace = self.workspace_api.get_one_by_label(label)
             workspace_path = '%s%s%s' % (self.path, '' if self.path == '/' else '/', transform_to_display(workspace.label))
 
-            return Workspace(workspace_path, self.environ, workspace)
+            return WorkspaceResource(
+                workspace_path,
+                self.environ,
+                workspace,
+                session=self.session,
+                user=self.user,
+            )
         except AttributeError:
             return None
 
@@ -146,7 +152,13 @@ class Root(DAVCollection):
         self.path, '' if self.path == '/' else '/', transform_to_display(new_workspace.label))
 
         transaction.commit()
-        return Workspace(workspace_path, self.environ, new_workspace)
+        return WorkspaceResource(
+            workspace_path,
+            self.environ,
+            new_workspace,
+            user=self.user,
+            session=self.session,
+        )
 
     def getMemberList(self):
         """
@@ -158,18 +170,19 @@ class Root(DAVCollection):
         for workspace in self.workspace_api.get_all():
             workspace_path = '%s%s%s' % (self.path, '' if self.path == '/' else '/', workspace.label)
             members.append(
-                Workspace(
+                WorkspaceResource(
                     path=workspace_path,
                     environ=self.environ,
                     workspace=workspace,
                     user=self.user,
-                    session=self.session)
+                    session=self.session,
+                )
             )
 
         return members
 
 
-class Workspace(DAVCollection):
+class WorkspaceResource(DAVCollection):
     """
     Workspace resource corresponding to tracim's workspaces.
     Direct children can only be folders, though files might come later on and are supported
@@ -182,7 +195,7 @@ class Workspace(DAVCollection):
                  user: User,
                  session: Session
     ) -> None:
-        super(Workspace, self).__init__(path, environ)
+        super(WorkspaceResource, self).__init__(path, environ)
 
         self.workspace = workspace
         self.content = None
@@ -262,7 +275,7 @@ class Workspace(DAVCollection):
             path=self.path + '/' + file_name
         )
 
-    def createCollection(self, label: str) -> 'Folder':
+    def createCollection(self, label: str) -> 'FolderResource':
         """
         Create a new folder for the current workspace. As it's not possible for the user to choose
         which types of content are allowed in this folder, we allow allow all of them.
@@ -292,9 +305,13 @@ class Workspace(DAVCollection):
 
         transaction.commit()
 
-        return Folder('%s/%s' % (self.path, transform_to_display(label)),
-                      self.environ, folder,
-                      self.workspace)
+        return FolderResource('%s/%s' % (self.path, transform_to_display(label)),
+                              self.environ,
+                              content=folder,
+                              session=self.session,
+                              user=self.user,
+                              workspace=self.workspace,
+                              )
 
     def delete(self):
         """For now, it is not possible to delete a workspace through the webdav client."""
@@ -320,7 +337,7 @@ class Workspace(DAVCollection):
 
             if content.type == ContentType.Folder:
                 members.append(
-                    Folder(
+                    FolderResource(
                         path=content_path,
                         environ=self.environ,
                         workspace=self.workspace,
@@ -332,55 +349,68 @@ class Workspace(DAVCollection):
             elif content.type == ContentType.File:
                 self._file_count += 1
                 members.append(
-                    File(
+                    FileResource(
                         path=content_path,
                         environ=self.environ,
                         content=content,
                         user=self.user,
-                        session=self.session
+                        session=self.session,
                     )
                 )
             else:
                 self._file_count += 1
-                members.append(OtherFile(content_path, self.environ, content))
+                members.append(
+                    OtherFileResource(
+                        content_path,
+                        self.environ,
+                        content,
+                        session=self.session,
+                        user=self.user,
+                    ))
 
         if self._file_count > 0 and self.provider.show_history():
             members.append(
-                HistoryFolder(
+                HistoryFolderResource(
                     path=self.path + '/' + ".history",
                     environ=self.environ,
                     content=self.content,
                     workspace=self.workspace,
-                    type=HistoryType.Standard
+                    type=HistoryType.Standard,
+                    session=self.session,
+                    user=self.user,
                 )
             )
 
         if self.provider.show_delete():
             members.append(
-                DeletedFolder(
+                DeletedFolderResource(
                     path=self.path + '/' + ".deleted",
                     environ=self.environ,
                     content=self.content,
-                    workspace=self.workspace
+                    workspace=self.workspace,
+                    session=self.session,
+                    user=self.user,
                 )
             )
 
         if self.provider.show_archive():
             members.append(
-                ArchivedFolder(
+                ArchivedFolderResource(
                     path=self.path + '/' + ".archived",
                     environ=self.environ,
                     content=self.content,
-                    workspace=self.workspace
+                    workspace=self.workspace,
+                    user=self.user,
+                    session=self.session,
                 )
             )
 
         return members
 
 
-class Folder(Workspace):
+class FolderResource(WorkspaceResource):
     """
-    Folder resource corresponding to tracim's folders.
+    FolderResource resource corresponding to tracim's folders.
     Direct children can only be either folder, files, pages or threads
     By default when creating new folders, we allow them to contain all types of content
     """
@@ -394,7 +424,7 @@ class Folder(Workspace):
             user: User,
             session: Session
     ):
-        super(Folder, self).__init__(
+        super(FolderResource, self).__init__(
             path=path,
             environ=environ,
             workspace=workspace,
@@ -482,7 +512,10 @@ class Folder(Workspace):
 
     def move_folder(self, destpath):
 
-        workspace_api = WorkspaceApi(self.user)
+        workspace_api = WorkspaceApi(
+            current_user=self.user,
+            session=self.session,
+        )
         workspace = self.provider.get_workspace_from_path(
             normpath(destpath), workspace_api
         )
@@ -493,7 +526,11 @@ class Folder(Workspace):
             workspace
         )
 
-        with new_revision(self.content):
+        with new_revision(
+            content=self.content,
+            tm=transaction.manager,
+            session=self.session,
+        ):
             if basename(destpath) != self.getDisplayName():
                 self.content_api.update_content(self.content, transform_to_bdd(basename(destpath)))
                 self.content_api.save(self.content)
@@ -524,7 +561,7 @@ class Folder(Workspace):
             try:
                 if content.type == ContentType.Folder:
                     members.append(
-                        Folder(
+                        FolderResource(
                             path=content_path,
                             environ=self.environ,
                             workspace=self.workspace,
@@ -536,7 +573,7 @@ class Folder(Workspace):
                 elif content.type == ContentType.File:
                     self._file_count += 1
                     members.append(
-                        File(
+                        FileResource(
                             path=content_path,
                             environ=self.environ,
                             content=content,
@@ -546,7 +583,7 @@ class Folder(Workspace):
                 else:
                     self._file_count += 1
                     members.append(
-                        OtherFile(
+                        OtherFileResource(
                             path=content_path,
                             environ=self.environ,
                             content=content,
@@ -563,39 +600,48 @@ class Folder(Workspace):
 
         if self._file_count > 0 and self.provider.show_history():
             members.append(
-                HistoryFolder(
+                HistoryFolderResource(
                     path=self.path + '/' + ".history",
                     environ=self.environ,
                     content=self.content,
                     workspace=self.workspace,
-                    type=HistoryType.Standard
+                    type=HistoryType.Standard,
+                    user=self.user,
+                    session=self.session,
                 )
             )
 
         if self.provider.show_delete():
             members.append(
-                DeletedFolder(
+                DeletedFolderResource(
                     path=self.path + '/' + ".deleted",
                     environ=self.environ,
                     content=self.content,
-                    workspace=self.workspace
+                    workspace=self.workspace,
+                    user=self.user,
+                    session=self.session,
                 )
             )
 
         if self.provider.show_archive():
             members.append(
-                ArchivedFolder(
+                ArchivedFolderResource(
                     path=self.path + '/' + ".archived",
                     environ=self.environ,
                     content=self.content,
-                    workspace=self.workspace
+                    workspace=self.workspace,
+                    user=self.user,
+                    session=self.session,
                 )
             )
 
         return members
 
+# TODO - G.M - 02-05-2018 - Check these object (History/Deleted/Archived Folder)
+# Those object are now not in used by tracim and also not tested,
+
 
-class HistoryFolder(Folder):
+class HistoryFolderResource(FolderResource):
     """
     A virtual resource which contains a sub-folder for every files (DAVNonCollection) contained in the parent
     folder
@@ -610,7 +656,7 @@ class HistoryFolder(Folder):
                  content: Content=None,
                  type: str=HistoryType.Standard
     ) -> None:
-        super(HistoryFolder, self).__init__(
+        super(HistoryFolderResource, self).__init__(
             path=path,
             environ=environ,
             workspace=workspace,
@@ -625,11 +671,13 @@ class HistoryFolder(Folder):
         self.content_api = ContentApi(
             current_user=self.user,
             show_archived=self._is_archived,
-            show_deleted=self._is_deleted
+            show_deleted=self._is_deleted,
+            session=self.session,
+            config=self.provider.app_config,
         )
 
     def __repr__(self) -> str:
-        return "<DAVCollection: HistoryFolder (%s)>" % self.content.file_name
+        return "<DAVCollection: HistoryFolderResource (%s)>" % self.content.file_name
 
     def getCreationDate(self) -> float:
         return mktime(datetime.now().timetuple())
@@ -646,10 +694,13 @@ class HistoryFolder(Folder):
             content_parent=self.content
         )
 
-        return HistoryFileFolder(
+        return HistoryFileFolderResource(
             path='%s/%s' % (self.path, content.get_label_as_file()),
             environ=self.environ,
-            content=content)
+            content=content,
+            session=self.session,
+            user=self.user,
+        )
 
     def getMemberNames(self) -> [str]:
         ret = []
@@ -692,15 +743,18 @@ class HistoryFolder(Folder):
         
         for content in children:
             if content.is_archived == self._is_archived and content.is_deleted == self._is_deleted:
-                members.append(HistoryFileFolder(
+                members.append(HistoryFileFolderResource(
                     path='%s/%s' % (self.path, content.get_label_as_file()),
                     environ=self.environ,
-                    content=content))
+                    content=content,
+                    user=self.user,
+                    session=self.session,
+                ))
 
         return members
 
 
-class DeletedFolder(HistoryFolder):
+class DeletedFolderResource(HistoryFolderResource):
     """
     A virtual resources which exists for every folder or workspaces which contains their deleted children
     """
@@ -714,7 +768,7 @@ class DeletedFolder(HistoryFolder):
             session: Session,
             content: Content=None
     ):
-        super(DeletedFolder, self).__init__(
+        super(DeletedFolderResource, self).__init__(
             path=path,
             environ=environ,
             workspace=workspace,
@@ -727,7 +781,7 @@ class DeletedFolder(HistoryFolder):
         self._file_count = 0
 
     def __repr__(self):
-        return "<DAVCollection: DeletedFolder (%s)" % self.content.file_name
+        return "<DAVCollection: DeletedFolderResource (%s)" % self.content.file_name
 
     def getCreationDate(self) -> float:
         return mktime(datetime.now().timetuple())
@@ -780,30 +834,54 @@ class DeletedFolder(HistoryFolder):
                 content_path = '%s/%s' % (self.path, transform_to_display(content.get_label_as_file()))
 
                 if content.type == ContentType.Folder:
-                    members.append(Folder(content_path, self.environ, self.workspace, content))
+                    members.append(
+                        FolderResource(
+                            content_path,
+                            self.environ,
+                            self.workspace,
+                            content,
+                            user=self.user,
+                            session=self.session,
+                        ))
                 elif content.type == ContentType.File:
                     self._file_count += 1
-                    members.append(File(content_path, self.environ, content))
+                    members.append(
+                        FileResource(
+                            content_path,
+                            self.environ,
+                            content,
+                            user=self.user,
+                            session=self.session,
+                        )
+                    )
                 else:
                     self._file_count += 1
-                    members.append(OtherFile(content_path, self.environ, content))
+                    members.append(
+                        OtherFileResource(
+                            content_path,
+                            self.environ,
+                            content,
+                            user=self.user,
+                            session=self.session,
+                    ))
 
         if self._file_count > 0 and self.provider.show_history():
             members.append(
-                HistoryFolder(
+                HistoryFolderResource(
                     path=self.path + '/' + ".history",
                     environ=self.environ,
                     content=self.content,
                     workspace=self.workspace,
                     user=self.user,
                     type=HistoryType.Standard,
+                    session=self.session,
                 )
             )
 
         return members
 
 
-class ArchivedFolder(HistoryFolder):
+class ArchivedFolderResource(HistoryFolderResource):
     """
     A virtual resources which exists for every folder or workspaces which contains their archived children
     """
@@ -816,7 +894,7 @@ class ArchivedFolder(HistoryFolder):
             session: Session,
             content: Content=None
     ):
-        super(ArchivedFolder, self).__init__(
+        super(ArchivedFolderResource, self).__init__(
             path=path,
             environ=environ,
             workspace=workspace,
@@ -829,7 +907,7 @@ class ArchivedFolder(HistoryFolder):
         self._file_count = 0
 
     def __repr__(self) -> str:
-        return "<DAVCollection: ArchivedFolder (%s)" % self.content.file_name
+        return "<DAVCollection: ArchivedFolderResource (%s)" % self.content.file_name
 
     def getCreationDate(self) -> float:
         return mktime(datetime.now().timetuple())
@@ -877,30 +955,53 @@ class ArchivedFolder(HistoryFolder):
                 content_path = '%s/%s' % (self.path, transform_to_display(content.get_label_as_file()))
 
                 if content.type == ContentType.Folder:
-                    members.append(Folder(content_path, self.environ, self.workspace, content))
+                    members.append(
+                        FolderResource(
+                            content_path,
+                            self.environ,
+                            self.workspace,
+                            content,
+                            user=self.user,
+                            session=self.session,
+                        ))
                 elif content.type == ContentType.File:
                     self._file_count += 1
-                    members.append(File(content_path, self.environ, content))
+                    members.append(
+                        FileResource(
+                            content_path,
+                            self.environ,
+                            content,
+                            user=self.user,
+                            session=self.session,
+                        ))
                 else:
                     self._file_count += 1
-                    members.append(OtherFile(content_path, self.environ, content))
+                    members.append(
+                        OtherFileResource(
+                            content_path,
+                            self.environ,
+                            content,
+                            user=self.user,
+                            session=self.session,
+                        ))
 
         if self._file_count > 0 and self.provider.show_history():
             members.append(
-                HistoryFolder(
+                HistoryFolderResource(
                     path=self.path + '/' + ".history",
                     environ=self.environ,
                     content=self.content,
                     workspace=self.workspace,
                     user=self.user,
-                    type=HistoryType.Standard
+                    type=HistoryType.Standard,
+                    session=self.session,
                 )
             )
 
         return members
 
 
-class HistoryFileFolder(HistoryFolder):
+class HistoryFileFolderResource(HistoryFolderResource):
     """
     A virtual resource that contains for a given content (file/page/thread) all its revisions
     """
@@ -913,7 +1014,7 @@ class HistoryFileFolder(HistoryFolder):
             user: User,
             session: Session
     ) -> None:
-        super(HistoryFileFolder, self).__init__(
+        super(HistoryFileFolderResource, self).__init__(
             path=path,
             environ=environ,
             workspace=content.workspace,
@@ -924,7 +1025,7 @@ class HistoryFileFolder(HistoryFolder):
         )
 
     def __repr__(self) -> str:
-        return "<DAVCollection: HistoryFileFolder (%s)" % self.content.file_name
+        return "<DAVCollection: HistoryFileFolderResource (%s)" % self.content.file_name
 
     def getDisplayName(self) -> str:
         return self.content.get_label_as_file()
@@ -951,17 +1052,23 @@ class HistoryFileFolder(HistoryFolder):
         left_side = '%s/(%d - %s) ' % (self.path, revision.revision_id, revision.revision_type)
 
         if self.content.type == ContentType.File:
-            return HistoryFile(
+            return HistoryFileResource(
                 path='%s%s' % (left_side, transform_to_display(revision.file_name)),
                 environ=self.environ,
                 content=self.content,
-                content_revision=revision)
+                content_revision=revision,
+                session=self.session,
+                user=self.user,
+            )
         else:
             return HistoryOtherFile(
                 path='%s%s' % (left_side, transform_to_display(revision.get_label_as_file())),
                 environ=self.environ,
                 content=self.content,
-                content_revision=revision)
+                content_revision=revision,
+                session=self.session,
+                user=self.user,
+            )
 
     def getMemberList(self) -> [_DAVResource]:
         members = []
@@ -971,26 +1078,32 @@ class HistoryFileFolder(HistoryFolder):
             left_side = '%s/(%d - %s) ' % (self.path, content.revision_id, content.revision_type)
 
             if self.content.type == ContentType.File:
-                members.append(HistoryFile(
+                members.append(HistoryFileResource(
                     path='%s%s' % (left_side, transform_to_display(content.file_name)),
                     environ=self.environ,
                     content=self.content,
-                    content_revision=content)
+                    content_revision=content,
+                    user=self.user,
+                    session=self.session,
+                    )
                 )
             else:
                 members.append(HistoryOtherFile(
                     path='%s%s' % (left_side, transform_to_display(content.file_name)),
                     environ=self.environ,
                     content=self.content,
-                    content_revision=content)
+                    content_revision=content,
+                    user=self.user,
+                    session=self.session,
+                    )
                 )
 
         return members
 
 
-class File(DAVNonCollection):
+class FileResource(DAVNonCollection):
     """
-    File resource corresponding to tracim's files
+    FileResource resource corresponding to tracim's files
     """
     def __init__(
             self,
@@ -1000,7 +1113,7 @@ class File(DAVNonCollection):
             user: User,
             session: Session,
     ) -> None:
-        super(File, self).__init__(path, environ)
+        super(FileResource, self).__init__(path, environ)
 
         self.content = content
         self.user = user
@@ -1016,7 +1129,7 @@ class File(DAVNonCollection):
         # self.setPropertyValue('Win32FileAttributes', '00000021')
 
     def __repr__(self) -> str:
-        return "<DAVNonCollection: File (%d)>" % self.content.revision_id
+        return "<DAVNonCollection: FileResource (%d)>" % self.content.revision_id
 
     def getContentLength(self) -> int:
         return self.content.depot_file.file.content_length
@@ -1230,16 +1343,16 @@ class File(DAVNonCollection):
         ).action()
 
 
-class HistoryFile(File):
+class HistoryFileResource(FileResource):
     """
     A virtual resource corresponding to a specific tracim's revision's file
     """
     def __init__(self, path: str, environ: dict, content: Content, user: User, session: Session, content_revision: ContentRevisionRO):
-        super(HistoryFile, self).__init__(path, environ, content, user=user, session=session)
+        super(HistoryFileResource, self).__init__(path, environ, content, user=user, session=session)
         self.content_revision = content_revision
 
     def __repr__(self) -> str:
-        return "<DAVNonCollection: HistoryFile (%s-%s)" % (self.content.content_id, self.content.file_name)
+        return "<DAVNonCollection: HistoryFileResource (%s-%s)" % (self.content.content_id, self.content.file_name)
 
     def getDisplayName(self) -> str:
         left_side = '(%d - %s) ' % (self.content_revision.revision_id, self.content_revision.revision_type)
@@ -1268,12 +1381,12 @@ class HistoryFile(File):
         raise DAVError(HTTP_FORBIDDEN)
 
 
-class OtherFile(File):
+class OtherFileResource(FileResource):
     """
-    File resource corresponding to tracim's page and thread
+    FileResource resource corresponding to tracim's page and thread
     """
     def __init__(self, path: str, environ: dict, content: Content, user:User, session: Session):
-        super(OtherFile, self).__init__(path, environ, content, user=user, session=session)
+        super(OtherFileResource, self).__init__(path, environ, content, user=user, session=session)
 
         self.content_revision = self.content.revision
 
@@ -1292,7 +1405,7 @@ class OtherFile(File):
         return self.path
 
     def __repr__(self) -> str:
-        return "<DAVNonCollection: OtherFile (%s)" % self.content.file_name
+        return "<DAVNonCollection: OtherFileResource (%s)" % self.content.file_name
 
     def getContentLength(self) -> int:
         return len(self.content_designed)
@@ -1318,12 +1431,24 @@ class OtherFile(File):
             )
 
 
-class HistoryOtherFile(OtherFile):
+class HistoryOtherFile(OtherFileResource):
     """
     A virtual resource corresponding to a specific tracim's revision's page and thread
     """
-    def __init__(self, path: str, environ: dict, content: Content, user:User, content_revision: ContentRevisionRO):
-        super(HistoryOtherFile, self).__init__(path, environ, content, user=user, session=self.session)
+    def __init__(self,
+                 path: str,
+                 environ: dict,
+                 content: Content,
+                 user:User,
+                 content_revision: ContentRevisionRO,
+                 session: Session):
+        super(HistoryOtherFile, self).__init__(
+            path,
+            environ,
+            content,
+            user=user,
+            session=session
+        )
         self.content_revision = content_revision
         self.content_designed = self.design()
 

tracim/models/__init__.py

@@ -48,8 +48,21 @@ def get_tm_session(session_factory, transaction_manager):
 
     """
     dbsession = session_factory()
+    # FIXME - G.M - 02-05-2018 - Check Zope/Sqlalchemy session conf.
+    # We use both keep_session=True for zope and
+    # expire_on_commit=False for sessionmaker to keep session alive after
+    # commit ( in order  to not have trouble like
+    # https://github.com/tracim/tracim_backend/issues/52
+    # or detached objects problems).
+    # These problem happened because we use "commit" in our current code.
+    # Understand what those params really mean and check if it can cause
+    # troubles somewhere else.
+    # see https://stackoverflow.com/questions/16152241/how-to-get-a-sqlalchemy-session-managed-by-zope-transaction-that-has-the-same-sc  # nopep8
     zope.sqlalchemy.register(
-        dbsession, transaction_manager=transaction_manager)
+        dbsession,
+        transaction_manager=transaction_manager,
+        keep_session=True,
+    )
     listen(dbsession, 'before_flush', prevent_content_revision_delete)
     return dbsession
 

tracim/tests/library/test_webdav.py

@@ -8,7 +8,7 @@ from tracim.lib.core.user import UserApi
 from tracim.tests import eq_
 from tracim.lib.core.notifications import DummyNotifier
 from tracim.lib.webdav.dav_provider import Provider
-from tracim.lib.webdav.resources import Root
+from tracim.lib.webdav.resources import RootResource
 from tracim.models import Content
 from tracim.models import ContentRevisionRO
 from tracim.tests import StandardTest
@@ -88,8 +88,8 @@ class TestWebDav(StandardTest):
                 'bob@fsf.local',
             )
         )
-        assert root, 'Path / should return a Root instance'
-        assert isinstance(root, Root)
+        assert root, 'Path / should return a RootResource instance'
+        assert isinstance(root, RootResource)
 
     def test_unit__list_workspaces_with_user__ok(self):
         provider = self._get_provider(self.app_config)
@@ -100,14 +100,14 @@ class TestWebDav(StandardTest):
                 'bob@fsf.local',
             )
         )
-        assert root, 'Path / should return a Root instance'
-        assert isinstance(root, Root), 'Path / should return a Root instance'
+        assert root, 'Path / should return a RootResource instance'
+        assert isinstance(root, RootResource), 'Path / should return a RootResource instance'
 
         children = root.getMemberList()
         eq_(
             2,
             len(children),
-            msg='Root should return 2 workspaces instead {0}'.format(
+            msg='RootResource should return 2 workspaces instead {0}'.format(
                 len(children),
             )
         )
@@ -130,14 +130,14 @@ class TestWebDav(StandardTest):
                 'admin@admin.admin',
             )
         )
-        assert root, 'Path / should return a Root instance'
-        assert isinstance(root, Root), 'Path / should return a Root instance'
+        assert root, 'Path / should return a RootResource instance'
+        assert isinstance(root, RootResource), 'Path / should return a RootResource instance'
 
         children = root.getMemberList()
         eq_(
             2,
             len(children),
-            msg='Root should return 2 workspaces instead {0}'.format(
+            msg='RootResource should return 2 workspaces instead {0}'.format(
                 len(children),
             )
         )
@@ -311,8 +311,6 @@ class TestWebDav(StandardTest):
             )
         )
 
-    @pytest.mark.xfail(raises=InvalidRequestError,
-                       reason='Sqlalchemy session and transaction bug')
     def test_unit__create_delete_and_create_file__ok(self):
         provider = self._get_provider(self.app_config)
         environ = self._get_environ(
@@ -559,8 +557,6 @@ class TestWebDav(StandardTest):
                 content_to_move.parent.label
         )
 
-    @pytest.mark.xfail(raises=InvalidRequestError,
-                       reason='Sqlalchemy session and transaction bug')
     def test_unit__update_content__ok(self):
         provider = self._get_provider(self.app_config)
         environ = self._get_environ(