Attachment 153114 Details for Bug 34218 – Bug 34218: Send a record copy to avoid loss of information and display problems

[patch] Bug 34218: Send a record copy to avoid loss of information and display problems

Bug-34218-Send-a-record-copy-to-avoid-loss-of-info.patch (text/plain), 1.50 KB, created by Marcel de Rooy on 2023-07-07 06:15:46 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2023-07-07 06:15:46 UTC

Size: 1.50 KB

patch

obsolete

>From 3079ace49786238a8d3c803b5e509936ab942cf3 Mon Sep 17 00:00:00 2001
>From: Thibaud Guillot <thibaud.guillot@biblibre.com>
>Date: Thu, 6 Jul 2023 13:46:47 +0200
>Subject: [PATCH] Bug 34218: Send a record copy to avoid loss of information
> and display problems
>Content-Type: text/plain; charset=utf-8
>
>As I wrote in the bug description, I encountered this problem in a particular case,
>so I'll explain it in the test plan, but it's certainly likely to be encountered in different ways.
>
>Test plan:
>
>1) Link authorised values with  a field on a MARC framework, like a 942a related to branches
>2) Set it on a record with no items
>3) Activate syspref "AlternateHoldingsField" and set it to field 942a
>4) Go to OPAC View and you will see "Holdings: " without any values
>5) Apply this patch
>6) Do step 4) again and now it's fixed
>
>Signed-off-by: Sam Lau <samalau@gmail.com>
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>---
> opac/opac-detail.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/opac/opac-detail.pl b/opac/opac-detail.pl
>index 116694e267..52e161205a 100755
>--- a/opac/opac-detail.pl
>+++ b/opac/opac-detail.pl
>@@ -639,7 +639,7 @@ $variables->{show_analytics_link} = $show_analytics;
> $template->param(
>     XSLTBloc => XSLTParse4Display({
>         biblionumber   => $biblionumber,
>-        record         => $record,
>+        record         => $record->clone(),
>         xsl_syspref    => 'OPACXSLTDetailsDisplay',
>         fix_amps       => 1,
>         xslt_variables => $variables,
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 34218: 153091 | 153103 | 153114