Attachment 70769 Details for Bug 19957 – Bug 19957 - Allow continued editing of report after saving

[patch] Bug 19957 - Allow continued editing of report after saving

Bug-19957---Allow-continued-editing-of-report-afte.patch (text/plain), 4.47 KB, created by Owen Leonard on 2018-01-19 18:00:21 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2018-01-19 18:00:21 UTC

Size: 4.47 KB

patch

obsolete

>From 57c5c6d4a0941987449b478ae70d34a00a508404 Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Thu, 11 Jan 2018 18:19:33 +0000
>Subject: [PATCH] Bug 19957 - Allow continued editing of report after saving
>
>To test:
> 1 - Edit or create a report
> 2 - Save it
> 3 - Note the report is saved but you can't keep editing
> 4 - Clicking 'Edit' returns you to editing
> 5 - Apply patch
> 6 - Save report
> 7 - Note success message remains, but report is still open for editing
> 8 - Note when you click the saved/updated message dissappears
> 9 - Try again after inserting erros into report (No Select, bad
>        authorised value, etc.)
>10 - Note errors stil display as expected
>11 - Verify message works on update of existing report and save of new
>      new report
>
>Signed-off-by: Owen Leonard <oleonard@myacpl.org>
>---
> .../en/modules/reports/guided_reports_start.tt     | 30 +++++++++++-----------
> reports/guided_reports.pl                          | 14 ++++++++++
> 2 files changed, 29 insertions(+), 15 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 1d5e9f4..b8ff88d 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
>@@ -812,6 +812,17 @@ Sub report:<select name="subreport">
> </fieldset>
> [% END %]
> 
>+[% IF ( save_successful ) %]
>+[% UNLESS ( errors ) %]
>+</br>
>+<div id="report_updated">
>+    <div class="dialog message">
>+        <p>Your report "[% reportname %]" has been saved</p>
>+    </div>
>+</div>
>+[% END %]
>+[% END %]
>+
> [% IF ( editsql ) %]
> <form action="/cgi-bin/koha/reports/guided_reports.pl" method="post" class="validated">
> <input type="hidden" name="phase" value="Update SQL" />
>@@ -854,21 +865,6 @@ Sub report:<select name="subreport">
> 
> [% END %]
> 
>-[% IF ( save_successful ) %]
>-[% UNLESS ( errors ) %]
>-<h2>Your report has been saved</h2>
>-<h4>[% reportname %]</h4>
>-<ul>
>-    [% IF ( id ) %]
>-    <li><a href="/cgi-bin/koha/reports/guided_reports.pl?reports=[% id %]&amp;phase=Run%20this%20report">Run this report</a></li>
>-    [% END %]
>-    <li>Access this report from the: <a href="/cgi-bin/koha/reports/guided_reports.pl?phase=Use%20saved">Saved reports page</a></li>
>-    <li>Schedule this report to run using the: <a href="/cgi-bin/koha/tools/scheduler.pl">Scheduler tool</a></li>
>-    <li>Return to: <a href="/cgi-bin/koha/reports/guided_reports.pl?phase=Use%20saved">Guided reports</a></li>
>-</ul>
>-[% END %]
>-[% END %]
>-
> [% IF ( errors ) %]
> <form action="/cgi-bin/koha/reports/guided_reports.pl" method="post">
> <div class="dialog alert">
>@@ -963,6 +959,10 @@ Sub report:<select name="subreport">
>                 $('#limitselect').submit();
>             });
> 
>+            $(document).click(function() {
>+                $('#report_updated').hide();
>+            });
>+
>             $(".goback").on("click",function(e){
>                 e.preventDefault();
>                 window.history.back();
>diff --git a/reports/guided_reports.pl b/reports/guided_reports.pl
>index 106ddb5..39bca80 100755
>--- a/reports/guided_reports.pl
>+++ b/reports/guided_reports.pl
>@@ -279,6 +279,13 @@ elsif ( $phase eq 'Update SQL'){
>                 'save_successful'       => 1,
>                 'reportname'            => $reportname,
>                 'id'                    => $id,
>+                'editsql'               => 1,
>+                'sql'                   => $sql,
>+                'groups_with_subgroups' => groups_with_subgroups($group, $subgroup),
>+                'notes'                 => $notes,
>+                'cache_expiry'          => $cache_expiry,
>+                'public'                => $public,
>+                'usecache'              => $usecache,
>             );
>             logaction( "REPORTS", "MODIFY", $id, "$reportname | $sql" ) if C4::Context->preference("ReportsLog");
>         }
>@@ -648,6 +655,13 @@ elsif ( $phase eq 'Save Report' ) {
>                 'save_successful' => 1,
>                 'reportname'      => $name,
>                 'id'              => $id,
>+                'editsql'         => 1,
>+                'sql'             => $sql,
>+                'groups_with_subgroups' => groups_with_subgroups($group, $subgroup),
>+                'notes'      => $notes,
>+                'cache_expiry' => $cache_expiry,
>+                'public' => $public,
>+                'usecache' => $usecache,
>             );
>         }
>     }
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 19957: 70439 | 70443 | 70604 | 70605 | 70767 | 70769 | 70776 | 73242