X-Git-Url: http://koha-dev.rot13.org:8081/gitweb/?a=blobdiff_plain;f=serials%2Fsubscription-detail.pl;h=009360b77cd9221ea566e1301e176a8b8f3ed633;hb=ea1aa7a0d906d583375618e37be60e9f0d62d939;hp=ae85c511d9af96ba061a65cf993e74e52a87c023;hpb=0d4acbba5c5db238eb42ed37dc9ba3dc2f36974b;p=koha_fer diff --git a/serials/subscription-detail.pl b/serials/subscription-detail.pl index ae85c511d9..009360b77c 100755 --- a/serials/subscription-detail.pl +++ b/serials/subscription-detail.pl @@ -34,6 +34,13 @@ my $issueconfirmed = $query->param('issueconfirmed'); my $dbh = C4::Context->dbh; my ($template, $loggedinuser, $cookie, $hemisphere); my $subscriptionid = $query->param('subscriptionid'); + +if ( $op and $op eq "close" ) { + C4::Serials::CloseSubscription( $subscriptionid ); +} elsif ( $op and $op eq "reopen" ) { + C4::Serials::ReopenSubscription( $subscriptionid ); +} + my $subs = GetSubscription($subscriptionid); $subs->{enddate} = GetExpirationDate($subscriptionid); @@ -100,7 +107,7 @@ my $hasRouting = check_routing($subscriptionid); # COMMENT hdl : IMHO, we should think about passing more and more data hash to template->param rather than duplicating code a new coding Guideline ? -for my $date qw(startdate enddate firstacquidate histstartdate histenddate){ +for my $date ( qw(startdate enddate firstacquidate histstartdate histenddate) ) { $$subs{$date} = format_date($$subs{$date}) if $date && $$subs{$date}; } $subs->{location} = GetKohaAuthorisedValueLib("LOC",$subs->{location}); @@ -119,6 +126,34 @@ if (! $subs->{periodicity}) { $subs->{periodicity} = '0'; } my $default_bib_view = get_default_view(); + +my ( $order, $bookseller, $tmpl_infos ); +# FIXME = see http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5335#c52 +#if ( defined $subscriptionid ) { +# my $lastOrderNotReceived = GetLastOrderNotReceivedFromSubscriptionid $subscriptionid; +# my $lastOrderReceived = GetLastOrderReceivedFromSubscriptionid $subscriptionid; +# if ( defined $lastOrderNotReceived ) { +# my $basket = GetBasket $lastOrderNotReceived->{basketno}; +# my $bookseller = GetBookSellerFromId $basket->{booksellerid}; +# ( $tmpl_infos->{valuegsti_ordered}, $tmpl_infos->{valuegste_ordered} ) = get_value_with_gst_params ( $lastOrderNotReceived->{ecost}, $lastOrderNotReceived->{gstrate}, $bookseller ); +# $tmpl_infos->{valuegsti_ordered} = sprintf( "%.2f", $tmpl_infos->{valuegsti_ordered} ); +# $tmpl_infos->{valuegste_ordered} = sprintf( "%.2f", $tmpl_infos->{valuegste_ordered} ); +# $tmpl_infos->{budget_name_ordered} = GetBudgetName $lastOrderNotReceived->{budget_id}; +# $tmpl_infos->{basketno} = $lastOrderNotReceived->{basketno}; +# $tmpl_infos->{ordered_exists} = 1; +# } +# if ( defined $lastOrderReceived ) { +# my $basket = GetBasket $lastOrderReceived->{basketno}; +# my $bookseller = GetBookSellerFromId $basket->{booksellerid}; +# ( $tmpl_infos->{valuegsti_spent}, $tmpl_infos->{valuegste_spent} ) = get_value_with_gst_params ( $lastOrderReceived->{unitprice}, $lastOrderReceived->{gstrate}, $bookseller ); +# $tmpl_infos->{valuegsti_spent} = sprintf( "%.2f", $tmpl_infos->{valuegsti_spent} ); +# $tmpl_infos->{valuegste_spent} = sprintf( "%.2f", $tmpl_infos->{valuegste_spent} ); +# $tmpl_infos->{budget_name_spent} = GetBudgetName $lastOrderReceived->{budget_id}; +# $tmpl_infos->{invoicenumber} = $lastOrderReceived->{booksellerinvoicenumber}; +# $tmpl_infos->{spent_exists} = 1; +# } +#} + $template->param( subscriptionid => $subscriptionid, serialslist => \@serialslist, @@ -138,7 +173,8 @@ $template->param( intranetcolorstylesheet => C4::Context->preference('intranetcolorstylesheet'), irregular_issues => scalar @irregular_issues, default_bib_view => $default_bib_view, - (uc(C4::Context->preference("marcflavour"))) => 1 + (uc(C4::Context->preference("marcflavour"))) => 1, + show_acquisition_details => defined $tmpl_infos->{ordered_exists} || defined $tmpl_infos->{spent_exists} ? 1 : 0, ); output_html_with_http_headers $query, $cookie, $template->output;