Merge branch 'dev' of https://github.com/tienthienhd/tc-edoc into feature/folder-fe

src/documents/filters.py

@@ -7,6 +7,7 @@ from django.db.models.functions import Cast
 from django_filters.rest_framework import BooleanFilter
 from django_filters.rest_framework import Filter
 from django_filters.rest_framework import FilterSet
+from django_filters import CharFilter, NumberFilter
 from guardian.utils import get_group_obj_perms_model
 from guardian.utils import get_user_obj_perms_model
 from rest_framework_guardian.filters import ObjectPermissionsFilter
@@ -207,6 +208,28 @@ class DocumentFilterSet(FilterSet):
 
     shared_by__id = SharedByUser()
     
+    warehouse__id = NumberFilter(method='filter_by_warehouse')
+
+    def filter_by_warehouse(self, queryset, name, value):
+        warehouse = Warehouse.objects.get(id=value)
+        return self.get_warehouse_documents(warehouse)
+
+    def get_warehouse_documents(self, warehouse):
+        if warehouse.type == Warehouse.BOXCASE:
+            return Document.objects.filter(warehouse=warehouse)
+        elif warehouse.type == Warehouse.SHELF:
+            boxcases = Warehouse.objects.filter(parent_warehouse=warehouse)
+            return Document.objects.filter(warehouse__in=[b.id for b in boxcases])
+        elif warehouse.type == Warehouse.WAREHOUSE:
+            shelves = Warehouse.objects.filter(parent_warehouse=warehouse)
+            boxcases = Warehouse.objects.filter(parent_warehouse__in=[s.id for s in shelves])
+            return Document.objects.filter(warehouse__in=[b.id for b in boxcases])
+        else:
+            return Document.objects.none()
+
+    
+    
+
     class Meta:
         model = Document
         fields = {

src/documents/migrations/1055_folder_checksum.py

@@ -0,0 +1,18 @@
+# Generated by Django 4.2.11 on 2024-06-12 02:10
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('documents', '1054_folder_document_folder_and_more'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='folder',
+            name='checksum',
+            field=models.CharField(editable=False, help_text='The checksum of the original folder.', max_length=32, null=True, unique=True, verbose_name='checksum'),
+        ),
+    ]

src/documents/migrations/1056_remove_folder_documents_folder_unique_name_owner_and_more.py

@@ -0,0 +1,21 @@
+# Generated by Django 4.2.11 on 2024-06-17 08:36
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('documents', '1055_folder_checksum'),
+    ]
+
+    operations = [
+        migrations.RemoveConstraint(
+            model_name='folder',
+            name='documents_folder_unique_name_owner',
+        ),
+        migrations.RemoveConstraint(
+            model_name='folder',
+            name='documents_folder_name_uniq',
+        ),
+    ]

src/documents/migrations/1057_remove_warehouse_documents_warehouse_unique_name_owner_and_more.py

@@ -0,0 +1,21 @@
+# Generated by Django 4.2.11 on 2024-06-17 08:56
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('documents', '1056_remove_folder_documents_folder_unique_name_owner_and_more'),
+    ]
+
+    operations = [
+        migrations.RemoveConstraint(
+            model_name='warehouse',
+            name='documents_warehouse_unique_name_owner',
+        ),
+        migrations.RemoveConstraint(
+            model_name='warehouse',
+            name='documents_warehouse_name_uniq',
+        ),
+    ]

src/documents/models.py

@@ -149,6 +149,7 @@ class Warehouse(MatchingModel):
     class Meta(MatchingModel.Meta): 
         verbose_name = _("warehouse")
         verbose_name_plural = _("warehouses")
+        constraints = []
     
     def __str__(self):
         return self.name
@@ -156,10 +157,20 @@ class Warehouse(MatchingModel):
 class Folder(MatchingModel):
     parent_folder = models.ForeignKey('self', on_delete=models.CASCADE, null=True, blank=True )
     path = models.TextField(_("path"), null=True, blank=True)
+    checksum = models.CharField(
+        _("checksum"),
+        max_length=32,
+        editable=False,
+        unique=True,
+        null=True,
+        help_text=_("The checksum of the original folder."),
+    )
 
     class Meta(MatchingModel.Meta):
+        
         verbose_name = _("folder")
         verbose_name_plural = _("folders")
