X-Git-Url: http://git.rot13.org/?a=blobdiff_plain;f=serials%2Fsubscription-detail.pl;h=b6538fe1558616638d59d400efbb0469be6fc1dc;hb=f245998ac964881e9cc2b36434860c55698e8701;hp=4c44c889fb5f6452c4a761933593caf57fd68233;hpb=247e04d204697126208d6142e02bd49307ef9e29;p=koha.git

diff --git a/serials/subscription-detail.pl b/serials/subscription-detail.pl
index 4c44c889fb..b6538fe155 100755
--- a/serials/subscription-detail.pl
+++ b/serials/subscription-detail.pl
@@ -1,144 +1,159 @@
 #!/usr/bin/perl
 
+# This file is part of Koha.
+#
+# Koha is free software; you can redistribute it and/or modify it under the
+# terms of the GNU General Public License as published by the Free Software
+# Foundation; either version 2 of the License, or (at your option) any later
+# version.
+#
+# Koha is distributed in the hope that it will be useful, but WITHOUT ANY
+# WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
+# A PARTICULAR PURPOSE.  See the GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License along
+# with Koha; if not, write to the Free Software Foundation, Inc.,
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+
 use strict;
+use warnings;
 use CGI;
 use C4::Auth;
 use C4::Koha;
-use C4::Date;
-use C4::Bull;
+use C4::Dates qw/format_date/;
+use C4::Serials;
 use C4::Output;
-use C4::Interface::CGI::Output;
 use C4::Context;
-use HTML::Template;
+use Date::Calc qw/Today Day_of_Year Week_of_Year Add_Delta_Days/;
+use Carp;
 
 my $query = new CGI;
-my $op = $query->param('op') || '';
+my $op = $query->param('op') || q{};
+my $issueconfirmed = $query->param('issueconfirmed');
 my $dbh = C4::Context->dbh;
-my $sth;
-# my $id;
-my ($template, $loggedinuser, $cookie, $subs, $user, $sessionID, $flags);
-my ($subscriptionid,$auser,$librarian,$cost,$aqbooksellerid, $aqbooksellername,$aqbudgetid, $bookfundid, $startdate, $periodicity,
-	$dow, $numberlength, $weeklength, $monthlength,
-	$add1,$every1,$whenmorethan1,$setto1,$lastvalue1,$innerloop1,
-	$add2,$every2,$whenmorethan2,$setto2,$lastvalue2,$innerloop2,
-	$add3,$every3,$whenmorethan3,$setto3,$lastvalue3,$innerloop3,
-	$numberingmethod, $status, $biblionumber, $bibliotitle, $notes,$letter);
-
-$subscriptionid = $query->param('subscriptionid');
-
-if ($op eq 'modsubscription') {
-	$auser = $query->param('user');
-	$librarian = $query->param('librarian');
-	$cost = $query->param('cost');
-	$aqbooksellerid = $query->param('aqbooksellerid');
-	$biblionumber = $query->param('biblionumber');
-	$aqbudgetid = $query->param('aqbudgetid');
-	$startdate = format_date_in_iso($query->param('startdate'));
-	$periodicity = $query->param('periodicity');
-	$dow = $query->param('dow');
-	$numberlength = $query->param('numberlength');
-	$weeklength = $query->param('weeklength');
-	$monthlength = $query->param('monthlength');
-	$add1 = $query->param('add1');
-	$every1 = $query->param('every1');
-	$whenmorethan1 = $query->param('whenmorethan1');
-	$setto1 = $query->param('setto1');
-	$lastvalue1 = $query->param('lastvalue1');
-	$innerloop1 = $query->param('innerloop1');
-	$add2 = $query->param('add2');
-	$every2 = $query->param('every2');
-	$whenmorethan2 = $query->param('whenmorethan2');
-	$setto2 = $query->param('setto2');
-	$lastvalue2 = $query->param('lastvalue2');
-	$innerloop2 = $query->param('innerloop2');
-	$add3 = $query->param('add3');
-	$every3 = $query->param('every3');
-	$whenmorethan3 = $query->param('whenmorethan3');
-	$setto3 = $query->param('setto3');
-	$lastvalue3 = $query->param('lastvalue3');
-	$innerloop3 = $query->param('innerloop3');
-	$numberingmethod = $query->param('numberingmethod');
-	$status = 1;
-	$notes = $query->param('notes');
-	$letter = $query->param('letter');
-    
-	&modsubscription($auser,$aqbooksellerid,$cost,$aqbudgetid,$startdate,
-					$periodicity,$dow,$numberlength,$weeklength,$monthlength,
-					$add1,$every1,$whenmorethan1,$setto1,$lastvalue1,$innerloop1,
-					$add2,$every2,$whenmorethan2,$setto2,$lastvalue2,$innerloop2,
-					$add3,$every3,$whenmorethan3,$setto3,$lastvalue3,$innerloop3,
-					$numberingmethod, $status, $biblionumber, $notes, $letter, $subscriptionid);
-}
+my ($template, $loggedinuser, $cookie, $hemisphere);
+my $subscriptionid = $query->param('subscriptionid');
+my $subs = GetSubscription($subscriptionid);
 
