gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 42695: Minor language and markup corrections to reports error messages 

This patch makes a vew minor corrections to guided_reports_start.tt.

To test apply the patch and go to Reports.

- Try to create an SQL report which uses UPDATE, e.g.
  UPDATE items SET barcode = 1 WHERE itemnumber = 1
- Confirm that the text of the error message reads well and is formatted
  well.

Sponsored-by: Athens County Public Libraries
Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Laura_Escamilla <laura.escamilla@bywatersolutions.com>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
This commit is contained in:
Owen Leonard 2026-05-26 13:50:15 -04:00 committed by Lucas Gass
parent 19bc16654a
commit 6fa6228f73
No known key found for this signature in database
GPG Key ID: 60B447398B0D36CA
1 changed files with 13 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt
View File

@ -1636,25 +1636,26 @@
[% IF ( errors ) %]
<form action="/cgi-bin/koha/reports/guided_reports.pl" method="get">
<div class="alert alert-warning">
<strong>The following error was encountered:</strong><br />
<h1>The following error was encountered:</h1>
[% FOREACH error IN errors %]
[% IF ( error.duplicate_running_report ) %]
This report is already running. Please wait for it to finish before running it again.
<p>This report is already running. Please wait for it to finish before running it again.</p>
[% ELSIF ( error.total_running_reports_limit_exceeded ) %]
You have too many reports running at the moment. Please wait for some of them to finish before running this report again.
<p>You have too many reports running at the moment. Please wait for some of them to finish before running this report again.</p>
[% ELSIF ( error.sqlerr ) %]
This report contains the SQL keyword <strong>[% error.sqlerr | html %]</strong>.<br />
Use of this keyword is not allowed in Koha reports due to security and data integrity risks. Only SELECT queries are allowed.<br />
Please return to the &quot;Saved Reports&quot; screen and delete this report or retry creating a new one.
<p>
This report contains an SQL keyword: <strong>[% error.sqlerr | html %]</strong>. Use of this keyword is not allowed in Koha reports due to security and data integrity risks. Only SELECT queries are allowed.
</p>
<p>Please delete this report or retry creating a new one.</p>
[% ELSIF ( error.queryerr ) %]
<span>The database returned the following error:</span> <br />
[% error.queryerr | html %]<br />Please check the log for further details.
<h2>SQL error:</h2>
<p>[% error.queryerr | html %]</p>
<p>Please check the log for further details.</p>
[% ELSIF ( error.cache_expiry ) %]
Please select a cache expiry less than 30 days.
<p>Please select a cache expiry less than 30 days.</p>
[% ELSIF ( error.passworderr ) %]
The column selection in this report includes a password field.<br />
The report cannot be executed due to security risks.<br />
Please edit this report and ensure no password columns have been selected.
<p>The column selection in this report includes a password field.</p>
<p>The report cannot be executed due to security risks. Please edit this report and ensure no password columns have been selected.</p>
[% ELSE %]
[% END %]