From 2ea5163970fad485d3dd37c715966fe5d73bf3cc Mon Sep 17 00:00:00 2001
From: Magnus Enger <magnus@libriotech.no>
Date: Sat, 1 Apr 2023 11:27:26 +0200
Subject: [PATCH] Bug 23824: Add a "Save and run" button to reports

This will save one click every time you want to save and then run
a report.

To test:
- Apply this patch
- Create a new report
- Click the new "Update and run SQL" button
- Verify the report was saved and the results are displayed

Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
---
 .../prog/en/modules/reports/guided_reports_start.tt          | 4 ++--
 reports/guided_reports.pl                                    | 5 ++++-
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt
index cc14f25773..363890a229 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt
@@ -1324,7 +1324,6 @@
 
                 [% IF ( editsql ) %]
                     <form action="/cgi-bin/koha/reports/guided_reports.pl" method="post" id="sql_report_form">
-                        <input type="hidden" name="phase" value="Update SQL" />
                         <input type="hidden" name="id" value="[% id | html %]"/>
                         <fieldset class="rows">
                             <legend><h1>Edit SQL report</h1></legend>
@@ -1382,7 +1381,8 @@
                         </fieldset>
 
                         <fieldset class="action">
-                            <input type="submit" name="submit" class="btn btn-primary" value="Update SQL" />
+                            <input type="submit" name="phase" class="btn btn-primary" value="Update SQL" />
+                            <input type="submit" name="phase" class="btn btn-primary" value="Update and run SQL" />
                             <a href="/cgi-bin/koha/reports/guided_reports.pl?phase=Use%20saved" class="cancel">Cancel</a>
                         </fieldset>
                     </form>
diff --git a/reports/guided_reports.pl b/reports/guided_reports.pl
index 94cff9fe21..fb51ab27fa 100755
--- a/reports/guided_reports.pl
+++ b/reports/guided_reports.pl
@@ -209,7 +209,7 @@ elsif ( $phase eq 'Edit SQL'){
     );
 }
 
-elsif ( $phase eq 'Update SQL'){
+elsif ( $phase eq 'Update SQL' || $phase eq 'Update and run SQL' ){
     my $id         = $input->param('id');
     my $sql        = $input->param('sql');
     my $reportname = $input->param('reportname');
@@ -301,6 +301,9 @@ elsif ( $phase eq 'Update SQL'){
                 cache_expiry_units => $cache_expiry_units,
             );
         }
+        if ( $phase eq 'Update and run SQL' ) {
+            print $input->redirect("/cgi-bin/koha/reports/guided_reports.pl?reports=$id&phase=Run%20this%20report");
+        }
     }
 }
 
-- 
2.39.5