endoreg-db 0.8.8.0__py3-none-any.whl → 0.8.8.9__py3-none-any.whl

endoreg_db/views/media/pdf_media.py

@@ -1,7 +1,7 @@
 """
-PDF Media Management View (Phase 1.2)
+report Media Management View (Phase 1.2)
 
-Provides standardized REST API for PDF files including listing, detail retrieval,
+Provides standardized REST API for report files including listing, detail retrieval,
 and streaming for the media management system.
 
 This is separate from the existing pdf.PDFMediaView which handles legacy workflows.
@@ -27,14 +27,14 @@ logger = logging.getLogger(__name__)
 
 class PdfMediaView(APIView):
     """
-    PDF Media Management API for CRUD operations on PDF files.
+    report Media Management API for CRUD operations on report files.
 
     Endpoints:
     - GET /api/media/pdfs/ - List all PDFs with filtering
-    - GET /api/media/pdfs/{id}/ - Get PDF details
-    - GET /api/media/pdfs/{id}/stream/ - Stream PDF file (same as detail for PDFs)
-    - PATCH /api/media/pdfs/{id}/ - Update PDF metadata (future)
-    - DELETE /api/media/pdfs/{id}/ - Delete PDF (future)
+    - GET /api/media/pdfs/{id}/ - Get report details
+    - GET /api/media/pdfs/{id}/stream/ - Stream report file (same as detail for PDFs)
+    - PATCH /api/media/pdfs/{id}/ - Update report metadata (future)
+    - DELETE /api/media/pdfs/{id}/ - Delete report (future)
 
     Query Parameters:
     - status: Filter by processing status (not_started, done, validated)
@@ -49,7 +49,7 @@ class PdfMediaView(APIView):
 
     Phase 1.2 Implementation:
     - List and detail views implemented
-    - PDF streaming functionality
+    - report streaming functionality
     - Filtering and search functionality
     - Pagination support
     - Error handling with proper HTTP status codes
@@ -59,17 +59,17 @@ class PdfMediaView(APIView):
 
     def get(self, request, pk=None):
         """
-        Handle GET requests for PDF listing, detail retrieval, or streaming.
+        Handle GET requests for report listing, detail retrieval, or streaming.
 
         Args:
             request: HTTP request object
-            pk: Optional PDF ID for detail view or streaming
+            pk: Optional report ID for detail view or streaming
 
         Returns:
-            Response or FileResponse: JSON response with PDF data or PDF file stream
+            Response or FileResponse: JSON response with report data or report file stream
 
         Raises:
-            Http404: If specific PDF not found
+            Http404: If specific report not found
         """
         if pk is not None:
             # Check if this is a streaming request
@@ -84,38 +84,46 @@ class PdfMediaView(APIView):
 
     def _get_pdf_detail(self, pk):
         """
-        Get detailed information for a specific PDF.
+        Get detailed information for a specific report.
 
         Args:
-            pk: PDF primary key
+            pk: report primary key
 
         Returns:
-            Response: JSON response with PDF details
+            Response: JSON response with report details
 
         Raises:
-            Http404: If PDF not found
+            Http404: If report not found
         """
         try:
             # Validate pdf_id is numeric
             try:
                 pdf_id_int = int(pk)
             except (ValueError, TypeError):
-                raise Http404("Invalid PDF ID format")
+                raise Http404("Invalid report ID format")
 
-            # Fetch PDF with related data
+            # Fetch report with related data
             pdf = RawPdfFile.objects.select_related("sensitive_meta").get(pk=pdf_id_int)
 
-            # Build PDF details
+            # Build report details
             pdf_data = {
                 "id": pdf.pk,
                 "filename": getattr(pdf.file, "name", "Unknown"),
                 "file_size": getattr(pdf.file, "size", 0),
                 "pdf_hash": pdf.pdf_hash,
-                "uploaded_at": pdf.date_created.isoformat() if getattr(pdf, "date_created", None) else None,
+                "uploaded_at": pdf.date_created.isoformat()
+                if getattr(pdf, "date_created", None)
+                else None,
                 "anonymized_text": pdf.anonymized_text,
-                "has_anonymized_text": bool(pdf.anonymized_text and pdf.anonymized_text.strip()),
-                "is_validated": getattr(pdf.sensitive_meta, "is_verified", False) if pdf.sensitive_meta else False,
-                "stream_url": self.request.build_absolute_uri(f"/api/media/pdfs/{pdf.pk}/stream/"),
+                "has_anonymized_text": bool(
+                    pdf.anonymized_text and pdf.anonymized_text.strip()
+                ),
+                "is_validated": getattr(pdf.sensitive_meta, "is_verified", False)
+                if pdf.sensitive_meta
+                else False,
+                "stream_url": self.request.build_absolute_uri(
+                    f"/api/media/pdfs/{pdf.pk}/stream/"
+                ),
             }
 
             # Add patient metadata if available