-if ($op eq 'del') {
-	&delsubscription($subscriptionid);
-	print "Content-Type: text/html\n\n<META HTTP-EQUIV=Refresh CONTENT=\"0; URL=../bull-home.pl\"></html>";
-	exit;
+$subs->{enddate} = GetExpirationDate($subscriptionid);
 
+if ($op && $op eq 'del') {
+	if ($subs->{'cannotedit'}){
+		carp "Attempt to delete subscription $subscriptionid by ".C4::Context->userenv->{'id'}." not allowed";
+		print $query->redirect("/cgi-bin/koha/serials/subscription-detail.pl?subscriptionid=$subscriptionid");
+	}
+	DelSubscription($subscriptionid);
+	print "Content-Type: text/html\n\n<META HTTP-EQUIV=Refresh CONTENT=\"0; URL=serials-home.pl\"></html>";
+	exit;
 }
-$subs = &getsubscription($subscriptionid);
-# html'ize distributedto
-$subs->{distributedto}=~ s/\n/<br \/>/g;
-my ($totalissues,@serialslist) = getserials($subscriptionid);
+my ($routing, @routinglist) = getroutinglist($subscriptionid);
+my ($totalissues,@serialslist) = GetSerials($subscriptionid);
 $totalissues-- if $totalissues; # the -1 is to have 0 if this is a new subscription (only 1 issue)
+# the subscription must be deletable if there is NO issues for a reason or another (should not happend, but...)
+
+# Permission needed if it is a deletion (del) : delete_subscription
+# Permission needed otherwise : *
+my $permission = ($op eq "del") ? "delete_subscription" : "*";
 
 ($template, $loggedinuser, $cookie)
 = get_template_and_user({template_name => "serials/subscription-detail.tmpl",
-				query => $query,
-				type => "intranet",
-				authnotrequired => 0,
-				flagsrequired => {catalogue => 1},
-				debug => 1,
-				});
+                query => $query,
+                type => "intranet",
+                authnotrequired => 0,
+                flagsrequired => {serials => $permission},
+                debug => 1,
+                });
 
-($user, $cookie, $sessionID, $flags) = checkauth($query, 0, {catalogue => 1}, "intranet");
+$$subs{enddate} ||= GetExpirationDate($subscriptionid);
 
