Bug 37667 - q_errors in Reports export phase is wrong data type
Summary: q_errors in Reports export phase is wrong data type
Status: NEW
Alias: None
Product: Koha
Classification: Unclassified
Component: Reports (show other bugs)
Version: Main
Hardware: All All
: P5 - low minor
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-08-19 03:35 UTC by David Cook
Modified: 2024-08-19 03:36 UTC (History)
0 users

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description David Cook 2024-08-19 03:35:49 UTC
In reports/guided_reports.pl, when you do an export, there is a test for "$q_errors and @$q_errors", but in most cases $q_errors will actually be a hashref returned by execute_query and not an arrayref. 

I think this probably hasn't been caught because it's hard to initiate the Export phase for a report with errors, since it won't do the Run phase correctly.

But if you have a bad report and manually go to the export link, you can get guided_reports.pl to return a 500 because of the $q_errors data type being wrong.

Anyway, quite low severity, but raising it anyway...