From 19264af3f4cb75f8111b9d095efde6fd44038cb3 Mon Sep 17 00:00:00 2001 From: Jared Camins-Esakov Date: Fri, 15 Jun 2012 14:43:03 -0400 Subject: [PATCH] Bug 8256: Teach webservice to select reports by name Adds the ability to pass a hash to C4::Reports::Guided::get_saved_report which specifies a name or id to select the report. Test plan: 1. Create a report (or choose an existing one), and mark it public 2. Run the report using the web service: [IntranetBaseURL]/cgi-bin/koha/svc/report?id=whatever 3. Confirm you get the expected results 4. Run the report by name using the web service: [IntranetBaseURL]/cgi-bin/koha/svc/report?name=[Report name] (keep spaces in the name) 5. Confirm you get the same results 6. Run the report using the public web service: [OPACBaseURL]/cgi-bin/koha/svc/report?id=whatever 7. Confirm you get the same results 8. Run the report by name using the public web service: [OPACBaseURL]/cgi-bin/koha/svc/report?name=[Report name] (keep spaces in the name) 9. Confirm you get the same results 10. Sign off Signed-off-by: Katrin Fischer - Adding, editing and deleting reports works - id parameter works - new name parameter works - public and non-public works Signed-off-by: Paul Poulain --- C4/Reports/Guided.pm | 28 +++++++++++++++++++++++----- opac/svc/report | 32 ++++++++++++++++++++------------ svc/report | 41 +++++++++++++++++++++++++++++------------ 3 files changed, 72 insertions(+), 29 deletions(-) diff --git a/C4/Reports/Guided.pm b/C4/Reports/Guided.pm index f1ebe0c634..fbc25d8b8f 100644 --- a/C4/Reports/Guided.pm +++ b/C4/Reports/Guided.pm @@ -607,13 +607,31 @@ sub get_saved_reports { } sub get_saved_report { - my ($id) = @_; my $dbh = C4::Context->dbh(); - my $query = " SELECT * FROM saved_sql WHERE id = ?"; - my $sth = $dbh->prepare($query); - $sth->execute($id); + my $query; + my $sth; + my $report_arg; + if ($#_ == 0 && ref $_[0] ne 'HASH') { + ($report_arg) = @_; + $query = " SELECT * FROM saved_sql WHERE id = ?"; + } elsif (ref $_[0] eq 'HASH') { + my ($selector) = @_; + if ($selector->{name}) { + $query = " SELECT * FROM saved_sql WHERE report_name = ?"; + $report_arg = $selector->{name}; + } elsif ($selector->{id} || $selector->{id} eq '0') { + $query = " SELECT * FROM saved_sql WHERE id = ?"; + $report_arg = $selector->{id}; + } else { + return; + } + } else { + return; + } + $sth = $dbh->prepare($query); + $sth->execute($report_arg); my $data = $sth->fetchrow_hashref(); - return ( $data->{'savedsql'}, $data->{'type'}, $data->{'report_name'}, $data->{'notes'}, $data->{'cache_expiry'}, $data->{'public'} ); + return ( $data->{'savedsql'}, $data->{'type'}, $data->{'report_name'}, $data->{'notes'}, $data->{'cache_expiry'}, $data->{'public'}, $data->{'id'} ); } =item create_compound($masterID,$subreportID) diff --git a/opac/svc/report b/opac/svc/report index 0f9d5fb708..86b7a008aa 100755 --- a/opac/svc/report +++ b/opac/svc/report @@ -28,18 +28,24 @@ use CGI; use Koha::Cache; my $query = CGI->new(); -my $report = $query->param('id'); +my $report_id = $query->param('id'); +my $report_name = $query->param('name'); my $cache; +my $sql; +my $type; +my $notes; +my $cache_expiry; +my $public; -my ( $sql, $type, $name, $notes, $cache_expiry, $public ) = - get_saved_report($report); +( $sql, $type, $report_name, $notes, $cache_expiry, $public, $report_id ) = + get_saved_report($report_name ? { 'name' => $report_name } : { 'id' => $report_id } ); die "Sorry this report is not public\n" unless $public; if (Koha::Cache->is_cache_active) { $cache = Koha::Cache->new( ); - my $page = $cache->get_from_cache("opac:report:$report"); + my $page = $cache->get_from_cache("opac:report:$report_id"); if ($page) { print $query->header; print $page; @@ -48,13 +54,15 @@ if (Koha::Cache->is_cache_active) { } print $query->header; -my $offset = 0; -my $limit = C4::Context->preference("SvcMaxReportRows") || 10; -my ( $sth, $errors ) = execute_query( $sql, $offset, $limit ); -my $lines = $sth->fetchall_arrayref; -my $json_text = to_json($lines); -print $json_text; +if ($sql) { + my $offset = 0; + my $limit = C4::Context->preference("SvcMaxReportRows") || 10; + my ( $sth, $errors ) = execute_query( $sql, $offset, $limit ); + my $lines = $sth->fetchall_arrayref; + my $json_text = to_json($lines); + print $json_text; -if (Koha::Cache->is_cache_active) { - $cache->set_in_cache( "opac:report:$report", $json_text, $cache_expiry ); + if (Koha::Cache->is_cache_active) { + $cache->set_in_cache( "opac:report:$report_id", $json_text, $cache_expiry ); + } } diff --git a/svc/report b/svc/report index 18e1986b25..b0507bd33d 100755 --- a/svc/report +++ b/svc/report @@ -30,9 +30,15 @@ use Koha::Cache; my $query = CGI->new(); -my $report = $query->param('id'); +my $report_id = $query->param('id'); +my $report_name = $query->param('name'); my $cache; +my $sql; +my $type; +my $notes; +my $cache_expiry; +my $public; my ( $template, $loggedinuser, $cookie ) = get_template_and_user( { @@ -45,8 +51,15 @@ my ( $template, $loggedinuser, $cookie ) = get_template_and_user( ); if (Koha::Cache->is_cache_active) { + if ($report_name) { # When retrieving by name, we have to hit the + # database to get the ID before we can check + # the cache. Yuck. + ( $sql, $type, $report_name, $notes, $cache_expiry, $public, $report_id ) = + get_saved_report( { 'name' => $report_name } ); + } + $cache = Koha::Cache->new(); - my $page = $cache->get_from_cache("intranet:report:$report"); + my $page = $cache->get_from_cache("intranet:report:$report_id"); if ($page) { print $query->header; print $page; @@ -57,15 +70,19 @@ if (Koha::Cache->is_cache_active) { print $query->header; # $public isnt used for intranet -my ( $sql, $type, $name, $notes, $cache_expiry, $public ) = - get_saved_report($report); -my $offset = 0; -my $limit = C4::Context->preference("SvcMaxReportRows") || 10; -my ( $sth, $errors ) = execute_query( $sql, $offset, $limit ); -my $lines = $sth->fetchall_arrayref; -my $json_text = to_json($lines); -print $json_text; +unless ($sql) { + ( $sql, $type, $report_name, $notes, $cache_expiry, $public, $report_id ) = + get_saved_report($report_name ? { 'name' => $report_name } : { 'id' => $report_id } ); +} +if ($sql) { + my $offset = 0; + my $limit = C4::Context->preference("SvcMaxReportRows") || 10; + my ( $sth, $errors ) = execute_query( $sql, $offset, $limit ); + my $lines = $sth->fetchall_arrayref; + my $json_text = to_json($lines); + print $json_text; -if (Koha::Cache->is_cache_active) { - $cache->set_in_cache( "intranet:report:$report", $json_text, $cache_expiry ); + if (Koha::Cache->is_cache_active) { + $cache->set_in_cache( "intranet:report:$report_id", $json_text, $cache_expiry ); + } } -- 2.39.5