GNOME Bugzilla – Bug 696152
Register to Treeview - Dupe and Reverse trans
Last modified: 2018-06-29 23:14:21 UTC
Created attachment 239274 [details] [review] Register rewrite patch This is an update to the Possible Register migration to Treeview - 673193 As before, do not open same account register in old style and the tree-view one and it may also be required to delete the saved pages in gconf under register2 heading. This update fixes duplicate transaction and reversing transaction. Also included is a minor change to the model, added the test for unbalanced transactions along with some other dialogs. Tab key navigation has been revised but is still a work in progress. Account key separator works along with new account creation.
Comment on attachment 239274 [details] [review] Register rewrite patch Thank you for the patch. Unfortunately, for me gnucash crashes when I try to duplicate a transaction. I will attach a backtrace. What I did: - applied both of your latest patches (update11 and update12) - run gnucash - open an asset account - click on one transaction - hit the duplicate button in the toolbar - leave the suggested date as is (shows 26.03.2013 in my example), leave the number field blank. - click ok => Crash
Created attachment 239890 [details] Backtrace while duplicating a transaction
Thank you for the back trace, it all pointed to some bad g_free's which pointed to an incorrect implementation of setting the num column entries when being duplicated. I have corrected this in the second patch on the latest bug 696868.
Comment on attachment 239274 [details] [review] Register rewrite patch Committed with the rest, having confirmed that patch 14 fixed the crash.
The crash is indeed fixed after the commit of bug 696868. Closing the report.
GnuCash bug tracking has moved to a new Bugzilla host. This bug has been copied to https://bugs.gnucash.org/show_bug.cgi?id=696152. Please update any external references or bookmarks.