@@ -124,62 +132,71 @@ class PdfMediaView(APIView):
                     {
                         "patient_first_name": pdf.sensitive_meta.patient_first_name,
                         "patient_last_name": pdf.sensitive_meta.patient_last_name,
-                        "patient_dob": pdf.sensitive_meta.patient_dob.strftime("%d.%m.%Y") if pdf.sensitive_meta.patient_dob else None,
-                        "examination_date": pdf.sensitive_meta.examination_date.strftime("%d.%m.%Y") if pdf.sensitive_meta.examination_date else None,
+                        "patient_dob": pdf.sensitive_meta.patient_dob.strftime(
+                            "%d.%m.%Y"
+                        )
+                        if pdf.sensitive_meta.patient_dob
+                        else None,
+                        "examination_date": pdf.sensitive_meta.examination_date.strftime(
+                            "%d.%m.%Y"
+                        )
+                        if pdf.sensitive_meta.examination_date
+                        else None,
                     }
                 )
 
             return Response(pdf_data)
 
         except RawPdfFile.DoesNotExist:
-            raise Http404(f"PDF with ID {pk} not found")
+            raise Http404(f"report with ID {pk} not found")
 
         except Exception as e:
-            logger.error(f"Unexpected error in PDF detail view for ID {pk}: {str(e)}")
-            return Response({"error": "Failed to retrieve PDF details"}, status=status.HTTP_500_INTERNAL_SERVER_ERROR)
+            logger.error(
+                f"Unexpected error in report detail view for ID {pk}: {str(e)}"
+            )
+            return Response(
+                {"error": "Failed to retrieve report details"},
+                status=status.HTTP_500_INTERNAL_SERVER_ERROR,
+            )
 
     @xframe_options_exempt
     def _stream_pdf(self, pk):
         """
-        Stream PDF file content for viewing/download.
+        Stream report file content for viewing/download.
 
         Args:
-            pk: PDF primary key
+            pk: report primary key
 
         Returns:
-            FileResponse: PDF file stream
+            FileResponse: report file stream
 
         Raises:
-            Http404: If PDF not found or file cannot be accessed
+            Http404: If report not found or file cannot be accessed
         """
         try:
             # Validate pdf_id is numeric
             try:
                 pdf_id_int = int(pk)
             except (ValueError, TypeError):
-                raise Http404("Invalid PDF ID format")
+                raise Http404("Invalid report ID format")
 
-            # Fetch PDF
+            # Fetch report
             pdf = RawPdfFile.objects.get(pk=pdf_id_int)
 
             file_field = pdf.file
+            file_path = file_field.path
 
             if not file_field or not file_field.name:
-                raise Http404("PDF file not found")
+                raise Http404("report file not found")
             if not file_exists(file_field):
-                raise Http404("PDF file does not exist in storage")
+                raise Http404("report file does not exist in storage")
 
-            try:
-                file_field.open("rb")
-            except Exception as exc:  # pragma: no cover - backend-specific failure
-                logger.error("File access error for PDF %s: %s", pk, exc)
-                raise Http404("PDF file cannot be accessed")
-
-            response = FileResponse(
-                file_field,
-                content_type="application/pdf",
-                as_attachment=False,
-            )
+            with open(file_path, "rb") as file_handle:
+                response = FileResponse(
+                    file_handle,
+                    content_type="application/pdf",
+                    as_attachment=False,
+                )
 
             filename = Path(file_field.name).name
             response["Content-Disposition"] = f'inline; filename="{filename}"'
@@ -191,11 +208,11 @@ class PdfMediaView(APIView):
             return response
 
         except RawPdfFile.DoesNotExist:
-            raise Http404(f"PDF with ID {pk} not found")
+            raise Http404(f"report with ID {pk} not found")
 
         except Exception as e:
-            logger.error(f"Unexpected error in PDF streaming for ID {pk}: {str(e)}")
-            raise Http404("PDF file cannot be streamed")
+            logger.error(f"Unexpected error in report streaming for ID {pk}: {str(e)}")
+            raise Http404("report file cannot be streamed")
 
     def _list_pdfs(self, request):
         """
@@ -205,7 +222,7 @@ class PdfMediaView(APIView):
             request: HTTP request with query parameters
 
         Returns:
-            Response: JSON response with paginated PDF list
+            Response: JSON response with paginated report list
         """
         try:
             # Start with all PDFs
@@ -240,9 +257,15 @@ class PdfMediaView(APIView):
                     "filename": getattr(pdf.file, "name", "Unknown"),
                     "file_size": self._safe_get_file_size(pdf.file),
                     "pdf_hash": pdf.pdf_hash,
