BUG 1580, subscription irregularities and volume type, not passed when editing subscr...
[koha.git] / serials / subscription-add.pl
index ccc94b9..47430cd 100755 (executable)
@@ -1,7 +1,5 @@
 #!/usr/bin/perl
 
-# Copyright 2000-2003 Katipo Communications
-#
 # This file is part of Koha.
 #
 # Koha is free software; you can redistribute it and/or modify it under the
 # Koha; if not, write to the Free Software Foundation, Inc., 59 Temple Place,
 # Suite 330, Boston, MA  02111-1307 USA
 
-# $Id$
-
-=head1 NAME
-
-subscription-add.pl
-
-=head1 DESCRIPTION
-
-this script add a subscription into the database.
-
-=head1 PARAMETERS
-
-=over 4
-
-=item op
-op use to know the operation to do on this template.
- * mod : to modify an existing subscription
- * addsubscription : to add a subscription
-
-Note that if op = mod or addsubscription there are a lot of other params.
-
-
-=back
-
-=cut
-
-
 use strict;
 use CGI;
+use Date::Calc qw(Today Day_of_Year Week_of_Year Add_Delta_Days);
 use C4::Koha;
 use C4::Auth;
-use C4::Date;
+use C4::Dates qw/format_date format_date_in_iso/;
+use C4::Acquisition;
 use C4::Output;
-use C4::Serials;
-use C4::Bookfund;
-use C4::Interface::CGI::Output;
 use C4::Context;
-use HTML::Template;
+use C4::Branch; # GetBranches
+use C4::Serials;
 use C4::Letters;
-use C4::Members;
+
+#use Smart::Comments;
 
 my $query = new CGI;
 my $op = $query->param('op');
 my $dbh = C4::Context->dbh;
