http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=1633 --- Comment #26 from Jared Camins-Esakov <jcamins@cpbibliography.com> 2011-12-14 15:17:22 UTC --- (In reply to comment #25)

From an IRC discussion regarding the RetrieveImage sub:

<magnuse> chris_n: http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=6755 * chris_n notes that this patch adds a what appears to be unrelated line to Makefile.PL

Without the added line to Makefile.PL, production-mode Koha installations fail dramatically because of the Koha/Images.pm file not being installed.

<chris_n> yeah, that sub needs to be changed to return a null value if there is a db error <chris_n> it already logs it <chris_n> ie. if ($sth->errstr) { log error; return;} <chris_n> and then let the caller handle it

I can make that change. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.