bug fixing :
[koha-ffzg.git] / admin / branches.pl
index 3ee2c65..4e730e8 100755 (executable)
 #        TMPL_LOOP'ized; doing this properly will fix bug 130; need to
 #        notify translators
 # FIXME: need to implement the branch categories stuff
-# FIXME: heading() need to be moved to templates, need to notify translators
 # FIXME: there are too many TMPL_IF's; the proper way to do it is to have
 #        separate templates for each individual action; need to notify
 #        translators
 # FIXME: there are lots of error messages exported to the template; a lot
 #        of these should be converted into exported booleans / counters etc
 #        so that the error messages can be localized; need to notify translators
+#
+# NOTE:  heading() should now be called like this:
+#        1. Use heading() as before
+#        2. $template->param('heading-LISPISHIZED-HEADING-p' => 1);
+#        3. $template->param('use-heading-flags-p' => 1);
+#        This ensures that both converted and unconverted templates work
 
 # Finlay working on this file from 26-03-2002
 # Reorganising this branches admin page.....
@@ -46,6 +51,7 @@ use C4::Context;
 use C4::Output;
 use C4::Interface::CGI::Output;
 use HTML::Template;
+use C4::Koha;
 
 # Fixed variables
 my $linecolor1='#ffffcc';
@@ -59,15 +65,16 @@ my $pagesize=20;
 # Main loop....
 my $input = new CGI;
 my $branchcode=$input->param('branchcode');
+my $branchname=$input->param('branchname');
 my $categorycode = $input->param('categorycode');
 my $op = $input->param('op');
 
 my ($template, $borrowernumber, $cookie)