-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);
-
-    my @budgets;
+my ($subscriptionid,$auser,$branchcode,$librarian,$cost,$aqbooksellerid, $aqbooksellername,$aqbudgetid, $bookfundid, $startdate, $periodicity,
+       $firstacquidate, $dow, $irregularity, $numberpattern, $numberlength, $weeklength, $monthlength, $sublength,
+       $add1,$every1,$whenmorethan1,$setto1,$lastvalue1,$innerloop1,
+       $add2,$every2,$whenmorethan2,$setto2,$lastvalue2,$innerloop2,
+       $add3,$every3,$whenmorethan3,$setto3,$lastvalue3,$innerloop3,
+       $numberingmethod, $status, $biblionumber, 
+       $bibliotitle, $callnumber, $notes, $hemisphere, $letter, $manualhistory);
+
+       my @budgets;
 my ($template, $loggedinuser, $cookie)
 = get_template_and_user({template_name => "serials/subscription-add.tmpl",
-                query => $query,
-                type => "intranet",
-                authnotrequired => 0,
-                flagsrequired => {catalogue => 1},
-                debug => 1,
-                });
-
-
-#FIXME : If Budgets are never used, then these lines are useless.
-$dbh = C4::Context->dbh;
-my $sthtemp = GetBranchCodeFromBorrowers();
-$sthtemp->execute($loggedinuser);
-my ($flags, $homebranch)=$sthtemp->fetchrow;
-#FIXME : END added by hdl on July,14 2005
-
-if ($op eq 'mod') {
+                               query => $query,
+                               type => "intranet",
+                               authnotrequired => 0,
+                               flagsrequired => {serials => 1},
+                               debug => 1,
+                               });
+
+
+my $weekarrayjs='';
+my $count = 0;
+my ($year, $month, $day) = Today;
+my $firstday   =  Day_of_Year($year,$month,$day);
+my ($wkno,$yr) = Week_of_Year($year,$month,$day); # week starting monday
+my $weekno = $wkno;
+for(my $i=$firstday;$i<($firstday+365);$i=$i+7){
+        $count = $i;
+        if($wkno > 52){$year++; $wkno=1;}
+        if($count>365){$count=$i-365;}    
+        my ($y,$m,$d) = Add_Delta_Days(1,1,1,$i - 1);
+
+        #BUGFIX padding add_delta_days() date
+        my $output  = sprintf("%04d-%02d-%02d",$y , $m, $d );
+
+        $weekarrayjs .= "'Wk $wkno: ".format_date($output)."',";
+        $wkno++;    
+}
+chop($weekarrayjs);
+# warn $weekarrayjs;
+
+my $sub_on;
+my @subscription_types = (
+            'issues', 'weeks', 'months'
+        ); 
+my @sub_type_data;
+
+my $letters = GetLetters('serial');
+my @letterloop;
+foreach my $thisletter (keys %$letters) {
+    my $selected = 1 if $thisletter eq $letter;
+    my %row =(value => $thisletter,
+                selected => $selected,
+                lettername => $letters->{$thisletter},
+            );
+    push @letterloop, \%row;
+}
+$template->param(letterloop => \@letterloop);
+
+my $onlymine=C4::Context->preference('IndependantBranches') && 
+             C4::Context->userenv && 
+             C4::Context->userenv->{flags}!=1 && 
+             C4::Context->userenv->{branch};
+my $branches = GetBranches($onlymine);
+my @branchloop;
+foreach my $thisbranch (keys %$branches) {
+    my $selected = 1 if $thisbranch eq C4::Context->userenv->{'branch'};
+    my %row =(value => $thisbranch,
+                selected => $selected,
+                branchname => $branches->{$thisbranch}->{'branchname'},
+            );
+    push @branchloop, \%row;
+}
+$template->param(branchloop => \@branchloop,
+    DHTMLcalendar_dateformat => C4::Dates->DHTMLcalendar(),
+);
+
+if ($op eq 'mod'||$op eq 'dup') {
+
     my $subscriptionid = $query->param('subscriptionid');
+#     warn "irregularity :$irregularity numberpattern : $numberpattern, callnumber :$callnumber, firstacquidate :$firstacquidate";
     my $subs = &GetSubscription($subscriptionid);
-    $auser = $subs->{'user'};
-    $librarian = $subs->{'librarian'};
-    $cost = $subs->{'cost'};
-    $aqbooksellerid = $subs->{'aqbooksellerid'};
-    $aqbooksellername = $subs->{'aqbooksellername'};
-    $bookfundid = $subs->{'bookfundid'};
-    $aqbudgetid = $subs->{'aqbudgetid'};
-    defined $aqbudgetid or $aqbudgetid='';
-    $startdate = $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'};
-    $setto1 = $subs->{'setto1'};
-    $lastvalue1 = $subs->{'lastvalue1'};
-    $innerloop1 = $subs->{'innerloop1'};
-    $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'};
-    defined $letter or $letter='';
-    $template->param(
-        $op => 1,
-        user => $auser,
-        librarian => $librarian,
-        aqbooksellerid => $aqbooksellerid,
-        aqbooksellername => $aqbooksellername,
-        cost => $cost,
-        aqbudgetid => $aqbudgetid,
-        bookfundid => $bookfundid,
-        startdate => format_date($startdate),
-        periodicity => $periodicity,
-        dow => $dow,
-        numberlength => $numberlength,
-        weeklength => $weeklength,
-        monthlength => $monthlength,
-        add1 => $add1,
-        every1 => $every1,
-        whenmorethan1 => $whenmorethan1,
-        setto1 => $setto1,
-        lastvalue1 => $lastvalue1,
-        innerloop1 => $innerloop1,
-        add2 => $add2,
-        every2 => $every2,
-        whenmorethan2 => $whenmorethan2,
-        setto2 => $setto2,
-        lastvalue2 => $lastvalue2,
-        innerloop2 => $innerloop2,
-        add3 => $add3,
-        every3 => $every3,
-        whenmorethan3 => $whenmorethan3,
-        setto3 => $setto3,
-        lastvalue3 => $lastvalue3,
-        innerloop3 => $innerloop3,
-        numberingmethod => $numberingmethod,
-        status => $status,
-        biblionumber => $biblionumber,
-        bibliotitle => $bibliotitle,
-        notes => $notes,
-        letter => $letter,
-        subscriptionid => $subscriptionid,
-        "periodicity$periodicity" => 1,
-        "dow$dow" => 1,
-        );
-}
-@budgets = GetBookFunds($homebranch);
-my $temp = scalar(@budgets);
+## FIXME : Check rights to edit if mod. Could/Should display an error message.
+    if ($subs->{'cannotedit'} && $op eq 'mod'){
+      warn "Attempt to modify subscription $subscriptionid by ".C4::Context->userenv->{'id'}." not allowed";
+      print $query->redirect("/cgi-bin/koha/serials/subscription-detail.pl?subscriptionid=$subscriptionid");
+    }  
+       for (qw(startdate firstacquidate histstartdate enddate)) {
+       $subs->{$_} = format_date($subs->{$_});
+       }
+    $subs->{'letter'}='' unless($subs->{'letter'});
+    $irregularity   = $subs->{'irregularity'};
+    $numberpattern  = $subs->{'numberpattern'};
 
