If you have an item level hold the 'Holds to pull' list needs to tell you which barcode in the case of several items being on the shelf
Created attachment 92404 [details] [review] Bug 23485: Add barocde to holds to pull This adds a barcode column, it tries to show 'Only' in the case of a single item, however, group by problems here may mean that you see several items that will fill a next available hold on this record instead of the single item to fill a patrons hold (i.e. the patron shown may not match the item list) As the group by is a bigger problem, I don't think we should block this one by that To test: 1 - Place an item level hold on a bib with several items with the same callnumber 2 - View the holds to pull report 3 - Try to guess which one on the shelf is right? 4 - Apply patch 5 - See the barcode in holds to pull report 6 - You can now grab the correct item (but don't yet) 7 - Place a next available hold on the same title 8 - See the report now lists all the barcodes on the item, but shows the borrower with a single hold 9 - Check in a different item that fills the next available hold 10 - Now the report shows the single item for the borrower
Why is there this line ? +[% IF ( reserveloo.itemnumber ) %]Only[% END %]
(In reply to Fridolin SOMERS from comment #2) > Why is there this line ? > > +[% IF ( reserveloo.itemnumber ) %]Only[% END %] Oh its to say hold is not on record but on a specific item ? Should this be also on all item-level information Libraries, Call numbers, ... ?
I'm concerned that the list of barcodes will be unmanageable for records which have a large number of items. On the holds queue report we say "31000000234552 or any available." Would that work here? Also: If it is an item-level hold I think the barcode should not be displayed as a list item, so that "Only 32000000202225" displays together.
(In reply to Owen Leonard from comment #4) > I'm concerned that the list of barcodes will be unmanageable for records > which have a large number of items. On the holds queue report we say > "31000000234552 or any available." Would that work here? > > Also: If it is an item-level hold I think the barcode should not be > displayed as a list item, so that "Only 32000000202225" displays together. +1 Some libraries have a lot of copies for text books and similar. Agreeing with Owen.
Created attachment 110172 [details] [review] Bug 23485: Show barcode in holds to pull Updated to only show one barcode number, with either an "only" or an "or any available" depending on whether it's an item or bib hold. To test: 1 - Place an item level hold on a bib with several items with the same callnumber 2 - View the holds to pull report 3 - Try to guess which one on the shelf is right? 4 - Apply patch 5 - See the barcode in holds to pull report 6 - You can now grab the correct item (but don't yet) 7 - Place a next available hold on the same title 8 - See the report now shows one possible valid barcode with the text "or any available." 9 - Check in a different item that fills the next available hold 10 - Now the report shows the single item for the borrower
Applying: Bug 23485: Show barcode in holds to pull error: sha1 information is lacking or useless (koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt). error: could not build fake ancestor Patch failed at 0001 Bug 23485: Show barcode in holds to pull
Created attachment 110195 [details] [review] Bug 23485: Show barcode in holds to pull Updated to only show one barcode number, with either an "only" or an "or any available" depending on whether it's an item or bib hold. To test: 1 - Place an item level hold on a bib with several items with the same callnumber 2 - View the holds to pull report 3 - Try to guess which one on the shelf is right? 4 - Apply patch 5 - See the barcode in holds to pull report 6 - You can now grab the correct item (but don't yet) 7 - Place a next available hold on the same title 8 - See the report now shows one possible valid barcode with the text "or any available." 9 - Check in a different item that fills the next available hold 10 - Now the report shows the single item for the borrower
I did something wrong here. Will have to sort it out :)
Created attachment 110200 [details] [review] Bug 23485: Show barcode in holds to pull Updated to only show one barcode number, with either an "only" or an "or any available" depending on whether it's an item or bib hold. To test: 1 - Place an item level hold on a bib with several items with the same callnumber 2 - View the holds to pull report 3 - Try to guess which one on the shelf is right? 4 - Apply patch 5 - See the barcode in holds to pull report 6 - You can now grab the correct item (but don't yet) 7 - Place a next available hold on the same title 8 - See the report now shows one possible valid barcode with the text "or any available." 9 - Check in a different item that fills the next available hold 10 - Now the report shows the single item for the borrower
Created attachment 110246 [details] [review] Bug 23485: Show barcode in holds to pull Updated to only show one barcode number, with either an "only" or an "or any available" depending on whether it's an item or bib hold. To test: 1 - Place an item level hold on a bib with several items with the same callnumber 2 - View the holds to pull report 3 - Try to guess which one on the shelf is right? 4 - Apply patch 5 - See the barcode in holds to pull report 6 - You can now grab the correct item (but don't yet) 7 - Place a next available hold on the same title 8 - See the report now shows one possible valid barcode with the text "or any available." 9 - Check in a different item that fills the next available hold 10 - Now the report shows the single item for the borrower Signed-off-by: Michal Denar <black23@gmail.com>
Comment on attachment 110246 [details] [review] Bug 23485: Show barcode in holds to pull Review of attachment 110246 [details] [review]: ----------------------------------------------------------------- ::: koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt @@ +105,5 @@ > <td> > + [% IF ( reserveloo.barcode ) %] > + [% IF ( reserveloo.itemnumber ) %]Only[% END %] > + <ul> > + [% FOREACH barcode IN reserveloo.barcode.first %] Not sure this makes sense.. does reserveloo.barcord.first really return an array to loop on? I would think you could simplify to just [% reserveloo.barcode.first %] Also.. I'm not sure this needs to be nested in a list.. I think I'd expect simply [% IF ( reserveloo.itemnumber ) %]Only [% reserveloo.barcode.first %][% ELSE %][% reserveloo.barcode.first %] or any available[% END %] With the filters added in where appropriate of course ;) Does that make sense, or am I misunderstanding the logic
Created attachment 110391 [details] [review] Bug 23485: Show barcode in holds to pull Updated to only show one barcode number, with either an "only" or an "or any available" depending on whether it's an item or bib hold. Also incorporating feedback to simplify the TT logic and remove list formatting. To test: 1 - Place an item level hold on a bib with several items with the same callnumber 2 - View the holds to pull report 3 - Try to guess which one on the shelf is right? 4 - Apply patch 5 - See the barcode in holds to pull report 6 - You can now grab the correct item (but don't yet) 7 - Place a next available hold on the same title 8 - See the report now shows one possible valid barcode with the text "or any available." 9 - Check in a different item that fills the next available hold 10 - Now the report shows the single item for the borrower Signed-off-by: Michal Denar <black23@gmail.com>
> Does that make sense, or am I misunderstanding the logic Nope, you're right on. Thanks, Martin! I was trying to nudge the existing patch just enough to make it work, but this is cleaner.
A minor fix required: FAIL koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt FAIL filters missing_filter at line 107
Created attachment 110400 [details] [review] Bug 23485: Show barcode in holds to pull Updated to only show one barcode number, with either an "only" or an "or any available" depending on whether it's an item or bib hold. Also incorporating feedback to simplify the TT logic and remove list formatting. To test: 1 - Place an item level hold on a bib with several items with the same callnumber 2 - View the holds to pull report 3 - Try to guess which one on the shelf is right? 4 - Apply patch 5 - See the barcode in holds to pull report 6 - You can now grab the correct item (but don't yet) 7 - Place a next available hold on the same title 8 - See the report now shows one possible valid barcode with the text "or any available." 9 - Check in a different item that fills the next available hold 10 - Now the report shows the single item for the borrower Signed-off-by: Michal Denar <black23@gmail.com>
Created attachment 110411 [details] [review] Bug 23485: Show barcode in holds to pull Updated to only show one barcode number, with either an "only" or an "or any available" depending on whether it's an item or bib hold. Also incorporating feedback to simplify the TT logic and remove list formatting. To test: 1 - Place an item level hold on a bib with several items with the same callnumber 2 - View the holds to pull report 3 - Try to guess which one on the shelf is right? 4 - Apply patch 5 - See the barcode in holds to pull report 6 - You can now grab the correct item (but don't yet) 7 - Place a next available hold on the same title 8 - See the report now shows one possible valid barcode with the text "or any available." 9 - Check in a different item that fills the next available hold 10 - Now the report shows the single item for the borrower Signed-off-by: Michal Denar <black23@gmail.com>
Created attachment 110436 [details] [review] Bug 23485: Show barcode in holds to pull Updated to only show one barcode number, with either an "only" or an "or any available" depending on whether it's an item or bib hold. Also incorporating feedback to simplify the TT logic and remove list formatting. To test: 1 - Place an item level hold on a bib with several items with the same callnumber 2 - View the holds to pull report 3 - Try to guess which one on the shelf is right? 4 - Apply patch 5 - See the barcode in holds to pull report 6 - You can now grab the correct item (but don't yet) 7 - Place a next available hold on the same title 8 - See the report now shows one possible valid barcode with the text "or any available." 9 - Check in a different item that fills the next available hold 10 - Now the report shows the single item for the borrower Signed-off-by: Michal Denar <black23@gmail.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
I feel this patch is helpful as it is, but I could imagine more improvements in the future. If I have a record with 3 holds on it, first is record level and 2 are item level, the report will tell me only about the next hold: Get any item. This is correct, but it might be even nicer if it told me to get barcode a, barcode b and any other other item. I think this would be not easy to implment, so we should have this patch here first.
Pushed to master for 20.11, thanks to everybody involved!
backported to 20.05.x for 20.05.05
enhancement, not backported to 19.11.x