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

Bug 7288: (follow-up) various fixes

Browse source 
- Fix syntax error in supplied test
- remove subscriptionid as a field returned by GetInvoices(), as
  the is_linked_to_subscriptions Boolean takes its place.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
This commit is contained in:
Galen Charlton 2014-04-21 05:06:14 +00:00
parent c2461cf171
commit afa6a33449
2 changed files with 2 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
C4/Acquisition.pm
View file

@ -2579,7 +2579,6 @@ sub GetInvoices {
my $dbh = C4::Context->dbh;
my $query = qq{
SELECT aqinvoices.*, aqbooksellers.name AS suppliername,
aqorders.subscriptionid,
COUNT(
DISTINCT IF(
aqorders.datereceived IS NOT NULL,

4
t/db_dependent/Acquisition/Invoices.t
View file

@ -9,7 +9,7 @@ use warnings;
use C4::Bookseller qw( GetBookSellerFromId );
use C4::Biblio qw( AddBiblio );
use Test::More tests => 21;
use Test::More tests => 22;
BEGIN {
use_ok('C4::Acquisition');
@ -184,7 +184,7 @@ my @invoices_linked_to_subscriptions = map{
? $_
: ()
} @invoices;
is_deeply( \@invoices_linked_to_subscriptions, [], "GetInvoices return linked_to_subscriptions: there is no invoices linked to subscriptions yet" )
is_deeply( \@invoices_linked_to_subscriptions, [], "GetInvoices return linked_to_subscriptions: there is no invoices linked to subscriptions yet" );
END {
$dbh and $dbh->rollback;