I was cataloging a new book today. I followed these steps: 1. Z39 search of LOC 2. Choose title 3. Add summary 520 4. Add url 856 5. Choose authority for author 100 6. Hit save I got this error: Software error: Can't locate object method "new" via package "C4::Linker::FirstMatch" (perhaps you forgot to load "C4::Linker::FirstMatch"?) at /home/nengard/kohaclone/C4/Biblio.pm line 503. For help, please send mail to the webmaster (webmaster@debian), giving this error message and the time and date of the error.
I tried again skipping steps 3-5 and still get the same error.
Finally I changed my linker settings and just got a different error: Software error: Can't locate object method "new" via package "C4::Linker::Default" (perhaps you forgot to load "C4::Linker::Default"?) at /home/nengard/kohaclone/C4/Biblio.pm line 503. For help, please send mail to the webmaster (webmaster@debian), giving this error message and the time and date of the error.
Created attachment 12511 [details] [review] Bug 8818: make sure we load modules before using them An eval { eval "require $module;" }; was replaced with eval { eval { require $module; }; }; which is a no-op, meaning that the linker was not getting loaded, and the catalog module was throwing up a big nasty error every time someone tried to save a record with a heading. This patch replaces the require with can_load from Module::Load::Conditional, which is PBP-friendly, and offers equivalent functionality.
Created attachment 12512 [details] [review] [SIGNED-OFF] Bug 8818: make sure we load modules before using them An eval { eval "require $module;" }; was replaced with eval { eval { require $module; }; }; which is a no-op, meaning that the linker was not getting loaded, and the catalog module was throwing up a big nasty error every time someone tried to save a record with a heading. This patch replaces the require with can_load from Module::Load::Conditional, which is PBP-friendly, and offers equivalent functionality. Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
(In reply to comment #4) > Created attachment 12512 [details] [review] > [SIGNED-OFF] Bug 8818: make sure we load modules before using them patch looks good, passing QA $ koha-qa.pl -c 1 testing 1 commit(s) (applied to commit eb6effd) * 0a60190 Bug 8818: make sure we load modules before using them C4/Biblio.pm misc/link_bibs_to_authorities.pl * C4/Biblio.pm OK * misc/link_bibs_to_authorities.pl OK
updating severity since I can't catalog at all until this is pushed.
Patch pushed to master
Released in 3.10.0