Uploaded image for project: 'OFBiz'
  1. OFBiz
  2. OFBIZ-5416

Sales order cart items already assigned to ship groups are incorrectly able to be modified to quantities/prices that break the ProductStoreShipmentMeth rules

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • Trunk
    • 14.12.01, 12.04.06, 13.07.02
    • order
    • None

    Description

      When modifying sales order cart items that already have been assigned to ship groups, no logic exists to check that the modifications follow the ProductStoreShipmentMeth quantity/price rules of the shipment method.

      To reproduce:
      1. Add a maxTot of 16 to productStoreShipmentMethId=9016 (FEDEX/Ground Home Delivery) on the Catalog->Store->Shipping page
      2. Create a sales order for DemoCustCompany
      3. Add 1 GZ-1000 to the cart and notice the total is under 16
      4. Finalize the cart until reaching the shipping options page
      5. Choose the "FedEx Home Delivery" option and click "Continue"
      6. Click "Cart Items" to return to the cart page
      7. Change the quantity of GZ-1000 to 2 and click "Recalculate Order"
      8. Notice the total price of the cart has exceeded the 16 but the shipment method is still associated with the ship group

      Logic needs to be added to do something such as remove the shipment methods from ship groups when modifications of associated order items violate their rules. This would cause the shipping options page to reappear for entry of required missing ship group shipment methods when the "Finalize" button is clicked.

      Another option would be to keep the shipment method and throw an error when modifications violate ship group rules.

      Attachments

        1. OFBIZ-5416.patch
          2 kB
          Arun Patidar

        Issue Links

          Activity

            People

              toashishvijay Ashish Vijaywargiya
              ofbizzer Christian Carlow
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: