After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 796614 - Reconciliation report contains incorrect transactions
Reconciliation report contains incorrect transactions
Status: RESOLVED OBSOLETE
Product: GnuCash
Classification: Other
Component: Reports
3.1
Other Windows
: Normal normal
: future
Assigned To: gnucash-reports-maint
gnucash-reports-maint
Depends on:
Blocks:
 
 
Reported: 2018-06-18 00:37 UTC by troysmith31
Modified: 2018-06-30 00:12 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Screenshot of reconciliation report with error. (52.87 KB, image/png)
2018-06-18 00:37 UTC, troysmith31
Details
Normal trace (1.54 KB, text/plain)
2018-06-19 19:38 UTC, troysmith31
Details
--debug trace (472.05 KB, text/plain)
2018-06-19 19:39 UTC, troysmith31
Details
--debug --extra trace (438.84 KB, text/plain)
2018-06-19 19:39 UTC, troysmith31
Details
Normal trace2 (1.54 KB, text/plain)
2018-06-20 00:49 UTC, troysmith31
Details
transaction.scm special build (100.55 KB, text/plain)
2018-06-21 05:14 UTC, Chris
Details
Normal trace3 (1.75 KB, text/plain)
2018-06-22 01:37 UTC, troysmith31
Details
render-options-changed and addto! inlined too (102.75 KB, application/octet-stream)
2018-06-22 01:42 UTC, Chris
Details
Reconciliation screenshot (135.71 KB, image/jpeg)
2018-06-22 20:11 UTC, troysmith31
Details

Description troysmith31 2018-06-18 00:37:13 UTC
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.
Comment 1 Chris 2018-06-18 02:20:28 UTC
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.
Comment 2 Chris 2018-06-18 05:59:14 UTC
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
Comment 3 troysmith31 2018-06-18 19:19:50 UTC
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.
Comment 4 Chris 2018-06-19 10:31:48 UTC
please try again; if still fails please include tracefile
https://wiki.gnucash.org/wiki/Tracefile
Comment 5 troysmith31 2018-06-19 19:38:20 UTC
Created attachment 372724 [details]
Normal trace
Comment 6 troysmith31 2018-06-19 19:39:19 UTC
Created attachment 372725 [details]
--debug trace
Comment 7 troysmith31 2018-06-19 19:39:51 UTC
Created attachment 372726 [details]
--debug --extra trace
Comment 8 troysmith31 2018-06-19 19:41:07 UTC
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
Comment 9 Chris 2018-06-19 23:29:10 UTC
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...
Comment 10 troysmith31 2018-06-20 00:49:27 UTC
Created attachment 372730 [details]
Normal trace2
Comment 11 troysmith31 2018-06-20 00:50:00 UTC
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
Comment 12 Chris 2018-06-21 05:14:01 UTC
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.
Comment 13 troysmith31 2018-06-22 01:37:21 UTC
Created attachment 372749 [details]
Normal trace3

The same report error observed, tracefile attached.
Comment 14 Chris 2018-06-22 01:42:42 UTC
Created attachment 372750 [details]
render-options-changed and addto! inlined too
Comment 15 troysmith31 2018-06-22 02:12:37 UTC
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.
Comment 16 Chris 2018-06-22 04:48:40 UTC
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.
Comment 17 Chris 2018-06-22 07:32:05 UTC
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.
Comment 18 troysmith31 2018-06-22 20:11:07 UTC
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.
Comment 19 Chris 2018-06-23 10:07:04 UTC
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
Comment 20 Chris 2018-06-23 13:08:47 UTC
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.
Comment 21 John Ralls 2018-06-30 00:12:04 UTC
GnuCash bug tracking has moved to a new Bugzilla host. The new URL for this bug is https://bugs.gnucash.org/show_bug.cgi?id=796614. Please continue processing the bug there and please update any external references or bookmarks.