[Bug 42637] New: Check if biblio exists before checking if it is a fast add framework
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42637 Bug ID: 42637 Summary: Check if biblio exists before checking if it is a fast add framework Initiative type: --- Sponsorship --- status: Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: koha-bugs@lists.koha-community.org Reporter: blawlor@clamsnet.org QA Contact: testopia@bugs.koha-community.org Bug 32773 introduced a regression, when a biblionumber does not exist. Instead of "The record you requested does not exist (99999999)." You get a 500 error, or in ktd "Can't call method "frameworkcode" on an undefined value at /kohadevbox/koha/catalogue/detail.pl line 94" -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42637 Brendan Lawlor <blawlor@clamsnet.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |32773 Status|NEW |Needs Signoff Patch complexity|--- |Trivial patch Assignee|koha-bugs@lists.koha-commun |blawlor@clamsnet.org |ity.org | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32773 [Bug 32773] Have the ability to have more than 1 Fast Add framework -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42637 --- Comment #1 from Brendan Lawlor <blawlor@clamsnet.org> --- Created attachment 199260 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=199260&action=edit Bug 42637: Check if biblio exists before checking if it is fast add This patch reorders code to make sure the biblio record exists before checking if the framework is a fast add framework. To test: 1. Go to a non existing biblio record like: http://localhost:8081/cgi-bin/koha/catalogue/detail.pl?biblionumber=99999999 2. See a 500 error or in ktd: Can't call method "frameworkcode" on an undefined value 3. Apply patch and restart_all 4. Repeat step 1 5. Now you get the nice error message: The record you requested does not exist (99999999). -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42637 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42637 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #199260|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42637 --- Comment #2 from David Nind <david@davidnind.com> --- Created attachment 199267 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=199267&action=edit Bug 42637: Check if biblio exists before checking if it is fast add This patch reorders code to make sure the biblio record exists before checking if the framework is a fast add framework. To test: 1. Go to a non existing biblio record like: http://localhost:8081/cgi-bin/koha/catalogue/detail.pl?biblionumber=99999999 2. See a 500 error or in ktd: Can't call method "frameworkcode" on an undefined value 3. Apply patch and restart_all 4. Repeat step 1 5. Now you get the nice error message: The record you requested does not exist (99999999). Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42637 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |This fixes a regression release notes| |from Bug 32773 - Have the | |ability to have more than 1 | |Fast Add framework, added | |to Koha 26.05. | | | |If you tried to access a | |record that doesn't exist, | |you got a 500 error "Can't | |call method "frameworkcode" | |on an undefined value at | |/kohadevbox/koha/catalogue/ | |detail.pl line 94". | | | |With this fix, you now get | |a message saying "The | |record you requested does | |not exist (XXXXXX)." (where | |XXXX is the biblionumber | |you were trying to access). CC| |david@davidnind.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42637 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lucas@bywatersolutions.com QA Contact|testopia@bugs.koha-communit |lucas@bywatersolutions.com |y.org | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42637 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42637 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #199267|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42637 --- Comment #3 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 199279 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=199279&action=edit Bug 42637: Check if biblio exists before checking if it is fast add This patch reorders code to make sure the biblio record exists before checking if the framework is a fast add framework. To test: 1. Go to a non existing biblio record like: http://localhost:8081/cgi-bin/koha/catalogue/detail.pl?biblionumber=99999999 2. See a 500 error or in ktd: Can't call method "frameworkcode" on an undefined value 3. Apply patch and restart_all 4. Repeat step 1 5. Now you get the nice error message: The record you requested does not exist (99999999). Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42637 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to main Version(s)| |26.05.00 released in| | --- Comment #4 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Pushed to main for 26.05.00. Great work all! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42637 Jacob O'Mara <jacob.omara@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to main |Needs documenting --- Comment #5 from Jacob O'Mara <jacob.omara@openfifth.co.uk> --- This will not be backported to 25.11.x due to it being deemed an enhancement or a dependancy not being met -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42637 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|Needs documenting |RESOLVED --- Comment #6 from David Nind <david@davidnind.com> --- Bug fix, no changes to the documentation required. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org