[Bug 11634] New: Allow renewal of item with unfilled holds if other available items can fill that hold
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Bug ID: 11634 Summary: Allow renewal of item with unfilled holds if other available items can fill that hold Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Hold requests Assignee: koha-bugs@lists.koha-community.org Reporter: kyle@bywatersolutions.com QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com The current holds behavior in Koha allows a situation like this: - Patron A has an item currently checked out. - Patron B places a hold on the next available copy of that title. - Then Patron A will not be able to renew his item, even if there are other available copies of that title that could potentially fill Patron B's hold. Since this seems unfair to Patron A, we should allow renewal of items even if there are unfilled holds, but those holds could all be filled with currently available items. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Allow renewal of item with |Allow renewal of item with |unfilled holds if other |unfilled holds if other |available items can fill |available items can fill |that hold |those holds -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff Patch complexity|--- |Medium patch -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 --- Comment #1 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 24866 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=24866&action=edit Bug 11634 - Allow renewal of item with unfilled holds if other available items can fill those holds The current holds behavior in Koha allows a situation like this: - Patron A has an item currently checked out. - Patron B places a hold on the next available copy of that title. - Then Patron A will not be able to renew his item, even if there are other available copies of that title that could potentially fill Patron B's hold. Since this seems unfair to Patron A, we should allow renewal of items even if there are unfilled holds, but those holds could all be filled with currently available items. Test Plan: 1) Apply this patch 2) Create a record with two items 3) Check out the item to a patron 4) Place a hold on the record 5) Note you cannot renew the item for the patron 6) Enable the new system preference AllowRenewalIfOtherItemsAvailable 7) Note you can now renew the item, as all the holds can be satisfied by available items. 8) Place a second hold on the record 9) Note you can no longer renew the item, as all the holds *cannot* be filled by currently available items -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Holger Meißner <h.meissner.82@web.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #24866|0 |1 is obsolete| | --- Comment #2 from Holger Meißner <h.meissner.82@web.de> --- Created attachment 25032 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25032&action=edit Bug 11634 - Allow renewal of item with unfilled holds if other available items can fill those holds The current holds behavior in Koha allows a situation like this: - Patron A has an item currently checked out. - Patron B places a hold on the next available copy of that title. - Then Patron A will not be able to renew his item, even if there are other available copies of that title that could potentially fill Patron B's hold. Since this seems unfair to Patron A, we should allow renewal of items even if there are unfilled holds, but those holds could all be filled with currently available items. Test Plan: 1) Apply this patch 2) Create a record with two items 3) Check out the item to a patron 4) Place a hold on the record 5) Note you cannot renew the item for the patron 6) Enable the new system preference AllowRenewalIfOtherItemsAvailable 7) Note you can now renew the item, as all the holds can be satisfied by available items. 8) Place a second hold on the record 9) Note you can no longer renew the item, as all the holds *cannot* be filled by currently available items Signed-off-by: Holger Meißner <h.meissner.82@web.de> -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Holger Meißner <h.meissner.82@web.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |h.meissner.82@web.de -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 --- Comment #3 from Holger Meißner <h.meissner.82@web.de> --- Works as described. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |m.de.rooy@rijksmuseum.nl Assignee|koha-bugs@lists.koha-commun |kyle@bywatersolutions.com |ity.org | -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Chris Rohde <crohde@roseville.ca.us> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |crohde@roseville.ca.us --- Comment #4 from Chris Rohde <crohde@roseville.ca.us> --- Tested and works as described. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Melissa Lefebvre <melissa@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25032|0 |1 is obsolete| | --- Comment #5 from Melissa Lefebvre <melissa@bywatersolutions.com> --- Created attachment 25148 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25148&action=edit Bug 11634 - Allow renewal of item with unfilled holds if other available items can fill those holds -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |In Discussion CC| |jonathan.druart@biblibre.co | |m --- Comment #6 from Jonathan Druart <jonathan.druart@biblibre.com> --- Hi Kyle, To me the new syspref does not make sense. Don't you consider the existing behavior as a bug? I don't see a situation where the renew should be refused. Maybe could you send an email to the koha mailing list in order to ask what librarians think about the existing behavior? Switch to in discussion. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de --- Comment #7 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- I am not sure everyone will agree that this is a bug - it seems to have worked like this forever. I think with a pref we would be on the safe side. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 --- Comment #8 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- In this discussion I first tended to agree with Jonathan. We should not need a syspref to have odd behavior stay in Koha. But on the other hand will the potential items really be available? If there is only one other item, and you cannot longer find it? So maybe we should indeed stay on the safer side with Katrin :) yasp.. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|In Discussion |Signed Off --- Comment #9 from Kyle M Hall <kyle@bywatersolutions.com> --- Agreed, I also think it's safer to pref it than to not. We can always remove the preference at a later date. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #10 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Kyle, could you please add the pref? Failing for now so it shows up in your queue :) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 --- Comment #11 from Kyle M Hall <kyle@bywatersolutions.com> --- (In reply to Katrin Fischer from comment #10)
Hi Kyle, could you please add the pref? Failing for now so it shows up in your queue :)
This already has a system preference, AllowRenewalIfOtherItemsAvailable! Is there something else that needs to be taken care of? Also, since this is being consider bug-ish, I think maybe we should enable the feature by default. I'll post a followup for that. If you don't think that's a good idea you can just deprecate the patch. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 --- Comment #12 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 25822 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25822&action=edit Bug 11634 [QA Followup] - Enable by default -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 --- Comment #13 from Galen Charlton <gmcharlt@gmail.com> --- (In reply to Kyle M Hall from comment #11)
(In reply to Katrin Fischer from comment #10)
Hi Kyle, could you please add the pref? Failing for now so it shows up in your queue :)
This already has a system preference, AllowRenewalIfOtherItemsAvailable! Is there something else that needs to be taken care of?
AllowRenewalIfOtherItemsAvailable is the minimum, IMO, as I would prefer it as a per-library setting, but as a YASP, it suffices for now.
Also, since this is being consider bug-ish, I think maybe we should enable the feature by default. I'll post a followup for that. If you don't think that's a good idea you can just deprecate the patch.
I think we've just exposed a difference in library practices, because I know of libraries that absolutely do not want renewals to be allowed if there are pending requests, and who would very much consider it a bug if they were suddenly allowed following an upgrade. Consequently, I will /not/ be pushing the follow-up as it stands now, as it changes existing behavior upon upgrade. I am neutral on the question of the default value for this one for new installations. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25822|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=11634 --- Comment #14 from Kyle M Hall <kyle@bywatersolutions.com> --- (In reply to Galen Charlton from comment #13) Thanks for the comments! I've depped my followup. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Galen Charlton <gmcharlt@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #15 from Galen Charlton <gmcharlt@gmail.com> --- Setting to failed QA, nonetheless; such a change to the holds API needs unit tests. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 --- Comment #16 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 25825 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25825&action=edit Bug 11634 [QA Followup] - Unit Tests These new unit tests will fail due to the fact that Koha::Database uses a separate dbh handle than C4::Context->dbh -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 --- Comment #17 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 25826 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25826&action=edit Make Koha::Database use C4::Context->dbh -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25826|0 |1 is obsolete| | --- Comment #18 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 25827 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25827&action=edit Make Koha::Database use C4::Context->dbh -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25827|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=11634 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off Depends on| |11891 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 --- Comment #19 from Galen Charlton <gmcharlt@gmail.com> --- Comment on attachment 25148 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25148 Bug 11634 - Allow renewal of item with unfilled holds if other available items can fill those holds Review of attachment 25148: --> (http://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=11634&attachment=25148) ----------------------------------------------------------------- ::: C4/Circulation.pm @@ +2516,5 @@
+ # Get all other reserves that could have been filled by this item + my @borrowernumbers; + while (1) { + my ( $reserve_found, $reserve, undef ) = + C4::Reserves::CheckReserves( $itemnumber, undef, undef,
I'm not convinced that CheckReserves() is a sufficient test -- IsAvailableForItemLevelRequest() may need to be checked as well. Otherwise, damaged, lost, and withdrawn items could "count" as available for filling the hold when that is not actually the case. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 --- Comment #20 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 25837 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25837&action=edit Bug 11634 [QA Followup 2] - Implement check for IsAvailableForItemLevelRequest -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #21 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Some older tests fail on applying these patches: # Biblio-level hold, renewal test not ok 21 - (Bug 10663) Cannot renew, reserved # Failed test '(Bug 10663) Cannot renew, reserved' # at t/db_dependent/Circulation.t line 266. # got: '0' # expected: '1' not ok 22 - (Bug 10663) Cannot renew, reserved # Failed test '(Bug 10663) Cannot renew, reserved' # at t/db_dependent/Circulation.t line 268. # got: '0' # expected: '1' -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 --- Comment #22 from Kyle M Hall <kyle@bywatersolutions.com> --- These unit tests will continue to fail until both C4::Context::dbh and Koha::Database share a database handle. Galen is planning a patch to make C4::Context use the database handle from Koha::Database, but hasn't posted his patch yet. He has told me it should be ready by this Friday. (In reply to Katrin Fischer from comment #21)
Some older tests fail on applying these patches:
# Biblio-level hold, renewal test not ok 21 - (Bug 10663) Cannot renew, reserved # Failed test '(Bug 10663) Cannot renew, reserved' # at t/db_dependent/Circulation.t line 266. # got: '0' # expected: '1' not ok 22 - (Bug 10663) Cannot renew, reserved # Failed test '(Bug 10663) Cannot renew, reserved' # at t/db_dependent/Circulation.t line 268. # got: '0' # expected: '1'
-- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25148|0 |1 is obsolete| | --- Comment #23 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 28212 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=28212&action=edit Bug 11634 - Allow renewal of item with unfilled holds if other available items can fill those holds The current holds behavior in Koha allows a situation like this: - Patron A has an item currently checked out. - Patron B places a hold on the next available copy of that title. - Then Patron A will not be able to renew his item, even if there are other available copies of that title that could potentially fill Patron B's hold. Since this seems unfair to Patron A, we should allow renewal of items even if there are unfilled holds, but those holds could all be filled with currently available items. Test Plan: 1) Apply this patch 2) Create a record with two items 3) Check out the item to a patron 4) Place a hold on the record 5) Note you cannot renew the item for the patron 6) Enable the new system preference AllowRenewalIfOtherItemsAvailable 7) Note you can now renew the item, as all the holds can be satisfied by available items. 8) Place a second hold on the record 9) Note you can no longer renew the item, as all the holds *cannot* be filled by currently available items Signed-off-by: Holger Meissner <h.meissner.82@web.de> Signed-off-by: Chris Rohde <crohde@roseville.ca.us> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25825|0 |1 is obsolete| | --- Comment #24 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 28213 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=28213&action=edit Bug 11634 [QA Followup] - Unit Tests These new unit tests will fail due to the fact that Koha::Database uses a separate dbh handle than C4::Context->dbh -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25837|0 |1 is obsolete| | --- Comment #25 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 28214 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=28214&action=edit Bug 11634 [QA Followup 2] - Implement check for IsAvailableForItemLevelRequest -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nengard@gmail.com -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #28212|0 |1 is obsolete| | Attachment #28213|0 |1 is obsolete| | Attachment #28214|0 |1 is obsolete| | --- Comment #26 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 31706 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=31706&action=edit Bug 11634 - Allow renewal of item with unfilled holds if other available items can fill those holds The current holds behavior in Koha allows a situation like this: - Patron A has an item currently checked out. - Patron B places a hold on the next available copy of that title. - Then Patron A will not be able to renew his item, even if there are other available copies of that title that could potentially fill Patron B's hold. Since this seems unfair to Patron A, we should allow renewal of items even if there are unfilled holds, but those holds could all be filled with currently available items. Test Plan: 1) Apply this patch 2) Create a record with two items 3) Check out the item to a patron 4) Place a hold on the record 5) Note you cannot renew the item for the patron 6) Enable the new system preference AllowRenewalIfOtherItemsAvailable 7) Note you can now renew the item, as all the holds can be satisfied by available items. 8) Place a second hold on the record 9) Note you can no longer renew the item, as all the holds *cannot* be filled by currently available items Signed-off-by: Holger Meissner <h.meissner.82@web.de> Signed-off-by: Chris Rohde <crohde@roseville.ca.us> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 --- Comment #27 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 31707 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=31707&action=edit Bug 11634 [QA Followup] - Unit Tests These new unit tests will fail due to the fact that Koha::Database uses a separate dbh handle than C4::Context->dbh -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 --- Comment #28 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 31708 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=31708&action=edit Bug 11634 [QA Followup 2] - Implement check for IsAvailableForItemLevelRequest -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 --- Comment #29 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 31709 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=31709&action=edit Bug 11634 [QA Followup 3] - Found holds should be considered unavailable -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Patch doesn't apply --- Comment #30 from Jonathan Druart <jonathan.druart@biblibre.com> --- Applying: Bug 11634 [QA Followup 3] - Found holds should be considered unavailable fatal: sha1 information is lacking or useless (C4/Reserves.pm). Repository lacks necessary blobs to fall back on 3-way merge. Cannot fall back to three-way merge. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #31706|0 |1 is obsolete| | Attachment #31707|0 |1 is obsolete| | Attachment #31708|0 |1 is obsolete| | Attachment #31709|0 |1 is obsolete| | --- Comment #31 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 32034 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=32034&action=edit Bug 11634 - Allow renewal of item with unfilled holds if other available items can fill those holds The current holds behavior in Koha allows a situation like this: - Patron A has an item currently checked out. - Patron B places a hold on the next available copy of that title. - Then Patron A will not be able to renew his item, even if there are other available copies of that title that could potentially fill Patron B's hold. Since this seems unfair to Patron A, we should allow renewal of items even if there are unfilled holds, but those holds could all be filled with currently available items. Test Plan: 1) Apply this patch 2) Create a record with two items 3) Check out the item to a patron 4) Place a hold on the record 5) Note you cannot renew the item for the patron 6) Enable the new system preference AllowRenewalIfOtherItemsAvailable 7) Note you can now renew the item, as all the holds can be satisfied by available items. 8) Place a second hold on the record 9) Note you can no longer renew the item, as all the holds *cannot* be filled by currently available items Signed-off-by: Holger Meissner <h.meissner.82@web.de> Signed-off-by: Chris Rohde <crohde@roseville.ca.us> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 --- Comment #32 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 32035 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=32035&action=edit Bug 11634 [QA Followup] - Unit Tests These new unit tests will fail due to the fact that Koha::Database uses a separate dbh handle than C4::Context->dbh -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 --- Comment #33 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 32036 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=32036&action=edit Bug 11634 [QA Followup 2] - Implement check for IsAvailableForItemLevelRequest -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 --- Comment #34 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 32037 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=32037&action=edit Bug 11634 [QA Followup 3] - Found holds should be considered unavailable -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 --- Comment #35 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Working on this now. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 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.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #32034|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=11634 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #32035|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=11634 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #32036|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=11634 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #32037|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=11634 --- Comment #36 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 32875 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=32875&action=edit [PASSED QA] Bug 11634 - Allow renewal of item with unfilled holds if other available items can fill those holds The current holds behavior in Koha allows a situation like this: - Patron A has an item currently checked out. - Patron B places a hold on the next available copy of that title. - Then Patron A will not be able to renew his item, even if there are other available copies of that title that could potentially fill Patron B's hold. Since this seems unfair to Patron A, we should allow renewal of items even if there are unfilled holds, but those holds could all be filled with currently available items. Test Plan: 1) Apply this patch 2) Create a record with two items 3) Check out the item to a patron 4) Place a hold on the record 5) Note you cannot renew the item for the patron 6) Enable the new system preference AllowRenewalIfOtherItemsAvailable 7) Note you can now renew the item, as all the holds can be satisfied by available items. 8) Place a second hold on the record 9) Note you can no longer renew the item, as all the holds *cannot* be filled by currently available items Signed-off-by: Holger Meissner <h.meissner.82@web.de> Signed-off-by: Chris Rohde <crohde@roseville.ca.us> 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=11634 --- Comment #37 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 32876 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=32876&action=edit [PASSED QA] Bug 11634 [QA Followup] - Unit Tests These new unit tests will fail due to the fact that Koha::Database uses a separate dbh handle than C4::Context->dbh 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=11634 --- Comment #38 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 32877 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=32877&action=edit [PASSED QA] Bug 11634 [QA Followup 2] - Implement check for IsAvailableForItemLevelRequest 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=11634 --- Comment #39 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 32878 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=32878&action=edit [PASSED QA] Bug 11634 [QA Followup 3] - Found holds should be considered unavailable 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=11634 --- Comment #40 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- *** Bug 10207 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 --- Comment #41 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 33492 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=33492&action=edit Bug 11634 [QA Followup] - Make unit tests pass * Allow on shelf holds needed to be enabled * Added some error supression code for undefined string comparison -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 --- Comment #42 from Jonathan Druart <jonathan.druart@biblibre.com> --- Comment on attachment 33492 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=33492 Bug 11634 [QA Followup] - Make unit tests pass Review of attachment 33492: --> (http://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=11634&attachment=33492) ----------------------------------------------------------------- ::: Koha/DateUtils.pm @@ +128,4 @@
my $pref = defined $force_pref ? $force_pref : C4::Context->preference('dateformat');
+ my $time_format = $force_time || C4::Context->preference('TimeFormat') || q{};
Hum, why? I am not sure it is a good idea to consider that TimeFormat can be '', 0 or undef. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master CC| |tomascohen@gmail.com --- Comment #43 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Patches pushed to master. Thanks Kyle! -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11634 Heather Braum <hbraum@nekls.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |hbraum@nekls.org -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org