+        constraints = []
     def __str__(self): 
         return self.name
     

src/documents/serialisers.py

@@ -1826,13 +1826,40 @@ class WorkflowSerializer(serializers.ModelSerializer):
         return instance
 
 
+class AdjustedNameField(serializers.CharField): 
+    def to_internal_value(self, data): 
+        model = self.parent.Meta.model
+        print(data) 
+        if hasattr(model, 'name'): 
+            parent_folder = self.parent.initial_data.get('parent_folder')
+            type = self.parent.initial_data.get('type')
             
+            if type: 
+                existing_names = model.objects.filter(type=type).values_list('name', flat=True) 
+            elif parent_folder: 
+                existing_names = model.objects.filter(parent_folder=parent_folder).values_list('name', flat=True) 
+            
+            else: 
+                existing_names = model.objects.filter(name__startswith=data).values_list('name', flat=True) 
+             
+            if data in existing_names: 
+                data = self.generate_unique_name(data, existing_names) 
+         
+        return data 
+     
+    def generate_unique_name(self, name, existing_names): 
+        i = 1 
+        new_name = name 
+        while new_name in existing_names: 
+            new_name = f"{name}({i})" 
+            i += 1 
+        return new_name
 
 
 
 class WarehouseSerializer(MatchingModelSerializer, OwnedObjectSerializer):
     document_count = serializers.SerializerMethodField()
-    
+    name = AdjustedNameField()
     class Meta:
         model = Warehouse
         fields = '__all__'
@@ -1860,7 +1887,7 @@ class WarehouseSerializer(MatchingModelSerializer, OwnedObjectSerializer):
     
     
 class FolderSerializer(MatchingModelSerializer, OwnedObjectSerializer):
-        
+    name = AdjustedNameField()
     class Meta:
         model = Folder
         fields = '__all__'   

src/documents/views.py

@@ -1,3 +1,4 @@
+import hashlib
 import itertools
 import json
 import logging
@@ -752,32 +753,6 @@ class DocumentViewSet(
             ]
             return Response(links)
 
-    def get_queryset(self):
-        queryset = self.queryset
-        warehouse_id = self.request.query_params.get('warehouse_id', None)
- 
-        if warehouse_id is not None:
-            queryset = self.get_warehouse(warehouse_id)
-    
-        return queryset
-
-    def get_warehouse(self, warehouse_id):
-        warehouse = Warehouse.objects.get(id=int(warehouse_id))
-        return self.get_warehouse_documents(warehouse)
-
-    def get_warehouse_documents(self, warehouse):
-        if warehouse.type == Warehouse.BOXCASE:
-            return Document.objects.filter(warehouse=warehouse)
-        elif warehouse.type == Warehouse.SHELF:
-            boxcases = Warehouse.objects.filter(parent_warehouse=warehouse)
-            return Document.objects.filter(warehouse__in=[b.id for b in boxcases])
-        elif warehouse.type == Warehouse.WAREHOUSE:
-            shelves = Warehouse.objects.filter(parent_warehouse=warehouse)
-            boxcases = Warehouse.objects.filter(parent_warehouse__in=[s.id for s in shelves])
-            return Document.objects.filter(warehouse__in=[b.id for b in boxcases])
-        else:
-            return Document.objects.none()
-            
             
 class SearchResultSerializer(DocumentSerializer, PassUserMixin):
     def to_representation(self, instance):
@@ -1635,6 +1610,8 @@ class BulkEditObjectsView(PassUserMixin):
                     shelves.delete()
                     warehouse.delete()
                     
+            return Response(status=status.HTTP_204_NO_CONTENT)
+        
         elif operation == "delete" and object_type == "folders":
             for folder_id in object_ids:
                 folder = Folder.objects.get(id=int(folder_id))
@@ -1651,7 +1628,7 @@ class BulkEditObjectsView(PassUserMixin):
                 
                 delete_folder_hierarchy(folder)
             
-            return Response(status=status.HTTP_200_OK)            
+            return Response(status=status.HTTP_204_NO_CONTENT)            
                           
         elif operation == "delete":
 
@@ -1920,16 +1897,16 @@ class WarehouseViewSet(ModelViewSet, PermissionsAwareDocumentCountMixin):
     ordering_fields = ("name", "type", "parent_warehouse", "document_count")
     
     def getWarehouseDoc(self, wareh):
