]> git.koha-community.org Git - koha.git/commit
Bug 8175 - check logs error or displays incorrectly in details.pl
authorMark Tompsett <mtompset@hotmail.com>
Sun, 2 Sep 2012 12:46:03 +0000 (20:46 +0800)
committerChris Cormack <chrisc@catalyst.net.nz>
Sun, 14 Oct 2012 07:32:54 +0000 (20:32 +1300)
commit91fb5fd2bcc78cc31f39c6a5ac985f5c22136cf1
tree5a0b7b1dcee9b404983d835a0146fcb5b39a07db
parent68c01e58e93e039ae664e1754105506d9a270373
Bug 8175 - check logs error or displays incorrectly in details.pl

Changed "$item->{'materials'} ne ''" to
"defined($item->{'materials'}) && $item->{'materials'} =~ /\S/"
in if condition to prevent error when it is NULL, and to
properly capture the intent of printing if there is something
visible.

Cases tested include NULL, '0', '', ' ', and 'blah' by using
 UPDATE items SET materials=NULL where biblionumber=19158;
where the biblionumber was chosen randomly, because only NULL
was in the items.materials field. The NULL case triggers an
error, but it does display correctly.

The ' ' case displays an apparently empty column, which does not
seem to be the intent of the flag that is being set. This is why
a simple $var check is not sufficient.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
catalogue/detail.pl