Ability to link directly from an item in the Staff client to budget information.
Hello Amit Did you see 5336 and 9780 ? M. Saby
Created attachment 17598 [details] [review] Ability to link directly from an item in the Staff client to budget information To Test: 1) Create budget and fund under budget administration. 2) Create Vendor in acquisitons module. 3) Create basket under vendor. 4) Create order and choose budget while creating order. 5) Receive shipment. 6) Check in catalog details, in holding tab budget information column is displayed , on click it should display the budget information
Comment on attachment 17598 [details] [review] Ability to link directly from an item in the Staff client to budget information Review of attachment 17598 [details] [review]: ----------------------------------------------------------------- Hi Amit, I found a couple of problems with this patch - I have noted them using the splinter review below. Please take a look at the coding guidelines in the wiki (http://wiki.koha-community.org/wiki/Coding_Guidelines). Also, I am not sure about the workflow here. Can you perhaps explain the use case for the link to the budget? ::: C4/Acquisition.pm @@ +75,1 @@ > Capitalization seems a bit off here. Also, unit tests are mandatory for every new sub in the C4 and Koha namespaces. ::: koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt @@ +370,4 @@ > [% IF ( SpineLabelShowPrintOnBibDetails ) %]<th>Spine label</th>[% END %] > [% IF ( hostrecords ) %]<th>Host records</th>[% END %] > [% IF ( analyze ) %]<th>Used in</th><th></th>[% END %] > + [% IF ( itemdata_booksellerid ) %]<th>Budget Information</th>[% END %] Wrong capitalization. @@ +523,5 @@ > <td><a href="/cgi-bin/koha/cataloguing/addbiblio.pl?hostbiblionumber=[% item.biblionumber %]&hostitemnumber=[% item.itemnumber %]">Create analytics</a></td> > + [% END %] > + [% IF ( itemdata_booksellerid ) %] > + <td><a href="/cgi-bin/koha/admin/aqbudgets.pl?budget_period_id=[% item.budget_period_id %]" >[% item.budget_name %]</a></td> > + [% END %] I am not sure this is the right place for this information. If I read the code correctly, this will appear in the holdings table on the detail page. All other acq related information is shown on the holdings tab.
Ah, I see I got the wrong line - the first comment refers to the capitalization of the new sub's name.
Hello Amit. I will take a look at your patch later, but I think its goal is partially the same as 6777 , 9780 (I did this one), and 7294. Maybe it could be interesting for you to take a look at them ? For 9780, it was supposed to show a link between record and Acq, but in a different way than your patch did (not in tabs, but under the record description). It is also forbidding librarians to suppress a record if it is used in an order. BZ9780 was signed off and waiting for QA, but I think I am going to rewrite it, keeping only the last point and removing the display of the link between record and Acq, as maybe your tab solution is better on UX point of view. For 6777, it is adding a new column in the holding tabs, showing acq info. For 7294, it is also using the tabs, but doing other things. Mathieu Saby
The use case for the for this is the desire of a number of sites to access the funding information for copies of titles held from the results of standard staff search.
Yes, it is a very needed feature! For the moment, in my library, we made a hack in JQuery for displaying this information. I just made previous comment to remark that there are 4 bugs BZ for the same subject. Maybe some ideas developped of the other patchs could be interesting for this one? M. Saby
Ok, I see :) I thought that the link was not very helpful as it will just show you the totals, but I can see that displaying the fund itself would be useful. I still think acq related information should be grouped together in one place tho - meaning not to add columns to items, but have that in a separate place. The items table already has a lot of columns and display won't get more readable adding more information that is not needed for circ.
I can join a screenshot of what we made in JQuery in Rennes 2.
Created attachment 17610 [details] Screenshot of a dev made in Rennes2 with JQuery I don't know if it could be helpful, but I join a screenshot of a dev made in Rennes2 with JQuery. We create items only when we receive books. So we need a way to show information about pending orders before reception. We made a little dev in jquery that 1 - creates a new tab for acquisition information : # order with a link # basket with a link Budget Note Collection code (our stat1) Level of public (our stat2) Date of order Date of reception Quantity (received/ordered) Date of cancellation 2 - show a summary of those information in "item" tab, ONLY if no real items have been created. 3 - show the same information in OPAC M. Saby
Created attachment 17623 [details] [review] Bug 10013 - Ability to link directly from an item in the Staff client to budget information. Hi Kartrin, Colin has already provided the use case and i have followed the "Capitalization strings consistently" guidelines. To Test: 1) Create budget and fund under budget administration. 2) Create Vendor in acquisitons module. 3) Create basket under vendor. 4) Create order and choose budget while creating order. 5) Receive shipment. 6) Check in catalog details, in holding tab budget information column is displayed , on click it should display the budget information
Hi Amit, thank you for fixing the capitalization problems. There is still one open problem - we need a unit test for the new &GetBudgetInfoFromItemnumber.
Created attachment 17645 [details] [review] Add new unit tests for new routine: GetBudgetInfoFromItemnumber Add new unit tests for new routine: GetBudgetInfoFromItemnumber
Hello, I'm questionning about the use of this patch. Why made a link to budget ? I've maybe misunderstood the goal of this patch. Can you give me more explaination ?
Hi, The purpose of this patch will show under which budget you ordered/purchased the item.
Ok. I think the proposals that Mathieu did are more complete. Have you seen them ? It will be a good idea to work to include what Rennes2 did. If evebody think this patch (10013) is essential, I can sign it (it works ^^).
Created attachment 18598 [details] [review] Ability to link directly from an item in the Staff client to budget information. To Test: 1) Create budget and fund under budget administration. 2) Create Vendor in acquisitons module. 3) Create basket under vendor. 4) Create order and choose budget while creating order. 5) Receive shipment. 6) Check in catalog details, in holding tab budget information column is displayed , on click it should display the budget information
Created attachment 18600 [details] [review] New unit tests for routine: GetBudgetInfoFromItemnumber New unit tests for routine: GetBudgetInfoFromItemnumber
Created attachment 20061 [details] [review] Bug 10013 - Ability to link directly from an item in the Staff client to budget information. To Test: 1) Create budget and fund under budget administration. 2) Create Vendor in acquisitons module. 3) Create basket under vendor. 4) Create order and choose budget while creating order. 5) Receive shipment. 6) Check in catalog details, in holding tab budget information column is displayed , on click it should display the budget information. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 20063 [details] [review] Bug-10013-New unit tests for routine: GetBudgetInfoFromItemnumber Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 20064 [details] [review] Bug-10013-New unit tests for routine: GetBudgetInfoFromItemnumber
The feature patch works as advertised, but I'm a bit confused by the unit test. There is language in it that says it's a stub, it it's not. Additionally, it doesn't mock any data, and fails for me. It makes invalid assumptions.
Created attachment 22061 [details] [review] Unit Test for newly created routine: GetBudgetInfoFromItemnumber Kyle, I have added the Test case for newly created routine: GetBudgetInfoFromItemnumber
Applying: Bug 10013 - Ability to link directly from an item in the Staff client to budget information. Using index info to reconstruct a base tree... M C4/Acquisition.pm M catalogue/detail.pl M koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt Falling back to patching base and 3-way merge... Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt Auto-merging catalogue/detail.pl Auto-merging C4/Acquisition.pm CONFLICT (content): Merge conflict in C4/Acquisition.pm Patch failed at 0001 Bug 10013 - Ability to link directly from an item in the Staff client to budget information. The copy of the patch that failed is found in: /home/christopher/git/koha/.git/rebase-apply/patch When you have resolved this problem, run "git am --continue". If you prefer to skip this patch, run "git am --skip" instead. To restore the original branch and stop patching, run "git am --abort".
This has been selected by someone for the hackfest in Marseille, but the patches are not testable. Any chance to see this rebased?
I feel like this is covered by the acquisitions tab added to the detail page after this bug was files. Please reopen if I am mistaken with information on what else is needed.