Bug 15550: Fix authority search and detail at the OPAC

The author of Bug 15381 (me!) has mismatch the objects he was
manipulating and forgotten to update a template.
for the opac-aythorities-home, the variable is named authority_types,
not authtypesloop.
In opac-authoritiesdetail.pl, the $record is a MARC::Record and there is
no authtypecode method. We need to retrieve the authtypecode using
the new Koha::Authorities module.

Test plan:
Search for authorities at the OPAC and click on the detail link to go to
the detail page.
Without this patch, you will get errors.
With this patch, the errors should have gone and everything should work
as before bug 15381.

Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com>
Works as described. Pull down for choosing the auth_type displayed again

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Brendan Gallagher brendan@bywatersolutions.com

diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-authorities-home.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-authorities-home.tt
index 85ecf8d..d3dfcf8 100644 (file)
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-authorities-home.tt
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-authorities-home.tt
@@ -35,11 +35,11 @@
                                     <li>
                                         <label for="authtypecode">Authority type: </label>
                                         <select name="authtypecode" id="authtypecode">
-                                            [% FOREACH authtypesloo IN authtypesloop %]
-                                                [% IF ( authtypesloo.selected ) %]
-                                                    <option value="[% authtypesloo.value %]" selected="selected">[% authtypesloo.authtypetext %]</option>
+                                            [% FOREACH authority_type IN authority_types %]
+                                                [% IF authority_type.authtypecode == authtypecode %]
+                                                    <option value="[% authority_type.authtypecode%]" selected="selected">[% authority_type.authtypetext %]</option>
                                                 [% ELSE %]
-                                                    <option value="[% authtypesloo.value %]"> [% authtypesloo.authtypetext %]</option>
+                                                    <option value="[% authority_type.authtypecode%]">[% authority_type.authtypetext %]</option>
                                                 [% END %]
                                             [% END %]
                                         </select>

diff --git a/opac/opac-authoritiesdetail.pl b/opac/opac-authoritiesdetail.pl
index b867012..f46c511 100755 (executable)
--- a/opac/opac-authoritiesdetail.pl
+++ b/opac/opac-authoritiesdetail.pl
@@ -47,6 +47,7 @@ use CGI qw ( -utf8 );
 use MARC::Record;
 use C4::Koha;
 
+use Koha::Authorities;
 use Koha::Authority::Types;
 
 my $query = new CGI;
@@ -75,7 +76,8 @@ if ( ! $record ) {
     exit;
 }
 
-my $authtypecode = $record->authtypecode;
+my $authority = Koha::Authorities->find( $authid );
+my $authtypecode = $authority->authtypecode;
 
 if ($display_hierarchy){
     $template->{VARS}->{'displayhierarchy'} = $display_hierarchy;