Bug 19399 - Show reports screen should not display query in textarea
Summary: Show reports screen should not display query in textarea
Status: RESOLVED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Reports (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-10-02 15:14 UTC by Barton Chittenden
Modified: 2023-09-30 20:36 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Barton Chittenden 2017-10-02 15:14:13 UTC
The Guided reports "Show SQL" e.g.

https://intranet.bywatersolutions.com/cgi-bin/koha/reports/guided_reports.pl?reports=120&phase=Show%20SQL

Looks very similar to the "Edit SQL" screen.

https://intranet.bywatersolutions.com/cgi-bin/koha/reports/guided_reports.pl?reports=120&phase=Edit%20SQL

Because of the visual similarity, there is a strong temptation to edit the SQL on the Show SQL screen. This is an annoyance; it would be better UX to display the SQL in a <code> tag.
Comment 1 Owen Leonard 2020-01-14 12:48:31 UTC
I wonder if this is improved by the switch to CodeMirror for viewing and editing SQL?
Comment 2 Katrin Fischer 2020-01-14 20:16:33 UTC
(In reply to Owen Leonard from comment #1)
> I wonder if this is improved by the switch to CodeMirror for viewing and
> editing SQL?

I think it's still valid, if you go to:
http://localhost:8081/cgi-bin/koha/reports/guided_reports.pl?reports=1&phase=Show%20SQL

It does look like the edit screen and you can put your cursor into it, even if you cannot type. Not sure if there is a 'just display' mode for codemirror?
Comment 3 Katrin Fischer 2023-09-30 20:36:54 UTC
I feel like the modal with the edit button below is probably making this clear enough now.