+if ($op eq 'del') {
+	if ($$subs{'cannotedit'}){
+		carp "Attempt to delete subscription $subscriptionid by ".C4::Context->userenv->{'id'}." not allowed";
+		print $query->redirect("/cgi-bin/koha/serials/subscription-detail.pl?subscriptionid=$subscriptionid");
+		exit;
+	}
+	
+    # Asking for confirmation if the subscription has not strictly expired yet or if it has linked issues
+    my $strictlyexpired = HasSubscriptionStrictlyExpired($subscriptionid);
+    my $linkedissues = CountIssues($subscriptionid);
+    my $countitems   = HasItems($subscriptionid);
+    if ($strictlyexpired == 0 || $linkedissues > 0 || $countitems>0) {
+		$template->param(NEEDSCONFIRMATION => 1);
+		if ($strictlyexpired == 0) { $template->param("NOTEXPIRED" => 1); }
+		if ($linkedissues     > 0) { $template->param("LINKEDISSUES" => 1); }
+		if ($countitems       > 0) { $template->param("LINKEDITEMS"  => 1); }
+    } else {
+		$issueconfirmed = "1";
+    }
+    # If it's ok to delete the subscription, we do so
+    if ($issueconfirmed eq "1") {
+		&DelSubscription($subscriptionid);
+		print "Content-Type: text/html\n\n<META HTTP-EQUIV=Refresh CONTENT=\"0; URL=serials-home.pl\"></html>";
+		exit;
+    }
+}
+my $hasRouting = check_routing($subscriptionid);
+my ($totalissues,@serialslist) = GetSerials($subscriptionid);
+$totalissues-- if $totalissues; # the -1 is to have 0 if this is a new subscription (only 1 issue)
+# the subscription must be deletable if there is NO issues for a reason or another (should not happend, but...)
+
+my ($user, $sessionID, $flags);
+($user, $cookie, $sessionID, $flags)
+    = checkauth($query, 0, {catalogue => 1}, "intranet");
+
+# 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){
+    $$subs{$date}      = format_date($$subs{$date}) if $date && $$subs{$date};
+}
+$subs->{abouttoexpire}  = abouttoexpire($subs->{subscriptionid});
+
+$template->param($subs);
+$template->param(biblionumber_for_new_subscription => $subs->{bibnum});
+my @irregular_issues = split /,/, $subs->{irregularity};
+
+if (! $subs->{numberpattern}) {
+    $subs->{numberpattern} = q{};
+}
+if (! $subs->{dow}) {
+    $subs->{dow} = q{};
+}
+if (! $subs->{periodicity}) {
+    $subs->{periodicity} = '0';
+}
+my $default_bib_view = get_default_view();
 $template->param(
-	user => $subs->{auser},
-	librarian => $subs->{librarian},
-	aqbooksellerid => $subs->{aqbooksellerid},
-	aqbooksellername => $subs->{aqbooksellername},
-	cost => $subs->{cost},
-	aqbudgetid => $subs->{aqbudgetid},
-	bookfundid => $subs->{bookfundid},
-	startdate => format_date($subs->{startdate}),
-	periodicity => $subs->{periodicity},
-	dow => $subs->{dow},
-	numberlength => $subs->{numberlength},
-	weeklength => $subs->{weeklength},
-	monthlength => $subs->{monthlength},
-	add1 => $subs->{add1},
-	every1 => $subs->{every1},
-	whenmorethan1 => $subs->{whenmorethan1},
-	innerloop1 => $subs->{innerloop1},
-	setto1 => $subs->{setto1},
-	lastvalue1 => $subs->{lastvalue1},
-	add2 => $subs->{add2},
-	every2 => $subs->{every2},
-	whenmorethan2 => $subs->{whenmorethan2},
-	setto2 => $subs->{setto2},
-	lastvalue2 => $subs->{lastvalue2},
-	innerloop2 => $subs->{innerloop2},
-	add3 => $subs->{add3},
-	every3 => $subs->{every3},
-	whenmorethan3 => $subs->{whenmorethan3},
-	setto3 => $subs->{setto3},
-	lastvalue3 => $subs->{lastvalue3},
-	innerloop3 => $subs->{innerloop3},
-	numberingmethod => $subs->{numberingmethod},
-	status => $subs->{status},
-	biblionumber => $subs->{biblionumber},
-	bibliotitle => $subs->{bibliotitle},
-	notes => $subs->{notes},
-	letter => $subs->{letter},
-	distributedto => $subs->{distributedto},
-	subscriptionid => $subs->{subscriptionid},
-	serialslist => \@serialslist,
-	totalissues => $totalissues,
-	);
-$template->param(
-			"periodicity$subs->{periodicity}" => 1,
-			"arrival$subs->{dow}" => 1,
-			);
+	subscriptionid => $subscriptionid,
+    serialslist => \@serialslist,
+    hasRouting  => $hasRouting,
+    totalissues => $totalissues,
+    hemisphere => $hemisphere,
+    cannotedit =>(C4::Context->preference('IndependantBranches') &&
+                C4::Context->userenv &&
+                C4::Context->userenv->{flags} % 2 !=1  &&
+                C4::Context->userenv->{branch} && $subs->{branchcode} &&
+                (C4::Context->userenv->{branch} ne $subs->{branchcode})),
+    'periodicity' . $subs->{periodicity} => 1,
+    'arrival' . $subs->{dow} => 1,
+    'numberpattern' . $subs->{numberpattern} => 1,
+    intranetstylesheet => C4::Context->preference('intranetstylesheet'),
+    intranetcolorstylesheet => C4::Context->preference('intranetcolorstylesheet'),
+    irregular_issues => scalar @irregular_issues,
+    default_bib_view => $default_bib_view,
+    );
 
 output_html_with_http_headers $query, $cookie, $template->output;
+
+sub get_default_view {
+    my $defaultview = C4::Context->preference('IntranetBiblioDefaultView');
+    my $views = { C4::Search::enabled_staff_search_views };
+    if ($defaultview eq 'isbd' && $views->{can_view_ISBD}) {
+        return 'ISBDdetail';
+    } elsif  ($defaultview eq 'marc' && $views->{can_view_MARC}) {
+        return 'MARCdetail';
+    } elsif  ($defaultview eq 'labeled_marc' && $views->{can_view_labeledMARC}) {
+        return 'labeledMARCdetail';
+    } else {
+        return 'detail';
+    }
+}