Implement logic

Author: ahmedxgouda

backend/apps/owasp/api/v1/project_health_metrics.py

@@ -0,0 +1,30 @@
+"""API endpoint for OWASP project health metrics."""
+
+from pathlib import Path
+
+from django.conf import settings
+from django.core.management import call_command
+from django.http import FileResponse
+from ninja import Router
+from ninja.errors import HttpError
+
+from apps.owasp.models.project_health_metrics import ProjectHealthMetrics
+
+router = Router()
+
+
+@router.get("/{project_key}/pdf")
+def get_project_health_metrics_pdf(request, project_key: str):
+    """Generate and return a PDF report of project health metrics."""
+    if not ProjectHealthMetrics.objects.filter(project__key=f"www-project-{project_key}").exists():
+        raise HttpError(404, f"No health metrics found for project '{project_key}'")
+
+    file_name = f"{project_key}_health_metrics_report.pdf"
+    pdf_path = Path(settings.BASE_DIR) / "reports" / file_name
+    call_command(
+        "owasp_generate_project_health_metrics_pdf",
+        project_key=project_key,
+    )
+    if not pdf_path.exists():
+        raise HttpError(404, f"PDF report for project '{project_key}' not found")
+    return FileResponse(Path.open(pdf_path, "rb"), as_attachment=True, filename=file_name)

backend/apps/owasp/api/v1/urls.py

@@ -6,10 +6,12 @@
 from apps.owasp.api.v1.committee import router as committee_router
 from apps.owasp.api.v1.event import router as event_router
 from apps.owasp.api.v1.project import router as project_router
+from apps.owasp.api.v1.project_health_metrics import router as project_health_metrics_router
 
 router = Router()
 
 router.add_router(r"/chapters", chapter_router)
 router.add_router(r"/committees", committee_router)
 router.add_router(r"/events", event_router)
 router.add_router(r"/projects", project_router)
+router.add_router(r"/project-health-metrics", project_health_metrics_router)

frontend/src/app/projects/dashboard/metrics/[projectKey]/page.tsx

@@ -63,8 +63,8 @@ const ProjectHealthMetricsDetails: FC = () => {
               color="primary"
               onPress={async () => {
                 await fetchMetricsPDF(
-                  `owasp/projects-health-metrics/${projectKey}/pdf`,
-                  `${metrics.projectName}-health-metrics.pdf`
+                  `owasp/project-health-metrics/${projectKey}/pdf`,
+                  `${projectKey}-health-metrics.pdf`
                 )
               }}
             >

frontend/src/server/fetchMetricsPDF.ts

@@ -9,12 +9,14 @@ export const fetchMetricsPDF = async (path: string, fileName: string): Promise<v
     },
   })
   if (!response.ok) {
-    throw new Error(`Failed to fetch PDF: ${response.statusText}`)
+    handleAppError(new Error(`Failed to fetch PDF: ${response.status} ${response.statusText}`))
+    return
   }
   try {
     const blob = await response.blob()
     if (!blob) {
-      throw new Error('No PDF data received')
+      handleAppError(new Error('No data received from the server'))
+      return
     }
     const url = window.URL.createObjectURL(blob)
     const link = document.createElement('a')
@@ -23,6 +25,7 @@ export const fetchMetricsPDF = async (path: string, fileName: string): Promise<v
     document.body.appendChild(link)
     link.click()
     document.body.removeChild(link)
+    window.URL.revokeObjectURL(url)
   } catch (error) {
     handleAppError(error)
   }