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

Quote to order - manual adjustments amount not included in cart grand total

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • Trunk
    • 14.12.01, 12.04.06, 13.07.02, 16.11.01
    • order
    • None
    • Ubuntu 8.10, Firefox 3.0.5Quote to order - manual adjustments amount not included in cart grand total

    • Bug Crush Event - 21/2/2015, Community Day 1 - 2015

    Description

      While creating a order from a quote with a manual adjustment of shipping charges, the adjustment amount is not included in the cart grand total. Although manual adjustments are copied to order adjustment and added to cart adjustment (ShoppingCartService.java, lines 676-781), they are filtered out while calculating the maxAmount for OrderPaymentPreference when calling the cart.getOrderOtherAdjustmentTotal() method which in turns calls the OrderReadHelper.calcOrderAdjustments(List orderHeaderAdjustments, BigDecimal subTotal, boolean includeOther, boolean includeTax, boolean includeShipping). This method filters out the SHIPPING_CHARGES order adjustment as includeShipping parameter is always true.

      One way to solve the problem is to use the value of includeInShipping field from OrderAdjsutment entity in OrderHelper#filterOrderAdjustments(List adjustments, boolean includeOther, boolean includeTax, boolean includeShipping, boolean forTax, boolean forShipping) method and setting this to true in QuoteAdjustment entity. (quote adjustments are copied to order adjustment by loadCartFromQuote service). However, doing so wrongly shows the shipping charges twice in the Order view and also add the shipping charges to the total tax whereas the grand total remains correct.

      Attachments

        1. order.diff
          4 kB
          Bilgin Ismet Ibryam
        2. OFBIZ-2146.patch
          4 kB
          Arun Patidar

        Activity

          People

            deepak Deepak Dixit
            rajsaini Raj Saini
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: