From 6a73a3ddf57b242cf097f6c867eb897c47987c66 Mon Sep 17 00:00:00 2001 From: Henri-Damien LAURENT Date: Wed, 26 Aug 2009 01:40:25 +0200 Subject: [PATCH] Adding some checking in facet branches When using search via commandline scripts, branch facets are sometimes failing badly. Not a HASH reference at C4/Search.pm line 557, This patch prevents this Signed-off-by: Galen Charlton --- C4/Search.pm | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/C4/Search.pm b/C4/Search.pm index 83a8455a3a..b794f749d0 100644 --- a/C4/Search.pm +++ b/C4/Search.pm @@ -513,8 +513,17 @@ sub getRecords { # if it's a branch, label by the name, not the code, if ( $link_value =~ /branch/ ) { - $facet_label_value = - $branches->{$one_facet}->{'branchname'}; + if (defined $branches + && ref($branches) eq "HASH" + && defined $branches->{$one_facet} + && ref ($branches->{$one_facet}) eq "HASH") + { + $facet_label_value = + $branches->{$one_facet}->{'branchname'}; + } + else { + $facet_label_value = "*"; + } } # but we're down with the whole label being in the link's title. -- 2.20.1