From 7d293b2628a0e21f7bfdb7dcad9a0fafa28334d0 Mon Sep 17 00:00:00 2001
From: David Gustafsson <david.gustafsson@ub.gu.se>
Date: Wed, 19 Oct 2022 15:59:22 +0200
Subject: [PATCH] Bug 31856: Don't fetch additional field valudes if there are
 no fields

---
 C4/Serials.pm | 24 +++++++++++++++---------
 1 file changed, 15 insertions(+), 9 deletions(-)

diff --git a/C4/Serials.pm b/C4/Serials.pm
index 8878354a247..adf88da2f34 100644
--- a/C4/Serials.pm
+++ b/C4/Serials.pm
@@ -643,16 +643,22 @@ sub SearchSubscriptions {
     for my $subscription ( @$results ) {
         $subscription->{cannotedit} = not can_edit_subscription( $subscription );
         $subscription->{cannotdisplay} = not can_show_subscription( $subscription );
-        my @additional_field_values = Koha::AdditionalFieldValues->search(
-            {
-                field_id => { -in => [ keys %additional_fields_by_id ] },
-                record_id => $subscription->{subscriptionid}
-            }
-        )->as_list;
 
-        $subscription->{additional_fields} = {
-            map { $additional_fields_by_id{$_->field_id} => $_->value } @additional_field_values
-        };
+        if (%additional_fields_by_id) {
+            my @additional_field_values = Koha::AdditionalFieldValues->search(
+                {
+                    field_id => { -in => [ keys %additional_fields_by_id ] },
+                    record_id => $subscription->{subscriptionid}
+                }
+            )->as_list;
+
+            $subscription->{additional_fields} = {
+                map { $additional_fields_by_id{$_->field_id} => $_->value } @additional_field_values
+            };
+        }
+        else {
+            $subscription->{additional_fields} = {};
+        }
     }
 
     return wantarray ? @{$results} : { results => $results, total => $total_results };
-- 
2.48.1