http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=1633

--- Comment #26 from Jared Camins-Esakov <[email protected]> 
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.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://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/

Reply via email to