http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Depends on|12168 | -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 --- Comment #2 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 27906 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=27906&action=edit Bug 12164: On cloning budget period, mark original budget as inactive This patch adds a checkbox "mark original budget as inactive" (budget period). If it is checked, the original budget will be marked as inactive. -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 --- Comment #4 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 27908 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=27908&action=edit Bug 12164: Close a budget period (budget) This is the main patch. On closing a budget period, all unreceived orders are moved from the old/previous fiscal year into the new fiscal year. You can rollover funds unused in the previous fiscal year to the new fiscal year. This patch set is based on bug 12168 (bugfix) and can be tested on top of bug 11578 (easier to see the fund structure). The patch set is cut in 6 main patches: - Move the budget period clone logic into C4::Budgets The code is moved from the pl to Budgets.pm and unit tests are provided. The original code should certainly be buggy since a typo existed. - On cloning budget period, mark original budget as inactive Cloning a budget period is already possible in Koha, this patch adds a checkbox to mark as inactive the original budget. That avoids to edit the budget and click the "inactive" checkbox. Both do the same action. - On cloning budget periods, add a "reset all funds" option Same as before, a new checkbox is added on cloning a budget period. If you check it, all fund amounts will be set to 0. Otherwise, no change compared to the existing behavior. - Close a budget period (budget) The goal of this patch set is to move unreceived orders from a budget to another. This patch adds a C4::Budgets::MoveOrders routine which does this job. This action is only possible if the fund structure is the same for both budgets, the budget_code field should be the same. - On closing budget period, move unspent amount Unspent amount will be move from the previous budget structure to the new one. - Add UI report This patch only adds a report when closing a budget is done. Test plan: Wording: below, budget is a "budget period" and fund is a "budget". Prerequisite: Having 1 active budget with some funds (with different levels and different amounts). Order and receive some orders (not all). 1/ Go on the budgets administration page (admin/aqbudgetperiods.pl) and duplicate the structure of this budget ("Duplicate" link in the "Actions" column). 2/ Enter start and end date for this budget and mark the original budget as inactive. 3/ Note that a new budget is created, with the same fund structures (and same value) and that the old one is marked as inactive (see admin/aqbudgets.pl page with patches from bug 11578). 4/ Try to close the new budget: it is not possible, there is no unreceived orders for this budget. 5/ You can close the inactive budget ("Close" link in the "Actions" column). 6/ Verify the number of "Unreceived orders" is correct and select the new budget in the budget list. Click on the "Move remaining unspent funds" if you want to move unspent amounts. 7/ A report view is displayed and show you the ordernumber which have been impacted (grouped by fund). 8/ Try different configuration, depending on the selected checkboxes. -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 --- Comment #6 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 27910 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=27910&action=edit Bug 12164: Add UI report -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 --- Comment #8 from Jonathan Druart <jonathan.druart@biblibre.com> --- Note: To complete this feature, a CSV export should be available on the report view. -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Change sponsored?|--- |Sponsored -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 --- Comment #11 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 28589 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=28589&action=edit Bug 12164: On cloning budget period, mark original budget as inactive This patch adds a checkbox "mark original budget as inactive" (budget period). If it is checked, the original budget will be marked as inactive. -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 --- Comment #13 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 28591 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=28591&action=edit Bug 12164: Close a budget period (budget) This is the main patch. On closing a budget period, all unreceived orders are moved from the old/previous fiscal year into the new fiscal year. You can rollover funds unused in the previous fiscal year to the new fiscal year. This patch set is based on bug 12168 (bugfix) and can be tested on top of bug 11578 (easier to see the fund structure). The patch set is cut in 6 main patches: - Move the budget period clone logic into C4::Budgets The code is moved from the pl to Budgets.pm and unit tests are provided. The original code should certainly be buggy since a typo existed. - On cloning budget period, mark original budget as inactive Cloning a budget period is already possible in Koha, this patch adds a checkbox to mark as inactive the original budget. That avoids to edit the budget and click the "inactive" checkbox. Both do the same action. - On cloning budget periods, add a "reset all funds" option Same as before, a new checkbox is added on cloning a budget period. If you check it, all fund amounts will be set to 0. Otherwise, no change compared to the existing behavior. - Close a budget period (budget) The goal of this patch set is to move unreceived orders from a budget to another. This patch adds a C4::Budgets::MoveOrders routine which does this job. This action is only possible if the fund structure is the same for both budgets, the budget_code field should be the same. - On closing budget period, move unspent amount Unspent amount will be move from the previous budget structure to the new one. - Add UI report This patch only adds a report when closing a budget is done. Test plan: Wording: below, budget is a "budget period" and fund is a "budget". Prerequisite: Having 1 active budget with some funds (with different levels and different amounts). Order and receive some orders (not all). 1/ Go on the budgets administration page (admin/aqbudgetperiods.pl) and duplicate the structure of this budget ("Duplicate" link in the "Actions" column). 2/ Enter start and end date for this budget and mark the original budget as inactive. 3/ Note that a new budget is created, with the same fund structures (and same value) and that the old one is marked as inactive (see admin/aqbudgets.pl page with patches from bug 11578). 4/ Try to close the new budget: it is not possible, there is no unreceived orders for this budget. 5/ You can close the inactive budget ("Close" link in the "Actions" column). 6/ Verify the number of "Unreceived orders" is correct and select the new budget in the budget list. Click on the "Move remaining unspent funds" if you want to move unspent amounts. 7/ A report view is displayed and show you the ordernumber which have been impacted (grouped by fund). 8/ Try different configuration, depending on the selected checkboxes. -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 --- Comment #15 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 28593 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=28593&action=edit Bug 12164: Add UI report -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 --- Comment #17 from Jonathan Druart <jonathan.druart@biblibre.com> --- (In reply to Paola Rossi from comment #16) Hello Paola, thanks for testing!