-# find default value & set it for the template
-for (my $i=0;$i<$#budgets;$i++) {
-    if ($budgets[$i]->{'aqbudgetid'} eq $aqbudgetid) {
-        $budgets[$i]->{'selected'}=1;
+
+    if($subs->{numberlength} > 0){
+        $sublength = $subs->{numberlength};
+        $sub_on = $subscription_types[0];
+    } elsif ($subs->{weeklength}>0){
+        $sublength = $subs->{weeklength};
+        $sub_on = $subscription_types[1];
+    } else {
+        $sublength = $subs->{monthlength};
+        $sub_on = $subscription_types[2];
+    }
+    while (@subscription_types) {
+        my $sub_type = shift @subscription_types;
+        my %row = ( 'name' => $sub_type );
+        if ( $sub_on eq $sub_type ) {
+            $row{'selected'} = ' selected';
+        } else {
+            $row{'selected'} = '';
+        }
+        push( @sub_type_data, \%row );
     }
-}
-$template->param(budgets => \@budgets);
-#FIXME : END Added by hdl on July, 14 2005
 
-my @letterlist = GetLetterList('serial');
-for (my $i=0;$i<=$#letterlist;$i++) {
-    $letterlist[$i]->{'selected'} =1 if $letterlist[$i]->{'code'} eq $letter;
+    $template->param($subs);
+    $template->param(
+            $op => 1,
+            subtype => \@sub_type_data,
+            sublength =>$sublength,
+            history => ($op eq 'mod' && ($subs->{recievedlist}||$subs->{missinglist}||$subs->{opacnote}||$subs->{librariannote}))
+            );
+    $template->param(
+                "periodicity".$subs->{'periodicity'} => 1,
+                "dow".$subs->{'dow'} => 1,
+                "numberpattern".$subs->{'numberpattern'} => 1,
+                );
 }
-$template->param(letters => \@letterlist);
 
 if ($op eq 'addsubscription') {
+
     my $auser = $query->param('user');
+    my $branchcode = $query->param('branchcode');
     my $aqbooksellerid = $query->param('aqbooksellerid');
     my $cost = $query->param('cost');
-    my $aqbudgetid = $query->param('aqbudgetid');
+    my $aqbudgetid = $query->param('aqbudgetid'); 
     my $startdate = $query->param('startdate');
+    my $firstacquidate = $query->param('firstacquidate');    
     my $periodicity = $query->param('periodicity');
     my $dow = $query->param('dow');
-    my $numberlength = $query->param('numberlength');
-    my $weeklength = $query->param('weeklength');
-    my $monthlength = $query->param('monthlength');
+    my @irregularity = $query->param('irregular');
+    my $numberlength = 0;
+    my $weeklength = 0;
+    my $monthlength = 0;
+    my $numberpattern = $query->param('numbering_pattern');
+    my $sublength = $query->param('sublength');
+    my $subtype = $query->param('subtype');
+    if ($subtype eq 'months'){
+        $monthlength = $sublength;
+    } elsif ($subtype eq 'weeks'){
+        $weeklength = $sublength;
+    } else {
+        $numberlength = $sublength;
+    }
     my $add1 = $query->param('add1');
     my $every1 = $query->param('every1');
     my $whenmorethan1 = $query->param('whenmorethan1');
     my $setto1 = $query->param('setto1');
     my $lastvalue1 = $query->param('lastvalue1');
+    my $innerloop1 =$query->param('innerloop1');
     my $add2 = $query->param('add2');
     my $every2 = $query->param('every2');
     my $whenmorethan2 = $query->param('whenmorethan2');
     my $setto2 = $query->param('setto2');
+    my $innerloop2 =$query->param('innerloop2');
     my $lastvalue2 = $query->param('lastvalue2');
     my $add3 = $query->param('add3');
     my $every3 = $query->param('every3');
     my $whenmorethan3 = $query->param('whenmorethan3');
     my $setto3 = $query->param('setto3');
     my $lastvalue3 = $query->param('lastvalue3');
+    my $innerloop3 =$query->param('innerloop3');
     my $numberingmethod = $query->param('numberingmethod');
     my $status = 1;
     my $biblionumber = $query->param('biblionumber');
+    my $callnumber = $query->param('callnumber');
+    my $notes = $query->param('notes');
+    my $internalnotes = $query->param('internalnotes');
+    my $hemisphere = $query->param('hemisphere') || 1;
+       my $letter = $query->param('letter');
+    # ## BugFIX : hdl doesnot know what innerloops or letter stand for but it seems necessary. So he adds them.
+    my $manualhistory = $query->param('manualhist');
+       my $subscriptionid = NewSubscription($auser,$branchcode,$aqbooksellerid,$cost,$aqbudgetid,$biblionumber,
+                                       $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, $notes,$letter,$firstacquidate,join(",",@irregularity),
+                    $numberpattern, $callnumber, $hemisphere,($manualhistory?$manualhistory:0),$internalnotes
+                               );
+
+    print $query->redirect("/cgi-bin/koha/serials/subscription-detail.pl?subscriptionid=$subscriptionid");
+} elsif ($op eq 'modsubscription') {
+    my $subscriptionid = $query->param('subscriptionid');
+    my @irregular = $query->param('irregular');
+    my $irregular_count = @irregular;
+    for(my $i =0;$i<$irregular_count;$i++){
+      $irregularity .=$irregular[$i].",";
+      warn "irregular : $irregular[$i] string :$irregularity";
+    }
+    $irregularity =~ s/\,$//;
+
+    my $auser = $query->param('user');
+    my $librarian => $query->param('librarian'),
+    my $branchcode = $query->param('branchcode');
+    my $cost = $query->param('cost');
+    my $aqbooksellerid = $query->param('aqbooksellerid');
+    my $biblionumber = $query->param('biblionumber');
+    my $aqbudgetid = $query->param('aqbudgetid');
+    my $startdate = format_date_in_iso($query->param('startdate'));
+    my $firstacquidate = format_date_in_iso($query->param('firstacquidate'));    
+    my $periodicity = $query->param('periodicity');
+    my $dow = $query->param('dow');
+    my $sublength = $query->param('sublength');
+    my $subtype = $query->param('subtype');
+
+    if($subtype eq 'months'){
+        $monthlength = $sublength;
+    } elsif ($subtype eq 'weeks'){
+        $weeklength = $sublength;
+    } else {
+        $numberlength = $sublength;
+    }
+    my $numberpattern = $query->param('numbering_pattern');
+    my $add1 = $query->param('add1');
+    my $every1 = $query->param('every1');
+    my $whenmorethan1 = $query->param('whenmorethan1');
+    my $setto1 = $query->param('setto1');
+    my $lastvalue1 = $query->param('lastvalue1');
+    my $innerloop1 = $query->param('innerloop1');
+    my $add2 = $query->param('add2');
+    my $every2 = $query->param('every2');
+    my $whenmorethan2 = $query->param('whenmorethan2');
+    my $setto2 = $query->param('setto2');
+    my $lastvalue2 = $query->param('lastvalue2');
+    my $innerloop2 = $query->param('innerloop2');
+    my $add3 = $query->param('add3');
+    my $every3 = $query->param('every3');
+    my $whenmorethan3 = $query->param('whenmorethan3');
+    my $setto3 = $query->param('setto3');
+    my $lastvalue3 = $query->param('lastvalue3');
+    my $innerloop3 = $query->param('innerloop3');
+    my $numberingmethod = $query->param('numberingmethod');
+    my $status = 1;
+    my $callnumber = $query->param('callnumber');
     my $notes = $query->param('notes');
+    my $internalnotes = $query->param('internalnotes');
+    my $hemisphere = $query->param('hemisphere');
     my $letter = $query->param('letter');
-    my $subscriptionid = NewSubscription($auser,$aqbooksellerid,$cost,$aqbudgetid,$biblionumber,
-                    $startdate,$periodicity,$dow,$numberlength,$weeklength,$monthlength,
-                    $add1,$every1,$whenmorethan1,$setto1,$lastvalue1,
-                    $add2,$every2,$whenmorethan2,$setto2,$lastvalue2,
-                    $add3,$every3,$whenmorethan3,$setto3,$lastvalue3,
-                    $numberingmethod, $status, $notes, $letter
-                );
+    my $manualhistory = $query->param('manualhist');
+    my $enddate = $query->param('enddate');
+    my $histstartdate = format_date_in_iso($query->param('histstartdate'));
+    my $recievedlist = $query->param('recievedlist');
+    my $missinglist = $query->param('missinglist');
+    my $opacnote = $query->param('opacnote');
+    my $librariannote = $query->param('librariannote');
+    &ModSubscription(
+        $auser,           $branchcode,   $aqbooksellerid, $cost,
+        $aqbudgetid,      $startdate,    $periodicity,    $firstacquidate,
+        $dow,             $irregularity, $numberpattern,  $numberlength,
+        $weeklength,      $monthlength,  $add1,           $every1,
+        $whenmorethan1,   $setto1,       $lastvalue1,     $innerloop1,
+        $add2,            $every2,       $whenmorethan2,  $setto2,
+        $lastvalue2,      $innerloop2,   $add3,           $every3,
+        $whenmorethan3,   $setto3,       $lastvalue3,     $innerloop3,
+        $numberingmethod, $status,       $biblionumber,   $callnumber,
+        $notes,           $letter,       $hemisphere,     $manualhistory,$internalnotes,
+        $subscriptionid);
+
+    ModSubscriptionHistory ($subscriptionid,$histstartdate,$enddate,$recievedlist,$missinglist,$opacnote,$librariannote);
     print $query->redirect("/cgi-bin/koha/serials/subscription-detail.pl?subscriptionid=$subscriptionid");
 } else {
-    output_html_with_http_headers $query, $cookie, $template->output;
+
+        while (@subscription_types) {
+           my $sub_type = shift @subscription_types;
+           my %row = ( 'name' => $sub_type );
+           if ( $sub_on eq $sub_type ) {
+            $row{'selected'} = ' selected';
+           } else {
+            $row{'selected'} = '';
+           }
+           push( @sub_type_data, \%row );
+        }    
+    $template->param(subtype => \@sub_type_data,
+                weekarrayjs => $weekarrayjs,
+                weekno => $weekno,
+       );
+       output_html_with_http_headers $query, $cookie, $template->output;
 }