#
# This file is part of Koha.
#
-# Koha is free software; you can redistribute it and/or modify it under the
-# terms of the GNU General Public License as published by the Free Software
-# Foundation; either version 2 of the License, or (at your option) any later
-# version.
+# Koha is free software; you can redistribute it and/or modify it
+# under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 3 of the License, or
+# (at your option) any later version.
#
-# Koha is distributed in the hope that it will be useful, but WITHOUT ANY
-# WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
-# A PARTICULAR PURPOSE. See the GNU General Public License for more details.
+# Koha is distributed in the hope that it will be useful, but
+# WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
#
-# You should have received a copy of the GNU General Public License along
-# with Koha; if not, write to the Free Software Foundation, Inc.,
-# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+# You should have received a copy of the GNU General Public License
+# along with Koha; if not, see <http://www.gnu.org/licenses>.
-use strict;
-use warnings;
+use Modern::Perl;
-use CGI;
+use CGI qw ( -utf8 );
use C4::Auth;
use C4::Koha;
use C4::Output;
use C4::Circulation;
-use C4::Review;
use C4::Biblio;
-use C4::Dates qw/format_date/;
-use C4::Members qw/GetMemberDetails/;
-use POSIX qw(ceil strftime);
+use Koha::DateUtils;
+use Koha::Patrons;
+use Koha::Reviews;
+use POSIX qw(ceil floor strftime);
my $template_name;
my $query = new CGI;
my $count = C4::Context->preference('OPACnumSearchResults') || 20;
my $results_per_page = $query->param('count') || $count;
my $offset = $query->param('offset') || 0;
-my $page = $query->param('page') || 1;
-$offset = ($page-1)*$results_per_page if $page>1;
+my $page = floor( $offset / $results_per_page ) + 1;
if ($format eq "rss") {
- $template_name = "opac-showreviews-rss.tmpl";
+ $template_name = "opac-showreviews-rss.tt";
} else {
- $template_name = "opac-showreviews.tmpl",
+ $template_name = "opac-showreviews.tt",
}
my ( $template, $borrowernumber, $cookie ) = &get_template_and_user(
);
if($format eq "rss"){
- my $lastbuilddate = C4::Dates->new();
- my $lastbuilddate_output = $lastbuilddate->output("rfc822");
+ my $lastbuilddate = dt_from_string;
+ my $lastbuilddate_output = $lastbuilddate->strftime("%a, %d %b %Y %H:%M:%S %z");
$template->param(
rss => 1,
timestamp => $lastbuilddate_output
}
}
-my $reviews = getallreviews(1,$offset,$results_per_page);
+my $reviews = Koha::Reviews->search(
+ { approved => 1 },
+ {
+ rows => $results_per_page,
+ page => $page,
+ order_by => { -desc => 'datereviewed' },
+ }
+)->unblessed;
my $marcflavour = C4::Context->preference("marcflavour");
-my $hits = numberofreviews(1);
+my $hits = Koha::Reviews->search({ approved => 1 })->count;
my $i = 0;
my $latest_comment_date;
for my $result (@$reviews){
my $biblionumber = $result->{biblionumber};
- my $bib = &GetBiblioData($biblionumber);
- my $record = GetMarcBiblio($biblionumber);
+ my $biblio = Koha::Biblios->find( $biblionumber );
+ my $biblioitem = $biblio->biblioitem;
+ my $record = GetMarcBiblio({ biblionumber => $biblionumber });
my $frameworkcode = GetFrameworkCode($biblionumber);
- my ( $borr ) = GetMemberDetails( $result->{borrowernumber} );
$result->{normalized_upc} = GetNormalizedUPC($record,$marcflavour);
$result->{normalized_ean} = GetNormalizedEAN($record,$marcflavour);
$result->{normalized_oclc} = GetNormalizedOCLCNumber($record,$marcflavour);
$result->{normalized_isbn} = GetNormalizedISBN(undef,$record,$marcflavour);
- $result->{title} = $bib->{'title'};
+ $result->{title} = $biblio->title;
$result->{subtitle} = GetRecordValue('subtitle', $record, $frameworkcode);
- $result->{author} = $bib->{'author'};
- $result->{place} = $bib->{'place'};
- $result->{publishercode} = $bib->{'publishercode'};
- $result->{copyrightdate} = $bib->{'copyrightdate'};
- $result->{pages} = $bib->{'pages'};
- $result->{size} = $bib->{'size'};
- $result->{notes} = $bib->{'notes'};
- $result->{timestamp} = $bib->{'timestamp'};
- $result->{firstname} = $borr->{'firstname'};
- $result->{surname} = $borr->{'surname'};
- if ($libravatar_enabled and $borr->{'email'}) {
- $result->{avatarurl} = libravatar_url(email => $borr->{'email'}, size => 40, https => $ENV{HTTPS});
- }
+ $result->{author} = $biblio->author;
+ $result->{place} = $biblioitem->place;
+ $result->{publishercode} = $biblioitem->publishercode;
+ $result->{copyrightdate} = $biblio->copyrightdate;
+ $result->{pages} = $biblioitem->pages;
+ $result->{size} = $biblioitem->size;
+ $result->{notes} = $biblioitem->notes;
+ $result->{timestamp} = $biblioitem->timestamp;
- if ($result->{borrowernumber} eq $borrowernumber) {
- $result->{your_comment} = 1;
- }
+ my $patron = Koha::Patrons->find( $result->{borrowernumber} );
+ if ( $patron ) {
+ $result->{borrtitle} = $patron->title;
+ $result->{firstname} = $patron->firstname;
+ $result->{surname} = $patron->surname;
+ $result->{userid} = $patron->userid;
+ if ($libravatar_enabled and $patron->email) {
+ $result->{avatarurl} = libravatar_url(email => $patron->email, size => 40, https => $ENV{HTTPS});
+ }
+
+ if ($result->{borrowernumber} eq $borrowernumber) {
+ $result->{your_comment} = 1;
+ }
+ }
if($format eq "rss"){
- my $rsstimestamp = C4::Dates->new($result->{datereviewed},"iso");
- my $rsstimestamp_output = $rsstimestamp->output("rfc822");
+ my $rsstimestamp = eval { dt_from_string( $result->{datereviewed} ); };
+ $rsstimestamp = dt_from_string unless ( $rsstimestamp ); #default to today if something went wrong
+ my $rsstimestamp_output = $rsstimestamp->strftime("%a, %d %b %Y %H:%M:%S %z");
$result->{timestamp} = $rsstimestamp_output;
- $result->{datereviewed} = format_date($result->{datereviewed});
- } else {
- $result->{datereviewed} = format_date($result->{datereviewed});
}
}
## Build the page numbers on the bottom of the page
# the page number for this page
my $this_page_number = $i;
# it should only be highlighted if it's the current page
- my $highlight = 1 if ($this_page_number == $current_page_number);
+ my $highlight;
+ $highlight = 1 if ($this_page_number == $current_page_number);
# put it in the array
push @page_numbers, { offset => $this_offset, pg => $this_page_number, highlight => $highlight };
for ($i=$current_page_number; $i<=($current_page_number + 20 );$i++) {
my $this_offset = ((($i-9)*$results_per_page)-$results_per_page);
my $this_page_number = $i-9;
- my $highlight = 1 if ($this_page_number == $current_page_number);
+ my $highlight;
+ $highlight = 1 if ($this_page_number == $current_page_number);
if ($this_page_number <= $pages) {
push @page_numbers, { offset => $this_offset, pg => $this_page_number, highlight => $highlight };
}
$template->param(
reviews => $reviews,
+ results_per_page => $results_per_page,
);
output_html_with_http_headers $query, $cookie, $template->output;