Disable ability to create empty revision from webdav and webui

tracim/tracim/controllers/content.py

@@ -174,7 +174,7 @@ class UserWorkspaceFolderFileRestController(TIMWorkspaceContentRestController):
 
     @property
     def _err_url(self):
-        return tg.url('/dashboard/workspaces/{}/folders/{}/file/{}')
+        return tg.url('/workspaces/{}/folders/{}/files/{}')
 
     @property
     def _item_type(self):
@@ -376,19 +376,13 @@ class UserWorkspaceFolderFileRestController(TIMWorkspaceContentRestController):
         try:
             api = ContentApi(tmpl_context.current_user)
             item = api.get_one(int(item_id), self._item_type, workspace)
-            label_changed = False
-            if label is not None and label != item.label:
-                label_changed = True
-
-            if label is None:
-                label = ''
-
-            # TODO - D.A. - 2015-03-19
-            # refactor this method in order to make code easier to understand
 
             with new_revision(item):
+                if label:
+                    # This case is the default "file title and description
+                    # update" In this case the file itself is not revisionned
 
-                if (comment and label) or (not comment and label_changed):
+                    # Update description and label
                     updated_item = api.update_content(
                         item, label if label else item.label,
                         comment if comment else ''
@@ -401,31 +395,21 @@ class UserWorkspaceFolderFileRestController(TIMWorkspaceContentRestController):
                         return render_invalid_integrity_chosen_path(
                             updated_item.get_label_as_file(),
                         )
-
                     api.save(updated_item, ActionDescription.EDITION)
-
-                    # This case is the default "file title and description
-                    # update" In this case the file itself is not revisionned
-
                 else:
                     # So, now we may have a comment and/or a file revision
-                    if comment and '' == label:
-                        comment_item = api.create_comment(workspace,
-                                                          item, comment,
-                                                          do_save=False)
-
-                        if not isinstance(file_data, FieldStorage):
-                            api.save(comment_item, ActionDescription.COMMENT)
-                        else:
-                            # The notification is only sent
-                            # if the file is NOT updated
-                            #
-                            # If the file is also updated,
-                            # then a 'file revision' notification will be sent.
-                            api.save(comment_item,
-                                     ActionDescription.COMMENT,
-                                     do_notify=False)
+                    comment_item = None
+                    file_revision = None
 
+                    # INFO - G.M - 20/03/2018 - Add new comment
+                    if comment:
+                        comment_item = api.create_comment(
+                            workspace,
+                            item,
+                            comment,
+                            do_save=False
+                        )
+                    # INFO - G.M - 20/03/2018 - Add new file-revision
                     if isinstance(file_data, FieldStorage):
                         api.update_file_data(item,
                                              file_data.filename,
@@ -440,8 +424,20 @@ class UserWorkspaceFolderFileRestController(TIMWorkspaceContentRestController):
                             return render_invalid_integrity_chosen_path(
                                 item.get_label_as_file(),
                             )
-
+                        file_revision = True
+
+                    # INFO - G.M - 20/03/2018 - Save revision/comment
+                    if comment_item and file_revision:
+                        api.save(
+                            comment_item,
+                            ActionDescription.COMMENT,
+                            do_notify= False
+                        )
+                        api.save(item, ActionDescription.REVISION)
+                    elif file_revision:
                         api.save(item, ActionDescription.REVISION)
+                    elif comment_item:
+                        api.save(comment_item, ActionDescription.COMMENT)
 
             msg = _('{} updated').format(self._item_type_label)
             tg.flash(msg, CST.STATUS_OK)
@@ -449,6 +445,14 @@ class UserWorkspaceFolderFileRestController(TIMWorkspaceContentRestController):
                                              tmpl_context.folder_id,
                                              item.content_id))
 
+        except SameValueError:
+            not_updated = '{} not updated: the content did not change'
+            msg = _(not_updated).format(self._item_type_label)
+            tg.flash(msg, CST.STATUS_WARNING)
+            tg.redirect(self._err_url.format(tmpl_context.workspace_id,
+                                             tmpl_context.folder_id,
+                                             item_id))
+
         except ValueError as e:
             error = '{} not updated - error: {}'
             msg = _(error).format(self._item_type_label,
@@ -459,6 +463,7 @@ class UserWorkspaceFolderFileRestController(TIMWorkspaceContentRestController):
                                              item_id))
 
 
