@@ -, +, @@ serials-collection.pl (3.12.x) --- C4/Serials.pm | 23 ++++++++++++-------- .../prog/en/modules/serials/serials-collection.tt | 4 ++-- serials/serials-collection.pl | 6 +++++ 3 files changed, 22 insertions(+), 11 deletions(-) --- a/C4/Serials.pm +++ a/C4/Serials.pm @@ -369,6 +369,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 @@ -378,15 +379,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); --- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt @@ -175,7 +175,7 @@ $(document).ready(function() { [% IF ( subscription.numberpattern7 ) %] None of the above [% END %] - [% subscription.branchcode %] + [% subscription.branchname %] [% subscription.callnumber %] [% subscription.notes %] [% UNLESS subscription.closed %] @@ -311,7 +311,7 @@ $(document).ready(function() { [% serial.notes %] - [% serial.branchcode %] + [% serial.branchname %] [% IF ( routing ) %] --- a/serials/serials-collection.pl +++ a/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; @@ -114,6 +115,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} = ''; } --