Attachment 161935 Details for Bug 31224 – Bug 31224: (QA follow-up) Fix syntax error

[patch] Bug 31224: (QA follow-up) Fix syntax error

Bug-31224-QA-follow-up-Fix-syntax-error.patch (text/plain), 1.75 KB, created by Martin Renvoize (ashimema) on 2024-02-08 16:53:35 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Renvoize (ashimema)

Created: 2024-02-08 16:53:35 UTC

Size: 1.75 KB

patch

obsolete

>From 59b96f1a34fe3aaa65dbc5a77bb20104509819fb Mon Sep 17 00:00:00 2001
>From: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>Date: Thu, 13 Oct 2022 09:30:06 +0100
>Subject: [PATCH] Bug 31224: (QA follow-up) Fix syntax error
>
>Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>---
> Koha/Biblio.pm | 13 ++++++++-----
> 1 file changed, 8 insertions(+), 5 deletions(-)
>
>diff --git a/Koha/Biblio.pm b/Koha/Biblio.pm
>index 80a90e7dcfc..392ad82d670 100644
>--- a/Koha/Biblio.pm
>+++ b/Koha/Biblio.pm
>@@ -140,13 +140,14 @@ type. Currently only I<MARC::Record> objects are returned.
> =cut
> 
> sub metadata_record {
>-    my ($self, $params) = @_;
>+    my ( $self, $params ) = @_;
> 
>     my $patron = $params->{patron};
> 
>     my $record = $self->metadata->record;
> 
>     if ( $params->{embed_items} or $params->{opac} ) {
>+
>         # There's need for a RecordProcessor, let's do it!
>         my @filters;
>         my $options = {
>@@ -154,12 +155,14 @@ sub metadata_record {
>             frameworkcode => $self->frameworkcode,
>         };
> 
>-        if ($params->{embed_items}) {
>+        if ( $params->{embed_items} ) {
>             push @filters, 'EmbedItems';
>-            if ($params->{opac}) {
>+            if ( $params->{opac} ) {
>                 $options->{items} = $self->items->filter_by_visible_in_opac(
>                     {
>-                        ( $params->{patron} ? patron => $params->{patron} : ())
>+                        (
>+                            $params->{patron} ? ( patron => $params->{patron} ) : ()
>+                        )
>                     }
>                 );
>             }
>@@ -183,7 +186,7 @@ sub metadata_record {
>             }
>         );
> 
>-        $rp->process( $record );
>+        $rp->process($record);
>     }
> 
>     return $record;
>-- 
>2.43.0

Actions: View | Diff | Splinter Review

Attachments on bug 31224: 138060 | 138061 | 138062 | 138063 | 138064 | 142137 | 142138 | 142139 | 142140 | 142141 | 142142 | 142143 | 161933 | 161934 | 161935 | 161936 | 161937 | 161938 | 161939 | 161940 | 161941 | 161942 | 161943 | 161944 | 173530 | 173531 | 173532 | 173533 | 173534 | 173535 | 173588 | 173597 | 173598 | 173636 | 173637 | 173638 | 173639 | 173640 | 173641 | 173642 | 173643 | 173644 | 174262 | 174263