From a9782f5970c15f1b7d2c595ecba71c976cc9f1a1 Mon Sep 17 00:00:00 2001 From: Henri-Damien LAURENT Date: Fri, 27 Mar 2009 17:21:35 +0100 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 --- C4/Search.pm | 13 +++++++++++-- 1 files changed, 11 insertions(+), 2 deletions(-) diff --git a/C4/Search.pm b/C4/Search.pm index 13a1e88..00e40ee 100644 --- a/C4/Search.pm +++ b/C4/Search.pm @@ -570,8 +570,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. -- 1.5.6.3