OFBiz
  1. OFBiz
  2. OFBIZ-482

Invoice generation from order fails if there is a percent item adjustment

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: Trunk
    • Fix Version/s: None
    • Component/s: accounting
    • Labels:
      None

      Description

      Invoice generation from an order checks whether the current invoiced amount for an adjustment exceeds the adjustment, but only works if the adjustment is a flat amount - it fails if the adjustment is a percentage.

      The attached patch addresses this issue.

      1. invoice.pdf
        11 kB
        Deepak Dixit
      2. InvoiceServices.patch
        2 kB
        Iain Fogg

        Activity

        Hide
        Jacopo Cappellato added a comment -

        Si,

        did you already review/commit this patch? Can we close the issue?

        Show
        Jacopo Cappellato added a comment - Si, did you already review/commit this patch? Can we close the issue?
        Hide
        Si Chen added a comment -

        No, I don't think I looked at this yet.

        Show
        Si Chen added a comment - No, I don't think I looked at this yet.
        Hide
        Deepak Dixit added a comment -

        IMO We can close this task. Invoice generating properly for percent item adjustment as well.
        I created an order for product GZ-1005 ( Promotion 20% off any one item, either GZ-1005 (.NIT Gizmo) or GZ-1006 (Open Gizmo) with a limit of 1 per order ) and invoice generated properly.

        Show
        Deepak Dixit added a comment - IMO We can close this task. Invoice generating properly for percent item adjustment as well. I created an order for product GZ-1005 ( Promotion 20% off any one item, either GZ-1005 (.NIT Gizmo) or GZ-1006 (Open Gizmo) with a limit of 1 per order ) and invoice generated properly.
        Hide
        Deepak Dixit added a comment -

        I think this has been fixed at r#814130.

        Show
        Deepak Dixit added a comment - I think this has been fixed at r#814130.
        Hide
        Ashish Vijaywargiya added a comment -

        Based on Deepak's recent comment we can safely close this issue.

        Show
        Ashish Vijaywargiya added a comment - Based on Deepak's recent comment we can safely close this issue.

          People

          • Assignee:
            Ashish Vijaywargiya
            Reporter:
            Iain Fogg
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development