From d1041dc0939f7b2e7b79b1614a82e3c55c0495bb Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Thu, 31 Dec 2015 10:32:35 +0000 Subject: [PATCH] Bug 15451: Koha::CsvProfiles - Remove GetCsvProfilesLoop Content-Type: text/plain; charset=utf-8 This subroutine returned the csv profiles for a given type. This could be done easily with the new Koha::CsvProfiles->search method. Test plan: To do at the OPAC and staff interface! 1/ Export a list using a CSV profile 2/ Export your CART using a CSV profile Note that only MARC profiles should be available. Signed-off-by: Bernardo Gonzalez Kriegel Tested on staff/opac & cart/list Small problem on filename extension fixed in followup. No errors Signed-off-by: Marcel de Rooy --- C4/Csv.pm | 19 ------------------- basket/basket.pl | 8 ++++---- basket/downloadcart.pl | 8 +++++--- opac/opac-downloadcart.pl | 8 +++++--- opac/opac-downloadshelf.pl | 6 +++--- virtualshelves/downloadshelf.pl | 6 +++--- virtualshelves/shelves.pl | 4 +++- 7 files changed, 23 insertions(+), 36 deletions(-) diff --git a/C4/Csv.pm b/C4/Csv.pm index ce4693a..275fb49 100644 --- a/C4/Csv.pm +++ b/C4/Csv.pm @@ -34,7 +34,6 @@ use vars qw(@ISA @EXPORT); &GetCsvProfiles &GetCsvProfile &GetCsvProfileId - &GetCsvProfilesLoop &GetMarcFieldsForCsv ); @@ -94,23 +93,5 @@ sub GetMarcFieldsForCsv { } -# Returns informations aboout csv profiles suitable for html templates -sub GetCsvProfilesLoop { - my ( $type ) = @_; - # List of existing profiles - my $dbh = C4::Context->dbh; - my $sth; - my $query = "SELECT export_format_id, profile FROM export_format"; - if ( $type ) { - $query .= " WHERE type = ?"; - } - - $sth = $dbh->prepare($query); - $sth->execute( $type ? $type : () ); - return $sth->fetchall_arrayref({}); - -} - - 1; diff --git a/basket/basket.pl b/basket/basket.pl index 1dfbdb8..ec5e485 100755 --- a/basket/basket.pl +++ b/basket/basket.pl @@ -16,15 +16,15 @@ # along with Koha; if not, see . -use strict; -use warnings; +use Modern::Perl; use CGI qw ( -utf8 ); use C4::Koha; use C4::Biblio; use C4::Items; use C4::Auth; use C4::Output; -use C4::Csv; + +use Koha::CsvProfiles; my $query = new CGI; @@ -121,7 +121,7 @@ my $resultsarray = \@results; $template->param( BIBLIO_RESULTS => $resultsarray, - csv_profiles => GetCsvProfilesLoop('marc'), + csv_profiles => [ Koha::CsvProfiles->search({ type => 'marc' }) ], bib_list => $bib_list, ); diff --git a/basket/downloadcart.pl b/basket/downloadcart.pl index 17e5617..0690da5 100755 --- a/basket/downloadcart.pl +++ b/basket/downloadcart.pl @@ -17,8 +17,7 @@ # You should have received a copy of the GNU General Public License # along with Koha; if not, see . -use strict; -use warnings; +use Modern::Perl; use CGI qw ( -utf8 ); use Encode qw(encode); @@ -30,6 +29,9 @@ use C4::Output; use C4::Record; use C4::Ris; use C4::Csv; + +use Koha::CsvProfiles; + use utf8; my $query = new CGI; @@ -89,7 +91,7 @@ if ($bib_list && $format) { print $output; } else { - $template->param(csv_profiles => GetCsvProfilesLoop('marc')); + $template->param(csv_profiles => [ Koha::CsvProfiles->search({ type => 'marc' }) ]); $template->param(bib_list => $bib_list); output_html_with_http_headers $query, $cookie, $template->output; } diff --git a/opac/opac-downloadcart.pl b/opac/opac-downloadcart.pl index c5574e3..ba4d4b2 100755 --- a/opac/opac-downloadcart.pl +++ b/opac/opac-downloadcart.pl @@ -17,8 +17,7 @@ # You should have received a copy of the GNU General Public License # along with Koha; if not, see . -use strict; -use warnings; +use Modern::Perl; use CGI qw ( -utf8 ); use Encode qw(encode); @@ -30,6 +29,9 @@ use C4::Output; use C4::Record; use C4::Ris; use C4::Csv; + +use Koha::CsvProfiles; + use utf8; my $query = new CGI; @@ -95,7 +97,7 @@ if ($bib_list && $format) { print $output; } else { - $template->param(csv_profiles => GetCsvProfilesLoop('marc')); + $template->param(csv_profiles => [ Koha::CsvProfiles->search({ type => 'marc' }) ]); $template->param(bib_list => $bib_list); output_html_with_http_headers $query, $cookie, $template->output; } diff --git a/opac/opac-downloadshelf.pl b/opac/opac-downloadshelf.pl index 6102082..8413e75 100755 --- a/opac/opac-downloadshelf.pl +++ b/opac/opac-downloadshelf.pl @@ -17,8 +17,7 @@ # You should have received a copy of the GNU General Public License # along with Koha; if not, see . -use strict; -use warnings; +use Modern::Perl; use CGI qw ( -utf8 ); use Encode qw(encode); @@ -31,6 +30,7 @@ use C4::Record; use C4::Ris; use C4::Csv; +use Koha::CsvProfiles; use Koha::Virtualshelves; use utf8; @@ -118,7 +118,7 @@ if ( $shelf and $shelf->can_be_viewed( $borrowernumber ) ) { } else { $template->param(fullpage => 1); } - $template->param(csv_profiles => GetCsvProfilesLoop('marc')); + $template->param(csv_profiles => [ Koha::CsvProfiles->search({ type => 'marc' }) ]); $template->param( shelf => $shelf ); output_html_with_http_headers $query, $cookie, $template->output; } diff --git a/virtualshelves/downloadshelf.pl b/virtualshelves/downloadshelf.pl index 5a00694..3db362a 100755 --- a/virtualshelves/downloadshelf.pl +++ b/virtualshelves/downloadshelf.pl @@ -17,8 +17,7 @@ # You should have received a copy of the GNU General Public License # along with Koha; if not, see . -use strict; -use warnings; +use Modern::Perl; use CGI qw ( -utf8 ); use Encode qw(encode); @@ -31,6 +30,7 @@ use C4::Record; use C4::Ris; use C4::Csv; +use Koha::CsvProfiles; use Koha::Virtualshelves; use utf8; @@ -101,7 +101,7 @@ if ($shelfid && $format) { $format = "csv" if ($format =~ m/^\d+$/); } else { - $template->param(csv_profiles => GetCsvProfilesLoop('marc')); + $template->param(csv_profiles => [ Koha::CsvProfiles->search({ type => 'marc' }) ]); $template->param(shelfid => $shelfid); } $template->param( messages => \@messages ); diff --git a/virtualshelves/shelves.pl b/virtualshelves/shelves.pl index 0540f3e..6a62750 100755 --- a/virtualshelves/shelves.pl +++ b/virtualshelves/shelves.pl @@ -27,6 +27,8 @@ use C4::Items; use C4::Members; use C4::Output; use C4::XSLT; + +use Koha::CsvProfiles; use Koha::Virtualshelves; my $query = new CGI; @@ -309,7 +311,7 @@ $template->param( messages => \@messages, category => $category, print => scalar $query->param('print') || 0, - csv_profiles => GetCsvProfilesLoop('marc'), + csv_profiles => [ Koha::CsvProfiles->search({ type => 'marc' }) ], ); output_html_with_http_headers $query, $cookie, $template->output; -- 1.7.10.4