Details

    • Type: Sub-task Sub-task
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: Release Branch 09.04, Trunk
    • Fix Version/s: None
    • Component/s: order
    • Labels:
      None

      Description

      if you create a sales order for quantity 5 of a product, and then partially pack and ship 2 of this item, then go back and cancel all the item, the quantity canceled its 5 but should be 3 because 2 had already been shipped. See enclosed screenshot.

      1. cancel quantitys 4.0.patch
        5 kB
        Albert Mayo
      2. jira-3126.patch
        6 kB
        Wickersheimer Jeremy
      3. Picture 9.png
        95 kB
        Si Chen

        Activity

        Hide
        Wickersheimer Jeremy added a comment -

        This fixes two issues:
        cancelOrderItem was not accounting for the issued quantity, the patched version only canceled the remaining quantity
        reserveInventory was not accounting for the issued quantity, the patched version only re reserve the remaining quantity

        Show
        Wickersheimer Jeremy added a comment - This fixes two issues: cancelOrderItem was not accounting for the issued quantity, the patched version only canceled the remaining quantity reserveInventory was not accounting for the issued quantity, the patched version only re reserve the remaining quantity
        Hide
        Scott Gray added a comment -

        Was this tested against the trunk? It doesn't compile.

        Show
        Scott Gray added a comment - Was this tested against the trunk? It doesn't compile.
        Hide
        Albert Mayo added a comment -

        By the way, I believe I had this same issue in release 4.0.

        Show
        Albert Mayo added a comment - By the way, I believe I had this same issue in release 4.0.
        Hide
        Albert Mayo added a comment -

        I cannot officially contribute, but hopefully the attached patch helps whoever takes it on. The patch is from my own local SVN repository so ignore the revision numbers, but the modifications were done on an OFBiz 4.0 release.

        Show
        Albert Mayo added a comment - I cannot officially contribute, but hopefully the attached patch helps whoever takes it on. The patch is from my own local SVN repository so ignore the revision numbers, but the modifications were done on an OFBiz 4.0 release.
        Hide
        Adrian Crum added a comment -

        Albert,

        We appreciate your help, but a patch that references a different repository will be unusable. It would help us if you could check out the R 4.0 version, copy your modified file(s) over to it, and then create a patch.

        Show
        Adrian Crum added a comment - Albert, We appreciate your help, but a patch that references a different repository will be unusable. It would help us if you could check out the R 4.0 version, copy your modified file(s) over to it, and then create a patch.

          People

          • Assignee:
            Unassigned
            Reporter:
            Si Chen
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:

              Development