Created attachment 372707 [details] Screenshot of reconciliation report with error. The reconciliation report contains incorrect transactions. Please see attached for a screenshot of a reconciliation report from 2018-04-01 to 2018-04-30. One transaction highlighted belong to the next month reconciliation as can be seen from the 2018-05-31 reconciled date. There is one other transaction that should be on the report and is not present. The transaction date is 2018-03-22 but was reconciled on 2018-04-30. Hope this helps.
Hello The first thought is that this is by design... the date filter is, by default, applied to the posted date only. However I think that applying date filter to the reconciled date wouldn't be too difficult. The filter would need to include all splits whereby the reconciled date falls within specified date, and also include splits whereby reconciled date doesn't exist.
try https://raw.githubusercontent.com/christopherlam/gnucash/maint-fix-796614/gnucash/report/standard-reports/transaction.scm overwriting the same filename somewhere in c:\program files\gnucash folders
I downloaded the file transaction.scm file and copied it to C:\Program Files (x86)\gnucash\share\gnucash\scm\gnucash\report\standard-reports folder. The following error message was displayed after running the reconciliation report. Report error An error occurred wile running the report.
please try again; if still fails please include tracefile https://wiki.gnucash.org/wiki/Tracefile
Created attachment 372724 [details] Normal trace
Created attachment 372725 [details] --debug trace
Created attachment 372726 [details] --debug --extra trace
I downloaded the transaction.scm file again and no change. I attached the following tracefiles: Normal trace: gnucash.trace.043NKZ.log --debug: gnucash.trace.15W1KZ.log --debub --extra gnucash.trace.4QN1KZ.log
The missing function is now in https://raw.githubusercontent.com/Gnucash/gnucash/maint/gnucash/report/report-system/html-utilities.scm As before, replace html-utilities.scm in your c:\program files\gnucash folder...
Created attachment 372730 [details] Normal trace2
I downloaded the html-utilities.scm and copied it to the C:\Program Files (x86)\gnucash\share\gnucash\scm folder. I am getting the same error. I uploaded the following trace file for review: gnucash.trace.SBW3KZ.log
Created attachment 372739 [details] transaction.scm special build this is a crafted transaction.scm - will not be applied to code. this contains gnc:render-options-changed inlined.
Created attachment 372749 [details] Normal trace3 The same report error observed, tracefile attached.
Created attachment 372750 [details] render-options-changed and addto! inlined too
This version worked! I was able to run the report for a one month period and the reconciled transactions now match! Great Job Chris! The report looks good with "Reconciled" selected in Report Options->Filter->Reconcile Status However, when "All" or "Unreconciled" is selected for report options, every transaction in the database is displayed under the Unreconciled section of the report and I think this is undesirable. I think the Unreconciled section should only contain transaction that were not reconciled before the reporting end period as in the original report.
I can't reproduce this one. Make sure you test your sorting/filtering options. The changes to this bug involve changing the split-date filter from Posted-Date to Reconciled-Date. I'd think this can now be reviewed.
I'm actually thinking that changing the date filtering strategy from posted-date to reconcile-date isn't such a great idea... the reconcile report displays the running balance by default (to mimic bank statement balance); this balance shows the running balance sorted by posted-date and will be completely jumbled up when sorting by reconcile-date. So, I think either remove running balance completely, or do not merge this change.
Created attachment 372778 [details] Reconciliation screenshot Remove the running balance because it is not necessary and unwanted. The reconciliation report should mimic the attached screenshot and ideally include the following: Date Num Description Amount Reconciled Total income Total expenses Statement Date Starting Balance Ending Balance Reconciled Balance Difference A reconciliation report should also contain the number of cleared transactions to quickly compare to the bank statement (used to quickly find errors) The attached is and example of information I have to submit to an auditor for verification of a good reconciliation. It would be my preference to produce a report in lieu of screenshots.
Most of the desired information can be gleaned from the Reconcilation Report. Running Balances can & will be disabled if sorting by reconciled date. So I'd submit the PR as is. The other fields are rather too specialised to calculate for now, and would require too complex change of transaction.scm
https://github.com/Gnucash/gnucash/pull/376 Addendum the one change I've added is the 'funds in' and 'funds out' can be offered, with their own subtotals. But the other fields are rather difficult.
will be fixed in 3.3
Relevant: I have started a new ticket bug 797772 that impacts this ticket.
Strongly considering reverting this change. Reasoning: the reconciled_date is a very inconsistently used field -- it may refer to the statement_date of reconciliation, or $today when downloading from AqBanking. Many users have resorted to *not* rely on this date to consider reconciliation data; they merely use the reconciled_status field. Would any major devs or OP object (strongly) to reverting this change? The OP request may be resolved as described in bug 797772 - sort by reconciled_status then sort by reconciled_date. If (any only if) a separate sorting via reconciled_date is required, it *could* be implemented via option: [x] Date filters on reconciled date rather than posted date.
Chris I agree with your points there, and your enhancement option suggestion.
