From 3cf4d92a5f2b7cd4a15913e96108f79323978000 Mon Sep 17 00:00:00 2001
From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Date: Tue, 16 Feb 2016 16:28:09 +0000
Subject: [PATCH] Bug 15839: Koha::Reviews - Remove getnumberofreviews

Signed-off-by: Marc Veron <veron@veron.ch>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
---
 C4/Review.pm              | 22 +---------------------
 mainpage.pl               |  4 ++--
 opac/opac-showreviews.pl  |  2 +-
 reviews/reviewswaiting.pl |  2 +-
 tools/tools-home.pl       |  7 +++----
 5 files changed, 8 insertions(+), 29 deletions(-)

diff --git a/C4/Review.pm b/C4/Review.pm
index 0c27833090..b76b1e99fa 100644
--- a/C4/Review.pm
+++ b/C4/Review.pm
@@ -27,7 +27,7 @@ use vars qw(@ISA @EXPORT);
 BEGIN {
     require Exporter;
     @ISA    = qw(Exporter);
-    @EXPORT = qw(savereview updatereview numberofreviews numberofreviewsbybiblionumber);
+    @EXPORT = qw(savereview updatereview numberofreviewsbybiblionumber);
 }
 
 =head1 NAME
@@ -40,7 +40,6 @@ C4::Review - Perl Module containing routines for dealing with reviews of items
 
   savereview($biblionumber,$borrowernumber,$review);
   updatereview($biblionumber,$borrowernumber,$review);
-  my $count=numberofreviews($status);
   my $count=numberofreviewsbybiblionumber($biblionumber);
 
 =head1 DESCRIPTION
@@ -83,25 +82,6 @@ sub updatereview {
     $sth->execute( $review, $borrowernumber, $biblionumber );
 }
 
-=head2 numberofreviews
-
-  my $count=numberofreviews( [$status] );
-
-Return the number of reviews where in the 'reviews' database : 'approved' = $status
-(By default $status = 1)
-
-=cut
-
-sub numberofreviews {
-    my ($param) = @_;
-    my $status = ( defined($param) ? $param : 1 );
-    my $dbh    = C4::Context->dbh;
-    my $query  = "SELECT count(*) FROM reviews WHERE approved=?";
-    my $sth    = $dbh->prepare($query);
-    $sth->execute($status);
-    return $sth->fetchrow;
-}
-
 =head2 numberofreviewsbybiblionumber
 
   my $count=numberofreviewsbybiblionumber($biblionumber);
diff --git a/mainpage.pl b/mainpage.pl
index f4efc653b8..258a9d5039 100755
--- a/mainpage.pl
+++ b/mainpage.pl
@@ -25,11 +25,11 @@ use C4::Output;
 use C4::Auth;
 use C4::Koha;
 use C4::NewsChannels; # GetNewsToDisplay
-use C4::Review qw/numberofreviews/;
 use C4::Suggestions qw/CountSuggestion/;
 use C4::Tags qw/get_count_by_tag_status/;
 use Koha::Patron::Modifications;
 use Koha::Patron::Discharge;
+use Koha::Reviews;
 
 my $query = new CGI;
 
@@ -62,7 +62,7 @@ my $branch =
   ? C4::Context->userenv()->{'branch'}
   : undef;
 
-my $pendingcomments    = numberofreviews(0);
+my $pendingcomments    = Koha::Reviews->search({ approved => 0 })->count;
 my $pendingtags        = get_count_by_tag_status(0);
 my $pendingsuggestions = CountSuggestion("ASKED");
 my $pending_borrower_modifications = Koha::Patron::Modifications->pending_count( $branch );
diff --git a/opac/opac-showreviews.pl b/opac/opac-showreviews.pl
index 69c0869787..a88a74e75a 100755
--- a/opac/opac-showreviews.pl
+++ b/opac/opac-showreviews.pl
@@ -85,7 +85,7 @@ my $reviews = Koha::Reviews->search(
     }
 )->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){
diff --git a/reviews/reviewswaiting.pl b/reviews/reviewswaiting.pl
index 74b92f5baa..017e1621d9 100755
--- a/reviews/reviewswaiting.pl
+++ b/reviews/reviewswaiting.pl
@@ -43,7 +43,7 @@ my $status   = $query->param('status') || 0;
 my $reviewid = $query->param('reviewid');
 my $page     = $query->param('page') || 1;
 my $count    = C4::Context->preference('numSearchResults') || 20;
-my $total    = numberofreviews($status);
+my $total    = Koha::Reviews->search({ approved => $status })->count;
 
 if ( $op eq 'approve' ) {
     my $review = Koha::Reviews->find( $reviewid );
diff --git a/tools/tools-home.pl b/tools/tools-home.pl
index 1a1ce303fb..86cfb0376f 100755
--- a/tools/tools-home.pl
+++ b/tools/tools-home.pl
@@ -15,14 +15,13 @@
 # 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 qw ( -utf8 );
 use C4::Auth;
 use C4::Output;
-use C4::Review qw/numberofreviews/;
 use C4::Tags qw/get_count_by_tag_status/;
+use Koha::Reviews;
 
 my $query = new CGI;
 my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
@@ -36,7 +35,7 @@ my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
     }
 );
 
-my $pendingcomments = numberofreviews(0);
+my $pendingcomments = Koha::Reviews->search({ approved => 0 })->count;
 my $pendingtags = get_count_by_tag_status(0);
 
 $template->param(
-- 
2.39.5