Attachment 70439 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.36 KB, created by Nick Clemens (kidclamp) on 2018-01-11 18:45:40 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2018-01-11 18:45:40 UTC

Size: 4.36 KB

patch

obsolete

>From b787319071ee949bc46cdb04030d9b31f4c83a01 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 - Try again after inserting erros into report (No Select, bad
>        authorised value, etc.)
>9 - Note errors stil display as expected
>---
> .../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..3071afc 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,21 @@ Sub report:<select name="subreport">
> </fieldset>
> [% 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 ( editsql ) %]
> <form action="/cgi-bin/koha/reports/guided_reports.pl" method="post" class="validated">
> <input type="hidden" name="phase" value="Update SQL" />
>@@ -854,21 +869,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">
>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