[Koha-bugs] [Bug 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 Nick Clemens changed: What|Removed |Added Blocks||25566 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25566 [Bug 25566] Change in DecreaseLoanHighHolds behaviour -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 Magnus Enger changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=22005 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 Hayley Mapley changed: What|Removed |Added CC||hayleymap...@catalyst.net.n ||z --- Comment #23 from Hayley Mapley --- Enhancement/patch will not be backported to 18.11.x series. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 Lucas Gass changed: What|Removed |Added CC||lu...@bywatersolutions.com Status|Pushed to stable|Pushed to oldstable Version(s)|20.05.00, 19.11.03 |20.05.00, 19.11.03, released in||19.05.08 --- Comment #22 from Lucas Gass --- backported to 19.05.x for 19.05.08 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 Joy Nelson changed: What|Removed |Added Version(s)|20.05.00|20.05.00, 19.11.03 released in|| CC||j...@bywatersolutions.com Status|Pushed to master|Pushed to stable --- Comment #21 from Joy Nelson --- Pushed to 19.11.x branch for 19.11.03 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 --- Comment #20 from Hans Pålsson --- (In reply to Martin Renvoize from comment #19) > Nice work everyone! > > Pushed to master for 20.05 :-) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 --- Comment #19 from Martin Renvoize --- Nice work everyone! Pushed to master for 20.05 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 Martin Renvoize changed: What|Removed |Added Version(s)||20.05.00 released in|| Status|Passed QA |Pushed to master -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 --- Comment #18 from Kyle M Hall --- Created attachment 97389 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=97389=edit Bug 20567: Count found holds for holds limit Signed-off-by: Hans Palsson Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 Kyle M Hall changed: What|Removed |Added Attachment #97386|0 |1 is obsolete|| Attachment #97387|0 |1 is obsolete|| --- Comment #17 from Kyle M Hall --- Created attachment 97388 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=97388=edit Bug 20567: Add tests Signed-off-by: Hans Palsson Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 Kyle M Hall 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 Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 --- Comment #16 from Kyle M Hall --- Created attachment 97387 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=97387=edit Bug 20567: Count found holds for holds limit Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 Kyle M Hall changed: What|Removed |Added Attachment #97336|0 |1 is obsolete|| Attachment #97337|0 |1 is obsolete|| --- Comment #15 from Kyle M Hall --- Created attachment 97386 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=97386=edit Bug 20567: Add tests Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 --- Comment #14 from Kyle M Hall --- Based on Ben's response, my opinion is that this change in behavior does not need to be syspref'ed at this time! (In reply to Katrin Fischer from comment #13) > (In reply to Benjamin Daeuber from comment #12) > > We are the sponsoring library Kyle refers to. While we hadn't noticed this, > > I guess I would agree with the opinion here, a hold in waiting status is > > still a hold. > > > > That said, if someone wants a syspref, that's fine, but I guess everyone > > here seems to agree waiting hold and pending holds ought to be counted > > together. > > Thx Benjamin for commenting! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 Katrin Fischer changed: What|Removed |Added Assignee|koha-b...@lists.koha-commun |jonathan.dru...@bugs.koha-c |ity.org |ommunity.org -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 --- Comment #13 from Katrin Fischer --- (In reply to Benjamin Daeuber from comment #12) > We are the sponsoring library Kyle refers to. While we hadn't noticed this, > I guess I would agree with the opinion here, a hold in waiting status is > still a hold. > > That said, if someone wants a syspref, that's fine, but I guess everyone > here seems to agree waiting hold and pending holds ought to be counted > together. Thx Benjamin for commenting! -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 Benjamin Daeuber changed: What|Removed |Added CC||bdaeu...@cityoffargo.com --- Comment #12 from Benjamin Daeuber --- We are the sponsoring library Kyle refers to. While we hadn't noticed this, I guess I would agree with the opinion here, a hold in waiting status is still a hold. That said, if someone wants a syspref, that's fine, but I guess everyone here seems to agree waiting hold and pending holds ought to be counted together. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 Hans Pålsson changed: What|Removed |Added Status|In Discussion |Signed Off --- Comment #11 from Hans Pålsson --- Signing of because patch solves initial problem. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 --- Comment #10 from Kyle M Hall --- (In reply to Jonathan Druart from comment #8) > Same as Katrin here, I submitted a patch that should modify the behavior as > you expect, but I am not sure it's correct (side effects). > That would be good to get Kyle's opinion on this, as he wrote that bit of > code (the comment on the line I remove especially). I'm researching the issue to see if the original development funder(s) had this behavior as a requirement or not. In any case it would be trivial to syspref this to satisfy both those who want the existing behavior and those who do not. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 --- Comment #9 from Hans Pålsson --- The problem is on item level more than on biblio level. If a library only allows 1 item to be reserved per biblio, then the current code creates the loophole. That is the problem we have and the same as reported in this bug. If the amount of holds placed should count towards a patrons total amount of holds placed on different biblios, then the current code makes sense. I think... -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 Jonathan Druart changed: What|Removed |Added CC||k...@bywatersolutions.com --- Comment #8 from Jonathan Druart --- Same as Katrin here, I submitted a patch that should modify the behavior as you expect, but I am not sure it's correct (side effects). That would be good to get Kyle's opinion on this, as he wrote that bit of code (the comment on the line I remove especially). -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 --- Comment #6 from Jonathan Druart --- Created attachment 97336 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=97336=edit Bug 20567: Add tests -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 --- Comment #7 from Jonathan Druart --- Created attachment 97337 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=97337=edit Bug 20567: Count found holds for holds limit -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 Katrin Fischer changed: What|Removed |Added CC||katrin.fisc...@bsz-bw.de --- Comment #5 from Katrin Fischer --- I understand why you would NOT want to count the waiting holds, but I wonder if we are missing a use case here if we change that too quickly. Maybe we coudl compare to how other hold counts, like the daily limit, work here? -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 --- Comment #4 from Hans Pålsson --- (In reply to Jonathan Druart from comment #2) > C4/Circulation.pm, in CanItemBeReserved: > 394 found => undef, # Found holds don't count against > a patron's holds limit I think it is C4/Reserves.pm. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 Hans Pålsson changed: What|Removed |Added CC||hans.pals...@hkr.se --- Comment #3 from Hans Pålsson --- (In reply to Jonathan Druart from comment #2) > It seems that it is the expected behavior. > > C4/Circulation.pm, in CanItemBeReserved: > 394 found => undef, # Found holds don't count against > a patron's holds limit > > Should not it be? No, a patron should only be allowed 1 hold per biblio, also after that hold is filled. It translates to patrons should not be able to place holds on biblios for which they have checked out items or already placed a hold. The current behaviour becomes a loophole. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 Jonathan Druart changed: What|Removed |Added Depends on||14695 Status|NEW |In Discussion CC||jonathan.dru...@bugs.koha-c ||ommunity.org --- Comment #2 from Jonathan Druart --- It seems that it is the expected behavior. C4/Circulation.pm, in CanItemBeReserved: 394 found => undef, # Found holds don't count against a patron's holds limit Should not it be? Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14695 [Bug 14695] Add ability to place multiple item holds on a given record per patron -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 Jonathan Druart changed: What|Removed |Added Version|17.05 |master -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 Christopher Brannon changed: What|Removed |Added Severity|minor |major CC||cbran...@cdalibrary.org --- Comment #1 from Christopher Brannon --- I would not classify this as a minor bug. If limits are not being enforced in all conditions, this is a problem. At the very least, there needs to be a preference to change this oversight, if for some reason there is justification to keep this unusual behavior. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 Margaretchanged: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=15404 -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 20567] "Holds per record (count)" limit is not enforced after item is captured for hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20567 Margaretchanged: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=17978 -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/