somes simples fixs: doc, one import per line, exception_name, ...

tracim/exceptions.py

@@ -117,5 +117,5 @@ class UserNotFoundInTracimRequest(TracimException):
     pass
 
 
-class NotSameWorkspace(TracimException):
-    pass
+class WorkspacesDoNotMatch(TracimException):
+    pass

tracim/lib/core/content.py

@@ -25,7 +25,7 @@
 from tracim.lib.utils.utils import cmp_to_key
 from tracim.lib.core.notifications import NotifierFactory
 from tracim.exceptions import SameValueError
-from tracim.exceptions import NotSameWorkspace
+from tracim.exceptions import WorkspacesDoNotMatch
 from tracim.lib.utils.utils import current_date_for_filename
 from tracim.models.revision_protection import new_revision
 from tracim.models.auth import User
@@ -878,7 +878,7 @@ def move(self,
             item.workspace = new_workspace
             if new_parent and \
                     new_parent.workspace_id != new_workspace.workspace_id:
-                raise NotSameWorkspace(
+                raise WorkspacesDoNotMatch(
                     'new parent workspace and new workspace should be the same.'
                 )
         else:

tracim/lib/utils/authorization.py

@@ -8,9 +8,8 @@
 except ImportError:  # python3.4
     JSONDecodeError = ValueError
 
-from tracim.exceptions import InsufficientUserWorkspaceRole, \
-    InsufficientUserProfile, WorkspaceNotFoundInTracimRequest
-
+from tracim.exceptions import InsufficientUserWorkspaceRole
+from tracim.exceptions import InsufficientUserProfile
 if TYPE_CHECKING:
     from tracim import TracimRequest
 ###

tracim/lib/utils/request.py

@@ -2,7 +2,8 @@
 from pyramid.request import Request
 from sqlalchemy.orm.exc import NoResultFound
 
-from tracim.exceptions import NotAuthenticated, WorkspaceNotFoundInTracimRequest
+from tracim.exceptions import NotAuthenticated
+from tracim.exceptions import WorkspaceNotFoundInTracimRequest
 from tracim.exceptions import UserNotFoundInTracimRequest
 from tracim.exceptions import UserDoesNotExist
 from tracim.exceptions import WorkspaceNotFound
@@ -59,7 +60,7 @@ def current_workspace(self) -> Workspace:
         :return: Workspace of the request
         """
         if self._current_workspace is None:
-            self.current_workspace = self._get_current_workspace(self.current_user, self)
+            self._current_workspace = self._get_current_workspace(self.current_user, self)
         return self._current_workspace
 
     @current_workspace.setter
@@ -108,10 +109,8 @@ def candidate_user(self) -> User:
     @property
     def candidate_workspace(self) -> Workspace:
         """
-        Get user from headers/body request. This user is not
-        the one found by authentication mecanism. This user
-        can help user to know about who one page is about in
-        a similar way as current_workspace.
+        Get workspace from headers/body request. This workspace is not
+        the one found from path. Its the one from json body.
         """
         if self._candidate_workspace is None:
             self._candidate_workspace = self._get_candidate_workspace(