Bugzilla – Attachment 5499 Details for
Bug 6875
de-nesting C4 packages
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 6875 de-nesting C4/Serials
Bug-6875-de-nesting-C4Serials.patch (text/plain), 2.56 KB, created by
Paul Poulain
on 2011-09-21 15:22:11 UTC
(
hide
)
Description:
Bug 6875 de-nesting C4/Serials
Filename:
MIME Type:
Creator:
Paul Poulain
Created:
2011-09-21 15:22:11 UTC
Size:
2.56 KB
patch
obsolete
>From a8fec2acc4bcd33e68c76b244cefb5bc94692365 Mon Sep 17 00:00:00 2001 >From: Paul Poulain <paul.poulain@biblibre.com> >Date: Fri, 16 Sep 2011 18:48:15 +0200 >Subject: [PATCH] Bug 6875 de-nesting C4/Serials > >* use C4::Branch is loaded only to call GetBranchNAme in PrepareSerialsData and set branchname in the hashresult. This sub PrepareSerialsData is used in the following scripts : >opac/opac-serial-issues.pl: my $subscriptioninformation=PrepareSerialsData($subscriptions); => no use of branchname after >serials/serials-collection.pl: $subscriptions=PrepareSerialsData(\@subscriptioninformation); => no use of branchname after >=> we can remove the ->{branchname} from the result, and remove the C4::Branch dependancy >* moves use C4::Items to require C4::Items, to call AddItemFromMarc, when receiving a serial, with Recieving create an item set in the subscription. >* removed use C4::Letters and C4::Search, that are useless >--- > C4/Serials.pm | 14 ++++---------- > 1 files changed, 4 insertions(+), 10 deletions(-) > >diff --git a/C4/Serials.pm b/C4/Serials.pm >index 8aa6ebc..07a1798 100644 >--- a/C4/Serials.pm >+++ b/C4/Serials.pm >@@ -23,13 +23,7 @@ use warnings; > use C4::Dates qw(format_date format_date_in_iso); > use Date::Calc qw(:all); > use POSIX qw(strftime); >-use C4::Suggestions; >-use C4::Koha; > use C4::Biblio; >-use C4::Branch; >-use C4::Items; >-use C4::Search; >-use C4::Letters; > use C4::Log; # logaction > use C4::Debug; > >@@ -417,7 +411,6 @@ sub PrepareSerialsData { > $subs->{$datefield} = format_date( $subs->{$datefield} ); > } > } >- $subs->{'branchname'} = GetBranchName( $subs->{'branchcode'} ); > $subs->{ "status" . $subs->{'status'} } = 1; > $subs->{"checked"} = $subs->{'status'} =~ /1|3|4|7/; > >@@ -1338,8 +1331,8 @@ sub ReNewSubscription { > my $biblio = $sth->fetchrow_hashref; > > if ( C4::Context->preference("RenewSerialAddsSuggestion") ) { >- >- NewSuggestion( >+ require C4::Suggestions; >+ C4::Suggestions::NewSuggestion( > { 'suggestedby' => $user, > 'title' => $subscription->{bibliotitle}, > 'author' => $biblio->{author}, >@@ -1556,7 +1549,8 @@ sub ItemizeSerials { > $marcrecord->insert_fields_ordered($newField); > } > } >- AddItemFromMarc( $marcrecord, $data->{'biblionumber'} ); >+ require C4::Items; >+ C4::Items::AddItemFromMarc( $marcrecord, $data->{'biblionumber'} ); > return 1; > } > return ( 0, @errors ); >-- >1.7.4.1
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 6875
:
5442
|
5443
|
5496
|
5497
|
5498
|
5499
|
5500
|
5501
|
5502
|
5503
|
5504
|
5505
|
5506
|
5507
|
5508
|
5509
|
5510
|
5511
|
5512
|
7359
|
7360
|
7361
|
7362
|
7363
|
7364
|
7365
|
7652
|
7654
|
7655
|
7656
|
7657
|
7658
|
7770
|
7771
|
7808
|
7844
|
8095
|
8133
|
8134
|
8135