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

Updates to poLine Cost

    XMLWordPrintable

Details

    • Story
    • Status: Closed (View Workflow)
    • P3
    • Resolution: Done
    • None
    • 4.0.0
    • None
    • ACQ Sprint 59
    • 3

    Description

      Overview:
      Adjustment information is being moved out of orders and into invoicing. This means we can remove adjustments from poLine. At the same time, we need to be able to specify separate list prices for physical and electronic. Here's a rundown of the changes required:

      • Remove poLine.adjustments
      • Rename cost.listPrice to cost.listUnitPrice
      • Add cost.listUnitPriceElectronic
      • Add cost.discount
      • Add cost.discountType - "percentage" or "amount" (default to "percentage")
      • Add cost.additionalCost - an amount
      • cost.poLineEstimatedPrice is calculated as follows:
        baseListPrice = (listUnitPrice * quantityPhysical) + (listUnitPriceElectronic * quantityElectronic)
        discountAmount = discountType == PERCENTAGE ? (baseListPrice * discount / 100) : discount
        poLineEstimatedPrice = baseListPrice + additonalCost - discountAmount
        
      • N.B. additionalCost is NOT factored into discount calculations
      • NOTE: The estimated price is always calculated by mod-orders upon creation/update, and stored... even if the estimated price is populated in the request.

      see UIOR-142

      Acceptance Criteria:

      • Schemas and examples are updated as described above
      • Unit and API tests have been updated as needed
      • poLineEstimatedPrice calculation has been updated

      TestRail: Results

        Attachments

          1. AddPoLineToEmptyOrder.gif
            AddPoLineToEmptyOrder.gif
            525 kB
          2. AddSecondLine.gif
            AddSecondLine.gif
            1.48 MB
          3. ApiTests_CostUpdates.png
            ApiTests_CostUpdates.png
            76 kB

          Issue Links

            Activity

              People

                cmcnally Craig McNally
                cmcnally Craig McNally
                Craig McNally Craig McNally
                Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved:

                  TestRail: Runs

                    TestRail: Cases