Attachment 53753 Details for Bug 16769 – Bug 16769: [QA Follow-up] Remove unsupported deepcopy parameter

[patch] Bug 16769: [QA Follow-up] Remove unsupported deepcopy parameter

Bug-16769-QA-Follow-up-Remove-unsupported-deepcopy.patch (text/plain), 1.02 KB, created by Marcel de Rooy on 2016-07-27 14:17:54 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2016-07-27 14:17:54 UTC

Size: 1.02 KB

patch

obsolete

>From 28c3909528fd5baab4c32b2d61e6b8faccf0983f Mon Sep 17 00:00:00 2001
>From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Date: Wed, 27 Jul 2016 16:00:43 +0200
>Subject: [PATCH] Bug 16769: [QA Follow-up] Remove unsupported deepcopy
> parameter
>Content-Type: text/plain; charset=utf-8
>
>Routine C4/Koha/GetAuthorisedValues adds deepcopy => 1 to the options of
>set_in_cache. This makes no sense (anymore?). Is not supported or passed
>further. So better remove it.
>
>Test plan:
>Run t/db_dependent/Koha.t (with Test::Deep available to minimize confusion)
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>---
> C4/Koha.pm |    2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/C4/Koha.pm b/C4/Koha.pm
>index bb35811..4e1e396 100644
>--- a/C4/Koha.pm
>+++ b/C4/Koha.pm
>@@ -1059,7 +1059,7 @@ sub GetAuthorisedValues {
>     }
>     $sth->finish;
> 
>-    $cache->set_in_cache( $cache_key, \@results, { deepcopy => 1, expiry => 5 } );
>+    $cache->set_in_cache( $cache_key, \@results, { expiry => 5 } );
>     return \@results;
> }
> 
>-- 
>1.7.10.4

Actions: View | Diff | Splinter Review

Attachments on bug 16769: 52512 | 53751 | 53752 | 53753 | 56013 | 56014 | 56015