Uploaded image for project: 'mod-orders'
  1. mod-orders
  2. MODORDERS-511

Do not allow changing the quantity or location of the POL, if the order already opened

    XMLWordPrintable

    Details

    • Template:
    • Sprint:
      ACQ Sprint 115
    • Story Points:
      2
    • Development Team:
      Thunderjet

      Description

      Purpose/Overview:
      Do not allow changing the quantity or location of the POL, if the order already opened. Please take a look logic in the Receiving with Holdings ID.png

      Requirements/Scope:

      If quantity or location of the POL was changed for Opened order with "Manually add pieces for receiving" = False, then throw org.folio.orders.rest.exceptions.HttpException

       Use Cases:

       Given : Opened order 

      When : User change location in POL when "Manually add pieces for receiving" = False**

      Then : Return Http response with code 400 and message "Please use the receiving App to update pieces and locations."

       

       Given : Opened order 

      When : User change quantity of the physical or electronic in POL when  "Manually add pieces for receiving" = False

      Then : Return Http response with code 400 and message "Please use the receiving App to update pieces and locations."

      Approach:

      Update logic org.folio.helper.PurchaseOrderLineHelper#updateOrderLine(...) :

      1. Replace method call org.folio.helper.PurchaseOrderLineHelper#checkLocationsAndPiecesConsistency  with new method with validation

      Acceptance criteria:

      • Logic updated
      • Unit tests
      • API Tests updated or created

        TestRail: Results

          Attachments

          1. image-2021-06-01-10-09-22-433.png
            19 kB
            Yauheni Kuzmianok
          2. Receiving with Holdings ID.png
            500 kB
            Andrei Makaranka
          3. screenshot-1.png
            88 kB
            Andrei Makaranka

            Issue Links

              Activity

                People

                Assignee:
                Andrei_Makaranka Andrei Makaranka
                Reporter:
                Andrei_Makaranka Andrei Makaranka
                Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                  Dates

                  Created:
                  Updated:
                  Resolved:

                    TestRail: Runs

                      TestRail: Cases