[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Fridolin Somers changed: What|Removed |Added Status|Pushed to stable|Needs documenting CC||[email protected] ||m --- Comment #49 from Fridolin Somers --- Enhancement not pushed to 24.11.x -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #48 from Paul Derscheid --- Nice work everyone! Pushed to 25.05.x -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Paul Derscheid changed: What|Removed |Added Version(s)|25.11.00|25.11.00,25.05.04 released in|| Status|Pushed to main |Pushed to stable -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Lucas Gass (lukeg) changed: What|Removed |Added Status|Passed QA |Pushed to main Version(s)||25.11.00 released in|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #47 from Lucas Gass (lukeg) --- Nice work everyone! Pushed to main for 25.11 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Marcel de Rooy changed: What|Removed |Added Status|Failed QA |Passed QA --- Comment #46 from Marcel de Rooy --- (In reply to Lucas Gass (lukeg) from comment #44) > With this patchset applied /kohadevbox/koha/t/db_dependent/Reserves.t fails. > Can you check on this? Fixed. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #45 from Marcel de Rooy --- Created attachment 185706 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185706&action=edit Bug 37651: (QA follow-up) Fix current_holds tests in Reserves.t Test plan: Run prove t/db_dependent/Reserves.t -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Lucas Gass (lukeg) changed: What|Removed |Added Status|Passed QA |Failed QA CC||[email protected] --- Comment #44 from Lucas Gass (lukeg) --- With this patchset applied /kohadevbox/koha/t/db_dependent/Reserves.t fails. Can you check on this? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #43 from Nick Clemens (kidclamp) --- Created attachment 185371 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185371&action=edit Bug 37651: (follow-up) Add GetAvailability test case for future hold Test plan: Run t/db_dependent/ILSDI_Services.t Signed-off-by: Marcel de Rooy Signed-off-by: Nick Clemens -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #42 from Nick Clemens (kidclamp) --- Created attachment 185370 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185370&action=edit Bug 37651: (follow-up) Still skip future holds for renewals We need a skip_future_holds flag in item->current_holds to make renewals act as they did before. There is no need to add it to biblio->current_holds. Test plan: Run t/db_dependent/Holds.t Interface test: Issue a book, place hold on tomorrow, set ConfirmFutureHolds to 2 days. Verify that renew is not blocked by this hold. Signed-off-by: Marcel de Rooy Signed-off-by: Nick Clemens -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #41 from Nick Clemens (kidclamp) --- Created attachment 185369 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185369&action=edit Bug 37651: (QA follow-up) Rewording for pref ConfirmFutureHolds As requested by QA. Signed-off-by: Marcel de Rooy [EDIT] Added an exception for renewals. Moving to new report. Signed-off-by: Nick Clemens -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #40 from Nick Clemens (kidclamp) --- Created attachment 185368 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185368&action=edit Bug 37651: (QA follow-up) Add POD for Biblio method We were missing POD here. Signed-off-by: Martin Renvoize Signed-off-by: Nick Clemens -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #39 from Nick Clemens (kidclamp) --- Created attachment 185367 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185367&action=edit Bug 37651: Unit tests Test plan: Run t/db_dependent/Holds.t (Without previous patches, it would fail.) Run t/db_dependent/Koha/Biblios.t Signed-off-by: Marcel de Rooy Signed-off-by: Martin Renvoize Signed-off-by: Nick Clemens -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #38 from Nick Clemens (kidclamp) --- Created attachment 185366 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185366&action=edit Bug 37651: Add ConfirmFutureHolds to biblio->current_holds Test plan: Easy way is to run the unit tests in the next patch. One call of biblio->current_holds is hidden in acqui/parcel.pl. The table column "Item holds / Total holds" should now make a difference for an order referring to an item having future holds. Signed-off-by: Marcel de Rooy Signed-off-by: Martin Renvoize Signed-off-by: Nick Clemens -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Nick Clemens (kidclamp) changed: What|Removed |Added Attachment #184438|0 |1 is obsolete|| Attachment #184439|0 |1 is obsolete|| Attachment #184440|0 |1 is obsolete|| Attachment #184441|0 |1 is obsolete|| Attachment #184442|0 |1 is obsolete|| Attachment #184443|0 |1 is obsolete|| Attachment #18|0 |1 is obsolete|| --- Comment #37 from Nick Clemens (kidclamp) --- Created attachment 185365 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185365&action=edit Bug 37651: Add ConfirmFutureHolds to item->current_holds Test plan: Add future item level hold for another branch. Check in. Confirm and transfer. Note that without this patch, there is no patron info for the column On hold for on transferstoreceive. Signed-off-by: Marcel de Rooy Signed-off-by: Martin Renvoize Signed-off-by: Nick Clemens -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Nick Clemens (kidclamp) changed: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Marcel de Rooy changed: What|Removed |Added QA Contact|[email protected] |[email protected] |y.org | Status|Failed QA |Signed Off --- Comment #36 from Marcel de Rooy --- Back into SO -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #35 from Marcel de Rooy --- (In reply to Nick Clemens (kidclamp) from comment #27) Thx for QA. > MoveReserve in this context is checking for future holds, but only to fill > the current patrons future holds - or cancel or remove waiting status from an > existing hold - it does not block the checkout in any way - and it looks like > this won't change after the patch. So the description seems right to me Well, we probably mean the same here. It sounds just different. Yes it does not block the checkout. But it does "interfere" in the sense that such a future hold from the same patron is filled. Same for reverting a waiting status or cancelling a future hold from someone else. So the description "does not interfere", which I would interpret as 'does not influence" is incorrect. Future holds are already respected/considered here. So they "interfere". > I don't know that this is true - a patron who has a back might be allowed to > renew as it is in their possession. I think future holds confirmation is > meant to capture a book that is returned - forcing others to return it > earlier is more akin to what bookings and recalls do. This is a significant > change in behavior I think, and should, at least, be added as a test case for > CanBookBeRenewed This is not a matter of forcing to return a book earlier. But it is about renewing when there is a hold within the next few days. In general, library policies will not allow renewal when there are future reservations (not even restricted to what we call future holds). Currently, Koha does not follow that pattern. It only looks at "current holds" (until today). I could even imagine that Koha should not only check the period of lookahead days in the ConfirmFutureHolds pref but should look at the whole renewal period. (Or limit the renewal period.) But that is not within the scope of this report. I understand your request of tests. And do think now that we might better move this change on its own report by adding a skip_future_holds flag for current holds in renewals. We could then remove that or make it a preference on a new report (opened 40435). What do you think? > C4::Circulation CanBookBeRenewed (will now be no) See above. > C4/ILSDI/Services GetAvailability (will now be on hold) This is display and looks good to me overall. Renewal goes via CanBookBeRenewed. Added a small subtest for status of GetAvailability. > C4/Reserves AddReserve (will now not be set to waiting if there is a future > hold) This depends on ReservesNeedReturns (1: "Don't automatically", 0: Automatically). It should be set to 0 or null (just 7 branches on HEA versus 15000+ who have value 1) TOGETHER WITH using future holds obviously. And note that this change seems to be in line with general policies (this new hold should not jump to waiting if there is already one within a few days). > Koha/Item first_hold (could now return a future hold over a non-future hold, > this seems okay to me, but I am not sure of all contexts of first_hold) Git grepped it. Looked a circ/transferstoreceive.pl; that seems fine. Occurrences are obscured by using $first_hold = $holds->next. Seems to be ok. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Marcel de Rooy changed: What|Removed |Added Blocks||40435 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40435 [Bug 40435] CanBookBeRenewed should respect future holds -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #34 from Marcel de Rooy --- Created attachment 18 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=18&action=edit Bug 37651: (follow-up) Add GetAvailability test case for future hold Test plan: Run t/db_dependent/ILSDI_Services.t Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #33 from Marcel de Rooy --- Created attachment 184443 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=184443&action=edit Bug 37651: (follow-up) Still skip future holds for renewals We need a skip_future_holds flag in item->current_holds to make renewals act as they did before. There is no need to add it to biblio->current_holds. Test plan: Run t/db_dependent/Holds.t Interface test: Issue a book, place hold on tomorrow, set ConfirmFutureHolds to 2 days. Verify that renew is not blocked by this hold. Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Marcel de Rooy changed: What|Removed |Added Attachment #178329|0 |1 is obsolete|| --- Comment #32 from Marcel de Rooy --- Created attachment 184442 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=184442&action=edit Bug 37651: (QA follow-up) Rewording for pref ConfirmFutureHolds As requested by QA. Signed-off-by: Marcel de Rooy [EDIT] Added an exception for renewals. Moving to new report. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Marcel de Rooy changed: What|Removed |Added Attachment #178328|0 |1 is obsolete|| --- Comment #31 from Marcel de Rooy --- Created attachment 184441 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=184441&action=edit Bug 37651: (QA follow-up) Add POD for Biblio method We were missing POD here. Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Marcel de Rooy changed: What|Removed |Added Attachment #178327|0 |1 is obsolete|| --- Comment #30 from Marcel de Rooy --- Created attachment 184440 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=184440&action=edit Bug 37651: Unit tests Test plan: Run t/db_dependent/Holds.t (Without previous patches, it would fail.) Run t/db_dependent/Koha/Biblios.t Signed-off-by: Marcel de Rooy Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Marcel de Rooy changed: What|Removed |Added Attachment #178326|0 |1 is obsolete|| --- Comment #29 from Marcel de Rooy --- Created attachment 184439 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=184439&action=edit Bug 37651: Add ConfirmFutureHolds to biblio->current_holds Test plan: Easy way is to run the unit tests in the next patch. One call of biblio->current_holds is hidden in acqui/parcel.pl. The table column "Item holds / Total holds" should now make a difference for an order referring to an item having future holds. Signed-off-by: Marcel de Rooy Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Marcel de Rooy changed: What|Removed |Added Attachment #178325|0 |1 is obsolete|| --- Comment #28 from Marcel de Rooy --- Created attachment 184438 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=184438&action=edit Bug 37651: Add ConfirmFutureHolds to item->current_holds Test plan: Add future item level hold for another branch. Check in. Confirm and transfer. Note that without this patch, there is no patron info for the column On hold for on transferstoreceive. Signed-off-by: Marcel de Rooy Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Nick Clemens (kidclamp) changed: What|Removed |Added Status|Signed Off |Failed QA --- Comment #27 from Nick Clemens (kidclamp) --- (In reply to Marcel de Rooy from comment #25) > The description "But it does not interfere with issuing, renewing or > transferring items" is not the most lucky one. I might have added it myself > long ago ;) > Note also that we are talking about a feature that say over 90% of the > libraries is not using. (HEA: 1040 AllowHoldDateinFuture 728 > OPACAllowHoldDateInFuture ) 10% is not an insignificant amount of libraries imo > Lets look at the processes mentioned. (When writing this patch, I reviewed > all occurrences of current_holds btw.) > > Checkout > AddIssue already calls MoveReserve and that does check ConfirmFutureHolds. > So this is an "existing interference". The description was wrong. MoveReserve in this context is checking for future holds, but only to fill the current patrons future holds - or cancel or remove waiting status from an existing hold - it does not block the checkout in any way - and it looks like this won't change after the patch. So the description seems right to me > Renew > CanBookBeRenewed calls current_holds. > This is a "new interference". If a library allows future holds and wants to > confirm them within a few days specified by ConfirmFutureHolds, currently > this is ignored by CanBookBeRenewed. So the book is renewed although there > is a hold for it on tomorrow. Not sure if we should write code for that? > Probably most do not want to renew here. I don't know that this is true - a patron who has a back might be allowed to renew as it is in their possession. I think future holds confirmation is meant to capture a book that is returned - forcing others to return it earlier is more akin to what bookings and recalls do. This is a significant change in behavior I think, and should, at least, be added as a test case for CanBookBeRenewed In my review, these are the effects: C4::Circulation CanBookBeRenewed (will now be no) C4/ILSDI/Services GetAvailability (will now be on hold) C4/Reserves AddReserve (will now not be set to waiting if there is a future hold) Koha/Item first_hold (could now return a future hold over a non-future hold, this seems okay to me, but I am not sure of all contexts of first_hold) I think below will just add the new holds to display without behaviour change: Koha/Acquisition/Order current_item_level_holds C4/SIP/ILS/Item new acqui/parcel.pl circ/transferstoreceive.pl opac/opac-reserve.pl reserve/request.pl I won't block this one, but can leave to the RM to decide - I would like to see tests added to CanBookBeRenewed and GetAvailability for QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Marcel de Rooy changed: What|Removed |Added Status|In Discussion |Signed Off --- Comment #26 from Marcel de Rooy --- Moving back to SO queue. Any thoughts, Nick? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #25 from Marcel de Rooy --- (In reply to Nick Clemens (kidclamp) from comment #24) > I am not convinced this is a desired behavior change for all libraries. When > they decided to set the pref, it explicitly said "does not affect" several > things > > Would you be open to a second setting within the pref: > > CircConfirmFutureHolds (at checkin, at renewal, at checkout - multi select?) > starting no later then XX days Hi Nick. Thanks for looking here ! The description "But it does not interfere with issuing, renewing or transferring items" is not the most lucky one. I might have added it myself long ago ;) Note also that we are talking about a feature that say over 90% of the libraries is not using. (HEA: 1040 AllowHoldDateinFuture 728 OPACAllowHoldDateInFuture ) Lets look at the processes mentioned. (When writing this patch, I reviewed all occurrences of current_holds btw.) Checkout AddIssue already calls MoveReserve and that does check ConfirmFutureHolds. So this is an "existing interference". The description was wrong. Renew CanBookBeRenewed calls current_holds. This is a "new interference". If a library allows future holds and wants to confirm them within a few days specified by ConfirmFutureHolds, currently this is ignored by CanBookBeRenewed. So the book is renewed although there is a hold for it on tomorrow. Not sure if we should write code for that? Probably most do not want to renew here. Transfer transferbook calls CheckReserves without lookahead days. So it does not block a transfer for a future hold (when not ignoring them altogether). Feels like a bug to me that could be solved on another report :) So there is currently no interference here. Discussion elsewhere? Checkin This is not relevant here. ConfirmFutureHolds already affects checkin (especially the popup). And this is already advertized accordingly. In conclusion, the new extra preference should only pertain to renewals. This would imo be a rather ugly exception. I would advise against it. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Nick Clemens (kidclamp) changed: What|Removed |Added Status|Signed Off |In Discussion CC||[email protected] --- Comment #24 from Nick Clemens (kidclamp) --- I am not convinced this is a desired behavior change for all libraries. When they decided to set the pref, it explicitly said "does not affect" several things Would you be open to a second setting within the pref: CircConfirmFutureHolds (at checkin, at renewal, at checkout - multi select?) starting no later then XX days -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #23 from Marcel de Rooy --- Rebased on tidied codebase -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Marcel de Rooy changed: What|Removed |Added Attachment #177372|0 |1 is obsolete|| --- Comment #22 from Marcel de Rooy --- Created attachment 178329 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178329&action=edit Bug 37651: (QA follow-up) Rewording for pref ConfirmFutureHolds As requested by QA. Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Marcel de Rooy changed: What|Removed |Added Attachment #176811|0 |1 is obsolete|| --- Comment #21 from Marcel de Rooy --- Created attachment 178328 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178328&action=edit Bug 37651: (QA follow-up) Add POD for Biblio method We were missing POD here. Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Marcel de Rooy changed: What|Removed |Added Attachment #176809|0 |1 is obsolete|| --- Comment #19 from Marcel de Rooy --- Created attachment 178326 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178326&action=edit Bug 37651: Add ConfirmFutureHolds to biblio->current_holds Test plan: Easy way is to run the unit tests in the next patch. One call of biblio->current_holds is hidden in acqui/parcel.pl. The table column "Item holds / Total holds" should now make a difference for an order referring to an item having future holds. Signed-off-by: Marcel de Rooy Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Marcel de Rooy changed: What|Removed |Added Attachment #176810|0 |1 is obsolete|| --- Comment #20 from Marcel de Rooy --- Created attachment 178327 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178327&action=edit Bug 37651: Unit tests Test plan: Run t/db_dependent/Holds.t (Without previous patches, it would fail.) Run t/db_dependent/Koha/Biblios.t Signed-off-by: Marcel de Rooy Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Marcel de Rooy changed: What|Removed |Added Attachment #176808|0 |1 is obsolete|| --- Comment #18 from Marcel de Rooy --- Created attachment 178325 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178325&action=edit Bug 37651: Add ConfirmFutureHolds to item->current_holds Test plan: Add future item level hold for another branch. Check in. Confirm and transfer. Note that without this patch, there is no patron info for the column On hold for on transferstoreceive. Signed-off-by: Marcel de Rooy Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #17 from Marcel de Rooy --- Created attachment 177372 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=177372&action=edit Bug 37651: (QA follow-up) Rewording for pref ConfirmFutureHolds As requested by QA. Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #16 from Marcel de Rooy --- (In reply to Jonathan Druart from comment #15) > This feels wrong to me. > > The pref description: "Note that this number of days will be used too in > calculating the default end date for the Holds to pull-report. But it does > not interfere with issuing, renewing or transferring items." > > It seems that we need a filter_by method, and use it only where it needs > this pref to be taken into account. You are right. We should adjust the wording of that pref. Will add a follow-up. The filter_by comment is valid too. But I dont think that it should be done here. It is an improvement on its own. There are more methods that could be reworked into filter_by constructs. Like for instance: Koha/Biblio.pm current_checkouts Koha/Acquisition/Order.pm current_item_level_holds The point of this report is refine the existing two current_holds methods where the pref should be taken into account. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Jonathan Druart changed: What|Removed |Added CC||[email protected] --- Comment #15 from Jonathan Druart --- This feels wrong to me. The pref description: "Note that this number of days will be used too in calculating the default end date for the Holds to pull-report. But it does not interfere with issuing, renewing or transferring items." It seems that we need a filter_by method, and use it only where it needs this pref to be taken into account. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #14 from Marcel de Rooy --- (In reply to Martin Renvoize (ashimema) from comment #12) > Signed-off-by: Martin Renvoize Thx ! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #13 from Martin Renvoize (ashimema) --- This is a fun one.. current_holds is called in a fair number of places; thus, whilst this is a small and clear patch, it could have some wide-reaching effects. I'm signing off and will seek some QA opinions. My gut says it makes sense and is a step toward being able to deprecate C4::Reserves::CheckReserve. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Martin Renvoize (ashimema) changed: What|Removed |Added Status|Needs Signoff |Signed Off CC||martin.renvoize@ptfs-europe ||.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #12 from Martin Renvoize (ashimema) --- Created attachment 176811 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=176811&action=edit Bug 37651: (QA follow-up) Add POD for Biblio method We were missing POD here. Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Martin Renvoize (ashimema) changed: What|Removed |Added Attachment #176656|0 |1 is obsolete|| --- Comment #11 from Martin Renvoize (ashimema) --- Created attachment 176810 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=176810&action=edit Bug 37651: Unit tests Test plan: Run t/db_dependent/Holds.t (Without previous patches, it would fail.) Run t/db_dependent/Koha/Biblios.t Signed-off-by: Marcel de Rooy Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Martin Renvoize (ashimema) changed: What|Removed |Added Attachment #176655|0 |1 is obsolete|| --- Comment #10 from Martin Renvoize (ashimema) --- Created attachment 176809 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=176809&action=edit Bug 37651: Add ConfirmFutureHolds to biblio->current_holds Test plan: Easy way is to run the unit tests in the next patch. One call of biblio->current_holds is hidden in acqui/parcel.pl. The table column "Item holds / Total holds" should now make a difference for an order referring to an item having future holds. Signed-off-by: Marcel de Rooy Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Martin Renvoize (ashimema) changed: What|Removed |Added Attachment #176654|0 |1 is obsolete|| --- Comment #9 from Martin Renvoize (ashimema) --- Created attachment 176808 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=176808&action=edit Bug 37651: Add ConfirmFutureHolds to item->current_holds Test plan: Add future item level hold for another branch. Check in. Confirm and transfer. Note that without this patch, there is no patron info for the column On hold for on transferstoreceive. Signed-off-by: Marcel de Rooy Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #8 from Marcel de Rooy --- Rebased. Waiting since September.. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Marcel de Rooy changed: What|Removed |Added Attachment #171250|0 |1 is obsolete|| --- Comment #7 from Marcel de Rooy --- Created attachment 176656 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=176656&action=edit Bug 37651: Unit tests Test plan: Run t/db_dependent/Holds.t (Without previous patches, it would fail.) Run t/db_dependent/Koha/Biblios.t Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Marcel de Rooy changed: What|Removed |Added Attachment #171248|0 |1 is obsolete|| --- Comment #6 from Marcel de Rooy --- Created attachment 176655 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=176655&action=edit Bug 37651: Add ConfirmFutureHolds to biblio->current_holds Test plan: Easy way is to run the unit tests in the next patch. One call of biblio->current_holds is hidden in acqui/parcel.pl. The table column "Item holds / Total holds" should now make a difference for an order referring to an item having future holds. Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Marcel de Rooy changed: What|Removed |Added Attachment #171247|0 |1 is obsolete|| --- Comment #5 from Marcel de Rooy --- Created attachment 176654 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=176654&action=edit Bug 37651: Add ConfirmFutureHolds to item->current_holds Test plan: Add future item level hold for another branch. Check in. Confirm and transfer. Note that without this patch, there is no patron info for the column On hold for on transferstoreceive. Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #2 from Marcel de Rooy --- Created attachment 171248 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=171248&action=edit Bug 37651: Add ConfirmFutureHolds to biblio->current_holds Test plan: Easy way is to run the unit tests in the next patch. One call of biblio->current_holds is hidden in acqui/parcel.pl. The table column "Item holds / Total holds" should now make a difference for an order referring to an item having future holds. Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Marcel de Rooy changed: What|Removed |Added Assignee|[email protected] |[email protected] |ity.org | -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Marcel de Rooy changed: What|Removed |Added Patch complexity|--- |Small patch Status|NEW |Needs Signoff -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Marcel de Rooy changed: What|Removed |Added Attachment #171249|0 |1 is obsolete|| --- Comment #4 from Marcel de Rooy --- Created attachment 171250 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=171250&action=edit Bug 37651: Unit tests Test plan: Run t/db_dependent/Holds.t (Without previous patches, it would fail.) Run t/db_dependent/Koha/Biblios.t Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #3 from Marcel de Rooy --- Created attachment 171249 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=171249&action=edit Bug 37651: Unit tests Test plan: Run t/db_dependent/Holds.t (Without previous patches, it would fail.) Run t/db_dependent/Koha/Biblios.t -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 --- Comment #1 from Marcel de Rooy --- Created attachment 171247 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=171247&action=edit Bug 37651: Add ConfirmFutureHolds to item->current_holds Test plan: Add future item level hold for another branch. Check in. Confirm and transfer. Note that without this patch, there is no patron info for the column On hold for on transferstoreceive. Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37651] biblio->current_holds and item->current_holds do not respect ConfirmFutureHolds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37651 Marcel de Rooy changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=37650 -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
