[Koha-bugs] [Bug 9580] Cover image from Coce, a remote image URL cache

[bugzilla-daemon at bugs.koha-community.org]

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

--- Comment #41 from Frédéric Demians <frederic at tamil.fr> ---
(In reply to M. Tompsett from comment #39)

> ::: installer/data/mysql/sysprefs.sql
> @@ +86,5 @@
> > +('CoceHost', 0, NULL, 'Coce server URL', 'Free'),
> > +('CoceProviders', 0, NULL, 'Coce providers, for example aws,gb', 'Free'),
> 
> This should be empty string ('') or NULL (which is what you put in the
> updatedatabase.pl) not 0!
> 
> Personally, I'm torn, because I checked out other URL entries in
> updatedatabase.pl and I have seen it both ways (NULL and empty string). That
> is, there is no standard. :(

Ok. My reasoning was that in Perl code, syspref are tested like that:

  if ( $my_syspref ) { }

And 0 is the false value in Perl (like undef and ''). But I can comply to
usage, of course.

-- 
You are receiving this mail because:
You are watching all bug changes.