Attachment 151760 Details for Bug 33844 – Bug 33844: Fix is_denied_renewal

[patch] Bug 33844: Fix is_denied_renewal

Bug-33844-Fix-isdeniedrenewal.patch (text/plain), 1.09 KB, created by Marcel de Rooy on 2023-05-26 13:03:51 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2023-05-26 13:03:51 UTC

Size: 1.09 KB

patch

obsolete

>From 76e1c9d9cd9cce1a21ecab5d5e2e5e6614feb7fd Mon Sep 17 00:00:00 2001
>From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Date: Fri, 26 May 2023 12:58:22 +0000
>Subject: [PATCH] Bug 33844: Fix is_denied_renewal
>Content-Type: text/plain; charset=utf-8
>
>Going via result_set->has_column does cost actually nothing if
>the object is already there.
>
>Test plan:
>Run again t/db_dependent/Koha/Item.t
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>---
> Koha/Item.pm | 2 ++
> 1 file changed, 2 insertions(+)
>
>diff --git a/Koha/Item.pm b/Koha/Item.pm
>index fe175a576c..619a307a61 100644
>--- a/Koha/Item.pm
>+++ b/Koha/Item.pm
>@@ -2069,6 +2069,8 @@ sub is_denied_renewal {
>     my $denyingrules = C4::Context->yaml_preference('ItemsDeniedRenewal');
>     return 0 unless $denyingrules;
>     foreach my $field (keys %$denyingrules) {
>+        # Silently ignore bad column names; TODO we should validate elsewhere
>+        next if !$self->_result->result_source->has_column($field);
>         my $val = $self->$field;
>         if( !defined $val) {
>             if ( any { !defined $_ }  @{$denyingrules->{$field}} ){
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 33844: 151759 | 151760 | 151785 | 151786 | 152110 | 152111