diff --git a/C4/Serials.pm b/C4/Serials.pm index 03d287d42f..a582d044a5 100644 --- a/C4/Serials.pm +++ b/C4/Serials.pm @@ -360,6 +360,7 @@ sub GetFullSubscription { aqbooksellers.name as aqbooksellername, biblio.title as bibliotitle, subscription.branchcode AS branchcode, + branches.branchname AS branchname, subscription.subscriptionid AS subscriptionid |; if ( C4::Context->preference('IndependantBranches') && C4::Context->userenv @@ -369,15 +370,19 @@ sub GetFullSubscription { , ((subscription.branchcode <>\"" . C4::Context->userenv->{'branch'} . "\") and subscription.branchcode <>\"\" and subscription.branchcode IS NOT NULL) as cannotedit "; } $query .= qq| - FROM serial - LEFT JOIN subscription ON - (serial.subscriptionid=subscription.subscriptionid ) - LEFT JOIN aqbooksellers on subscription.aqbooksellerid=aqbooksellers.id - LEFT JOIN biblio on biblio.biblionumber=subscription.biblionumber - WHERE serial.subscriptionid = ? - ORDER BY year DESC, - IF(serial.publisheddate="00-00-0000",serial.planneddate,serial.publisheddate) DESC, - serial.subscriptionid + FROM serial + LEFT JOIN subscription ON + ( serial.subscriptionid = subscription.subscriptionid ) + LEFT JOIN aqbooksellers ON + ( subscription.aqbooksellerid = aqbooksellers.id ) + LEFT JOIN biblio ON + ( biblio.biblionumber = subscription.biblionumber ) + LEFT JOIN branches ON + ( subscription.branchcode = branches.branchcode ) + WHERE serial.subscriptionid = ? + ORDER BY year DESC, + IF(serial.publisheddate="00-00-0000",serial.planneddate,serial.publisheddate) DESC, + serial.subscriptionid |; $debug and warn "GetFullSubscription query: $query"; my $sth = $dbh->prepare($query); diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt index 3d2e690037..9ca26fcee1 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt @@ -149,7 +149,7 @@ $(document).ready(function() { [% IF ( subscription.numberpattern7 ) %] None of the above [% END %] - [% subscription.branchcode %] + [% subscription.branchname %] [% subscription.callnumber %] [% subscription.notes %] [% IF ( subscription.subscriptionexpired ) %]
Subscription expired [% END %] @@ -240,7 +240,7 @@ $(document).ready(function() { [% serial.notes %] - [% serial.branchcode %] + [% serial.branchname %] [% IF ( routing ) %] diff --git a/serials/serials-collection.pl b/serials/serials-collection.pl index 43935467a6..24efe39a2d 100755 --- a/serials/serials-collection.pl +++ b/serials/serials-collection.pl @@ -23,6 +23,7 @@ use strict; use warnings; use CGI; use C4::Auth; +use C4::Branch qw/GetBranchName/; use C4::Koha; use C4::Dates qw/format_date/; use C4::Serials; @@ -112,6 +113,11 @@ if (@subscriptionid){ $subs->{ "status" . $subs->{'status'} } = 1; $subs->{startdate} = format_date( $subs->{startdate} ); $subs->{histstartdate} = format_date( $subs->{histstartdate} ); + + if ( defined $subs->{ branchcode } ) { + $subs->{ branchname } = GetBranchName( $subs->{ branchcode } ); + } + if ( !defined $subs->{enddate} || $subs->{enddate} eq '0000-00-00' ) { $subs->{enddate} = ''; }