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

Best Selling Products section in main order page takes into account cancelled orders

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: Release Branch 12.04, Release Branch 13.07, Release Branch 14.12, Trunk
    • Fix Version/s: 14.12.01, 12.04.06, 13.07.03, 16.11.01
    • Component/s: order
    • Labels:
      None

      Description

      To test with a clean installation with only demo data and no action yet, simply
      create an order,
      cancel it
      you will see the product ordered in the Best Selling Products section

        Activity

        Hide
        adityapatwa Aditya Patwa added a comment -

        Added condition to exclude cancelled orders and cancelled order items in the logic for fetching the best selling products.

        Show
        adityapatwa Aditya Patwa added a comment - Added condition to exclude cancelled orders and cancelled order items in the logic for fetching the best selling products.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Hi Mridul Pathak we crossed on wire , I was planning to commit just now, please reassign your self

        Show
        jacques.le.roux Jacques Le Roux added a comment - Hi Mridul Pathak we crossed on wire , I was planning to commit just now, please reassign your self
        Hide
        mridul.pathak Mridul Pathak added a comment -

        Sure Jacques

        Show
        mridul.pathak Mridul Pathak added a comment - Sure Jacques
        Hide
        mridul.pathak Mridul Pathak added a comment -

        The patch is now in,

        • Trunk r1715471.
        • 14.12 r1715472.
        • 13.07 r1715473 with some minor changes.
        • 12.04 r1715474 with some minor changes.
        Show
        mridul.pathak Mridul Pathak added a comment - The patch is now in, Trunk r1715471. 14.12 r1715472. 13.07 r1715473 with some minor changes. 12.04 r1715474 with some minor changes.
        Hide
        mridul.pathak Mridul Pathak added a comment - - edited

        Thanks Jacques for reporting the issue and Aditya Patwa for providing the patch.

        Show
        mridul.pathak Mridul Pathak added a comment - - edited Thanks Jacques for reporting the issue and Aditya Patwa for providing the patch.

          People

          • Assignee:
            mridul.pathak Mridul Pathak
            Reporter:
            jacques.le.roux Jacques Le Roux
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development