GNOME Bugzilla – Bug 568327
Using most "budget" reports, without a budget defined crashes gnucash
Last modified: 2018-06-29 22:16:19 UTC
Patch http://svn.gnucash.org/repo/gnucash/trunk@17829 adds many more budget reports. There does not appear to be proper validation of whether any budgets are defined. If none are then gnucash crashes. See also: https://lists.gnucash.org/pipermail/gnucash-devel/2009-January/024630.html? and http://lists.gnucash.org/logs/2009/01/2009-01-19.html#T11:10:26
I'll cook up a patch for this as soon as I have time.
Confirmed by Frank Ellenberger.
Created attachment 127218 [details] [review] Patch fixing bug Changes Budget Balance Sheet, Budget Income Statement to check for invalid budget (probably due to no budget existing).
(In reply to comment #3) > Created an attachment (id=127218) [edit] > Patch fixing bug > > Changes Budget Balance Sheet, Budget Income Statement to check for invalid > budget (probably due to no budget existing). > Stops crashes correctly. However, budget barchart just gives a blank sheet now. No nothing. No indication of anything being wrong.
This patch doesn't modify Budget Barchart. The behavior you describe was already present.
Created attachment 127377 [details] [review] Fixes crashes and also improves error message when no budgets exists (for all budget reports). This patch replaces the previous one.
(In reply to comment #6) > Created an attachment (id=127377) [edit] > Fixes crashes and also improves error message when no budgets exists (for all > budget reports). > > This patch replaces the previous one. > Confirmed that this works in the case without a budget defined. Please apply.
Trunk, r17851. Should this be back-ported as well? Probably yes, at least for the budget.scm report.
The consensus in IRC seems to be that the fix for budget.scm is cosmetic and therefore should not be backported.
O.k. The primary bug, introduced recently, is fixed and the concensus seems to be not to backport the the cosmetic parts, so marking fixed.
Deleting the backport block.
Oh, obviously I read the comments here only after I already back-ported the commit. 2.2-branch, r17873. I hope this doesn't make any harm, does it?
GnuCash bug tracking has moved to a new Bugzilla host. This bug has been copied to https://bugs.gnucash.org/show_bug.cgi?id=568327. Please update any external references or bookmarks.