From f5ac0500f7e35f3aeb3e6c6022477ebb888d2833 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 --- C4/Items.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/C4/Items.pm b/C4/Items.pm index 14774e48e1..b93d073663 100644 --- a/C4/Items.pm +++ b/C4/Items.pm @@ -808,7 +808,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.20.1