From ed04035e9ffb00a12f84161dc016b2c307282167 Mon Sep 17 00:00:00 2001
From: Nick Clemens <nick@bywatersolutions.com>
Date: Mon, 12 Aug 2024 12:10:12 +0000
Subject: [PATCH] Bug 37508: Don't return Internal server error when running
 report

To test:
1 - Create a report like:
SELECT "a"
FROM borrowers
WHERE <<Test>> != ''
2 - Run report
3 - Enter "password"
4 - Internal server error / stacktrace
5 - Apply patch
6 - Repeat
7 - Get a yellow warning box

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
---
 reports/guided_reports.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/reports/guided_reports.pl b/reports/guided_reports.pl
index b7c722945a..b85b36a860 100755
--- a/reports/guided_reports.pl
+++ b/reports/guided_reports.pl
@@ -953,7 +953,7 @@ if ($op eq 'run'){
             );
             my $total;
             if (!$sth) {
-                die "execute_query failed to return sth for report $report_id: $sql";
+                push @errors, "Report could not be run " . $errors;
             } elsif ( !$errors ) {
                 $total = nb_rows($sql) || 0;
                 my $headers = header_cell_loop($sth);
-- 
2.39.5