Koha-community/Koha
13
7
Fork 0
Code Issues Releases Wiki Activity

Update for bug 1545 allowing for the title to be edited

Browse source 
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
This commit is contained in:
Chris Cormack 2009-04-19 09:44:30 +12:00 committed by Galen Charlton
parent 696fc0ff22
commit 60fc079cfb
3 changed files with 21 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
C4/Reports/Guided.pm
View file

@ -449,11 +449,12 @@ sub save_report {
sub update_sql {
my $id = shift || croak "No Id given";
my $sql = shift;
my $reportname = shift;
my $dbh = C4::Context->dbh();
$sql =~ s/(\s*\;\s*)$//; # removes trailing whitespace and /;/
my $query = "UPDATE saved_sql SET savedsql = ?, last_modified = now() WHERE id = ? ";
my $query = "UPDATE saved_sql SET savedsql = ?, last_modified = now(), report_name = ? WHERE id = ? ";
my $sth = $dbh->prepare($query);
$sth->execute( $sql, $id );
$sth->execute( $sql, $reportname, $id );
$sth->finish();
}

9
koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tmpl
View file

@ -112,8 +112,8 @@ canned reports and writing custom SQL reports.</p>
</td>
<td><a href="/cgi-bin/koha/reports/guided_reports.pl?reports=<!-- TMPL_VAR NAME="id" -->&amp;phase=Show%20SQL">Show SQL</a>
&nbsp; <a href="/cgi-bin/koha/reports/guided_reports.pl?reports=<!-- TMPL_VAR NAME="id" -->&amp;phase=Edit%20SQL">Edit SQL</a></td>
<td><a href="/cgi-bin/koha/reports/guided_reports.pl?reports=<!-- TMPL_VAR NAME="id" -->&amp;phase=Run%20this%20report">Run this Report</a></td>
<td><a href="/cgi-bin/koha/tools/scheduler.pl?id=<!-- TMPL_VAR NAME="id" -->">Schedule this Report</a></td>
<td><a href="/cgi-bin/koha/reports/guided_reports.pl?reports=<!-- TMPL_VAR NAME="id" -->&amp;phase=Run%20this%20report">Run</a></td>
<td><a href="/cgi-bin/koha/tools/scheduler.pl?id=<!-- TMPL_VAR NAME="id" -->">Schedule</a></td>
<td><a class="confirmdelete" title="Delete this saved report" href="/cgi-bin/koha/reports/guided_reports.pl?reports=<!-- TMPL_VAR NAME="id" -->&amp;phase=Delete%20Saved">Delete</a></td></tr>
<!-- /TMPL_LOOP -->
</table>
@ -461,7 +461,10 @@ Sub report:<select name="subreport">
<input type="hidden" name="id" value="<!-- TMPL_VAR NAME="id" -->">
<fieldset class="rows">
<legend>Edit SQL</legend>
<textarea id="sql" name="sql"><!-- TMPL_VAR NAME="sql" --></textarea>
<ol>
<li><label for="reportname">Report Name:</label><input type="text" id="reportname" name="reportname" value="<!-- TMPL_VAR NAME="reportname" -->" ></li>
<li><textarea id="sql" name="sql"><!-- TMPL_VAR NAME="sql" --></textarea></li>
</ol>
</fieldset>
<fieldset class="action">

6
reports/guided_reports.pl
View file

@ -92,9 +92,10 @@ elsif ( $phase eq 'Show SQL'){
elsif ( $phase eq 'Edit SQL'){
my $id = $input->param('reports');
my $sql = get_sql($id);
my ($sql,$type,$reportname,$notes) = get_saved_report($id);
$template->param(
'sql' => $sql,
'reportname' => $reportname,
'id' => $id,
'editsql' => 1,
);
@ -103,6 +104,7 @@ elsif ( $phase eq 'Edit SQL'){
elsif ( $phase eq 'Update SQL'){
my $id = $input->param('id');
my $sql = $input->param('sql');
my $reportname = $input->param('reportname');
my @errors;
if ($sql =~ /;?\W?(UPDATE|DELETE|DROP|INSERT|SHOW|CREATE)\W/i) {
push @errors, {sqlerr => $1};
@ -117,7 +119,7 @@ elsif ( $phase eq 'Update SQL'){
);
}
else {
update_sql( $id, $sql );
update_sql( $id, $sql, $reportname );
$template->param(
'save_successful' => 1,
);