[Koha-bugs] [Bug 38751] Creating ILL request through OPAC openURL explodes if same attribute defined twice
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38751 --- Comment #17 from Paul Derscheid --- Nice work everyone! Pushed to 24.11.x for 24.11.04 -- 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 38751] Creating ILL request through OPAC openURL explodes if same attribute defined twice
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38751 Paul Derscheid changed: What|Removed |Added Version(s)|25.05.00|25.05.00,24.11.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 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 38751] Creating ILL request through OPAC openURL explodes if same attribute defined twice
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38751 Katrin Fischer changed: What|Removed |Added Version(s)||25.05.00 released in|| Status|Passed QA |Pushed to main -- 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 38751] Creating ILL request through OPAC openURL explodes if same attribute defined twice
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38751 --- Comment #16 from Katrin Fischer --- Pushed for 25.05! Well done everyone, thank you! -- 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 38751] Creating ILL request through OPAC openURL explodes if same attribute defined twice
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38751 Martin Renvoize (ashimema) changed: What|Removed |Added CC||martin.renvoize@ptfs-europe ||.com QA Contact|testo...@bugs.koha-communit |martin.renvoize@ptfs-europe |y.org |.com Status|Signed Off |Passed QA --- Comment #15 from Martin Renvoize (ashimema) --- Clear bugfix and code improvement, well tested and clear. QA scripts all happy, Passing 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 38751] Creating ILL request through OPAC openURL explodes if same attribute defined twice
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38751 Martin Renvoize (ashimema) changed: What|Removed |Added Attachment #176124|0 |1 is obsolete|| --- Comment #14 from Martin Renvoize (ashimema) --- Created attachment 176253 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=176253&action=edit Bug 38751: Update Standard ILL backend migrate Make use of newly updated ILL::Request->extended_attributes method instead of creating the query manually The test plan in bug 38819 fails without this patch (page explodes with duplicate ID error). It's the same cause as the one being fixed here in bug 38751 so I'm submitting this here. Signed-off-by: David Nind Signed-off-by: Martin Renvoize -- 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 38751] Creating ILL request through OPAC openURL explodes if same attribute defined twice
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38751 Martin Renvoize (ashimema) changed: What|Removed |Added Attachment #175905|0 |1 is obsolete|| --- Comment #13 from Martin Renvoize (ashimema) --- Created attachment 176252 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=176252&action=edit Bug 38751: Update Standard ILL backend add_request Make use of newly updated ILL::Request->extended_attributes method instead of creating the query manually Signed-off-by: David Nind Signed-off-by: Martin Renvoize -- 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 38751] Creating ILL request through OPAC openURL explodes if same attribute defined twice
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38751 Martin Renvoize (ashimema) changed: What|Removed |Added Attachment #175904|0 |1 is obsolete|| --- Comment #12 from Martin Renvoize (ashimema) --- Created attachment 176251 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=176251&action=edit Bug 38751: Add tests prove t/db_dependent/Koha/ILL/Request.t Signed-off-by: David Nind Signed-off-by: Martin Renvoize -- 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 38751] Creating ILL request through OPAC openURL explodes if same attribute defined twice
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38751 Martin Renvoize (ashimema) changed: What|Removed |Added Attachment #175903|0 |1 is obsolete|| --- Comment #11 from Martin Renvoize (ashimema) --- Created attachment 176250 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=176250&action=edit Bug 38751: Update ILL::Request->extended_attributes This updates the extended_attributes method to also act as setter. Additionally, it'll skip adding any attribute that result in error, preventing an ugly error 500. Test plan: 1) Enable ILLModule sys pref 2) Attempt to create the following openurl request through the OPAC: http://localhost:8080/cgi-bin/koha/opac-illrequests.pl?DOI=10.1001%2Fjama.2024.11146&atitle=Interventions%20for%20High%20Body%20Mass%20Index%20in%20Children%20and%20Adolescents%3A%20US%20Preventive%20Services%20Task%20Force%20Recommendation%20Statement.&aulast=Nicholson%2C%20Wanda%20K.&backend=FreeForm&cardnumber=53807898&date=20240716&genre=article&id=DOI%3A10.1001%2Fjama.2024.11146&issn=00987484&issue=3&method=create&opac=1&openurl=1 3) Notice it fails with a Duplicate ID error and the OPAC blows up. This is because Koha is trying to add the 'DOI' illrequestattribute twice and that is resulting in a duplicate ID 4) Apply patches, repeat test plan, notice the error no longer happens, the request is created correctly and all successful attribute have been added. Signed-off-by: David Nind Signed-off-by: Martin Renvoize -- 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 38751] Creating ILL request through OPAC openURL explodes if same attribute defined twice
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38751 --- Comment #10 from Pedro Amorim --- Created attachment 176124 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=176124&action=edit Bug 38751: Update Standard ILL backend migrate Make use of newly updated ILL::Request->extended_attributes method instead of creating the query manually The test plan in bug 38819 fails without this patch (page explodes with duplicate ID error). It's the same cause as the one being fixed here in bug 38751 so I'm submitting this here. -- 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 38751] Creating ILL request through OPAC openURL explodes if same attribute defined twice
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38751 Pedro Amorim changed: What|Removed |Added Blocks||38819 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38819 [Bug 38819] ILL - 'Switch provider' should use AutoILLBackendPriority -- 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 38751] Creating ILL request through OPAC openURL explodes if same attribute defined twice
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38751 David Nind changed: What|Removed |Added Attachment #175705|0 |1 is obsolete|| --- Comment #9 from David Nind --- Created attachment 175905 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=175905&action=edit Bug 38751: Update Standard ILL backend add_request Make use of newly updated ILL::Request->extended_attributes method instead of creating the query manually Signed-off-by: David Nind -- 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 38751] Creating ILL request through OPAC openURL explodes if same attribute defined twice
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38751 David Nind changed: What|Removed |Added Attachment #175704|0 |1 is obsolete|| --- Comment #8 from David Nind --- Created attachment 175904 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=175904&action=edit Bug 38751: Add tests prove t/db_dependent/Koha/ILL/Request.t Signed-off-by: David Nind -- 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 38751] Creating ILL request through OPAC openURL explodes if same attribute defined twice
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38751 David Nind changed: What|Removed |Added Attachment #175703|0 |1 is obsolete|| --- Comment #7 from David Nind --- Created attachment 175903 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=175903&action=edit Bug 38751: Update ILL::Request->extended_attributes This updates the extended_attributes method to also act as setter. Additionally, it'll skip adding any attribute that result in error, preventing an ugly error 500. Test plan: 1) Enable ILLModule sys pref 2) Attempt to create the following openurl request through the OPAC: http://localhost:8080/cgi-bin/koha/opac-illrequests.pl?DOI=10.1001%2Fjama.2024.11146&atitle=Interventions%20for%20High%20Body%20Mass%20Index%20in%20Children%20and%20Adolescents%3A%20US%20Preventive%20Services%20Task%20Force%20Recommendation%20Statement.&aulast=Nicholson%2C%20Wanda%20K.&backend=FreeForm&cardnumber=53807898&date=20240716&genre=article&id=DOI%3A10.1001%2Fjama.2024.11146&issn=00987484&issue=3&method=create&opac=1&openurl=1 3) Notice it fails with a Duplicate ID error and the OPAC blows up. This is because Koha is trying to add the 'DOI' illrequestattribute twice and that is resulting in a duplicate ID 4) Apply patches, repeat test plan, notice the error no longer happens, the request is created correctly and all successful attribute have been added. Signed-off-by: David Nind -- 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 38751] Creating ILL request through OPAC openURL explodes if same attribute defined twice
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38751 David Nind 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 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 38751] Creating ILL request through OPAC openURL explodes if same attribute defined twice
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38751 Pedro Amorim changed: What|Removed |Added Attachment #175701|0 |1 is obsolete|| --- Comment #6 from Pedro Amorim --- Created attachment 175705 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=175705&action=edit Bug 38751: Update Standard ILL backend add_request Make use of newly updated ILL::Request->extended_attributes method instead of creating the query manually -- 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 38751] Creating ILL request through OPAC openURL explodes if same attribute defined twice
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38751 Pedro Amorim changed: What|Removed |Added Attachment #175700|0 |1 is obsolete|| --- Comment #5 from Pedro Amorim --- Created attachment 175704 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=175704&action=edit Bug 38751: Add tests prove t/db_dependent/Koha/ILL/Request.t -- 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 38751] Creating ILL request through OPAC openURL explodes if same attribute defined twice
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38751 Pedro Amorim changed: What|Removed |Added Attachment #175702|0 |1 is obsolete|| --- Comment #4 from Pedro Amorim --- Created attachment 175703 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=175703&action=edit Bug 38751: Update ILL::Request->extended_attributes This updates the extended_attributes method to also act as setter. Additionally, it'll skip adding any attribute that result in error, preventing an ugly error 500. Test plan: 1) Enable ILLModule sys pref 2) Attempt to create the following openurl request through the OPAC: http://localhost:8080/cgi-bin/koha/opac-illrequests.pl?DOI=10.1001%2Fjama.2024.11146&atitle=Interventions%20for%20High%20Body%20Mass%20Index%20in%20Children%20and%20Adolescents%3A%20US%20Preventive%20Services%20Task%20Force%20Recommendation%20Statement.&aulast=Nicholson%2C%20Wanda%20K.&backend=FreeForm&cardnumber=53807898&date=20240716&genre=article&id=DOI%3A10.1001%2Fjama.2024.11146&issn=00987484&issue=3&method=create&opac=1&openurl=1 3) Notice it fails with a Duplicate ID error and the OPAC blows up. This is because Koha is trying to add the 'DOI' illrequestattribute twice and that is resulting in a duplicate ID 4) Apply patches, repeat test plan, notice the error no longer happens, the request is created correctly and all successful attribute have been added. -- 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 38751] Creating ILL request through OPAC openURL explodes if same attribute defined twice
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38751 --- Comment #3 from Pedro Amorim --- Created attachment 175702 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=175702&action=edit Bug 38751: Update ILL::Request->extended_attributes This updates the extended_attributes method to also act as setter. Additionally, it'll skip adding any attribute that result in error, preventing an ugly error 500. Test plan: 1) Enable ILLModule sys pref 2) Attempt to create the following openurl request through the OPAC: http://localhost:8080/cgi-bin/koha/opac-illrequests.pl?DOI=10.1001%2Fjama.2024.11146&atitle=Interventions%20for%20High%20Body%20Mass%20Index%20in%20Children%20and%20Adolescents%3A%20US%20Preventive%20Services%20Task%20Force%20Recommendation%20Statement.&aulast=Nicholson%2C%20Wanda%20K.&backend=FreeForm&cardnumber=53807898&date=20240716&genre=article&id=DOI%3A10.1001%2Fjama.2024.11146&issn=00987484&issue=3&method=create&opac=1&openurl=1 3) Notice it fails with a Duplicate ID error and the OPAC blows up. This is because Koha is trying to add the 'DOI' illrequestattribute twice and that is resulting in a duplicate ID 4) Apply patches, repeat test plan, notice the error no longer happens, the request is created correctly and all successful attribute have been added. -- 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 38751] Creating ILL request through OPAC openURL explodes if same attribute defined twice
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38751 --- Comment #2 from Pedro Amorim --- Created attachment 175701 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=175701&action=edit Bug 38751: Update Standard ILL backend add_request Make use of newly updated ILL::Request->extended_attributes method instead of creating the query manually -- 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 38751] Creating ILL request through OPAC openURL explodes if same attribute defined twice
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38751 --- Comment #1 from Pedro Amorim --- Created attachment 175700 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=175700&action=edit Bug 38751: Add tests prove t/db_dependent/Koha/ILL/Request.t -- 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 38751] Creating ILL request through OPAC openURL explodes if same attribute defined twice
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38751 Pedro Amorim changed: What|Removed |Added Patch complexity|--- |Trivial patch Status|NEW |Needs Signoff -- 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/