-                    "has_anonymized_text": bool(pdf.anonymized_text and pdf.anonymized_text.strip()),
-                    "is_validated": getattr(pdf.sensitive_meta, "is_verified", False) if pdf.sensitive_meta else False,
-                    "stream_url": request.build_absolute_uri(f"/api/media/pdfs/{pdf.pk}/stream/"),
+                    "has_anonymized_text": bool(
+                        pdf.anonymized_text and pdf.anonymized_text.strip()
+                    ),
+                    "is_validated": getattr(pdf.sensitive_meta, "is_verified", False)
+                    if pdf.sensitive_meta
+                    else False,
+                    "stream_url": request.build_absolute_uri(
+                        f"/api/media/pdfs/{pdf.pk}/stream/"
+                    ),
                 }
 
                 # Determine status based on anonymization and validation
@@ -265,11 +288,17 @@ class PdfMediaView(APIView):
             )
 
         except ValueError as e:
-            return Response({"error": f"Invalid query parameter: {str(e)}"}, status=status.HTTP_400_BAD_REQUEST)
+            return Response(
+                {"error": f"Invalid query parameter: {str(e)}"},
+                status=status.HTTP_400_BAD_REQUEST,
+            )
 
         except Exception as e:
-            logger.error(f"Unexpected error in PDF list view: {str(e)}")
-            return Response({"error": "Failed to retrieve PDF list"}, status=status.HTTP_500_INTERNAL_SERVER_ERROR)
+            logger.error(f"Unexpected error in report list view: {str(e)}")
+            return Response(
+                {"error": "Failed to retrieve report list"},
+                status=status.HTTP_500_INTERNAL_SERVER_ERROR,
+            )
 
     def _safe_get_file_size(self, file_field):
         """
@@ -292,7 +321,7 @@ class PdfMediaView(APIView):
 
     def _apply_filters(self, queryset, query_params):
         """
-        Apply status and other filters to PDF queryset.
+        Apply status and other filters to report queryset.
 
         Args:
             queryset: Base queryset to filter
@@ -306,15 +335,24 @@ class PdfMediaView(APIView):
         if status_filter:
             if status_filter == "not_started":
                 # PDFs without anonymized text
-                queryset = queryset.filter(Q(anonymized_text__isnull=True) | Q(anonymized_text__exact=""))
+                queryset = queryset.filter(
+                    Q(anonymized_text__isnull=True) | Q(anonymized_text__exact="")
+                )
             elif status_filter == "done":
                 # PDFs with anonymized text but not validated
                 queryset = queryset.filter(
-                    ~Q(anonymized_text__isnull=True), ~Q(anonymized_text__exact=""), Q(sensitive_meta__is_verified=False) | Q(sensitive_meta__isnull=True)
+                    ~Q(anonymized_text__isnull=True),
+                    ~Q(anonymized_text__exact=""),
+                    Q(sensitive_meta__is_verified=False)
+                    | Q(sensitive_meta__isnull=True),
                 )
             elif status_filter == "validated":
                 # PDFs with anonymized text and validated
-                queryset = queryset.filter(~Q(anonymized_text__isnull=True), ~Q(anonymized_text__exact=""), sensitive_meta__is_verified=True)
+                queryset = queryset.filter(
+                    ~Q(anonymized_text__isnull=True),
+                    ~Q(anonymized_text__exact=""),
+                    sensitive_meta__is_verified=True,
+                )
 
         return queryset
 
@@ -348,20 +386,26 @@ class PdfMediaView(APIView):
     # Future implementation placeholders
     def patch(self, request, pk):
         """
-        Update PDF metadata (Phase 1.2+ future enhancement).
+        Update report metadata (Phase 1.2+ future enhancement).
 
         Currently returns 501 Not Implemented.
         """
-        return Response({"error": "PDF metadata updates not yet implemented"}, status=status.HTTP_501_NOT_IMPLEMENTED)
+        return Response(
+            {"error": "report metadata updates not yet implemented"},
+            status=status.HTTP_501_NOT_IMPLEMENTED,
+        )
 
     def delete(self, request, pk):
         """
-        Delete PDF file (Phase 1.2+ future enhancement).
+        Delete report file (Phase 1.2+ future enhancement).
 
         Currently returns 501 Not Implemented.
         Use /api/media-management/force-remove/{id}/ instead.
         """
         return Response(
-            {"error": "PDF deletion not yet implemented", "alternative": f"Use DELETE /api/media-management/force-remove/{pk}/ instead"},
+            {
+                "error": "report deletion not yet implemented",
+                "alternative": f"Use DELETE /api/media-management/force-remove/{pk}/ instead",
+            },
             status=status.HTTP_501_NOT_IMPLEMENTED,
         )