[Koha-bugs] [Bug 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Martin Renvoize changed: What|Removed |Added Keywords|RM_priority | -- 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Lucas Gass changed: What|Removed |Added CC||lu...@bywatersolutions.com --- Comment #39 from Lucas Gass --- backported Kyle's rebase for 18.11.x for 18.11.08 -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Fridolin SOMERS changed: What|Removed |Added Version(s)|19.11.00|19.11.00,19.05.02 released in|| CC||fridolin.som...@biblibre.co ||m Status|Pushed to master|Pushed to stable --- Comment #38 from Fridolin SOMERS --- Pushed to 19.05.x for 19.05.02 -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #37 from Kyle M Hall --- (In reply to David Cook from comment #36) > (In reply to Kyle M Hall from comment #34) > > (In reply to David Cook from comment #33) > > > Kyle, how are folks managing the terrors I caused with 15221? > > > > lol, at ByWater we backported this patch to out build of Koha 18.11. I'd be > > happy to provide that patch for community backporting once we are sure it is > > stable. In fact, I'll post it right now for anyone to use. > > That's good to hear. So it's already running in the wild and not causing > chaos? As far as I know, that is correct! :) -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #36 from David Cook --- (In reply to Kyle M Hall from comment #34) > (In reply to David Cook from comment #33) > > Kyle, how are folks managing the terrors I caused with 15221? > > lol, at ByWater we backported this patch to out build of Koha 18.11. I'd be > happy to provide that patch for community backporting once we are sure it is > stable. In fact, I'll post it right now for anyone to use. That's good to hear. So it's already running in the wild and not causing chaos? -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #35 from Kyle M Hall --- Created attachment 91644 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=91644=edit [Koha v18.11.x] Bug 23057: If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason Acknowledgements: Patch also contains original code by David Cook. Patches were squashed for ease of use. This patch fixes a bug from Bug 15221 and hopefully also makes it so that the alert flag is suppressed when checked_in_ok is set and an item is not checked out. To test: 0) Create patron in web interface with a cardnumber and userid of "staff" with a password that matches the account in SIPconfig.xml. Also set their branch to CPL (also matching SIPconfig.xml). 1) Create an item with a barcode of 'test' 2) Choose a patron to check out to and record their borrowernumber In one terminal: 3) cd to your git directory (e.g. /home/koha/koha) 4) perl ./C4/SIP/SIPServer.pm ~/koha-dev/etc/SIPconfig.xml In another terminal: 5) cd to your git directory (e.g. /home/koha/koha) 6) perl ./misc/sip_cli_emulator.pl -l CPL -su staff -sp --port=6001 --address=localhost --item test -m checkin --patron NOTE: You need to replace with the borrowernumber from Step 2, and with the password from Step 0. -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #34 from Kyle M Hall --- (In reply to David Cook from comment #33) > Kyle, how are folks managing the terrors I caused with 15221? lol, at ByWater we backported this patch to out build of Koha 18.11. I'd be happy to provide that patch for community backporting once we are sure it is stable. In fact, I'll post it right now for anyone to use. -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #33 from David Cook --- Kyle, how are folks managing the terrors I caused with 15221? -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #32 from David Cook --- Hoping folks managing 19.05.x and 18.11.x see this one, as they must be bitten by https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15221 -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #31 from Martin Renvoize --- Nice work! Pushed to master for 19.11.00 -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Martin Renvoize changed: What|Removed |Added Status|Passed QA |Pushed to master Version(s)||19.11.00 released in|| -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Martin Renvoize changed: What|Removed |Added Status|Signed Off |Passed QA --- Comment #30 from Martin Renvoize --- I'm happy without the final followup, the code is nice and terse and I don't feel the followup clarifies it greatly personally.. lets move forward :) -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #29 from Kyle M Hall --- No problem! A second pair of eyes never hurts :) (In reply to Marcel de Rooy from comment #28) > (In reply to Kyle M Hall from comment #26) > > Comment on attachment 91326 [details] [review] [review] > > Bug 23057: (QA follow-up) Stay closer to old code > > > > I agree with Liz of course ;) > > > > The old code is messy and confusing. I put a lot of thought into the changes > > I made to make the code clean, terse and understandable. I don't think we > > should put back bad code just for the sake of leaving it unchanged in this > > case. There is very real benefit to the new code consolidating logic for the > > benefit of future developers who have to deal with this code. > > No problem. But in that case it may not harm to have another QAer look at it > too. Thanks for your feedback. -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #28 from Marcel de Rooy --- (In reply to Kyle M Hall from comment #26) > Comment on attachment 91326 [details] [review] > Bug 23057: (QA follow-up) Stay closer to old code > > I agree with Liz of course ;) > > The old code is messy and confusing. I put a lot of thought into the changes > I made to make the code clean, terse and understandable. I don't think we > should put back bad code just for the sake of leaving it unchanged in this > case. There is very real benefit to the new code consolidating logic for the > benefit of future developers who have to deal with this code. No problem. But in that case it may not harm to have another QAer look at it too. Thanks for your feedback. -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #27 from Marcel de Rooy --- (In reply to Liz Rea from comment #25) > I don't understand why the last patch is necessary? Explained in the commit message. Not much more to say -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Kyle M Hall changed: What|Removed |Added Attachment #91326|0 |1 is obsolete|| --- Comment #26 from Kyle M Hall --- Comment on attachment 91326 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=91326 Bug 23057: (QA follow-up) Stay closer to old code I agree with Liz of course ;) The old code is messy and confusing. I put a lot of thought into the changes I made to make the code clean, terse and understandable. I don't think we should put back bad code just for the sake of leaving it unchanged in this case. There is very real benefit to the new code consolidating logic for the benefit of future developers who have to deal with this code. (In reply to Marcel de Rooy from comment #24) > Kyle, > Please have a look at the last follow-up. And let me know if you are > comfortable with it. I am at least, since we stay closer to the old code > while resolving the bug. If you are too, I will pass QA. > > Marcel -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Liz Rea changed: What|Removed |Added CC||wizzy...@gmail.com --- Comment #25 from Liz Rea --- I don't understand why the last patch is necessary? -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #24 from Marcel de Rooy --- Kyle, Please have a look at the last follow-up. And let me know if you are comfortable with it. I am at least, since we stay closer to the old code while resolving the bug. If you are too, I will pass QA. Marcel -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Marcel de Rooy changed: What|Removed |Added Patch complexity|--- |Small patch -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Marcel de Rooy changed: What|Removed |Added Version|unspecified |master -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Marcel de Rooy changed: What|Removed |Added QA Contact|testo...@bugs.koha-communit |m.de.r...@rijksmuseum.nl |y.org | -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Marcel de Rooy changed: What|Removed |Added Status|BLOCKED |Signed Off -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #21 from Marcel de Rooy --- Created attachment 91324 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=91324=edit Bug 23057: Update do_checkin Signed-off-by: Kyle M Hall Signed-off-by: Matha Fuerst Signed-off-by: Marcel de Rooy -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #20 from Marcel de Rooy --- Created attachment 91323 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=91323=edit Bug 23057: Unit tests Signed-off-by: Kyle M Hall Signed-off-by: Matha Fuerst Signed-off-by: Marcel de Rooy -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #23 from Marcel de Rooy --- Created attachment 91326 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=91326=edit Bug 23057: (QA follow-up) Stay closer to old code Somehow it feels safer to me to alert for a !$return directly after calling AddReturn. And restore the old code for alert_type at the end. Signed-off-by: Marcel de Rooy -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #22 from Marcel de Rooy --- Created attachment 91325 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=91325=edit Bug 23057: (QA follow-up) Remove new AddReturn message ReturnOfLostItemBlocked, use existing pattern from returns.pl Signed-off-by: Kyle M Hall Signed-off-by: Matha Fuerst Signed-off-by: Marcel de Rooy -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Marcel de Rooy changed: What|Removed |Added Attachment #90505|0 |1 is obsolete|| Attachment #90506|0 |1 is obsolete|| Attachment #90507|0 |1 is obsolete|| Attachment #90508|0 |1 is obsolete|| --- Comment #19 from Marcel de Rooy --- Created attachment 91322 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=91322=edit Bug 23057: If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason This patch fixes a bug from Bug 15221 and hopefully also makes it so that the alert flag is suppressed when checked_in_ok is set and an item is not checked out. To test: 0) Create patron in web interface with a cardnumber and userid of "staff" with a password that matches the account in SIPconfig.xml. Also set their branch to CPL (also matching SIPconfig.xml). 1) Create an item with a barcode of 'test' 2) Choose a patron to check out to and record their borrowernumber In one terminal: 3) cd to your git directory (e.g. /home/koha/koha) 4) perl ./C4/SIP/SIPServer.pm ~/koha-dev/etc/SIPconfig.xml In another terminal: 5) cd to your git directory (e.g. /home/koha/koha) 6) perl ./misc/sip_cli_emulator.pl -l CPL -su staff -sp --port=6001 --address=localhost --item test -m checkin --patron NOTE: You need to replace with the borrowernumber from Step 2, and with the password from Step 0. Signed-off-by: Kyle M Hall Signed-off-by: Matha Fuerst Signed-off-by: Marcel de Rooy -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Marcel de Rooy changed: What|Removed |Added Status|Signed Off |BLOCKED --- Comment #18 from Marcel de Rooy --- QA: Looking here again -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Sally Healey changed: What|Removed |Added CC||sally.healey@cheshireshared ||services.gov.uk -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Marcel de Rooy changed: What|Removed |Added CC||m.de.r...@rijksmuseum.nl -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Bug 23057 depends on bug 15221, which changed state. Bug 15221 Summary: SIP server always sets the alert flag when item not returned https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15221 What|Removed |Added Status|Pushed to Stable|RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Martin Renvoize changed: What|Removed |Added Keywords||RM_priority CC||martin.renvoize@ptfs-europe ||.com -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #16 from Kyle M Hall --- Created attachment 90507 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=90507=edit Bug 23057: Update do_checkin Signed-off-by: Kyle M Hall Signed-off-by: Matha Fuerst -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #17 from Kyle M Hall --- Created attachment 90508 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=90508=edit Bug 23057: (QA Follow-up) Remove new AddReturn message ReturnOfLostItemBlocked, use existing pattern from returns.pl Signed-off-by: Kyle M Hall Signed-off-by: Matha Fuerst -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Kyle M Hall changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #15 from Kyle M Hall --- Created attachment 90506 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=90506=edit Bug 23057: Unit tests Signed-off-by: Kyle M Hall Signed-off-by: Matha Fuerst -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Kyle M Hall changed: What|Removed |Added Attachment #90344|0 |1 is obsolete|| Attachment #90384|0 |1 is obsolete|| Attachment #90388|0 |1 is obsolete|| Attachment #90480|0 |1 is obsolete|| --- Comment #14 from Kyle M Hall --- Created attachment 90505 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=90505=edit Bug 23057: If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason This patch fixes a bug from Bug 15221 and hopefully also makes it so that the alert flag is suppressed when checked_in_ok is set and an item is not checked out. To test: 0) Create patron in web interface with a cardnumber and userid of "staff" with a password that matches the account in SIPconfig.xml. Also set their branch to CPL (also matching SIPconfig.xml). 1) Create an item with a barcode of 'test' 2) Choose a patron to check out to and record their borrowernumber In one terminal: 3) cd to your git directory (e.g. /home/koha/koha) 4) perl ./C4/SIP/SIPServer.pm ~/koha-dev/etc/SIPconfig.xml In another terminal: 5) cd to your git directory (e.g. /home/koha/koha) 6) perl ./misc/sip_cli_emulator.pl -l CPL -su staff -sp --port=6001 --address=localhost --item test -m checkin --patron NOTE: You need to replace with the borrowernumber from Step 2, and with the password from Step 0. Signed-off-by: Kyle M Hall Signed-off-by: Matha Fuerst -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #13 from Kyle M Hall --- To follow up on my last comment, I did want to say that if QA feels that my ALT patch is ok as part of this bug, please by all means use it instead of my more terse followup! -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057
--- Comment #12 from Kyle M Hall ---
(In reply to Jonathan Druart from comment #8)
> Comment on attachment 90388 [details] [review]
> Bug 23057: Update do_checkin
>
> Review of attachment 90388 [details] [review]:
> -
>
> ::: C4/Circulation.pm
> @@ +1945,4 @@
> > }
> >
> > if ( $item->itemlost and
> > C4::Context->preference("BlockReturnOfLostItems") ) {
> > +$messages->{'ReturnOfLostItemBlocked'};
>
> Should we handle it from returns.pl as well?
So, implementing this turned out to be way more complicated than I expected.
The logic in AddReturn for lost items is "messy" to put it kindly. You can find
this patch in the obsolete patches on this bug.
As an alternative solution with far fewer changes to the code, I've added a
different followup that removes the new ReturnOfLostItemBlocked and follows the
existing pattern implemented by returns.pl where the logic is implemented
outside AddReturn.
I find this to be technically inferior, but re-implementing the
ReturnOfLostItemBlocked message properly is far outside the scope of a SIP bug
report. The now obsoleted patch could definitely be re-used on a bug dedicated
to this issue.
--
You are receiving this mail because:
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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Kyle M Hall changed: What|Removed |Added Assignee|koha-b...@lists.koha-commun |k...@bywatersolutions.com |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 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Kyle M Hall changed: What|Removed |Added Attachment #90479|0 |1 is obsolete|| --- Comment #11 from Kyle M Hall --- Created attachment 90480 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=90480=edit Bug 23057: (QA Follow-up) Remove new AddReturn message ReturnOfLostItemBlocked, use existing pattern from returns.pl -- 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 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #10 from Kyle M Hall --- Created attachment 90479 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=90479=edit Bug 23057: (QA Follow-up) [ALT PATCH] Handle new ReturnOfLostItemBlocked message in returns.pl -- 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057
--- Comment #9 from Kyle M Hall ---
(In reply to Jonathan Druart from comment #8)
> Comment on attachment 90388 [details] [review]
> Bug 23057: Update do_checkin
>
> Review of attachment 90388 [details] [review]:
> -
>
> ::: C4/Circulation.pm
> @@ +1945,4 @@
> > }
> >
> > if ( $item->itemlost and
> > C4::Context->preference("BlockReturnOfLostItems") ) {
> > +$messages->{'ReturnOfLostItemBlocked'};
>
> Should we handle it from returns.pl as well?
It would make sense. I find it odd that that is the *only* doreturn block
without a message.
--
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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057
--- Comment #8 from Jonathan Druart
---
Comment on attachment 90388
--> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=90388
Bug 23057: Update do_checkin
Review of attachment 90388:
-->
(https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html=23057=90388)
-
::: C4/Circulation.pm
@@ +1945,4 @@
> }
>
> if ( $item->itemlost and
> C4::Context->preference("BlockReturnOfLostItems") ) {
> +$messages->{'ReturnOfLostItemBlocked'};
Should we handle it from returns.pl as well?
--
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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Kyle M Hall changed: What|Removed |Added Attachment #90385|0 |1 is obsolete|| --- Comment #7 from Kyle M Hall --- Created attachment 90388 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=90388=edit Bug 23057: Update do_checkin -- 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 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #6 from Kyle M Hall --- Created attachment 90385 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=90385=edit Bug 23057: Update subroutine so tests pass -- 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 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Kyle M Hall changed: What|Removed |Added Attachment #90377|0 |1 is obsolete|| --- Comment #5 from Kyle M Hall --- Created attachment 90384 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=90384=edit Bug 23057: Unit 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 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Kyle M Hall changed: What|Removed |Added Status|NEW |Needs Signoff -- 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 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #4 from Kyle M Hall --- Created attachment 90377 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=90377=edit Bug 23057: Unit 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 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #3 from David Cook --- I haven't actually tested my patch or run the unit tests, but hopefully this is the solution. I have more urgent matters to attend to but I'll look back on this one as soon as I can. -- 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 --- Comment #2 from David Cook --- Created attachment 90344 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=90344=edit Bug 23057: If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason This patch fixes a bug from Bug 15221 and hopefully also makes it so that the alert flag is suppressed when checked_in_ok is set and an item is not checked out. To test: 0) Create patron in web interface with a cardnumber and userid of "staff" with a password that matches the account in SIPconfig.xml. Also set their branch to CPL (also matching SIPconfig.xml). 1) Create an item with a barcode of 'test' 2) Choose a patron to check out to and record their borrowernumber In one terminal: 3) cd to your git directory (e.g. /home/koha/koha) 4) perl ./C4/SIP/SIPServer.pm ~/koha-dev/etc/SIPconfig.xml In another terminal: 5) cd to your git directory (e.g. /home/koha/koha) 6) perl ./misc/sip_cli_emulator.pl -l CPL -su staff -sp --port=6001 --address=localhost --item test -m checkin --patron NOTE: You need to replace with the borrowernumber from Step 2, and with the password from Step 0. -- 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 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 David Cook changed: What|Removed |Added CC||dc...@prosentient.com.au --- Comment #1 from David Cook --- Uh oh, that's no good. When I was writing the patch for 15221, I was looking at implementing the alert suppression a different way...looking at sub do_checkin in C4/SIP/ILS/Transaction/Checkin.pm, I see the following lines: my ($return, $messages, $issue, $borrower) = AddReturn($barcode, $branch, undef, dt_from_string($return_date)); $self->alert(!$return); So we should revert the patch for 15221, and add $checked_in_ok to the params for sub do_checkin. And wrap that $self->alert(!$return); within a condition looking for $checked_in_ok. Not sure why I didn't do that originally. I was going to... but then I figured I'd try to centralize the changes with the existing use of $checked_in_ok. Bad call on my part. I'm super pressed for time at the moment, but I'll do up a quick patch with what I'm thinking and post it here for review... -- 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 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Martha Fuerst changed: What|Removed |Added CC||mfue...@hmcpl.org -- You are receiving this mail because: 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 23057] If checked_in_ok is set and item is not checked out, alert flag is supressed for *any* reason
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23057 Kyle M Hall changed: What|Removed |Added Depends on||15221 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15221 [Bug 15221] SIP server always sets the alert flag when item not returned -- 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 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/
