Attachment 10817 Details for Bug 7401 – Bug 7401 - Shelving Location facet instead of Branch facet when only 1 branch configured

[patch] Bug 7401 - Shelving Location facet instead of Branch facet when only 1 branch configured

Bug-7401---Shelving-Location-facet-instead-of-Bran.patch (text/plain), 8.83 KB, created by Kyle M Hall (khall) on 2012-07-13 12:20:12 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2012-07-13 12:20:12 UTC

Size: 8.83 KB

patch

obsolete

>From 4ac2e5f3f3a4d33f24f37a0003b2b33e9fe6d844 Mon Sep 17 00:00:00 2001
>From: Ian Walls <ian.walls@bywatersolutions.com>
>Date: Thu, 16 Feb 2012 12:09:44 -0500
>Subject: [PATCH] Bug 7401 - Shelving Location facet instead of Branch facet when only 1 branch configured
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>
>Enable for either
>A) SingleBranchMode is enabled
>or
>B) There is only one branch in the branches table
>
>Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
>
>Tested with one branch and with multiple branches, tested with
>SingleBranchMode on and off. All tests passed.
>
>Signed-off-by: Michael Davis <m.r.davis@cranfield.ac.uk>
>---
> C4/Branch.pm                                       |   10 ++++
> C4/Koha.pm                                         |   58 ++++++++++++++------
> C4/Search.pm                                       |    9 +++-
> .../intranet-tmpl/prog/en/includes/facets.inc      |    1 +
> .../opac-tmpl/prog/en/includes/opac-facets.inc     |    1 +
> opac/opac-search.pl                                |    2 +-
> 6 files changed, 61 insertions(+), 20 deletions(-)
>
>diff --git a/C4/Branch.pm b/C4/Branch.pm
>index 1959a3d..dcd4ef7 100644
>--- a/C4/Branch.pm
>+++ b/C4/Branch.pm
>@@ -46,6 +46,7 @@ BEGIN {
> 		&DelBranchCategory
> 	        &CheckCategoryUnique
> 		&mybranch
>+		&GetBranchesCount
> 	);
> 	@EXPORT_OK = qw( &onlymine &mybranch get_branch_code_from_name );
> }
>@@ -580,6 +581,15 @@ sub get_branch_code_from_name {
>    return $sth->fetchrow_array;
> }
> 
>+sub GetBranchesCount {
>+    my $dbh = C4::Context->dbh();
>+    my $query = "SELECT COUNT(*) AS branches_count FROM branches";
>+    my $sth = $dbh->prepare( $query );
>+    $sth->execute();
>+    my $row = $sth->fetchrow_hashref();
>+    return $row->{'branches_count'};
>+}
>+
> 1;
> __END__
> 
>diff --git a/C4/Koha.pm b/C4/Koha.pm
>index 4f386e4..b85351c 100644
>--- a/C4/Koha.pm
>+++ b/C4/Koha.pm
>@@ -24,7 +24,7 @@ use strict;
> #use warnings; FIXME - Bug 2505
> 
> use C4::Context;
>-
>+use C4::Branch qw(GetBranchesCount);
> use Memoize;
> use DateTime;
> use DateTime::Format::MySQL;
>@@ -713,14 +713,27 @@ sub getFacets {
>                 tags  => [ qw/ 225a / ],
>                 sep   => ', ',
>             },
>-        ];
>-        my $library_facet = {
>-            idx   => 'branch',
>-            label => 'Libraries',
>-            tags  => [ qw/ 995b / ],
>-            expanded => '1',
>-        };
>-        push @$facets, $library_facet unless C4::Context->preference("singleBranchMode");
>+            ];
>+
>+            my $library_facet;
>+            unless ( C4::Context->preference("singleBranchMode") || GetBranchesCount() == 1 ) {
>+                $library_facet = {
>+                    link_value  => 'branch',
>+                    label_value => 'Libraries',
>+                    tags        => [ '995', ],
>+                    subfield    => 'b',
>+                    expanded    => '1',
>+                };
>+            } else {
>+                $library_facet = {
>+                    link_value  => 'location',
>+                    label_value => 'Location',
>+                    tags        => [ '995' ],
>+                    subfield    => 'c',
>+                    expanded    => '1',
>+                };
>+            }
>+            push( @$facets, $library_facet );
>     }
>     else {
>         $facets = [
>@@ -767,15 +780,26 @@ sub getFacets {
>                 sep   => ', ',
>             },
>             ];
>+
>             my $library_facet;
>-            $library_facet = {
>-                idx   => 'branch',
>-                label => 'Libraries',
>-                tags  => [ qw/ 952b / ],
>-                sep   => ', ',
>-                expanded    => '1',
>-            };
>-            push @$facets, $library_facet unless C4::Context->preference("singleBranchMode");
>+            unless ( C4::Context->preference("singleBranchMode") || GetBranchesCount() == 1 ) {
>+                $library_facet = {
>+                    link_value  => 'branch',
>+                    label_value => 'Libraries',
>+                    tags        => [ '952', ],
>+                    subfield    => 'b',
>+                    expanded    => '1',
>+                };
>+            } else {
>+                $library_facet = {
>+                    link_value => 'location',
>+                    label_value => 'Location',
>+                    tags => [ '952' ],
>+                    subfield => 'c',
>+                    expanded => '1',
>+                };
>+            }
>+            push( @$facets, $library_facet );
>     }
>     return $facets;
> }
>diff --git a/C4/Search.pm b/C4/Search.pm
>index 4f61661..8928ad4 100644
>--- a/C4/Search.pm
>+++ b/C4/Search.pm
>@@ -301,8 +301,8 @@ See verbse embedded documentation.
> sub getRecords {
>     my (
>         $koha_query,       $simple_query, $sort_by_ref,    $servers_ref,
>-        $results_per_page, $offset,       $expanded_facet, $branches,$itemtypes,
>-        $query_type,       $scan
>+        $results_per_page, $offset,       $expanded_facet, $branches,
>+        $itemtypes,        $query_type,   $scan,           $opac
>     ) = @_;
> 
>     my @servers = @$servers_ref;
>@@ -567,6 +567,11 @@ sub getRecords {
>                                 }
>                             }
> 
>+                            # also, if it's a location code, use the name instead of the code
>+                            if ( $link_value =~ /location/ ) {
>+                                $facet_label_value = GetKohaAuthorisedValueLib('LOC', $one_facet, $opac) || '*';
>+                            }
>+
>                             # but we're down with the whole label being in the link's title.
>                             push @this_facets_array, {
>                                 facet_count       => $facets_counter->{$link_value}->{$one_facet},
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/facets.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/facets.inc
>index 5cf6a7f..1731bed 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/includes/facets.inc
>+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/facets.inc
>@@ -17,6 +17,7 @@
> [% IF ( facets_loo.type_label_Series ) %]Series[% END %]
> [% IF ( facets_loo.type_label_ItemTypes ) %]Item types[% END %]
> [% IF ( facets_loo.type_label_Libraries ) %]Libraries[% END %]
>+[% IF ( facets_loo.type_label_Location ) %]Locations[% END %]
> <ul>
>         [% FOREACH facet IN facets_loo.facets %]<li><a href="/cgi-bin/koha/catalogue/search.pl?[% query_cgi |html %][% limit_cgi |html %][% IF ( sort_by ) %]&amp;sort_by=[% sort_by %][% END %]&amp;limit=[% facet.type_link_value %]:[% facet.facet_link_value %]" title="[% facet.facet_title_value %]">[% facet.facet_label_value %]</a> [% IF ( displayFacetCount ) %]([% facet.facet_count %])[% END %]</li>[% END %][% IF ( facets_loo.expandable ) %]
>         <li class="showmore"><a href="/cgi-bin/koha/catalogue/search.pl?[% query_cgi |html %][% limit_cgi |html %][% IF ( sort_by ) %]&amp;sort_by=[% sort_by %][% END %][% IF ( offset ) %]&amp;offset=[% offset %][% END %]&amp;expand=[% facets_loo.expand %]#[% facets_loo.type_id %]">Show more</a></li>
>diff --git a/koha-tmpl/opac-tmpl/prog/en/includes/opac-facets.inc b/koha-tmpl/opac-tmpl/prog/en/includes/opac-facets.inc
>index d9e4577..082f846 100644
>--- a/koha-tmpl/opac-tmpl/prog/en/includes/opac-facets.inc
>+++ b/koha-tmpl/opac-tmpl/prog/en/includes/opac-facets.inc
>@@ -18,6 +18,7 @@
> [% UNLESS ( singleBranchMode ) %]
> [% IF ( facets_loo.type_label_Libraries ) %]Libraries[% END %]
> [% END %]
>+[% IF ( facets_loo.type_label_Location ) %]Locations[% END %]
> <ul>
>         [% FOREACH facet IN facets_loo.facets %]<li><a href="/cgi-bin/koha/opac-search.pl?[% query_cgi |html %][% limit_cgi |html %][% IF ( sort_by ) %]&amp;sort_by=[% sort_by %][% END %]&amp;limit=[% facet.type_link_value %]:[% facet.facet_link_value %]" title="[% facet.facet_title_value |html %]">[% facet.facet_label_value %]</a> [% IF ( displayFacetCount ) %]([% facet.facet_count %])[% END %]</li>[% END %][% IF ( facets_loo.expandable ) %]
>         <li class="showmore"><a href="/cgi-bin/koha/opac-search.pl?[% query_cgi |html %][% limit_cgi |html %][% IF ( sort_by ) %]&amp;sort_by=[% sort_by %][% END %][% IF ( offset ) %]&amp;offset=[% offset %][% END %]&amp;expand=[% facets_loo.expand %]#[% facets_loo.type_id %]">Show more</a></li>
>diff --git a/opac/opac-search.pl b/opac/opac-search.pl
>index e39e547..e430f5f 100755
>--- a/opac/opac-search.pl
>+++ b/opac/opac-search.pl
>@@ -487,7 +487,7 @@ elsif (C4::Context->preference('NoZebra')) {
>     $pasarParams .= '&amp;simple_query=' . $simple_query;
>     $pasarParams .= '&amp;query_type=' . $query_type if ($query_type);
>     eval {
>-        ($error, $results_hashref, $facets) = getRecords($query,$simple_query,\@sort_by,\@servers,$results_per_page,$offset,$expanded_facet,$branches,$itemtypes,$query_type,$scan);
>+        ($error, $results_hashref, $facets) = getRecords($query,$simple_query,\@sort_by,\@servers,$results_per_page,$offset,$expanded_facet,$branches,$itemtypes,$query_type,$scan,1);
>     };
> }
> # This sorts the facets into alphabetical order
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 7401: 7040 | 7700 | 7741 | 8188 | 8189 | 10154 | 10210 | 10756 | 10817 | 10877 | 12241 | 12253 | 12254