Created attachment 5648 [details] Screenshot of the new Add budget page This new feature adds a list of budgets in the Add budget page to create a budget based on a previous. An exemple use case for this is if you want to create a 2010-2011 budget and want the same funds as the 2009-2010 budget, with the same amounts. It also has an option called "transfer the order acquisitions" that transfers all the orders not received in the new budget copy.
Created attachment 5649 [details] [review] Patch to add feature
oh, and once again this was sponsored by our client, http://www.ccsr.qc.ca/.
Comment on attachment 5649 [details] [review] Patch to add feature I'm afraid that this was not meant to be sent yet. I will update this bug once we can really share it.
Maxime, can you clarify? The status is 'Needs Signoff' but from your comment I am not sure if this can be included into Koha at this point in time or not.
(In reply to comment #4) > Maxime, can you clarify? The status is 'Needs Signoff' but from your comment I > am not sure if this can be included into Koha at this point in time or not. Yes it can now be included.
Patch applies without conflicts, but I get a 404 error when trying to go to /cgi-bin/koha/admin/aqbudgetperiods.pl
End of year is near and I think it's the right time to give this a little push :)
This feature is being worked by catalyst, (sub-contracted by BibLibre, sponsored by french library). Code being validated atm. (I missed this bugzilla entry, too bad
As there is already code for that by Maxime now - perhaps some of it can be reused? I am not sure if the spec biblibre/catalyst have covers all of the functionality or more than described here. Could this made available? There is no RFC on the wiki I think?
Created attachment 7043 [details] [review] Patch to allow duplication of budgets This is the work from Catalyst that Paul mentioned. Patch allows for a user to duplicate a budget from the edit drop down menu while viewing a budget.
Hi Chris, does the work from Catalyst also include an option to move the not received orders to the new funds?
(In reply to comment #11) Hey Katrin No sorry, the patch only allows for a budget and all its funds to be duplicated (as if you created a new budget with all the same funds and allocated money).
Created attachment 7168 [details] [review] [SIGNED-OFF] Bug 6943: Added ability to duplicate a budget tree (from the edit sub menu) bug 6953 http://bugs.koha-community.org/show_bug.cgi?id=6943 Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> - all new javascript alerts have translations markers _() - hierarchies and permissions were copied correctly Possible enhancements: - make it possible to move orders from old to new funds - make it possible to change description while copying, saving 1 additional step
I looked at Maxime's patch too. It still applies but /cgi-bin/koha/admin/aqbudgetperiods.pl gives me a 404. [Mon Jan 16 07:51:05 2012] [error] [client 127.0.0.1] script not found or unable to stat: /home/katrin/kohaclone/admin/aqbudgetperiods.pl, referer: http://localhost:8080/cgi-bin/koha/acqui/acqui-home.pl
QA comment: Looking at your patch chris_h, I see that you've used 2 spaces for indentation. We use 4 spaces in Koha. I checked http://wiki.koha-community.org/wiki/Coding_Guidelines and saw it was not explicit (it was still in the "discussed"), so I updated the guidelines. Marking failed QA, please resubmit with 4 tabs I don't see any other problem
Created attachment 7309 [details] [review] Added ability to duplicate a budget Sorry about that paul, indentation spacing was something I was worrying about but i've seen both 2 and 4 used so I just used 2. Fixed editor settings so it won't happen again. Reattached patch, I modified my original patch and marked the singed off patch obsolete as I think my patch needs to be singed off again, I hope this is correct.
Created attachment 7330 [details] [review] [SIGNED-OFF] Bug 6943: Added ability to duplicate a budget tree (from the edit sub menu) http://bugs.koha-community.org/show_bug.cgi?id=6943 Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> - all new javascript alerts have translations markers _() - hierarchies and permissions were copied correctly Possible enhancements: - make it possible to move orders from old to new funds - make it possible to change description while copying, saving 1 additional step
QA comment: #2 The code does not respect the HTML3 coding guideline. however, it was submitted before the inclusion of this new behaviour, and there are also many dates managed elsewhere in the script, so maybe it's more consistent #1 the cloning does not let the user updating the description of the budget. Thus, the cloned budget has exactly the same name as the previous one, only the dates differ. As there is a form to enter the dates of the new budget, why not add the description of the budget ? (pre-filled with the previous value). Would you agree to add this feature to your patch ? Katrin, as you've signed-off, your opinion is welcomed ! Not marking failed QA, #2 is more a question.
Hi Paul, see comment 13 ;) I noted there that an option to edit the description while copying would be good. Still this feature works as advertised and it is useful - so didn't want to scope creep it. :)
OK, i'm convinced, also because duplicating a budget is something you do once a year, so it's not a big deal. I'll open a new bug for the enhancement
(In reply to comment #19) > Possible enhancements: > - make it possible to change description while copying, saving 1 additional > step I have created Bug 9155 for this.