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

Error on Accounting reports for Cash transactions.

    Details

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

      Description

      An error is displayed on Accounting reports for Cash transactions:
      Error running script at location component://accounting/webapp/accounting/WEB-INF/actions/reports/CashFlowStatement.groovy: java.lang.IllegalArgumentException: argument type mismatch

      How to reproduce: create a new cash transaction and go to the Cash Flow Statement report.
      eg https://demo-trunk-ofbiz.apache.org/accounting/control/CashFlowStatement?organizationPartyId=Company

      The attached patch addresses the issue. I believe this is related to OFBIZ-5781.

      1. OFBIZ-6765.patch
        1.0 kB
        Francis Douet

        Activity

        Hide
        pfm.smits Pierre Smits added a comment -

        Hi Francis Douet,

        Having tested your patch I see that the error won't occur when I apply the change regarding the first part of your patch (related to CashFlowStatement.groovy), however I don't see what the effect of the second part of the patch (related to ReportFinancialSummaryScreens.xml) is on the cash flow statement.

        Did that slip in by accident?

        Show
        pfm.smits Pierre Smits added a comment - Hi Francis Douet , Having tested your patch I see that the error won't occur when I apply the change regarding the first part of your patch (related to CashFlowStatement.groovy), however I don't see what the effect of the second part of the patch (related to ReportFinancialSummaryScreens.xml) is on the cash flow statement. Did that slip in by accident?
        Hide
        Francis Douet Francis Douet added a comment - - edited

        Hi Pierre, yes the second part isn't directly related to the original issue, I just thought killing two birds with one stone, and fix another issue we had when integrating the OrganizationAccountingReportsTabBar menu into our own screen.

        Edit: after having another look our screen, it's a problem on our end, not ReportFinancialSummaryScreens, I fixed the patch.

        Show
        Francis Douet Francis Douet added a comment - - edited Hi Pierre, yes the second part isn't directly related to the original issue, I just thought killing two birds with one stone, and fix another issue we had when integrating the OrganizationAccountingReportsTabBar menu into our own screen. Edit: after having another look our screen, it's a problem on our end, not ReportFinancialSummaryScreens, I fixed the patch.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Thanks Francis, Pierre,

        Fixed at revision: 1719857

        Show
        jacques.le.roux Jacques Le Roux added a comment - Thanks Francis, Pierre, Fixed at revision: 1719857

          People

          • Assignee:
            jacques.le.roux Jacques Le Roux
            Reporter:
            Francis Douet Francis Douet
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development