Bug 19537: (QA follow-up) Consistent structures
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Fri, 3 Nov 2017 07:24:15 +0000 (08:24 +0100)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Wed, 8 Nov 2017 16:34:48 +0000 (13:34 -0300)
The subfield_loop structure should contain things like:
$VAR1 = {
          'marc_subfield' => 'v',
          'marc_values' => [
                             'form'
                           ]
        };

This patches replaces the list by an array ref to restore consistency,
although Template Toolkit does not really care. Both work.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

authorities/blinddetail-biblio-search.pl

index 838e4ba..9e7930d 100755 (executable)
@@ -122,9 +122,9 @@ if ($authid) {
             $indicator2 = defined $thes_mapping{$thes_008_11} ? $thes_mapping{$thes_008_11} : $thes_008_11;
             if ($indicator2 eq '7') {
                 if ($thes_008_11 eq 'r') {
-                    push @subfield_loop, { marc_subfield => '2', marc_values => ('aat') };
+                    push @subfield_loop, { marc_subfield => '2', marc_values => [ 'aat' ] };
                 } elsif ($thes_008_11 eq 's') {
-                    push @subfield_loop, { marc_subfield => '2', marc_values => ('sears') };
+                    push @subfield_loop, { marc_subfield => '2', marc_values => [ 'sears' ] };
                 }
             }
         }