Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: Release Branch 11.04, Release Branch 12.04, Release Branch 13.07, Trunk
    • Fix Version/s: 16.11.01
    • Component/s: order
    • Labels:
      None
    • Sprint:
      Bug Crush Event - 21/2/2015

      Description

      If user partially received an order and then cancel order item, system cancel all the order item quantity.System does not check for shipped/received order item quantity.
      Expected behavior: Only remaining order item quantity should be cancelled and order item should marked as complete with partial received/shipped quantity.

      Steps to regenerate:

      • Crate an SO/PO .
      • Partially ship/received order item quantity.
      • Edit order and cancel the order item.
      • System should cancel the remaining quantity not ordered quantity and should mark order item as Complete because partial order item quantity has been shipped/received.
        But as per the current behavior system cancel the ordered quantity and ignore the shipped/received order item quantity and marked order item as cancelled.

      Thoughts required.

      1. OFBIZ-4954-trunk-updated.patch
        10 kB
        Yashwant Dhakad
      2. OFBIZ-4954-trunk.patch
        7 kB
        HotWax Media OFBiz Team
      3. OFBIZ-4954-RB1307.patch
        7 kB
        HotWax Media OFBiz Team

        Issue Links

          Activity

          Hide
          hwmofbizus HotWax Media OFBiz Team added a comment -

          Test Case:
          1. Create an SO/PO.
          2. Partially ship/received order item quantity.
          3. Edit order and cancel the order item.
          Actual Result: System cancel the ordered quantity and ignore the shipped/received order item quantity and marked order item as cancelled.
          Expected: System cancel the remaining order item quantity instead of canceling total order item quantity and mark order item as completed.

          Here is the patch for trunk and RB1307.

          Show
          hwmofbizus HotWax Media OFBiz Team added a comment - Test Case: 1. Create an SO/PO. 2. Partially ship/received order item quantity. 3. Edit order and cancel the order item. Actual Result: System cancel the ordered quantity and ignore the shipped/received order item quantity and marked order item as cancelled. Expected: System cancel the remaining order item quantity instead of canceling total order item quantity and mark order item as completed. Here is the patch for trunk and RB1307.
          Hide
          jacques.le.roux Jacques Le Roux added a comment -

          Is there a reason why this was not committed?

          Show
          jacques.le.roux Jacques Le Roux added a comment - Is there a reason why this was not committed?
          Hide
          deepak.dixit Deepak Dixit added a comment - - edited

          Hi Jacques,

          There are some more issues still need to handle, I did not get a chance to look into those issues, will try to fix these issues ASAP and will commit the patch.

          Show
          deepak.dixit Deepak Dixit added a comment - - edited Hi Jacques, There are some more issues still need to handle, I did not get a chance to look into those issues, will try to fix these issues ASAP and will commit the patch.
          Hide
          jacques.le.roux Jacques Le Roux added a comment -

          Thanks for feedback Deepak

          Show
          jacques.le.roux Jacques Le Roux added a comment - Thanks for feedback Deepak
          Hide
          yashwant.dhakad Yashwant Dhakad added a comment - - edited

          Hi Deepak,

          I have verified some more flows and now it is working as expected and also updated the patch for trunk. Please review it and let me know if you need more changes.

          Show
          yashwant.dhakad Yashwant Dhakad added a comment - - edited Hi Deepak, I have verified some more flows and now it is working as expected and also updated the patch for trunk. Please review it and let me know if you need more changes.
          Hide
          deepak.dixit Deepak Dixit added a comment -

          Thanks Yashwant, Your patch has been committed at trunk r#1721093

          Show
          deepak.dixit Deepak Dixit added a comment - Thanks Yashwant, Your patch has been committed at trunk r#1721093
          Hide
          yashwant.dhakad Yashwant Dhakad added a comment -

          Thanks, Deepak.

          Show
          yashwant.dhakad Yashwant Dhakad added a comment - Thanks, Deepak.
          Hide
          jacques.le.roux Jacques Le Roux added a comment -

          Could this not be backported?

          Show
          jacques.le.roux Jacques Le Roux added a comment - Could this not be backported?
          Hide
          deepak.dixit Deepak Dixit added a comment -

          Yes Jacques, this is an bug fix and this should be backported to 14.12.
          I am getting conflicts, I'll backport it to 14.12 ASAP.

          Show
          deepak.dixit Deepak Dixit added a comment - Yes Jacques, this is an bug fix and this should be backported to 14.12. I am getting conflicts, I'll backport it to 14.12 ASAP.
          Hide
          deepak.dixit Deepak Dixit added a comment -

          This has been merged at 14.12 at r#1721875

          Show
          deepak.dixit Deepak Dixit added a comment - This has been merged at 14.12 at r#1721875
          Hide
          jacques.le.roux Jacques Le Roux added a comment -

          Thanks Deepak!

          Show
          jacques.le.roux Jacques Le Roux added a comment - Thanks Deepak!

            People

            • Assignee:
              deepak.dixit Deepak Dixit
              Reporter:
              deepak.dixit Deepak Dixit
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development

                  Agile