1879 edit operation

bc_obps/common/management/commands/load_fixtures.py

@@ -47,6 +47,7 @@ def handle(self, *args, **options):
                 f'{fixture_base_dir}/transfer_event.json',
                 f'{fixture_base_dir}/parent_operator.json',
                 f'{fixture_base_dir}/partner_operator.json',
+                f'{fixture_base_dir}/registration_purpose.json',
             ]
 
         for fixture in fixtures:

bc_obps/registration/api/v2/_operations/_operation_id/_registration/operation.py

@@ -6,18 +6,16 @@
     OperationInformationIn,
     OperationUpdateOut,
     OperationRegistrationSubmissionIn,
-    OperationStatutoryDeclarationIn,
 )
 from service.operation_service_v2 import OperationServiceV2
-from registration.constants import OPERATION_TAGS, V2
+from registration.constants import V2
 from common.permissions import authorize
 from registration.api.utils.current_user_utils import get_current_user_guid
 from service.error_service.custom_codes_4xx import custom_codes_4xx
 from registration.models import Operation
 from registration.decorators import handle_http_errors
 from registration.api.router import router
 from registration.schema.generic import Message
-from ninja.responses import codes_4xx
 
 
 ##### PUT #####
@@ -38,22 +36,6 @@ def register_edit_operation_information(
     return 200, OperationServiceV2.register_operation_information(get_current_user_guid(request), operation_id, payload)
 
 
-@router.put(
-    "/v2/operations/{operation_id}/registration/statutory-declaration",
-    response={200: OperationUpdateOut, codes_4xx: Message},
-    tags=OPERATION_TAGS,
-    description="Creates or replaces a statutory declaration document for an Operation",
-    auth=authorize("approved_industry_user"),
-)
-@handle_http_errors()
-def create_or_replace_statutory_declarations(
-    request: HttpRequest, operation_id: UUID, payload: OperationStatutoryDeclarationIn
-) -> Tuple[Literal[200], Operation]:
-    return 200, OperationServiceV2.create_or_replace_statutory_declaration(
-        get_current_user_guid(request), operation_id, payload
-    )
-
-
 @router.patch(
     "/v2/operations/{uuid:operation_id}/registration/submission",
     response={200: OperationUpdateStatusOut, custom_codes_4xx: Message},

bc_obps/registration/api/v2/_operations/operation_id.py

@@ -1,10 +1,11 @@
 from typing import Literal, Tuple
 from uuid import UUID
-from registration.schema.v2.operation import OperationOutV2
+from registration.schema.v2.operation import OperationOutV2, OperationInformationIn, OperationOutWithDocuments
 from common.permissions import authorize
 from django.http import HttpRequest
 from registration.constants import OPERATION_TAGS
 from service.operation_service import OperationService
+from service.operation_service_v2 import OperationServiceV2
 from registration.api.utils.current_user_utils import get_current_user_guid
 from registration.decorators import handle_http_errors
 from registration.api.router import router
@@ -29,3 +30,33 @@
 @handle_http_errors()
 def get_operation_v2(request: HttpRequest, operation_id: UUID) -> Tuple[Literal[200], Operation]:
     return 200, OperationService.get_if_authorized(get_current_user_guid(request), operation_id)
+
+
+@router.get(
+    "/v2/operations/{uuid:operation_id}/with-documents",
+    response={200: OperationOutWithDocuments, codes_4xx: Message},
+    tags=OPERATION_TAGS,
+    description="""Retrieves the details of a specific operation by its ID along with it's documents""",
+    exclude_none=True,
+    auth=authorize("approved_authorized_roles"),
+)
+@handle_http_errors()
+def get_operation_with_documents(request: HttpRequest, operation_id: UUID) -> Tuple[Literal[200], Operation]:
+    return 200, OperationService.get_if_authorized(get_current_user_guid(request), operation_id)
+
+
+##### PUT ######
+
+
+@router.put(
+    "/v2/operations/{uuid:operation_id}",
+    response={200: OperationOutV2, codes_4xx: Message},
+    tags=OPERATION_TAGS,
+    description="Updates the details of a specific operation by its ID.",
+    auth=authorize("approved_industry_user"),
+)
+@handle_http_errors()
+def update_operation_v2(
+    request: HttpRequest, operation_id: UUID, payload: OperationInformationIn
+) -> Tuple[Literal[200], Operation]:
+    return 200, OperationServiceV2.update_operation(get_current_user_guid(request), payload, operation_id)

bc_obps/registration/fixtures/mock/operation.json

@@ -243,6 +243,7 @@
   },
   {
     "model": "registration.operation",
+    "pk": "df62d793-8cfe-4272-a93e-ea9c9139ff82",
     "fields": {
       "operator": "4242ea9d-b917-4129-93c2-db00b7451051",
       "documents": [],
@@ -260,6 +261,7 @@
   },
   {
     "model": "registration.operation",
+    "pk": "c5b3643b-c143-42f3-8a2b-03ccc7319cd9",
     "fields": {
       "operator": "4242ea9d-b917-4129-93c2-db00b7451051",
       "documents": [],
@@ -278,6 +280,7 @@
   },
   {
     "model": "registration.operation",
+    "pk": "954c0382-ff61-4e87-a8a0-873586534b54",
     "fields": {
       "operator": "4242ea9d-b917-4129-93c2-db00b7451051",
       "documents": [],
@@ -296,6 +299,7 @@
   },
   {
     "model": "registration.operation",
+    "pk": "6d07d02a-1ad2-46ed-ad56-2f84313e98bf",
     "fields": {
       "operator": "4242ea9d-b917-4129-93c2-db00b7451051",
       "documents": [],
@@ -314,6 +318,7 @@
   },
   {
     "model": "registration.operation",
+    "pk": "59d95661-c752-489b-9fd1-0c3fa3454dda",
     "fields": {
       "operator": "4242ea9d-b917-4129-93c2-db00b7451051",
       "documents": [],
@@ -332,6 +337,7 @@
   },
   {
     "model": "registration.operation",
+    "pk": "1bd04128-d070-4d3a-940a-0874c4956181",
     "fields": {
       "operator": "4242ea9d-b917-4129-93c2-db00b7451051",
       "documents": [],
@@ -350,6 +356,7 @@
   },
   {
     "model": "registration.operation",
+    "pk": "17f13f4d-29b4-45f4-b025-b21f2e126771",
     "fields": {
       "operator": "4242ea9d-b917-4129-93c2-db00b7451051",
       "documents": [],
@@ -367,6 +374,7 @@
   },
   {
     "model": "registration.operation",
+    "pk": "ef9044dd-2a27-4d26-86fe-02e51e0755f7",
     "fields": {
       "operator": "4242ea9d-b917-4129-93c2-db00b7451051",
       "documents": [],
@@ -383,6 +391,7 @@
   },
   {
     "model": "registration.operation",
+    "pk": "aeeb781e-a97b-4ab2-9a6e-02e4522add1a",
     "fields": {
       "operator": "4242ea9d-b917-4129-93c2-db00b7451051",
       "documents": [],
@@ -399,6 +408,7 @@
   },
   {
     "model": "registration.operation",
+    "pk": "02a3ab84-26c6-4a79-bf89-72f877ceef8e",
     "fields": {
       "operator": "4242ea9d-b917-4129-93c2-db00b7451051",
       "documents": [],
@@ -431,6 +441,7 @@
   },
   {
     "model": "registration.operation",
+    "pk": "0ac72fa9-2636-4f54-b378-af6b1a070787",
     "fields": {
       "operator": "4242ea9d-b917-4129-93c2-db00b7451051",
       "documents": [],

bc_obps/registration/fixtures/mock/registration_purpose.json

@@ -0,0 +1,170 @@
+[
+  {
+    "model": "registration.registrationpurpose",
+    "pk": 1,
+    "fields": {
+      "registration_purpose": "OBPS Regulated Operation",
+      "operation_id": "c0743c09-82fa-4186-91aa-4b5412e3415c"
+    }
+  },
+  {
+    "model": "registration.registrationpurpose",
+    "pk": 2,
+    "fields": {
+      "registration_purpose": "Potential Reporting Operation",
+      "operation_id": "c0743c09-82fa-4186-91aa-4b5412e3415c"
+    }
+  },
+  {
+    "model": "registration.registrationpurpose",
+    "pk": 3,
+    "fields": {
+      "registration_purpose": "OBPS Regulated Operation",
+      "operation_id": "1bd04128-d070-4d3a-940a-0874c4956181"
+    }
+  },
+  {
+    "model": "registration.registrationpurpose",
+    "pk": 4,
+    "fields": {
+      "registration_purpose": "Reporting Operation",
+      "operation_id": "1bd04128-d070-4d3a-940a-0874c4956181"
+    }
+  },
+  {
+    "model": "registration.registrationpurpose",
+    "pk": 5,
+    "fields": {
+      "registration_purpose": "New Entrant Operation",
+      "operation_id": "1bd04128-d070-4d3a-940a-0874c4956181"
+    }
+  },
+  {
+    "model": "registration.registrationpurpose",
+    "pk": 6,
+    "fields": {
+      "registration_purpose": "OBPS Regulated Operation",
+      "operation_id": "3b5b95ea-2a1a-450d-8e2e-2e15feed96c9"
+    }
+  },
+  {
+    "model": "registration.registrationpurpose",
+    "pk": 7,
+    "fields": {
+      "registration_purpose": "Reporting Operation",
+      "operation_id": "3b5b95ea-2a1a-450d-8e2e-2e15feed96c9"
+    }
+  },
+  {
+    "model": "registration.registrationpurpose",
+    "pk": 8,
+    "fields": {
+      "registration_purpose": "Electricity Import Operation",
+      "operation_id": "3b5b95ea-2a1a-450d-8e2e-2e15feed96c9"
+    }
+  },
+  {
+    "model": "registration.registrationpurpose",
+    "pk": 9,
+    "fields": {
+      "registration_purpose": "OBPS Regulated Operation",
+      "operation_id": "17f13f4d-29b4-45f4-b025-b21f2e126771"
+    }
+  },
+  {
+    "model": "registration.registrationpurpose",
+    "pk": 10,
+    "fields": {
+      "registration_purpose": "Reporting Operation",
+      "operation_id": "1bd04128-d070-4d3a-940a-0874c4956181"
+    }
+  },
+  {
+    "model": "registration.registrationpurpose",
+    "pk": 11,
+    "fields": {
+      "registration_purpose": "OBPS Regulated Operation",
+      "operation_id": "aeeb781e-a97b-4ab2-9a6e-02e4522add1a"
+    }
+  },
+  {
+    "model": "registration.registrationpurpose",
+    "pk": 12,
+    "fields": {
+      "registration_purpose": "Reporting Operation",
+      "operation_id": "02a3ab84-26c6-4a79-bf89-72f877ceef8e"
+    }
+  },
+  {
+    "model": "registration.registrationpurpose",
+    "pk": 13,
+    "fields": {
+      "registration_purpose": "OBPS Regulated Operation",
+      "operation_id": "0ac72fa9-2636-4f54-b378-af6b1a070787"
+    }
+  },
+  {
+    "model": "registration.registrationpurpose",
+    "pk": 14,
+    "fields": {
+      "registration_purpose": "Reporting Operation",
+      "operation_id": "df62d793-8cfe-4272-a93e-ea9c9139ff82"
+    }
+  },
+  {
+    "model": "registration.registrationpurpose",
+    "pk": 15,
+    "fields": {
+      "registration_purpose": "OBPS Regulated Operation",
+      "operation_id": "c5b3643b-c143-42f3-8a2b-03ccc7319cd9"
+    }
+  },
+  {
+    "model": "registration.registrationpurpose",
+    "pk": 16,
+    "fields": {
+      "registration_purpose": "OBPS Regulated Operation",
+      "operation_id": "954c0382-ff61-4e87-a8a0-873586534b54"
+    }
+  },
+  {
+    "model": "registration.registrationpurpose",
+    "pk": 14,
+    "fields": {
+      "registration_purpose": "Reporting Operation",
+      "operation_id": "ef9044dd-2a27-4d26-86fe-02e51e0755f7"
+    }
+  },
+  {
+    "model": "registration.registrationpurpose",
+    "pk": 15,
+    "fields": {
+      "registration_purpose": "Reporting Operation",
+      "operation_id": "59d95661-c752-489b-9fd1-0c3fa3454dda"
+    }
+  },
+  {
+    "model": "registration.registrationpurpose",
+    "pk": 14,
+    "fields": {
+      "registration_purpose": "Reporting Operation",
+      "operation_id": "6d07d02a-1ad2-46ed-ad56-2f84313e98bf"
+    }
+  },
+  {
+    "model": "registration.registrationpurpose",
+    "pk": 15,
+    "fields": {
+      "registration_purpose": "Potential Reporting Operation",
+      "operation_id": "df62d793-8cfe-4272-a93e-ea9c9139ff82"
+    }
+  },
+  {
+    "model": "registration.registrationpurpose",
+    "pk": 16,
+    "fields": {
+      "registration_purpose": "Electricity Import Operation",
+      "operation_id": "c5b3643b-c143-42f3-8a2b-03ccc7319cd9"
+    }
+  }
+]

bc_obps/registration/models/operation.py

@@ -191,6 +191,27 @@ def get_statutory_declaration(self) -> Optional[Document]:
             .first()
         )  # filter returns a queryset, so we use .first() to get the single record (there will only ever be one statutory declaration per operation)
 
+    def get_boundary_map(self) -> Optional[Document]:
+        """
+        Returns the boundary map document associated with the operation.
+        """
+
+        return self.documents.filter(type=DocumentType.objects.get(name="boundary_map")).only('file').first()
+
+    def get_process_flow_diagram(self) -> Optional[Document]:
+        """
+        Returns the process flow diagram document associated with the operation.
+        """
+
+        return self.documents.filter(type=DocumentType.objects.get(name="process_flow_diagram")).only('file').first()
+
+    def get_equipment_list(self) -> Optional[Document]:
+        """
+        Returns the equipment list document associated with the operation.
+        """
+
+        return self.documents.filter(type=DocumentType.objects.get(name="equipment_list")).only('file').first()
+
     def user_has_access(self, user_guid: UUID) -> bool:
         """
         Returns whether a user has access to the operation.