From f87e8aeab99c4e0da86a2f5349abfe172144c2a5 Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Wed, 28 Jul 2021 13:32:24 +0000 Subject: [PATCH] Bug 28776: Warns from GetItemsInfo when biblio marked as serial GetItemsInfo sorts the items using serial fields if the biblio is defined as a serial The fields may not be defined. We can add a fallback to blank in this case To test: 1 - Mark a record with items as serial in the marc (942$s) 2 - At least one of the items should have no enumchron defined 3 - Load the detail page 4 - Warns in logs: [WARN] Use of uninitialized value in string comparison (cmp) at /kohadevbox/koha/C4/Items.pm line 811. 5 - Apply patch and restart all 6 - Reload the details page 7 - No more warns Signed-off-by: Fridolin Somers Signed-off-by: Tomas Cohen Arazi Signed-off-by: Jonathan Druart Signed-off-by: Kyle M Hall (cherry picked from commit 6ea41db079ae2b8362608eb35ec64cbc958c315c) Signed-off-by: Fridolin Somers --- C4/Items.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/C4/Items.pm b/C4/Items.pm index 300e87db22..bd708a2e35 100644 --- a/C4/Items.pm +++ b/C4/Items.pm @@ -797,7 +797,7 @@ sub GetItemsInfo { } return $serial - ? sort { ($b->{'publisheddate'} || $b->{'enumchron'}) cmp ($a->{'publisheddate'} || $a->{'enumchron'}) } @results + ? sort { ($b->{'publisheddate'} || $b->{'enumchron'} || "") cmp ($a->{'publisheddate'} || $a->{'enumchron'} || "") } @results : @results; } -- 2.39.5