GNOME Bugzilla – Bug 589865
In the report "Budget flow" period it doesn't work ok
Last modified: 2018-06-29 22:25:29 UTC
Please describe the problem: The data in report refer to next period. I can't see the period 1. Steps to reproduce: 1. Acess the report "Budget Flow" 2. Select option 3. select general 4. keep period as 1 5. select Apply 6. The data in report refer to period 2 Actual results: The data in report refer to next period. I can't see the period 1. Expected results: I would like can see the correct period Does this happen every time? Yes Other information: I sugest that the label of the field, in portuguese, show up as "Período"
Still present in 2.4.4 (Windows). I.e. my budget starts on 1/1/2011 but the period 1 flow report gives the date as 1/2/2011 with associated budget values from 1/2/2011 to 28/2/2011.
Created attachment 194666 [details] [review] src/report/standard-reports/budget-flow.scm period - 1 correction
Comment on attachment 194666 [details] [review] src/report/standard-reports/budget-flow.scm period - 1 correction Also present in 2.4.7.
Comment on attachment 194666 [details] [review] src/report/standard-reports/budget-flow.scm period - 1 correction The problem is using period numbers started from 1 in budget values list that starts from zero.
Same here, agree with mikhail, a "-1" is needed somewhere in the code.
Created attachment 242456 [details] [review] patch file to be applied on budget-flow.scm in GnuCash 2.5.0 First of all, I can confirm - that this bug still occurs in GnuCash 2.5.0 (tested with Ubuntu 12.04) - the patch uploaded by Mikhail TSaplin is still applicable and does the trick I also noticed that when now e.g. period 1 is selected, the headline of the report still shows the date as defined with period 2. Therefore I upload this 2nd patch that fixes this headline issue by introducing the (- period 1) also at this point. So in order to fix it completely, both patches need to be applied. This bug should at least be set to status NEW.
Thank you both for your patches. The reason Mikhail's patch got unnoticed for so long is because it wasn't marked as a patch. That means that the usual bugzilla search queries for patches didn't pick it up. I have now changed this (and will apply the patch shortly).
Comment on attachment 194666 [details] [review] src/report/standard-reports/budget-flow.scm period - 1 correction Looks ok to me. I'll apply it shortly.
Comment on attachment 242456 [details] [review] patch file to be applied on budget-flow.scm in GnuCash 2.5.0 @Carsten: why do you change the name of the report in your patch ? Is this an oversight or intentional ?
Comment on attachment 194666 [details] [review] src/report/standard-reports/budget-flow.scm period - 1 correction Committed for the 2.5 series in r22966, and for the 2.4 series in r22973. Thank you very much !
Comment on attachment 242456 [details] [review] patch file to be applied on budget-flow.scm in GnuCash 2.5.0 I have reverted the unwanted name change and committed for the 2.5 series in r22967, and for the 2.4 series in r22974. Thank you very much !
GnuCash bug tracking has moved to a new Bugzilla host. This bug has been copied to https://bugs.gnucash.org/show_bug.cgi?id=589865. Please update any external references or bookmarks.