A) I run the script. The only funds in the database are the expected ones described below. But, on main Acquisitions' page [Manual 3.16, See Budget/Fund Tracking: Clicking on the linked amounts under spent or ordered will show you a summary of the titles ordered/received on that budget.], after selecting the cells of the column "Spent" of the table "All available funds for Centerville", I see "No data available in table" even when some orders are expected in the selected cell (not zero). [Perhaps does the script build the received orders in a not entirely correctly way?]

Hum yes, not exactly sure what is wrong here, but the script does not depend on the AcqCreateItem pref. I don't think it is useful for testing.

B) At 6-th step: "Verify the number of Unreceived orders is correct ...", I see this error: the numbers in "Unreceived orders"'s column are not the expected ones: they're the total numbers of received and unreceived orders.

Will be fixed.

C) At 6-th step, koha says: "Choose the funds you want to move unreceived orders:" but no checkbox is given to mark which funds I want to choose.

I changed the sentence. No checkbox should appear.

D) At 6-th step: I've selected the budget, but, after selecting the "Move unreceived orders" button, koha says: "You have chosen to move all unreceived orders from 'test_clone' to 'null'." So the name of the new budget is not shown correctly. [If I've already changed the name of the target budget, 'null' still remains as an error.]

Will be fixed.

E) I prefer to change the name of the "new" budget period, the target. I call it "test_clone_new". [it's the 2015's budget]. At 7-th step, the report shows: Report after moving unreceived orders from budget test_clone_new (01/01/2015 - 12/31/2015) to test_clone_new (01/01/2015 - 12/31/2015) So the old budget (name and dates) is not shown correctly. See also G) below.

Will be fixed.

F) Now the old budget is inactive and closed. It only has already received orders (OK). But I can close it again, marking the original budget as inactive (?), and so on. I choose the same target budget: no orders shall be moved (OK ...), but the base-level allocated, the base-level available and the total available of the target budget period are increased again. See Nota bene 1 below.

If you try to close an already "closed" budget, you should not be able to continue. You should arrive on a page with the following message: "There is no unreceived orders for this budget. Back" I just tried and it seems to work. Could you detail please?

G) I duplicate the "test_clone" budget into "test_2015". Then I duplicate the same "test_clone" again in "test_2016". Now I close the "test_clone" onto "test_2016", the last one. The report says: "Report after moving unreceived orders from budget test_2015 (01/01/2015 - 12/31/2016) to test_2016 (01/01/2016 - 12/31/2016)" The part "test_2015 (01/01/2015 - 12/31/2016)" is wrong.

This should be fixed with previous changes.

---------------------------------------------------------------------- "Closing" a budget period is new in koha: NB.1: Perhaps would the option be absent for a budget period without unreceived orders? Or for a budget period already closed?

Actually I don't know what could be the difference between a "closed" budget and an "inactive" budget. It is why it is recommended to mark as inactive a budget when you duplicate it. I don't want to display/hide the "close" link on on admin/aqbudgetperiods.pl because the way to know is bad for performances.

NB.2: How can I know that a budget period has been already closed? I don't see it [in "Edit"ing a budget period, f.i.].