+
 class UserWorkspaceFolderPageRestController(TIMWorkspaceContentRestController):
     """
     manage a path like this: /workspaces/1/folders/XXX/pages/4
@@ -606,7 +611,7 @@ class UserWorkspaceFolderPageRestController(TIMWorkspaceContentRestController):
             tg.redirect(self._std_url.format(tmpl_context.workspace_id,
                                              tmpl_context.folder_id,
                                              item.content_id))
-        except SameValueError as e:
+        except SameValueError:
             not_updated = '{} not updated: the content did not change'
             msg = _(not_updated).format(self._item_type_label)
             tg.flash(msg, CST.STATUS_WARNING)
@@ -621,7 +626,6 @@ class UserWorkspaceFolderPageRestController(TIMWorkspaceContentRestController):
                                              tmpl_context.folder_id,
                                              item_id))
 
-
 class UserWorkspaceFolderThreadRestController(TIMWorkspaceContentRestController):
     """
     manage a path like this: /workspaces/1/folders/XXX/pages/4

tracim/tracim/lib/content.py

@@ -920,7 +920,9 @@ class ContentApi(object):
 
     def update_content(self, item: Content, new_label: str, new_content: str=None) -> Content:
         if item.label==new_label and item.description==new_content:
-            raise SameValueError(_('The content did not changed'))
+            # TODO - G.M - 20-03-2018 - Fix internatization for webdav access.
+            # Internatization disabled in libcontent for now.
+            raise SameValueError('The content did not changed')
         item.owner = self._user
         item.label = new_label
         item.description = new_content if new_content else item.description # TODO: convert urls into links
@@ -928,6 +930,9 @@ class ContentApi(object):
         return item
 
     def update_file_data(self, item: Content, new_filename: str, new_mimetype: str, new_content: bytes) -> Content:
+        if new_mimetype == item.file_mimetype and \
+                new_content == item.depot_file.file.read():
+            raise SameValueError('The content did not changed')
         item.owner = self._user
         item.file_name = new_filename
         item.file_mimetype = new_mimetype

tracim/tracim/lib/webdav/__init__.py

@@ -5,11 +5,13 @@ from wsgidav import util
 from wsgidav import compat
 
 from tracim.lib.content import ContentApi
+from tracim.lib.utils import SameValueError
 from tracim.model import new_revision
 from tracim.model.data import ActionDescription
 from tracim.model.data import ContentType
 from tracim.model.data import Content
 from tracim.model.data import Workspace
+from wsgidav.dav_error import DAVError, HTTP_FORBIDDEN, HTTP_NOT_MODIFIED
 
 
 class HistoryType(object):
@@ -96,12 +98,20 @@ class FakeFileStream(object):
 
         self._file_stream.seek(0)
 
-        if self._content is None:
-            self.create_file()
-        else:
-            self.update_file()
-
-        transaction.commit()
+        try:
+            if self._content is None:
+                self.create_file()
+            else:
+                self.update_file()
+            transaction.commit()
+        except SameValueError:
+            transaction.abort()
+            transaction.begin()
+            raise DAVError(HTTP_NOT_MODIFIED)
+        except ValueError:
+            transaction.abort()
+            transaction.begin()
+            raise DAVError(HTTP_FORBIDDEN)
 
     def create_file(self):
         """
@@ -138,5 +148,4 @@ class FakeFileStream(object):
                 util.guessMimeType(self._content.file_name),
                 self._file_stream.read()
             )
-
             self._api.save(self._content, ActionDescription.REVISION)

tracim/tracim/model/__init__.py

@@ -8,7 +8,8 @@ from sqlalchemy.orm.unitofwork import UOWTransaction
 from zope.sqlalchemy import ZopeTransactionExtension
 
 from tracim.lib.exception import ContentRevisionUpdateError, ContentRevisionDeleteError
-
+from tracim.lib.utils import SameValueError
+import transaction
 
 class RevisionsIntegrity(object):
     """
@@ -136,5 +137,12 @@ def new_revision(
                 content.new_revision()
             RevisionsIntegrity.add_to_updatable(content.revision)
             yield content
+        except SameValueError or ValueError as e:
+            # INFO - 20-03-2018 - renew transaction when error happened
+            # This avoid bad session data like new "temporary" revision
+            # to be add when problem happen.
+            transaction.abort()
+            transaction.begin()
+            raise e
         finally:
             RevisionsIntegrity.remove_from_updatable(content.revision)