short descriptive message for every version bump

Author: ciur

papermerge/core/models/document.py

@@ -1,5 +1,6 @@
 import logging
 import os
+from typing import Optional
 from os.path import getsize
 from pikepdf import Pdf
 
@@ -346,7 +347,11 @@ def version_bump_from_pages(self, pages):
 
         return document_version
 
-    def version_bump(self, page_count=None):
+    def version_bump(
+        self,
+        page_count=None,
+        short_description: Optional[str] = ''
+    ):
         """
         Increment document's version.
 
@@ -370,6 +375,7 @@ def version_bump(self, page_count=None):
             file_name=last_doc_version.file_name,
             size=0,  # TODO: set to newly created file size
             page_count=new_page_count,
+            short_description=short_description,
             lang=last_doc_version.lang
         )
         new_doc_version.save()

papermerge/core/views/documents.py

@@ -162,7 +162,8 @@ def _merge_documents(src_uuid, dst_uuid):
 
     src_version = src.versions.last()
     dst_new_version = dst.version_bump(
-        page_count=src_version.pages.count()
+        page_count=src_version.pages.count(),
+        short_description='document merge'
     )
     total_merge(
         src_old_version=src_version,

papermerge/core/views/pages.py

@@ -575,23 +575,27 @@ def merge_to_document(self, data):
         )
         src_old_version = pages.first().document_version
         doc = src_old_version.document
+        pages_count = pages.count()
 
-        if src_old_version.pages.count() == pages.count():
+        if src_old_version.pages.count() == pages_count:
             # destination new version will have same
             # number of pages as source document count
             dst_new_version = dst_document.version_bump(
-                page_count=pages.count()
+                page_count=pages_count,
+                short_description=f'{pages_count} page(s) merged in'
             )
             total_merge(
                 src_old_version=src_old_version,
                 dst_new_version=dst_new_version
             )
         else:
             src_new_version = doc.version_bump(
-                page_count=src_old_version.pages.count() - pages.count()
+                page_count=src_old_version.pages.count() - pages_count,
+                short_description=f'{pages_count} page(s) merged out'
             )
             dst_new_version = dst_document.version_bump(
-                page_count=pages.count()
+                page_count=pages_count,
+                short_description=f'{pages_count} page(s) merged in'
             )
             partial_merge(
                 src_old_version=src_old_version,
@@ -610,13 +614,16 @@ def move_to_document(self, data):
         )
         src_old_version = pages.first().document_version
         dst_old_version = dst_document.versions.last()
+        pages_count = pages.count()
 
         doc = src_old_version.document
         src_new_version = doc.version_bump(
-            page_count=src_old_version.pages.count() - pages.count()
+            page_count=src_old_version.pages.count() - pages_count,
+            short_description=f'{pages_count} page(s) moved out'
         )
         dst_new_version = dst_document.version_bump(
-            page_count=dst_old_version.pages.count() + pages.count()
+            page_count=dst_old_version.pages.count() + pages_count,
+            short_description=f'{pages_count} page(s) moved in'
         )
 
         remove_pdf_pages(