-        
+        currentUser = self.request.user
         if wareh.type == Warehouse.BOXCASE:
-            return list(Document.objects.filter(warehouse=wareh).order_by("-created").values())
+            return list(Document.objects.filter(warehouse=wareh, owner=currentUser).order_by("-created").values())
         elif wareh.type == Warehouse.SHELF:
             boxcases = Warehouse.objects.filter(parent_warehouse=wareh)
-            return list(Document.objects.filter(warehouse__in=[b.id for b in boxcases]).order_by("-created").values())
+            return list(Document.objects.filter(warehouse__in=[b.id for b in boxcases], owner=currentUser).order_by("-created").values())
         elif wareh.type == Warehouse.WAREHOUSE:
             shelves = Warehouse.objects.filter(parent_warehouse=wareh)
             boxcases = Warehouse.objects.filter(parent_warehouse__in=[s.id for s in shelves])
-            return list(Document.objects.filter(warehouse__in=[b.id for b in boxcases]).order_by("-created").values())
+            return list(Document.objects.filter(warehouse__in=[b.id for b in boxcases], owner=currentUser).order_by("-created").values())
         else:
             return list(Document.objects.none())
     
@@ -1967,15 +1944,15 @@ class WarehouseViewSet(ModelViewSet, PermissionsAwareDocumentCountMixin):
         parent_warehouse = Warehouse.objects.filter(id=parent_warehouse.id if parent_warehouse else 0).first()   
         
         if serializer.validated_data.get("type") == Warehouse.WAREHOUSE and not parent_warehouse:
-            warehouse = serializer.save()
+            warehouse = serializer.save(owner=request.user)
             warehouse.path = str(warehouse.id)
             warehouse.save()
         elif serializer.validated_data.get("type", "") == Warehouse.SHELF and  getattr(parent_warehouse, 'type', "") == Warehouse.WAREHOUSE :
-            warehouse = serializer.save(type=Warehouse.SHELF, parent_warehouse=parent_warehouse)
+            warehouse = serializer.save(type=Warehouse.SHELF, parent_warehouse=parent_warehouse,owner=request.user)
             warehouse.path = f"{parent_warehouse.path}/{warehouse.id}"
             warehouse.save()
         elif serializer.validated_data.get("type", "") == Warehouse.BOXCASE and  getattr(parent_warehouse, 'type', "") == Warehouse.SHELF :
-            warehouse = serializer.save(type=Warehouse.BOXCASE, parent_warehouse=parent_warehouse)
+            warehouse = serializer.save(type=Warehouse.BOXCASE, parent_warehouse=parent_warehouse,owner=request.user)
             warehouse.path = f"{parent_warehouse.path}/{warehouse.id}"
             warehouse.save()
         else:
@@ -2089,17 +2066,40 @@ class FolderViewSet(ModelViewSet, PermissionsAwareDocumentCountMixin):
     ordering_fields = ("name", "path", "parent_folder", "document_count")
 
         
-    def getFolderDoc(self, fol):
-        
-        documents = list(Document.objects.filter(folder=fol).order_by("-created").values())
-        folders = list(Folder.objects.filter(parent_folder=fol).order_by("name").values())
+    def getFolderDoc(self, request):
+        currentUser = request.user
+        documents = list(Document.objects.filter(folder=None, owner=currentUser).order_by("-created").values())
+        folders = list(Folder.objects.filter(parent_folder=None, owner=currentUser).order_by("name").values())
         return {
             "documents": documents,
             "folders": folders,
         }    
         
+    @action(methods=["get"], detail=False)
+    def folders_documents(self, request):
+        
+        if request.method == "GET":
+            try:
+                return Response(self.getFolderDoc(request))
+            except Exception as e:
+                logger.warning(f"An error occurred retrieving folders: {e!s}")
+                return Response(
+                    {"error": "Error retrieving folders, check logs for more detail."},
+                )
+    
+    
+    def getFolderDocById(self, fol):
+        
+        documents = list(Document.objects.filter(folder=fol).order_by("-created").values())
+        child_folders = list(Folder.objects.filter(parent_folder=fol).order_by("name").values())
+        return {
+            "documents": documents,
+            "folders": child_folders,
+        }
+    
     @action(methods=["get"], detail=True)
