Attachment 169495 Details for Bug 37425 – Bug 37425: Check for existence of biblio object before fetching cover images

[patch] Bug 37425: Check for existence of biblio object before fetching cover images

Bug-37425-Check-for-existence-of-biblio-object-bef.patch (text/plain), 1.84 KB, created by Julian Maurice on 2024-07-24 14:39:15 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Julian Maurice

Created: 2024-07-24 14:39:15 UTC

Size: 1.84 KB

patch

obsolete

>From 165b84bd3f96a8d60247c756a32bdc0f58b4c31b Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Mon, 22 Jul 2024 19:46:33 +0000
>Subject: [PATCH] Bug 37425: Check for existence of biblio object before
> fetching cover images
>
>This patch simply adds a conditional to ensure the biblio object has been retrieved and assumes no cover images otherwise
>
>To test:
> 1 - Enable system preference LocalCoverImages
> 2 - Perform a search in staff interface
> 3 - Find the biblionumebr for one of the results and delete it via the SQL backend:
>     DELETE FROM biblio WHERE biblionumber=3;
> 4 - Search again.
> 5 - KO!
>     Can't call method "cover_images" on an undefined value at /usr/share/koha/intranet/cgi-bin/catalogue/search.pl line 671.
> 6 - Reindex, confirm error is gone
> 7 - Apply patch
> 8 - Search again
> 9 - Delete a record from the results via SQL
>10 - Reload and confirm no error
>11 - Reindex and repeat search and confirm no error
>
>Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
>Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
>---
> catalogue/search.pl | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
>diff --git a/catalogue/search.pl b/catalogue/search.pl
>index 7c9bb22126..ebb78bcfb6 100755
>--- a/catalogue/search.pl
>+++ b/catalogue/search.pl
>@@ -668,7 +668,8 @@ for (my $i=0;$i<@servers;$i++) {
>                 }
> 
>                 if ( C4::Context->preference('LocalCoverImages') ) {
>-                    $line->{has_local_cover_image} = $line->{biblio_object}->cover_images->count;
>+                    $line->{has_local_cover_image} =
>+                        $line->{biblio_object} ? $line->{biblio_object}->cover_images->count : 0;
>                 }
>             }
>             my( $page_numbers, $hits_to_paginate, $pages, $current_page_number, $previous_page_offset, $next_page_offset, $last_page_offset ) =
>-- 
>2.39.2

Actions: View | Diff | Splinter Review

Attachments on bug 37425: 169380 | 169425 | 169495