No, you can't. This information is not stored in the DB. -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 --- Comment #18 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 28695 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=28695&action=edit Bug 12164: follow-up: various fixes This follow-up fixes issues raised on bug 12164 comment 17. 1/ The unreceived orders columns should only display "pending" orders 2/ Fix some typo from_budget_period_id vs budget_period_id vs to_budget_period_id -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 Paola Rossi <paola.rossi@cineca.it> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #28588|0 |1 is obsolete| | --- Comment #20 from Paola Rossi <paola.rossi@cineca.it> --- Created attachment 28702 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=28702&action=edit Bug 12164: Move the budget period clone logic into C4::Budgets -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 Paola Rossi <paola.rossi@cineca.it> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #28590|0 |1 is obsolete| | --- Comment #22 from Paola Rossi <paola.rossi@cineca.it> --- Created attachment 28704 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=28704&action=edit Bug 12164: On cloning budget periods, add a "reset all funds" option -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 Paola Rossi <paola.rossi@cineca.it> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #28592|0 |1 is obsolete| | --- Comment #24 from Paola Rossi <paola.rossi@cineca.it> --- Created attachment 28706 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=28706&action=edit Bug 12164: On closing budget period, move unspent amount -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 Paola Rossi <paola.rossi@cineca.it> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #28695|0 |1 is obsolete| | --- Comment #26 from Paola Rossi <paola.rossi@cineca.it> --- Created attachment 28708 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=28708&action=edit Bug 12164: follow-up: various fixes -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 --- Comment #28 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Jonathan, I am still testing this, so far no bigger problems found. Some small issues and a question: 1) Some small typos found during code review: + There is no unreceived orders for this budget. There are no unreceived oreders for this budget. + <input type="submit" value="Move unreceive orders" /> Move unreceived orders + <td>Informations</td> Information, Details + <td>There was no unreceived orders for this fund.</td> There were no... 2) The report shown after moving funds shows the amounts with quite a ot of zeros :) 99 200.000000 3) Question: If you decide to roll over the funds with the orders into the next year - should the funds be removed from last year's budget? I am not sure about this, so it's really a question to libraries I think. -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 --- Comment #30 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 29354 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29354&action=edit Bug 12164: Fix some typos -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 --- Comment #32 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 29356 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29356&action=edit Bug 12164: Fix decimal number display If the funds are not moved, the amount is display like 123.000000. This patch hardcodes the number of decimal numbers to display (2). -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 --- Comment #34 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 29422 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29422&action=edit [PASSED QA] Bug 12164: Move the budget period clone logic into C4::Budgets Note the typo InsertInTable "aqcudgets", note sure about the existing behavior before this patch! This patch adds a link "Duplicate" in the action list for budget periods (budgets). Signed-off-by: Paola Rossi <paola.rossi@cineca.it> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 --- Comment #36 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 29424 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29424&action=edit [PASSED QA] Bug 12164: On cloning budget periods, add a "reset all funds" option This patch adds a checkbox "reset all funds" (budgets). If it is checked, the new created budgets (funds) will be reset. Signed-off-by: Paola Rossi <paola.rossi@cineca.it> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 --- Comment #38 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 29426 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29426&action=edit [PASSED QA] Bug 12164: On closing budget period, move unspent amount On closing a budget (budget period), the user will be presented with an option to move remaining unspent funds from the previous budget to the newly created one - adding to the amounts already entered in those funds. Signed-off-by: Paola Rossi <paola.rossi@cineca.it> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 --- Comment #40 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 29428 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29428&action=edit [PASSED QA] Bug 12164: follow-up: various fixes This follow-up fixes issues raised on bug 12164 comment 17. 1/ The unreceived orders columns should only display "pending" orders 2/ Fix some typo from_budget_period_id vs budget_period_id vs to_budget_period_id Signed-off-by: Paola Rossi <paola.rossi@cineca.it> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 --- Comment #42 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 29430 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29430&action=edit [PASSED QA] Bug 12164: Fix decimal number display If the funds are not moved, the amount is display like 123.000000. This patch hardcodes the number of decimal numbers to display (2). Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Passes QA script and tests, especially all acquisition tests. New routines in Budgets.pm: - CloneBudgetPeriod - CloneBudgetHierarchy - MoveOrders OK t/db_dependent/Budgets.t Some of the tests done: 1) Duplicate an existing budget - OK Existing budget is active, duplicate it and mark it inactive - OK Existing budget is inactive, duplicate and mark it inactive 2) Close budget - OK Budget was created without funds - no changes are made - OK Budget was created with funds, all orders moved correctly - moving the available money to the new budget is optional - OK Budget was created with funds, but one fund deleted, all orders moved correctly, one missing fund reported, orders unchanged 3) Partial receiced undone - OK Partial receive from last budget is undone, order lines are correctly recombined and fund is set to the current budget -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #28704|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #28706|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #28708|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #29356|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #28703|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Patch doesn't apply CC| |tomascohen@gmail.com --- Comment #44 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Jonathan, could you please rebase this to current master? I'm willing to push it, but I didn't notice it before, and now it doesn't apply :-( I tried myself but it wasn't a trivial one. -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 --- Comment #46 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 29734 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29734&action=edit Bug 12164: On cloning budget period, mark original budget as inactive This patch adds a checkbox "mark original budget as inactive" (budget period). If it is checked, the original budget will be marked as inactive. Signed-off-by: Paola Rossi <paola.rossi@cineca.it> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 --- Comment #48 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 29736 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29736&action=edit Bug 12164: Close a budget period (budget) This is the main patch. On closing a budget period, all unreceived orders are moved from the old/previous fiscal year into the new fiscal year. You can rollover funds unused in the previous fiscal year to the new fiscal year. This patch set is based on bug 12168 (bugfix) and can be tested on top of bug 11578 (easier to see the fund structure). The patch set is cut in 6 main patches: - Move the budget period clone logic into C4::Budgets The code is moved from the pl to Budgets.pm and unit tests are provided. The original code should certainly be buggy since a typo existed. - On cloning budget period, mark original budget as inactive Cloning a budget period is already possible in Koha, this patch adds a checkbox to mark as inactive the original budget. That avoids to edit the budget and click the "inactive" checkbox. Both do the same action. - On cloning budget periods, add a "reset all funds" option Same as before, a new checkbox is added on cloning a budget period. If you check it, all fund amounts will be set to 0. Otherwise, no change compared to the existing behavior. - Close a budget period (budget) The goal of this patch set is to move unreceived orders from a budget to another. This patch adds a C4::Budgets::MoveOrders routine which does this job. This action is only possible if the fund structure is the same for both budgets, the budget_code field should be the same. - On closing budget period, move unspent amount Unspent amount will be move from the previous budget structure to the new one. - Add UI report This patch only adds a report when closing a budget is done. Test plan: Wording: below, budget is a "budget period" and fund is a "budget". Prerequisite: Having 1 active budget with some funds (with different levels and different amounts). Order and receive some orders (not all). 1/ Go on the budgets administration page (admin/aqbudgetperiods.pl) and duplicate the structure of this budget ("Duplicate" link in the "Actions" column). 2/ Enter start and end date for this budget and mark the original budget as inactive. 3/ Note that a new budget is created, with the same fund structures (and same value) and that the old one is marked as inactive (see admin/aqbudgets.pl page with patches from bug 11578). 4/ Try to close the new budget: it is not possible, there is no unreceived orders for this budget. 5/ You can close the inactive budget ("Close" link in the "Actions" column). 6/ Verify the number of "Unreceived orders" is correct and select the new budget in the budget list. Click on the "Move remaining unspent funds" if you want to move unspent amounts. 7/ A report view is displayed and show you the ordernumber which have been impacted (grouped by fund). 8/ Try different configuration, depending on the selected checkboxes. Signed-off-by: Paola Rossi <paola.rossi@cineca.it> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 --- Comment #50 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 29738 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29738&action=edit Bug 12164: Add UI report Signed-off-by: Paola Rossi <paola.rossi@cineca.it> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 --- Comment #52 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 29740 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29740&action=edit Bug 12164: Fix some typos Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Passed QA --- Comment #54 from Jonathan Druart <jonathan.druart@biblibre.com> --- (In reply to Tomás Cohen Arazi from comment #44)

Jonathan, could you please rebase this to current master? I'm willing to push it, but I didn't notice it before, and now it doesn't apply :-(

I tried myself but it wasn't a trivial one.

The last patch set fixes the conflict with bug 12482. It was not so easy and I hope I did not forget something. I retested a big part of the feature and all looks good. -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12164 --- Comment #56 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Created attachment 30044 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=30044&action=edit Bug 12164: (followup) keys expects a hash parameter As of http://perldoc.perl.org/functions/keys.html we should avoid passing a scalar parameter as it is not backwards compatible with squeezze's Perl version (5.10). Cite: "Starting with Perl 5.14, keys can take a scalar EXPR, which must contain a reference to an unblessed hash or array. The argument will be dereferenced automatically. This aspect of keys is considered highly experimental. The exact behaviour may change in a future version of Perl." Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com> -- You are receiving this mail because: You are watching all bug changes.