-    def folders_documents(self, request, pk=None):
+    def folders_documents_by_id(self, request, pk=None):
+        print("dkljfskljf")
         currentUser = request.user
         try:
             fol = Folder.objects.get(pk=pk)
@@ -2114,7 +2114,7 @@ class FolderViewSet(ModelViewSet, PermissionsAwareDocumentCountMixin):
 
         if request.method == "GET":
             try:
-                return Response(self.getFolderDoc(fol))
+                return Response(self.getFolderDocById(fol))
             except Exception as e:
                 logger.warning(f"An error occurred retrieving folders: {e!s}")
                 return Response(
@@ -2132,12 +2132,14 @@ class FolderViewSet(ModelViewSet, PermissionsAwareDocumentCountMixin):
         parent_folder = Folder.objects.filter(id=parent_folder.id if parent_folder else 0).first()   
         
         if parent_folder == None:
-            folder = serializer.save()
+            folder = serializer.save(owner=request.user)
             folder.path = str(folder.id)
+            folder.checksum = hashlib.md5(f'{folder.id}.{folder.name}'.encode()).hexdigest()
             folder.save()
         elif parent_folder:
-            folder = serializer.save(parent_folder=parent_folder)
+            folder = serializer.save(parent_folder=parent_folder,owner=request.user)
             folder.path = f"{parent_folder.path}/{folder.id}"
+            folder.checksum = hashlib.md5(f'{folder.id}.{folder.name}'.encode()).hexdigest()
             folder.save()
         else:
             return Response(status=status.HTTP_400_BAD_REQUEST)
@@ -2147,6 +2149,17 @@ class FolderViewSet(ModelViewSet, PermissionsAwareDocumentCountMixin):
     def update(self, request, *args, **kwargs):
         partial = kwargs.pop('partial', False)
         instance = self.get_object()
+        
+        if 'parent_folder' in request.data and int(request.data['parent_folder']) == instance.id:
+            return Response(status=status.HTTP_400_BAD_REQUEST)
+        
+        if 'parent_folder' in request.data :
+            new_parent_folder = Folder.objects.get(id=int(request.data['parent_folder']))
+            if new_parent_folder.path.startswith(instance.path):
+                return Response(status=status.HTTP_400_BAD_REQUEST, data={'error': 'Cannot move a folder into one of its child folders.'})
+        else:
+            request.data['parent_folder'] = None
+        
         serializer = self.get_serializer(instance, data=request.data, partial=partial)
         serializer.is_valid(raise_exception=True)
 
@@ -2157,6 +2170,7 @@ class FolderViewSet(ModelViewSet, PermissionsAwareDocumentCountMixin):
         if old_parent_folder != instance.parent_folder:
             if instance.parent_folder:
                 instance.path = f"{instance.parent_folder.path}/{instance.id}"
+                
             else:
                 instance.path = f"{instance.id}"
             instance.save()
@@ -2168,6 +2182,17 @@ class FolderViewSet(ModelViewSet, PermissionsAwareDocumentCountMixin):
     def partial_update(self, request, *args, **kwargs):
         partial = kwargs.pop('partial', True)
         instance = self.get_object()
+        
+        if 'parent_folder' in request.data and int(request.data['parent_folder']) == instance.id:
+            return Response(status=status.HTTP_400_BAD_REQUEST)
+        
+        if 'parent_folder' in request.data :
+            new_parent_folder = Folder.objects.get(id=int(request.data['parent_folder']))
+            if new_parent_folder.path.startswith(instance.path):
+                return Response(status=status.HTTP_400_BAD_REQUEST, data={'error': 'Cannot move a folder into one of its child folders.'})
+        else:
+            request.data['parent_folder'] = None
+        
         serializer = self.get_serializer(instance, data=request.data, partial=partial)
         serializer.is_valid(raise_exception=True)
 
@@ -2212,5 +2237,5 @@ class FolderViewSet(ModelViewSet, PermissionsAwareDocumentCountMixin):
         
         delete_folder_hierarchy(folder)
         
-        return Response(status=status.HTTP_200_OK)
+        return Response(status=status.HTTP_204_NO_CONTENT)