[Bug 31692] New: Let librarians change item level holds to record level holds when possible
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Bug ID: 31692 Summary: Let librarians change item level holds to record level holds when possible Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Hold requests Assignee: koha-bugs@lists.koha-community.org Reporter: lari.taskula@hypernova.fi QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com Koha has item level holds (only a specific item was requested) and record level holds (any next available item). There can be a case when a librarian wishes to change an item level hold to a record level hold. This is currently not (easily) possible, as they would have to cancel the hold and then set another type of hold and place it to the previous point in hold queue. This Bug aims to add an easy way to change hold types. Only one type of hold is allowed simultaneously for a record and a patron. The two types cannot be mixed. In other words a patron can't have both item level and record level holds on a specific record at the same time. This must be kept in mind when designing such functionality. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #1 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 141388 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=141388&action=edit Bug 31692: Add Koha::Hold::change_type and unit tests To test: 1. prove t/db_dependent/Koha/Hold.t -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #2 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 141390 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=141390&action=edit Bug 31692: Allow librarians to change hold type in staff client To test: 1. Apply patch 2. Add item level hold to a record/item, make sure patron has no other holds on that record 3. Go to /cgi-bin/koha/reserve/request.pl?biblionumber=xxx where xxx is the record you placed the hold for 4. Under "Existing holds" table, in "Details" column you should see "Only item <barcode> X". Click the red X and click Update hold(s) 5. Observe "Only item <barcode>" has changed to "Next available" 6. Cancel the hold and add two item level holds 7. Under "Existing holds" table, in "Details" column you should see "Only item <barcode>", but no red X for either holds. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |lari.taskula@hypernova.fi |ity.org | -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lucas@bywatersolutions.com --- Comment #3 from Lucas Gass <lucas@bywatersolutions.com> --- I think the use of another red X on the page to change the item level hold into a bib level one is confusing. There are already a few red X's on the page and that particular one offers no explanation to the user as to what it will do. Maybe we need a dropdown? I'm not sure if that is the proper solution either. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA --- Comment #4 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- (In reply to Lucas Gass from comment #3)
I think the use of another red X on the page to change the item level hold into a bib level one is confusing. There are already a few red X's on the page and that particular one offers no explanation to the user as to what it will do.
Maybe we need a dropdown? I'm not sure if that is the proper solution either.
Lucas is making a good point, could you please have a look Lari? Maybe some other icon could be used? I have to admit my first idea was some kind of upwards arrow for (level upwards) but... there are also a lot of arrows already. A tooltip for sure makes sense. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #141388|0 |1 is obsolete| | --- Comment #5 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 142102 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=142102&action=edit Bug 31692: Add Koha::Hold::change_type and unit tests To test: 1. prove t/db_dependent/Koha/Hold.t -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #141390|0 |1 is obsolete| | --- Comment #6 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 142103 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=142103&action=edit Bug 31692: Allow librarians to change hold type in staff client To test: 1. Apply patch 2. Add item level hold to a record/item, make sure patron has no other holds on that record 3. Go to /cgi-bin/koha/reserve/request.pl?biblionumber=xxx where xxx is the record you placed the hold for 4. Under "Existing holds" table, in "Details" column you should see "Only item <barcode> X". Click the red X and click Update hold(s) 5. Observe "Only item <barcode>" has changed to "Next available" 6. Cancel the hold and add two item level holds 7. Under "Existing holds" table, in "Details" column you should see "Only item <barcode>", but no red X for either holds. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #7 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 142104 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=142104&action=edit Bug 31692: (follow-up) Add tooltip for changing hold type -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #8 from Lari Taskula <lari.taskula@hypernova.fi> --- (In reply to Lucas Gass from comment #3)
I think the use of another red X on the page to change the item level hold into a bib level one is confusing. There are already a few red X's on the page and that particular one offers no explanation to the user as to what it will do.
Maybe we need a dropdown? I'm not sure if that is the proper solution either.
(In reply to Katrin Fischer from comment #4)
Lucas is making a good point, could you please have a look Lari? Maybe some other icon could be used? I have to admit my first idea was some kind of upwards arrow for (level upwards) but... there are also a lot of arrows already. A tooltip for sure makes sense.
Thanks for the feedback, Lucas and Katrin. I've added a tooltip in the form of "title" attribute - if Koha has another way of showing tooltips then please correct me. Great suggestion! About the X, (by the way, it just became green with the new theme) my reasoning for choosing it was to represent something to "remove" the item from the hold. Perhaps an "unlinking" icon could work too. https://fontawesome.com/v4/icon/chain-broken Dropdown is also a nice idea in the sense that it could be extended to show other items as well and let librarians change the item if necessary. But that is unfortunately out of scope for what I am trying to achieve here. I have also rebased to resolve some git conflicts and patches can now be tested again. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #142103|0 |1 is obsolete| | --- Comment #9 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 142106 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=142106&action=edit Bug 31692: Allow librarians to change hold type in staff client To test: 1. Apply patch 2. Add item level hold to a record/item, make sure patron has no other holds on that record 3. Go to /cgi-bin/koha/reserve/request.pl?biblionumber=xxx where xxx is the record you placed the hold for 4. Under "Existing holds" table, in "Details" column you should see "Only item <barcode> X". Click the red X and click Update hold(s) 5. Observe "Only item <barcode>" has changed to "Next available" 6. Cancel the hold and add two item level holds 7. Under "Existing holds" table, in "Details" column you should see "Only item <barcode>", but no red X for either holds. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #142104|0 |1 is obsolete| | --- Comment #10 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 142107 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=142107&action=edit Bug 31692: (follow-up) Add tooltip for changing hold type -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #11 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 142108 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=142108&action=edit Bug 31692: (follow-up) Use fa-chain and fa-chain-broken icons -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #12 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 142109 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=142109&action=edit Bug 31692: (follow-up) Rewording tooltip Using "change to next available item" would require changing title and aria-label dynamically to "change to only item ..." when clicked. While possible, simply using "toggle hold type" covers both cases. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #142106|0 |1 is obsolete| | --- Comment #13 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 142114 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=142114&action=edit Bug 31692: Allow librarians to change hold type in staff client To test: 1. Apply patch 2. Add item level hold to a record/item, make sure patron has no other holds on that record 3. Go to /cgi-bin/koha/reserve/request.pl?biblionumber=xxx where xxx is the record you placed the hold for 4. Under "Existing holds" table, in "Details" column you should see "Only item <barcode> X". Click the red X and click Update hold(s) 5. Observe "Only item <barcode>" has changed to "Next available" 6. Cancel the hold and add two item level holds 7. Under "Existing holds" table, in "Details" column you should see "Only item <barcode>", but no red X for either holds. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #142107|0 |1 is obsolete| | --- Comment #14 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 142115 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=142115&action=edit Bug 31692: (follow-up) Add tooltip for changing hold type -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #142108|0 |1 is obsolete| | --- Comment #15 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 142116 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=142116&action=edit Bug 31692: (follow-up) Use fa-chain and fa-chain-broken icons -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #142109|0 |1 is obsolete| | --- Comment #16 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 142117 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=142117&action=edit Bug 31692: (follow-up) Rewording tooltip Using "change to next available item" would require changing title and aria-label dynamically to "change to only item ..." when clicked. While possible, simply using "toggle hold type" covers both cases. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #17 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 142118 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=142118&action=edit Bug 31692: (alternative look) Use select dropdown -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #18 from Lari Taskula <lari.taskula@hypernova.fi> --- In reply to Lucas Gass from comment #3)
Maybe we need a dropdown? I'm not sure if that is the proper solution either.
I actually wanted to see what it looks like so I implemented patch: Bug 31692: (alternative look) Use select dropdown I think it looks quite okay. It also allows me to get rid of all extra JavaScript and tooltip issues. Great idea Lucas. Thoughts? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #19 from Lucas Gass <lucas@bywatersolutions.com> --- (In reply to Lari Taskula from comment #18)
In reply to Lucas Gass from comment #3)
Maybe we need a dropdown? I'm not sure if that is the proper solution either.
I actually wanted to see what it looks like so I implemented patch:
Bug 31692: (alternative look) Use select dropdown
I think it looks quite okay. It also allows me to get rid of all extra JavaScript and tooltip issues. Great idea Lucas. Thoughts?
Lari, I do like that much better! Everything works great. I do wonder if there is still going to be some confusion on what the dropdown does. Do we need a hint or a tip to help describe what the dropdown is going to do? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #142102|0 |1 is obsolete| | Attachment #142115|0 |1 is obsolete| | Attachment #142116|0 |1 is obsolete| | Attachment #142117|0 |1 is obsolete| | Attachment #142118|0 |1 is obsolete| | --- Comment #20 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 142586 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=142586&action=edit Bug 31692: Add Koha::Hold::change_type and unit tests To test: 1. prove t/db_dependent/Koha/Hold.t -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #142114|0 |1 is obsolete| | --- Comment #21 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 142587 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=142587&action=edit Bug 31692: Allow librarians to change hold type in staff client To test: 1. Apply patch 2. Add item level hold to a record/item, make sure patron has no other holds on that record 3. Go to /cgi-bin/koha/reserve/request.pl?biblionumber=xxx where xxx is the record you placed the hold for 4. Under "Existing holds" table, in "Details" column you should see "Only item <barcode>" dropdown 5. Select "Next available" from the dropdown 6. Click Update hold(s) 7. Observe dropdown value has changed from "Only item <barcode>" to "Next available" 8. Cancel the hold and add two item level holds 9. Under "Existing holds" table, in "Details" column you should see "Only item <barcode>", but no select dropdown -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #22 from Lari Taskula <lari.taskula@hypernova.fi> --- (In reply to Lucas Gass from comment #19)
I do wonder if there is still going to be some confusion on what the dropdown does. Do we need a hint or a tip to help describe what the dropdown is going to do?
I think the dropdown values themselves explain what it does. If you think more detailed description is needed, feel free to make a suggestion. On another note I've squashed the follow-ups into the main patch and updated test plan. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #142586|0 |1 is obsolete| | --- Comment #23 from David Nind <david@davidnind.com> --- Created attachment 142595 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=142595&action=edit Bug 31692: Add Koha::Hold::change_type and unit tests To test: 1. prove t/db_dependent/Koha/Hold.t Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #142587|0 |1 is obsolete| | --- Comment #24 from David Nind <david@davidnind.com> --- Created attachment 142596 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=142596&action=edit Bug 31692: Allow librarians to change hold type in staff client To test: 1. Apply patch 2. Add item level hold to a record/item, make sure patron has no other holds on that record 3. Go to /cgi-bin/koha/reserve/request.pl?biblionumber=xxx where xxx is the record you placed the hold for 4. Under "Existing holds" table, in "Details" column you should see "Only item <barcode>" dropdown 5. Select "Next available" from the dropdown 6. Click Update hold(s) 7. Observe dropdown value has changed from "Only item <barcode>" to "Next available" 8. Cancel the hold and add two item level holds 9. Under "Existing holds" table, in "Details" column you should see "Only item <barcode>", but no select dropdown Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david@davidnind.com --- Comment #25 from David Nind <david@davidnind.com> --- I've signed off, as everything works as per the test plan. For step 8, I added two item level holds for the same patron - I hope that is what was required. My only suggestion would be to change the "Details" column label to "Hold type". -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #26 from Lari Taskula <lari.taskula@hypernova.fi> --- (In reply to David Nind from comment #25)
For step 8, I added two item level holds for the same patron - I hope that is what was required.
Yes that's correct.
My only suggestion would be to change the "Details" column label to "Hold type".
It looks like the details column can also contain a "Non priority hold" text. If we were to rename the column, can we consider "non priority hold" as a "hold type", or does hold type just mean an item/biblio hold? https://github.com/Koha-Community/Koha/blob/48bf9b1d91dbbc336f859ac1a83d7d25... -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #27 from David Nind <david@davidnind.com> --- (In reply to Lari Taskula from comment #26)
My only suggestion would be to change the "Details" column label to "Hold type".
It looks like the details column can also contain a "Non priority hold" text. If we were to rename the column, can we consider "non priority hold" as a "hold type", or does hold type just mean an item/biblio hold? https://github.com/Koha-Community/Koha/blob/ 48bf9b1d91dbbc336f859ac1a83d7d25b0e6a3c1/koha-tmpl/intranet-tmpl/prog/en/ includes/holds_table.inc#L189-L191
I wasn't aware of that :D Also noticed that the details column already exists, so happy to leave the same without any changes. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Small patch Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #142595|0 |1 is obsolete| | --- Comment #28 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 142818 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=142818&action=edit Bug 31692: Add Koha::Hold::change_type and unit tests To test: 1. prove t/db_dependent/Koha/Hold.t Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #142596|0 |1 is obsolete| | --- Comment #29 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 142819 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=142819&action=edit Bug 31692: Allow librarians to change hold type in staff client To test: 1. Apply patch 2. Add item level hold to a record/item, make sure patron has no other holds on that record 3. Go to /cgi-bin/koha/reserve/request.pl?biblionumber=xxx where xxx is the record you placed the hold for 4. Under "Existing holds" table, in "Details" column you should see "Only item <barcode>" dropdown 5. Select "Next available" from the dropdown 6. Click Update hold(s) 7. Observe dropdown value has changed from "Only item <barcode>" to "Next available" 8. Cancel the hold and add two item level holds 9. Under "Existing holds" table, in "Details" column you should see "Only item <barcode>", but no select dropdown Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #30 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- I was starting to doubt myself, but this patch broke my evil "FQA" streak. Thanks Lari, I think the discussion resulted in a nice clean GUI here! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |rel_22_11_candidate Status|Passed QA |Failed QA --- Comment #31 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Oh no... I found an issue :( You remove the itemnumber, which would have been enough on old versions, but now we also need to change reserves.item_level_hold Should be an easy fix - please follow-up. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #32 from Lari Taskula <lari.taskula@hypernova.fi> --- (In reply to Katrin Fischer from comment #31)
Oh no... I found an issue :(
You remove the itemnumber, which would have been enough on old versions, but now we also need to change reserves.item_level_hold
Should be an easy fix - please follow-up.
Good catch. Although it has been there for 3 years I didn't know of this "new" column :) Fix will follow. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #33 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 143151 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143151&action=edit Bug 31692: (follow-up) Adjust reserves.item_level_hold Original patch missed handling reserves.item_level_hold column. To test: 1. prove t/db_dependent/Koha/Hold.t -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #34 from David Nind <david@davidnind.com> --- The patches still apply, but something seems changed: 1. Step 7 - There is now no dropdown list (before there was a drop down list with Only item <barcode> and Next available); selecting next available and updating removes the drop down list). 2. Step 9 - There is still a dropdown list with the two options (Only item <barcode> and Next available). Happy to sign off again if that is what is expected. Tests pass both before and after patches are applied. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #35 from Lari Taskula <lari.taskula@hypernova.fi> --- (In reply to David Nind from comment #34)
The patches still apply, but something seems changed:
1. Step 7 - There is now no dropdown list (before there was a drop down list with Only item <barcode> and Next available); selecting next available and updating removes the drop down list).
The wording in this step is a bit silly, it appears I didn't properly rewrite it when I squashed the new dropdown menu with the original patch. What you describe seems expected behavior to me. After changing from item level hold to record level hold there should no longer be a dropdown. So at step 7 if you see just the text "Next available" then that is to be expected. This is because the patches here do not add GUI support for going from record level to item level hold, but only the other way around.
2. Step 9 - There is still a dropdown list with the two options (Only item <barcode> and Next available).
This is not expected, did you add the two item level holds for the same patron? I noticed the test plan does not clarify that this is required. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #36 from David Nind <david@davidnind.com> --- (In reply to Lari Taskula from comment #35)
(In reply to David Nind from comment #34)
The patches still apply, but something seems changed:
1. Step 7 - There is now no dropdown list (before there was a drop down list with Only item <barcode> and Next available); selecting next available and updating removes the drop down list).
The wording in this step is a bit silly, it appears I didn't properly rewrite it when I squashed the new dropdown menu with the original patch.
What you describe seems expected behavior to me. After changing from item level hold to record level hold there should no longer be a dropdown. So at step 7 if you see just the text "Next available" then that is to be expected.
This is because the patches here do not add GUI support for going from record level to item level hold, but only the other way around.
Thanks for clarifying!
2. Step 9 - There is still a dropdown list with the two options (Only item <barcode> and Next available).
This is not expected, did you add the two item level holds for the same patron? I noticed the test plan does not clarify that this is required.
I added for different patrons, so will use the same patron. Just re-testing, so should be signed off shortly! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #142818|0 |1 is obsolete| | --- Comment #37 from David Nind <david@davidnind.com> --- Created attachment 143160 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143160&action=edit Bug 31692: Add Koha::Hold::change_type and unit tests To test: 1. prove t/db_dependent/Koha/Hold.t Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #142819|0 |1 is obsolete| | --- Comment #38 from David Nind <david@davidnind.com> --- Created attachment 143161 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143161&action=edit Bug 31692: Allow librarians to change hold type in staff client To test: 1. Apply patch 2. Add item level hold to a record/item, make sure patron has no other holds on that record 3. Go to /cgi-bin/koha/reserve/request.pl?biblionumber=xxx where xxx is the record you placed the hold for 4. Under "Existing holds" table, in "Details" column you should see "Only item <barcode>" dropdown 5. Select "Next available" from the dropdown 6. Click Update hold(s) 7. Observe dropdown value has changed from "Only item <barcode>" to "Next available" 8. Cancel the hold and add two item level holds 9. Under "Existing holds" table, in "Details" column you should see "Only item <barcode>", but no select dropdown Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #143151|0 |1 is obsolete| | --- Comment #39 from David Nind <david@davidnind.com> --- Created attachment 143162 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143162&action=edit Bug 31692: (follow-up) Adjust reserves.item_level_hold Original patch missed handling reserves.item_level_hold column. To test: 1. prove t/db_dependent/Koha/Hold.t Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #40 from David Nind <david@davidnind.com> --- I've ended up adding sign off lines for the first two patches - not sure how to remove those! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Patch doesn't apply --- Comment #41 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- There is a conflict in the .t, can you please fix? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #143160|0 |1 is obsolete| | --- Comment #42 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 143614 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143614&action=edit Bug 31692: Add Koha::Hold::change_type and unit tests To test: 1. prove t/db_dependent/Koha/Hold.t -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #143161|0 |1 is obsolete| | --- Comment #43 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 143615 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143615&action=edit Bug 31692: Allow librarians to change hold type in staff client To test: 1. Apply patch 2. Add item level hold to a record/item, make sure patron has no other holds on that record 3. Go to /cgi-bin/koha/reserve/request.pl?biblionumber=xxx where xxx is the record you placed the hold for 4. Under "Existing holds" table, in "Details" column you should see "Only item <barcode>" dropdown 5. Select "Next available" from the dropdown 6. Click Update hold(s) 7. Observe dropdown value has changed from "Only item <barcode>" to "Next available" 8. Cancel the hold and add two item level holds 9. Under "Existing holds" table, in "Details" column you should see "Only item <barcode>", but no select dropdown -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #143162|0 |1 is obsolete| | --- Comment #44 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 143616 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143616&action=edit Bug 31692: (follow-up) Adjust reserves.item_level_hold Original patch missed handling reserves.item_level_hold column. To test: 1. prove t/db_dependent/Koha/Hold.t -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #45 from Lari Taskula <lari.taskula@hypernova.fi> --- I accidentally removed the signed-off lines from patches, fixing... -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #143614|0 |1 is obsolete| | --- Comment #46 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 143617 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143617&action=edit Bug 31692: Add Koha::Hold::change_type and unit tests To test: 1. prove t/db_dependent/Koha/Hold.t Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #143615|0 |1 is obsolete| | --- Comment #47 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 143618 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143618&action=edit Bug 31692: Allow librarians to change hold type in staff client To test: 1. Apply patch 2. Add item level hold to a record/item, make sure patron has no other holds on that record 3. Go to /cgi-bin/koha/reserve/request.pl?biblionumber=xxx where xxx is the record you placed the hold for 4. Under "Existing holds" table, in "Details" column you should see "Only item <barcode>" dropdown 5. Select "Next available" from the dropdown 6. Click Update hold(s) 7. Observe dropdown is gone and cell value has changed from "Only item <barcode>" to "Next available" 8. Cancel the hold and add two item level holds for the same patron 9. Under "Existing holds" table, in "Details" column you should see "Only item <barcode>", but no select dropdown Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #143616|0 |1 is obsolete| | --- Comment #48 from Lari Taskula <lari.taskula@hypernova.fi> --- Created attachment 143619 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143619&action=edit Bug 31692: (follow-up) Adjust reserves.item_level_hold Original patch missed handling reserves.item_level_hold column. To test: 1. prove t/db_dependent/Koha/Hold.t Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #49 from Lari Taskula <lari.taskula@hypernova.fi> --- (In reply to David Nind from comment #40)
I've ended up adding sign off lines for the first two patches - not sure how to remove those!
I've removed those for you. (In reply to Katrin Fischer from comment #41)
There is a conflict in the .t, can you please fix?
Fixed! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #143617|0 |1 is obsolete| | --- Comment #50 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 143627 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143627&action=edit Bug 31692: Add Koha::Hold::change_type and unit tests To test: 1. prove t/db_dependent/Koha/Hold.t Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #143618|0 |1 is obsolete| | --- Comment #51 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 143628 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143628&action=edit Bug 31692: Allow librarians to change hold type in staff client To test: 1. Apply patch 2. Add item level hold to a record/item, make sure patron has no other holds on that record 3. Go to /cgi-bin/koha/reserve/request.pl?biblionumber=xxx where xxx is the record you placed the hold for 4. Under "Existing holds" table, in "Details" column you should see "Only item <barcode>" dropdown 5. Select "Next available" from the dropdown 6. Click Update hold(s) 7. Observe dropdown is gone and cell value has changed from "Only item <barcode>" to "Next available" 8. Cancel the hold and add two item level holds for the same patron 9. Under "Existing holds" table, in "Details" column you should see "Only item <barcode>", but no select dropdown Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #143619|0 |1 is obsolete| | --- Comment #52 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 143629 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143629&action=edit Bug 31692: (follow-up) Adjust reserves.item_level_hold Original patch missed handling reserves.item_level_hold column. To test: 1. prove t/db_dependent/Koha/Hold.t Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #53 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Thx for the quick fix, Lari. This has some string changes, so not sure if we can still make it. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tomascohen@gmail.com --- Comment #54 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Can we please change the method signature? I'd expect $hold->change_type( { type => [biblio|item], [ item_id => $item->id, ] } ); Or even split it in two methods: $hold->make_biblio_level $hold->make_item_level( $item->id ) each of them checking the right things and throwing the required exceptions. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #55 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- (In reply to Tomás Cohen Arazi from comment #54)
Can we please change the method signature? I'd expect
$hold->change_type( { type => [biblio|item], [ item_id => $item->id, ] } );
Or even split it in two methods:
$hold->make_biblio_level $hold->make_item_level( $item->id )
each of them checking the right things and throwing the required exceptions.
Maybe use switch? switch_to_record_level (we want to avoid biblio) and switch_to_item_level -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords|rel_22_11_candidate |rel_23_05_candidate -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #56 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Lari, are you still willing to work on this? Thanks -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart+koha@gmail. | |com Keywords|rel_23_05_candidate |rel_23_11_candidate -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to master Version(s)| |23.11.00 released in| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #57 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Pushed to master for 23.11. Nice work everyone, thanks! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fridolin.somers@biblibre.co | |m --- Comment #58 from Fridolin Somers <fridolin.somers@biblibre.com> --- Enhancement not pushed to 23.05.x -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to master |Needs documenting -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Katie Bliss <kebliss@dmpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kebliss@dmpl.org -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |release-notes-needed -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #59 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Testing with a superlibrarian, but I don't get a pull down in the details column on my item level holds. Any idea what could go wrong there? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #60 from David Nind <david@davidnind.com> --- (In reply to Katrin Fischer from comment #59)
Testing with a superlibrarian, but I don't get a pull down in the details column on my item level holds. Any idea what could go wrong there?
I'm not seeing it either... -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #61 from Lari Taskula <lari.taskula@hypernova.fi> --- (In reply to Katrin Fischer from comment #59)
Testing with a superlibrarian, but I don't get a pull down in the details column on my item level holds. Any idea what could go wrong there?
Looks like the change introduced to koha-tmpl/intranet-tmpl/prog/en/includes/holds_table.inc in "Allow librarians to change hold type in staff client" patch was reverted by Bug 33575. So this feature is no longer working as of now. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lari Taskula <lari.taskula@hypernova.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |35483 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35483 [Bug 35483] Restore item level to record level hold switch in hold table -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 --- Comment #62 from Lari Taskula <lari.taskula@hypernova.fi> --- See Bug 35483 for restoring the missing feature. (In reply to Tomás Cohen Arazi from comment #56)
Lari, are you still willing to work on this? Thanks
Sorry I had either missed or forgot to answer your comment! (In reply to Tomás Cohen Arazi from comment #54)
Can we please change the method signature? I'd expect
$hold->change_type( { type => [biblio|item], [ item_id => $item->id, ] } );
Or even split it in two methods:
$hold->make_biblio_level $hold->make_item_level( $item->id )
Well, why not, if you feel it's still important even after pushing the Bug (sorry again for the late answer). I'm not extremely attached or strongly opinionated about the choice I made - I suppose I was thinking of it more as a toggle hence not introducing two methods. The itemnumber param in my sense specifies the type already, if given => item, if not given => biblio. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Philip Orr <philip.orr@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Documentation| |https://gitlab.com/koha-com submission| |munity/koha-manual/-/merge_ | |requests/832 Documentation| |Philip Orr contact| | CC| |philip.orr@lmscloud.de -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Philip Orr <philip.orr@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|Needs documenting |RESOLVED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31692 Lisette Scheer <lisette@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords|release-notes-needed | -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org