Bugzilla – Attachment 35873 Details for
Bug 7957
Routing lists: manage several routing list for each subscription, and export them as CSV
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[SIGNED OFF] Bug 7957: Remove uses of check_routing which doesn't exist anymore
0002-Bug-7957-Remove-uses-of-check_routing-which-doesn-t-.patch (text/plain), 3.48 KB, created by
Paola Rossi
on 2015-02-12 13:48:37 UTC
(
hide
)
Description:
[SIGNED OFF] Bug 7957: Remove uses of check_routing which doesn't exist anymore
Filename:
MIME Type:
Creator:
Paola Rossi
Created:
2015-02-12 13:48:37 UTC
Size:
3.48 KB
patch
obsolete
>From e48567be904b1820cc1f1f8fadcaca4fbf082996 Mon Sep 17 00:00:00 2001 >From: Julian Maurice <julian.maurice@biblibre.com> >Date: Tue, 10 Feb 2015 16:35:19 +0100 >Subject: [PATCH 2/4] Bug 7957: Remove uses of check_routing which doesn't > exist anymore > >Also remove a unit tests about C4::Serials::addroutingmember which >also no longer exists > >Signed-off-by: Paola Rossi <paola.rossi@cineca.it> >--- > acqui/newordersubscription.pl | 3 ++- > serials/serials-collection.pl | 2 +- > serials/serials-recieve.pl | 3 ++- > t/db_dependent/Serials.t | 6 +----- > 4 files changed, 6 insertions(+), 8 deletions(-) > >diff --git a/acqui/newordersubscription.pl b/acqui/newordersubscription.pl >index cb04c64..a8f5393 100755 >--- a/acqui/newordersubscription.pl >+++ b/acqui/newordersubscription.pl >@@ -25,6 +25,7 @@ use C4::Branch; > use C4::Context; > use C4::Output; > use C4::Serials; >+use C4::Serials::RoutingLists qw( GetRoutingListsCount ); > > use Koha::Acquisition::Bookseller; > >@@ -72,7 +73,7 @@ foreach my $sub (@subscriptions) { > > # to toggle between create or edit routing list options > if ($routing) { >- $sub->{routingedit} = check_routing( $sub->{subscriptionid} ); >+ $sub->{routingedit} = GetRoutingListsCount( $sub->{subscriptionid} ); > } > } > >diff --git a/serials/serials-collection.pl b/serials/serials-collection.pl >index e3f5ab3..ce03ff4 100755 >--- a/serials/serials-collection.pl >+++ b/serials/serials-collection.pl >@@ -129,7 +129,7 @@ if (@subscriptionid){ > my $numberpattern = C4::Serials::Numberpattern::GetSubscriptionNumberpattern($subs->{numberpattern}); > $subs->{frequency} = $frequency; > $subs->{numberpattern} = $numberpattern; >- $subs->{'hasRouting'} = check_routing($subscriptionid); >+ $subs->{'hasRouting'} = GetRoutingListsCount($subscriptionid); > push @$subscriptiondescs,$subs; > my $tmpsubscription= GetFullSubscription($subscriptionid); > @subscriptioninformation=(@$tmpsubscription,@subscriptioninformation); >diff --git a/serials/serials-recieve.pl b/serials/serials-recieve.pl >index 9fbce35..1c8aeaf 100755 >--- a/serials/serials-recieve.pl >+++ b/serials/serials-recieve.pl >@@ -73,6 +73,7 @@ use C4::Koha; > use C4::Output; > use C4::Context; > use C4::Serials; >+use C4::Serials::RoutingLists qw/GetRoutingListsCount/; > use C4::Branch; # GetBranches > > my $query = new CGI; >@@ -104,7 +105,7 @@ my $subscription=GetSubscription($subscriptionid); > > > my $auser = $subscription->{'librarian'}; # bob >-my $routing = check_routing($subscriptionid); # to see if routing list exists >+my $routing = GetRoutingListsCount($subscriptionid); # to see if routing list exists > my $manualdate =''; > my $manualissue =''; > my $manualstatus =0; >diff --git a/t/db_dependent/Serials.t b/t/db_dependent/Serials.t >index c255b9e..1e44ef8 100644 >--- a/t/db_dependent/Serials.t >+++ b/t/db_dependent/Serials.t >@@ -15,7 +15,7 @@ use C4::Bookseller; > use C4::Biblio; > use C4::Budgets; > use Koha::DateUtils; >-use Test::More tests => 43; >+use Test::More tests => 41; > > BEGIN { > use_ok('C4::Serials'); >@@ -179,10 +179,6 @@ is(C4::Serials::updateClaim(),undef, 'test updating claim'); > > is(C4::Serials::getsupplierbyserialid(),undef, 'test getting supplier idea'); > >-is(C4::Serials::check_routing(), undef, 'test checking route'); >- >-is(C4::Serials::addroutingmember(),undef, 'test adding route member'); >- > > # Unit tests for statuses management (Bug 11689) > $subscriptionid = NewSubscription( >-- >1.7.10.4 >
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 7957
:
9259
|
10973
|
13217
|
16614
|
21340
|
22726
|
35709
|
35801
|
35802
|
35803
|
35804
|
35842
|
35860
|
35872
|
35873
|
35874
|
35875
|
38425
|
41762
|
41763
|
41764
|
42427
|
42428
|
42429
|
47633
|
47634
|
47635
|
47659
|
47660
|
47661
|
47662
|
47663
|
47664
|
50226
|
50227
|
50228
|
50265
|
50266
|
50270
|
50273
|
97519
|
97520
|
97521
|
97522
|
97523
|
97524