[Koha-bugs] [Bug 25726] Holds to Pull made empty by pathological holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25726 Nick Clemens changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=27166 CC||n...@bywatersolutions.com --- Comment #15 from Nick Clemens --- possible cause on 27166 -- 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 25726] Holds to Pull made empty by pathological holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25726 Victor Grousset/tuxayo changed: What|Removed |Added CC||vic...@tuxayo.net Version(s)|20.11.00, 20.05.03, |20.11.00, 20.05.03, released in|19.11.09|19.11.09, 19.05.15 --- Comment #14 from Victor Grousset/tuxayo --- Backported to 19.05.x branch for 19.05.15 -- 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 25726] Holds to Pull made empty by pathological holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25726 Fridolin SOMERS changed: What|Removed |Added CC||fridolin.som...@biblibre.co ||m --- Comment #13 from Fridolin SOMERS --- Super I was looking for this exact fix. Looks like its the behavior of NOT IN () with NULL : I've found some explanations in : https://stackoverflow.com/questions/1001144/mysql-select-x-from-a-where-not-in-select-x-from-b-unexpected-result#1001197 . -- 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 25726] Holds to Pull made empty by pathological holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25726 Aleisha Amohia changed: What|Removed |Added CC||alei...@catalyst.net.nz Status|Pushed to stable|Pushed to oldstable Version(s)|20.11.00, 20.05.03 |20.11.00, 20.05.03, released in||19.11.09 --- Comment #12 from Aleisha Amohia --- backported to 19.11.x for 19.11.09 -- 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 25726] Holds to Pull made empty by pathological holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25726 Lucas Gass changed: What|Removed |Added Status|Pushed to master|Pushed to stable -- 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 25726] Holds to Pull made empty by pathological holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25726 Lucas Gass changed: What|Removed |Added Version(s)|20.11.00|20.11.00, 20.05.03 released in|| CC||lu...@bywatersolutions.com --- Comment #11 from Lucas Gass --- backported to 20.05.x for 20.05.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 25726] Holds to Pull made empty by pathological holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25726 --- Comment #10 from Jonathan Druart --- Pushed to master for 20.11, thanks to everybody involved! -- 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 25726] Holds to Pull made empty by pathological holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25726 Jonathan Druart changed: What|Removed |Added Version(s)||20.11.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 25726] Holds to Pull made empty by pathological holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25726 --- Comment #9 from Andrew Fuerste-Henry --- Thanks, all! I should add that I've only run into this on one library, so whatever got their holds into this state seems to be pretty vanishingly rare, but I'm keeping and eye out for more. If I can figure out how to reproduce it, I'll absolutely file a bug for it. -- 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 25726] Holds to Pull made empty by pathological holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25726 Marcel de Rooy changed: What|Removed |Added Status|Signed Off |BLOCKED QA Contact|testo...@bugs.koha-communit |m.de.r...@rijksmuseum.nl |y.org | 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 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 25726] Holds to Pull made empty by pathological holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25726 Marcel de Rooy changed: What|Removed |Added Attachment #105821|0 |1 is obsolete|| --- Comment #7 from Marcel de Rooy --- Created attachment 106493 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106493=edit Bug 25726: make holds to pull ignore holds where found is not null and itemnumber is null To test: 1- place 2 bib-level holds on available items 2- confirm they both show on Holds to Pull 3- edit one hold from the database to set found='T' 4- reload Holds to Pull, confirm it is now empty 5- apply patch 6- Reload Holds to Pull 7- confirm it now shows the hold you did not edit Signed-off-by: donnab Signed-off-by: Marcel de Rooy [EDIT] Using capitals for SQL reserved words; added a FIXME Tested with: update reserves set found='T', itemnumber=NULL where reserve_id=... This should be a workaround while we fix the underlying problem. -- 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 25726] Holds to Pull made empty by pathological holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25726 --- Comment #6 from Marcel de Rooy --- (In reply to Katrin Fischer from comment #5) > If I understand correctly this will make the items in 'Transit' status show > up in the list. But if they really are in transit - they can't be pulled... > so should they show up? I think the logic of the SQL might be right and we > need to take care to fix the holds, if the state of found = "T" and > itemnumber empty is unwanted. No the transit does not come up. Since it is in a itemnumber NOT IN ... construction. The only thing we exclude here is getting a NULL and then saying itemnumber NOT IN (NULL) making all biblio level holds disappear. -- 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 25726] Holds to Pull made empty by pathological holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25726 Marcel de Rooy changed: What|Removed |Added Status|BLOCKED |Passed QA 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 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 25726] Holds to Pull made empty by pathological holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25726 --- Comment #8 from Alex Arnaud --- (In reply to Katrin Fischer from comment #5) > > If I understand correctly this will make the items in 'Transit' status show > up in the list. But if they really are in transit - they can't be pulled... > so should they show up? That doesn't seem to make the items in transit show up in Holds to pull list. However, i'm not sure of any other side effects. > I think the logic of the SQL might be right and we > need to take care to fix the holds, if the state of found = "T" and > itemnumber empty is unwanted. Agreed. Maybe we should understand why we get this pathological holds in the first place. > > I've tried several things to get a 'pathological' hold, but haven't succeded > other than in turning up more oddities (see bug 25748). Unable to recreate neither... -- 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 25726] Holds to Pull made empty by pathological holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25726 --- Comment #5 from Katrin Fischer --- Hi Andrew, I am actually a bit confused by the SQL change and worried about unwanted side effects: -AND items.itemnumber NOT IN (select itemnumber FROM reserves where found IS NOT NULL) +AND items.itemnumber NOT IN (select itemnumber FROM reserves where found IS NOT NULL AND itemnumber IS NOT NULL) If I understand correctly this will make the items in 'Transit' status show up in the list. But if they really are in transit - they can't be pulled... so should they show up? I think the logic of the SQL might be right and we need to take care to fix the holds, if the state of found = "T" and itemnumber empty is unwanted. I've tried several things to get a 'pathological' hold, but haven't succeded other than in turning up more oddities (see bug 25748). -- 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 25726] Holds to Pull made empty by pathological holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25726 Katrin Fischer changed: What|Removed |Added Assignee|koha-b...@lists.koha-commun |and...@bywatersolutions.com |ity.org | -- 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 25726] Holds to Pull made empty by pathological holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25726 --- Comment #4 from Andrew Fuerste-Henry --- Me, too! We've only seen a couple examples and haven't yet tracked down how they got in that state. In both cases the hold had a reserves.found value of T, but was not actually in transit. In one of those cases, the patron had 2 holds on the same record, which suggests maybe bug 18958 is involved? But, yes, for sure, there is a larger issue of where this bad data came from, my patch is just alleviating the major symptom caused by the data's existence. (In reply to Katrin Fischer from comment #3) > Hi Andrew, > > I am a bit worried about the cause of these strange holds. You say > reserves.found is not null - what was the found value for these strange > holds? -- 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 25726] Holds to Pull made empty by pathological holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25726 --- Comment #3 from Katrin Fischer --- Hi Andrew, I am a bit worried about the cause of these strange holds. You say reserves.found is not null - what was the found value for these strange holds? -- 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 25726] Holds to Pull made empty by pathological holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25726 Donna changed: What|Removed |Added CC||bwsdo...@gmail.com Status|Needs Signoff |Signed Off -- 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 25726] Holds to Pull made empty by pathological holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25726 ByWater Sandboxes changed: What|Removed |Added Attachment #105800|0 |1 is obsolete|| --- Comment #2 from ByWater Sandboxes --- Created attachment 105821 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=105821=edit Bug 25726: make holds to pull ignore holds where found is not null and itemnumber is null To test: 1- place 2 bib-level holds on available items 2- confirm they both show on Holds to Pull 3- edit one hold from the database to set found='T' 4- reload Holds to Pull, confirm it is now empty 5- apply patch 6- Reload Holds to Pull 7- confirm it now shows the hold you did not edit Signed-off-by: donnab -- 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 25726] Holds to Pull made empty by pathological holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25726 --- Comment #1 from Andrew Fuerste-Henry --- Created attachment 105800 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=105800=edit Bug 25726: make holds to pull ignore holds where found is not null and itemnumber is null To test: 1- place 2 bib-level holds on available items 2- confirm they both show on Holds to Pull 3- edit one hold from the database to set found='T' 4- reload Holds to Pull, confirm it is now empty 5- apply patch 6- Reload Holds to Pull 7- confirm it now shows the hold you did not edit -- 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 25726] Holds to Pull made empty by pathological holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25726 Andrew Fuerste-Henry changed: What|Removed |Added Status|NEW |Needs Signoff -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ 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/