Attachment 128373 Details for Bug 29669 – Bug 29669: fix uninitialized value warnings when XSLTParse4Display is called

[patch] Bug 29669: fix uninitialized value warnings when XSLTParse4Display is called

Bug-29669-fix-uninitialized-value-warnings-when-XS.patch (text/plain), 2.73 KB, created by Peter Vashchuk on 2021-12-09 13:48:40 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Peter Vashchuk

Created: 2021-12-09 13:48:40 UTC

Size: 2.73 KB

patch

obsolete

>From d568b5ea1ee2027af8be83d04cdc3c23b5417dbc Mon Sep 17 00:00:00 2001
>From: Petro Vashchuk <stalkernoid@gmail.com>
>Date: Thu, 9 Dec 2021 15:47:07 +0200
>Subject: [PATCH] Bug 29669: fix uninitialized value warnings when
> XSLTParse4Display is called
>
>Wrong key 'xsl_filename' is present in opac-tags.pl and shelves.pl
>(the only places where it is used in the code even) instead of
>'xsl_syspref' that is actually used and this is the cause of those
>warnings.
>
>Also added honeypot to check if other calls in same way happens.
>Honeypot probably not needed, review when QA in the Community.
>
>To reproduce:
>1) Prepare some test list with items or use existing one.
>2) Go to the lists page (koha/virtualshelves/shelves.pl),
>notice multitude of new warnings in the "intranet-error.log" log file
>that mostly have to do with "Use of uninitialized value $xslsyspref".
>3) Apply the patch.
>4) Go back to same page and check it again, warns like that should not
>appear in the log file anymore.
>---
> C4/XSLT.pm                | 3 +++
> opac/opac-tags.pl         | 2 +-
> virtualshelves/shelves.pl | 2 +-
> 3 files changed, 5 insertions(+), 2 deletions(-)
>
>diff --git a/C4/XSLT.pm b/C4/XSLT.pm
>index 125f887cb0..4a89997bcc 100644
>--- a/C4/XSLT.pm
>+++ b/C4/XSLT.pm
>@@ -251,6 +251,9 @@ sub XSLTParse4Display {
>     my $variables    = $params->{xslt_variables};
>     my $items_rs     = $params->{items_rs};
> 
>+    die "Mandatory \$params->{xsl_syspref} does not provided, called with biblionumber $params->{biblionumber}"
>+        if not defined $params->{xsl_syspref};
>+
>     my $xslfilename = get_xsl_filename( $xslsyspref);
> 
>     # grab the XML, run it through our stylesheet, push it out to the browser
>diff --git a/opac/opac-tags.pl b/opac/opac-tags.pl
>index 63169e812a..515f4ccd2e 100755
>--- a/opac/opac-tags.pl
>+++ b/opac/opac-tags.pl
>@@ -292,7 +292,7 @@ if ($loggedinuser) {
>             {
>                 biblionumber   => $tag->{biblionumber},
>                 record         => $record,
>-                xsl_filename   => 'OPACXSLTResultsDisplay',
>+                xsl_syspref    => 'OPACXSLTResultsDisplay',
>                 fix_amps       => 1,
>                 hidden_items   => $hidden_items,
>                 xslt_variables => $variables
>diff --git a/virtualshelves/shelves.pl b/virtualshelves/shelves.pl
>index d1ce868e4b..2cc09766cb 100755
>--- a/virtualshelves/shelves.pl
>+++ b/virtualshelves/shelves.pl
>@@ -268,7 +268,7 @@ if ( $op eq 'view' ) {
>                     {
>                         biblionumber => $biblionumber,
>                         record       => $record,
>-                        xsl_filename => "XSLTListsDisplay",
>+                        xsl_syspref  => 'XSLTListsDisplay',
>                         fix_amps     => 1,
>                     }
>                 );
>-- 
>2.31.1

Actions: View | Diff | Splinter Review

Attachments on bug 29669: 128373 | 128544 | 128545 | 128566