-    = get_template_and_user({template_name => "parameters/branches.tmpl",
+    = get_template_and_user({template_name => "admin/branches.tmpl",
                             query => $input,
                             type => "intranet",
                             authnotrequired => 0,
-                            flagsrequired => {parameters => 1},
+                            flagsrequired => {parameters => 1, management => 1},
                             debug => 1,
                             });
 if ($op) {
@@ -78,26 +85,28 @@ if ($op) {
                                else        => 1); # we show only the TMPL_VAR names $op
 }
 $template->param(action => $script_name);
-
 if ($op eq 'add') {
        # If the user has pressed the "add new branch" button.
        heading("Branches: Add Branch");
+       $template->param('heading-branches-add-branch-p' => 1);
        editbranchform();
 
 } elsif ($op eq 'edit') {
        # if the user has pressed the "edit branch settings" button.
        heading("Branches: Edit Branch");
+       $template->param('heading-branches-edit-branch-p' => 1);
        $template->param(add => 1);
        editbranchform($branchcode);
 } elsif ($op eq 'add_validate') {
        # confirm settings change...
        my $params = $input->Vars;
        unless ($params->{'branchcode'} && $params->{'branchname'}) {
-               default ("Cannot change branch record: You must specify a Branchname and a Branchcode");
+               $template->param(else => 1);
+               default ("MESSAGE1");
        } else {
                setbranchinfo($params);
                $template->param(else => 1);
-               default ("Branch record changed for branch: $params->{'branchname'}");
+               default ("MESSAGE2");
        }
 } elsif ($op eq 'delete') {
        # if the user has pressed the "delete branch" button.
@@ -106,6 +115,7 @@ if ($op eq 'add') {
                $template->param(else => 1);
                default($message);
        } else {
+               $template->param(branchname => $branchname);
                $template->param(delete_confirm => 1);
                $template->param(branchcode => $branchcode);
        }
@@ -113,20 +123,21 @@ if ($op eq 'add') {
        # actually delete branch and return to the main screen....
        deletebranch($branchcode);
        $template->param(else => 1);
-       default("The branch with code $branchcode has been deleted.");
+       default("MESSAGE3");
 } elsif ($op eq 'editcategory') {
-       # If the user has pressed the "add new category" button.
-       heading("Branches: Edit Category");
+       # If the user has pressed the "add new category" or "modify" buttons.
+       $template->param('heading-branches-edit-category-p' => 1);
        editcatform($categorycode);
 } elsif ($op eq 'addcategory_validate') {
        # confirm settings change...
        my $params = $input->Vars;
        unless ($params->{'categorycode'} && $params->{'categoryname'}) {
-               default ("Cannot change branch record: You must specify a Branchname and a Branchcode");
+               $template->param(else => 1);
+               default ("MESSAGE4");
        } else {
                setcategoryinfo($params);
                $template->param(else => 1);
-               default ("Category record changed for category $params->{'categoryname'}");
+               default ("MESSAGE5");
        }
 } elsif ($op eq 'delete_category') {
        # if the user has pressed the "delete branch" button.
@@ -142,7 +153,7 @@ if ($op eq 'add') {
        # actually delete branch and return to the main screen....
        deletecategory($categorycode);
        $template->param(else => 1);
-       default("The category with code $categorycode has been deleted.");
+       default("MESSAGE6");
 
 } else {
        # if no operation has been set...
@@ -158,7 +169,8 @@ if ($op eq 'add') {
 sub default {
        my ($message) = @_;
        heading("Branches");
-       $template->param(message => $message);
+       $template->param('heading-branches-p' => 1);
+       $template->param("$message" => 1);
        $template->param(action => $script_name);
        branchinfotable();
 }
@@ -171,20 +183,52 @@ sub heading {
 
 sub editbranchform {
        # prepares the edit form...
+
+# initiate the scrolling-list to select the printers
+       my %env;
+       my $printers=getprinters(\%env);
+       my @printerloop;
+       my $printercount=0;
+       my $oldprinter;
+       my $CGIprinter;
        my ($branchcode) = @_;
        my $data;
+       
        if ($branchcode) {
                $data = getbranchinfo($branchcode);
                $data = $data->[0];
+       # get the old printer of the branch
+               $oldprinter = $data->{'branchprinter'}; 
+#      printer loop
+               foreach my $thisprinter (keys %$printers) {
+                       my $selected = 1 if $oldprinter eq $printers->{$thisprinter}->{'printqueue'};
+                       my %row =(value => $thisprinter,
+                                       selected => $selected,
+                                       branchprinter => $printers->{$thisprinter}->{'printqueue'},
+                       );
+               push @printerloop, \%row;
+               }
+               
+               $template->param(printerloop => \@printerloop );
                $template->param(branchcode => $data->{'branchcode'});
-               $template->param(branchname => $data->{'branchname'});
+               $template->param(branch_name => $data->{'branchname'});
                $template->param(branchaddress1 => $data->{'branchaddress1'});
                $template->param(branchaddress2 => $data->{'branchaddress2'});
                $template->param(branchaddress3 => $data->{'branchaddress3'});
                $template->param(branchphone => $data->{'branchphone'});
                $template->param(branchfax => $data->{'branchfax'});
                $template->param(branchemail => $data->{'branchemail'});
-    }
+               $template->param(branchip => $data->{'branchip'});
+       }
+       else { #case of an add branch select printer
+               foreach my $thisprinter (keys %$printers) {
+                       my %row =(value => $thisprinter,
+                               branchprinter => $printers->{$thisprinter}->{'printqueue'},
+                       );
+               push @printerloop, \%row;
+               }
+       $template->param(printerloop => \@printerloop );        
+       }
 
     # make the checkboxs.....
     #
@@ -201,7 +245,7 @@ sub editbranchform {
        my $checked = "";
        my $tmp = quotemeta($cat->{'categorycode'});
        if (grep {/^$tmp$/} @{$data->{'categories'}}) {
-               $checked = "CHECKED";
+               $checked = "checked=\"checked\"";
        }
        push @categoryloop, {
                categoryname    => $cat->{'categoryname'},
@@ -249,10 +293,11 @@ sub branchinfotable {
        } else {
                $branchinfo = getbranchinfo();
        }
-       my $color;
+       my $toggle;
+       my $i;
        my @loop_data =();
        foreach my $branch (@$branchinfo) {
-               ($color eq $linecolor1) ? ($color=$linecolor2) : ($color=$linecolor1);
+               ($i % 2) ? ($toggle = 1) : ($toggle = 0);
                #
                # We export the following fields to the template. These are not
                # pre-composed as a single "address" field because the template
@@ -280,7 +325,7 @@ sub branchinfotable {
                # Handle address fields separately
                my $address_empty_p = 1;
                for my $field ('branchaddress1', 'branchaddress2', 'branchaddress3',
-                       'branchphone', 'branchfax', 'branchemail') {
+                       'branchphone', 'branchfax', 'branchemail', 'branchip', 'branchprinter') {
                        $row{$field} = $branch->{$field};
                        if ( $branch->{$field} ) {
                                $address_empty_p = 0;
@@ -308,21 +353,27 @@ sub branchinfotable {
                # Handle all other fields
                $row{'branch_name'} = $branch->{'branchname'};
                $row{'branch_code'} = $branch->{'branchcode'};
-               $row{'color'} = $color;
+               $row{'toggle'} = $toggle;
                $row{'value'} = $branch->{'branchcode'};
                $row{'action'} = '/cgi-bin/koha/admin/branches.pl';
 
                push @loop_data, { %row };
+               $i++;
        }
        my @branchcategories =();
        my $catinfo = getcategoryinfo();
+       my $toggle;
+       my $i = 0;
        foreach my $cat (@$catinfo) {
+               ($i % 2) ? ($toggle = 1) : ($toggle = 0);
                push @branchcategories, {
+                       toggle => $toggle,
                        categoryname    => $cat->{'categoryname'},
                        categorycode    => $cat->{'categorycode'},
                        codedescription => $cat->{'codedescription'},
                };
-       }
+               $i++;
+       } 
 
        $template->param(branches => \@loop_data,
                                                        branchcategories => \@branchcategories);
@@ -353,19 +404,17 @@ sub getbranchinfo {
 
     my ($branchcode) = @_;
     my $dbh = C4::Context->dbh;
-    my ($query, @query_args);
+    my $sth;
     if ($branchcode) {
-       $query = "Select * from branches where branchcode = ?";
-       @query_args = ($branchcode);
+               $sth = $dbh->prepare("Select * from branches where branchcode = ? order by branchcode");
+               $sth->execute($branchcode);
     } else {
-       $query = "Select * from branches";
+               $sth = $dbh->prepare("Select * from branches order by branchcode");
+               $sth->execute();
     }
-    my $sth = $dbh->prepare($query);
-    $sth->execute(@query_args);
     my @results;
     while (my $data = $sth->fetchrow_hashref) {
-       $query = "select categorycode from branchrelations where branchcode = ?";
-       my $nsth = $dbh->prepare($query);
+       my $nsth = $dbh->prepare("select categorycode from branchrelations where branchcode = ?");
        $nsth->execute($data->{'branchcode'});;
        my @cats = ();
        while (my ($cat) = $nsth->fetchrow_array) {
@@ -384,17 +433,15 @@ sub getcategoryinfo {
 # returns a reference to an array of hashes containing branches,
        my ($catcode) = @_;
        my $dbh = C4::Context->dbh;
-       my ($query, @query_args);
+       my $sth;
        #    print DEBUG "getcategoryinfo: entry: catcode=".cvs($catcode)."\n";
        if ($catcode) {
-               $query = "select * from branchcategories where categorycode = ?";
-               @query_args = ($catcode);
+               $sth = $dbh->prepare("select * from branchcategories where categorycode = ?");
+               $sth->execute($catcode);
        } else {
-               $query = "Select * from branchcategories";
+               $sth = $dbh->prepare("Select * from branchcategories");
+               $sth->execute();
        }
-       #    print DEBUG "getcategoryinfo: query=".cvs($query)."\n";
-       my $sth = $dbh->prepare($query);
-       $sth->execute(@query_args);
        my @results;
        while (my $data = $sth->fetchrow_hashref) {
                push(@results, $data);
@@ -409,12 +456,11 @@ sub setbranchinfo {
 # sets the data from the editbranch form, and writes to the database...
        my ($data) = @_;
        my $dbh = C4::Context->dbh;
-       my $query = "replace branches (branchcode,branchname,branchaddress1,branchaddress2,branchaddress3,branchphone,branchfax,branchemail) values (?,?,?,?,?,?,?,?)";
-       my $sth=$dbh->prepare($query);
+       my $sth=$dbh->prepare("replace branches (branchcode,branchname,branchaddress1,branchaddress2,branchaddress3,branchphone,branchfax,branchemail,branchip,branchprinter) values (?,?,?,?,?,?,?,?,?,?)");
        $sth->execute(uc($data->{'branchcode'}), $data->{'branchname'},
                $data->{'branchaddress1'}, $data->{'branchaddress2'},
                $data->{'branchaddress3'}, $data->{'branchphone'},
-               $data->{'branchfax'}, $data->{'branchemail'});
+               $data->{'branchfax'}, $data->{'branchemail'}, $data->{'branchip'},$data->{'branchprinter'});
 
        $sth->finish;
        # sort out the categories....
@@ -442,17 +488,13 @@ sub setbranchinfo {
                        push(@addcats, $ccat);
                }
        }
-       # FIXME - There's already a $dbh in this scope.
-       my $dbh = C4::Context->dbh;
        foreach my $cat (@addcats) {
-               my $query = "insert into branchrelations (branchcode, categorycode) values(?, ?)";
-               my $sth = $dbh->prepare($query);
+               my $sth = $dbh->prepare("insert into branchrelations (branchcode, categorycode) values(?, ?)");
                $sth->execute($branchcode, $cat);
                $sth->finish;
        }
        foreach my $cat (@removecats) {
-               my $query = "delete from branchrelations where branchcode=? and categorycode=?";
-               my $sth = $dbh->prepare($query);
+               my $sth = $dbh->prepare("delete from branchrelations where branchcode=? and categorycode=?");
                $sth->execute($branchcode, $cat);
                $sth->finish;
        }
@@ -461,9 +503,8 @@ sub setbranchinfo {
 sub deletebranch {
 # delete branch...
     my ($branchcode) = @_;
-    my $query = "delete from branches where branchcode = ?";
     my $dbh = C4::Context->dbh;
-    my $sth=$dbh->prepare($query);
+    my $sth=$dbh->prepare("delete from branches where branchcode = ?");
     $sth->execute($branchcode);
     $sth->finish;
 }
@@ -472,8 +513,7 @@ sub setcategoryinfo {
 # sets the data from the editbranch form, and writes to the database...
        my ($data) = @_;
        my $dbh = C4::Context->dbh;
-       my $query = "replace branchcategories (categorycode,categoryname,codedescription) values (?,?,?)";
-       my $sth=$dbh->prepare($query);
+       my $sth=$dbh->prepare("replace branchcategories (categorycode,categoryname,codedescription) values (?,?,?)");
        $sth->execute(uc($data->{'categorycode'}), $data->{'categoryname'},$data->{'codedescription'});
 
        $sth->finish;
@@ -481,9 +521,8 @@ sub setcategoryinfo {
 sub deletecategory {
 # delete branch...
     my ($categorycode) = @_;
-    my $query = "delete from branchcategories where categorycode = ?";
     my $dbh = C4::Context->dbh;
-    my $sth=$dbh->prepare($query);
+    my $sth=$dbh->prepare("delete from branchcategories where categorycode = ?");
     $sth->execute($categorycode);
     $sth->finish;
 }
@@ -499,7 +538,7 @@ sub checkdatabasefor {
     my $message;
     if ($total) {
        # FIXME: need to be replaced by an exported boolean parameter
-       $message = "Branch cannot be deleted because there are $total items using that branch.";
+       $message = "MESSAGE7";
     }
     return $message;
 }