Fix for Bug 6872 - Can't set default SMS messaging options for patron category
[koha.git] / admin / branches.pl
index c6d0ea0..12c49e2 100755 (executable)
@@ -1,9 +1,5 @@
 #!/usr/bin/perl
 
-# Finlay working on this file from 26-03-2002
-# Reorganising this branches admin page.....
-
-
 # Copyright 2000-2002 Katipo Communications
 #
 # This file is part of Koha.
 # 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., 59 Temple Place,
-# Suite 330, Boston, MA  02111-1307 USA
+# 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.
+
+=head1 branches.pl
+
+ FIXME: individual fields in branch address need to be exported to templates,
+        in order to fix bug 180; need to notify translators
+ FIXME: looped html (e.g., list of checkboxes) need to be properly
+        TMPL_LOOP'ized; doing this properly will fix bug 130; need to
+        notify translators
+ FIXME: need to implement the branch categories stuff
+ 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
+
+ Finlay working on this file from 26-03-2002
+ Reorganising this branches admin page.....
+
+=cut
 
 use strict;
+use warnings;
 use CGI;
 use C4::Auth;
 use C4::Context;
 use C4::Output;
-use C4::Interface::CGI::Output;
-use HTML::Template;
+use C4::Koha;
+use C4::Branch;
 
 # Fixed variables
-my $linecolor1='#ffffcc';
-my $linecolor2='white';
-my $backgroundimage="/images/background-mem.gif";
-my $script_name="/cgi-bin/koha/admin/branches.pl";
-my $pagesize=20;
-
+my $script_name = "/cgi-bin/koha/admin/branches.pl";
 
-#######################################################################################
+################################################################################
 # Main loop....
-
-my $input = new CGI;
-my $branchcode=$input->param('branchcode');
-my $op = $input->param('op');
-
-my ($template, $borrowernumber, $cookie)
-    = get_template_and_user({template_name => "parameters/branches.tmpl",
-                            query => $input,
-                            type => "intranet",
-                            authnotrequired => 0,
-                            flagsrequired => {parameters => 1},
-                            debug => 1,
-                            });
-if ($op) {
-$template->param(script_name => $script_name,
-                                               $op              => 1); # we show only the TMPL_VAR names $op
-} else {
-$template->param(script_name => $script_name,
-                                               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");
-    editbranchform();
-
-} elsif ($op eq 'edit') {
-# if the user has pressed the "edit branch settings" button.
-    heading("Branches: Edit Branch");
-    $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");
-    } else {
-       setbranchinfo($params);
-       $template->param(else => 1);
-       default ("Branch record changed for branch: $params->{'branchname'}");
+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   => "admin/branches.tmpl",
+        query           => $input,
+        type            => "intranet",
+        authnotrequired => 0,
+        flagsrequired   => { parameters => 1},
+        debug           => 1,
     }
+);
+$template->param(
+     script_name => $script_name,
+     action      => $script_name,
+);
+$template->param( ($op || 'else') => 1 );
 
-} elsif ($op eq 'delete') {
-# if the user has pressed the "delete branch" button.
-    my $message = checkdatabasefor($branchcode);
-    if ($message) {
-       $template->param(else => 1);
-       default($message);
-    } else {
-       deleteconfirm($branchcode);
-        $template->param(delete_confirm => 1);
-       $template->param(branchcode => $branchcode);
-    }
+if ( $op eq 'add' ) {
 
-} elsif ($op eq 'delete_confirmed') {
-# actually delete branch and return to the main screen....
-    deletebranch($branchcode);
-    $template->param(else => 1);
-    default("The branch with code $branchcode has been deleted.");
+    # If the user has pressed the "add new branch" button.
+    $template->param( 'heading_branches_add_branch_p' => 1 );
+    editbranchform($branchcode,$template);
 
-} else {
-# if no operation has been set...
-    default();
 }
+elsif ( $op eq 'edit' ) {
 
-
-
-######################################################################################################
-#
-# html output functions....
-
-sub default {
-    my ($message) = @_;
-    heading("Branches");
-    $template->param(message => $message);
-    $template->param(action => $script_name);
-    branchinfotable();
-    
-    
+    # if the user has pressed the "edit branch settings" button.
+    $template->param( 'heading_branches_add_branch_p' => 0,
+                        'add' => 1, );
+    editbranchform($branchcode,$template);
 }
+elsif ( $op eq 'add_validate' ) {
 
-sub heading {
-    my ($head) = @_;
-    $template->param(head => $head);
+    # confirm settings change...
+    my $params = $input->Vars;
+    unless ( $params->{'branchcode'} && $params->{'branchname'} ) {
+        $template->param( else => 1 );
+        default("MESSAGE1",$template);
+    }
+    else {
+        my $mod_branch = 1;
+        if ($params->{add}) {
+            my ($existing) =
+                C4::Context->dbh->selectrow_array("SELECT count(*) FROM branches WHERE branchcode = ?", {}, $branchcode);
+            if ($existing > 0) {
+                $mod_branch = 0;
+                _branch_to_template($params, $template); # preserve most (FIXME) of user's input
+                $template->param( 'heading_branches_add_branch_p' => 1, 'add' => 1, 'ERROR1' => 1 );
+            }
+        }
+        if ($mod_branch) {
+            my $error = ModBranch($params); # FIXME: causes warnings to log on duplicate branchcode
+            # if error saving, stay on edit and rise error
+            if ($error) {
+                # copy input parameters back to form
+                # FIXME - doing this doesn't preserve any branch group selections, but good enough for now
+                editbranchform($branchcode,$template);
+                $template->param( 'heading_branches_add_branch_p' => 1, 'add' => 1, "ERROR$error" => 1 );
+            } else {
+                $template->param( else => 1);
+                default("MESSAGE2",$template);
+            }
+        }
+    }
 }
-
-sub editbranchform {
-# prepares the edit form...
-    my ($branchcode) = @_;
-    my $data;
-    if ($branchcode) {
-       $data = getbranchinfo($branchcode);
-       $data = $data->[0];
-       $template->param(branchcode => $data->{'branchcode'});
-        $template->param(branchname => $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'});
+elsif ( $op eq 'delete' ) {
+    # if the user has pressed the "delete branch" button.
+    
+    # check to see if the branchcode is being used in the database somewhere....
+    my $dbh = C4::Context->dbh;
+    my $sthitems     = $dbh->prepare("select count(*) from items where holdingbranch=? or homebranch=?");
+    my $sthborrowers = $dbh->prepare("select count(*) from borrowers where branchcode=?");
+    $sthitems->execute( $branchcode, $branchcode );
+    $sthborrowers->execute( $branchcode );
+    my ($totalitems)     = $sthitems->fetchrow_array;
+    my ($totalborrowers) = $sthborrowers->fetchrow_array;
+    if ($totalitems && !$totalborrowers) {
+        $template->param( else => 1 );
+        default("MESSAGE10", $template);
     }
-# make the checkboxs.....
-    my $catinfo = getcategoryinfo();
-    my $catcheckbox;
-    foreach my $cat (@$catinfo) {
-       my $checked = "";
-       my $tmp = $cat->{'categorycode'};
-       if (grep {/^$tmp$/} @{$data->{'categories'}}) {
-           $checked = "CHECKED";
-               }
-    $template->param(categoryname => $cat->{'categoryname'});
-    $template->param(categorycode => $cat->{'categorycode'});
-    $template->param(codedescription => $checked>$cat->{'codedescription'});    
+    elsif (!$totalitems && $totalborrowers){
+        $template->param( else => 1 );
+        default("MESSAGE11", $template);
+    }
+    elsif ($totalitems && $totalborrowers){
+        $template->param( else => 1 );
+        default("MESSAGE7", $template);
+    }
+    else {
+        $template->param( delete_confirm => 1 );
+        $template->param( branchname     => $branchname );
+        $template->param( branchcode     => $branchcode );
     }
-   
 }
+elsif ( $op eq 'delete_confirmed' ) {
 
-sub deleteconfirm {
-# message to print if the 
-    my ($branchcode) = @_;
+    # actually delete branch and return to the main screen....
+    DelBranch($branchcode);
+    $template->param( else => 1 );
+    default("MESSAGE3",$template);
 }
+elsif ( $op eq 'editcategory' ) {
 
+    # If the user has pressed the "add new category" or "modify" buttons.
+    $template->param( 'heading_branches_edit_category_p' => 1 );
+    editcatform($categorycode,$template);
+}
+elsif ( $op eq 'addcategory_validate' ) {
 
-sub branchinfotable {
-# makes the html for a table of branch info from reference to an array of hashs.
-
-    my ($branchcode) = @_;
-    my $branchinfo;
-    if ($branchcode) {
-       $branchinfo = getbranchinfo($branchcode);
-    } else {
-       $branchinfo = getbranchinfo();
+    $template->param( else => 1 );
+    # confirm settings change...
+    my $params = $input->Vars;
+    unless ( $params->{'categorycode'} && $params->{'categoryname'} ) {
+        default("MESSAGE4",$template);
     }
-    my $color;
-    my @loop_data =();
-    foreach my $branch (@$branchinfo) {
-       ($color eq $linecolor1) ? ($color=$linecolor2) : ($color=$linecolor1);
-       my $address = '';
-       $address .= $branch->{'branchaddress1'}          if ($branch->{'branchaddress1'});
-       $address .= '<br>'.$branch->{'branchaddress2'}   if ($branch->{'branchaddress2'});
-       $address .= '<br>'.$branch->{'branchaddress3'}   if ($branch->{'branchaddress3'});
-       $address .= '<br>ph: '.$branch->{'branchphone'}   if ($branch->{'branchphone'});
-       $address .= '<br>fax: '.$branch->{'branchfax'}    if ($branch->{'branchfax'});
-       $address .= '<br>email: '.$branch->{'branchemail'} if ($branch->{'branchemail'});
-       $address = '(nothing entered)' unless ($address);
-       my $categories = '';
-       foreach my $cat (@{$branch->{'categories'}}) {
-           my ($catinfo) = @{getcategoryinfo($cat)};
-           $categories .= $catinfo->{'categoryname'}."<br>";
+    elsif ($input->param('add')){
+       # doing an add must check the code is unique
+       if (CheckCategoryUnique($input->param('categorycode'))){
+           ModBranchCategoryInfo($params);
+        default("MESSAGE5",$template);
+       }
+       else {
+           default("MESSAGE9",$template);
        }
-       $categories = '(no categories set)' unless ($categories);
-       my @colors = ();
-       my @branch_name = ();
-       my @branch_code = ();
-       my @address = ();
-       my @categories = ();
-       my @value = ();
-       my @action =();
-       push(@colors,$color);
-       push(@branch_name,$branch->{'branchname'});
-       push(@branch_code,$branch->{'branchcode'});
-       push(@address,$address);
-       push(@categories,$categories);
-       push(@value,$branch->{'branchcode'});
-       push(@action,"/cgi-bin/koha/admin/branches.pl");
-       while (@colors and @branch_name and @branch_code and @address and @categories and @value and @action) {
-       my %row_data;
-       $row_data{color} = shift @colors;
-       $row_data{branch_name} = shift @branch_name;
-       $row_data{branch_code} = shift @branch_code;
-       $row_data{address} = shift @address;
-       $row_data{categories} = shift @categories;
-       $row_data{value} = shift @value;
-       $row_data{action} = shift @action;
-       push(@loop_data, \%row_data);
     }
-    
+    else {
+        ModBranchCategoryInfo($params);
+        default("MESSAGE5",$template);
     }
-    $template->param(branches => \@loop_data);
-
 }
+elsif ( $op eq 'delete_category' ) {
 
-sub branchcategoriestable {
-#Needs to be implemented...
-
-    my $categoryinfo = getcategoryinfo();
-my $color;
-    foreach my $cat (@$categoryinfo) {
-       ($color eq $linecolor1) ? ($color=$linecolor2) : ($color=$linecolor1);
-$template->param(color => $color);
-$template->param(categoryname => $cat->{'categoryname'});
-$template->param(categorycode => $cat->{'categorycode'});
-$template->param(codedescription => $cat->{'codedescription'});
+    # if the user has pressed the "delete branch" button.
+    if ( CheckBranchCategorycode($categorycode) ) {
+        $template->param( else => 1 );
+        default( 'MESSAGE8', $template );
+    } else {
+        $template->param( delete_category => 1 );
+        $template->param( categorycode    => $categorycode );
     }
 }
+elsif ( $op eq 'categorydelete_confirmed' ) {
 
-######################################################################################################
+    # actually delete branch and return to the main screen....
+    DelBranchCategory($categorycode);
+    $template->param( else => 1 );
+    default("MESSAGE6",$template);
+
+}
+else {
+    # if no operation has been set...
+    default("",$template);
+}
+
+################################################################################
 #
-# Database functions....
+# html output functions....
 
-sub getbranchinfo {
-# returns a reference to an array of hashes containing branches,
+sub default {
+    my $message       = shift || '';
+    my $innertemplate = shift or return;
+    $innertemplate->param($message => 1) if $message;
+    $innertemplate->param(
+        'heading_branches_p' => 1,
+    );
+    branchinfotable("",$innertemplate);
+}
+
+sub editbranchform {
+    my ($branchcode,$innertemplate) = @_;
+    # initiate the scrolling-list to select the printers
+    my $printers = GetPrinters();
+    my @printerloop;
+    my $data;
+    my $oldprinter = "";
 
-    my ($branchcode) = @_;
-    my $dbh = C4::Context->dbh;
-    my $query;
     if ($branchcode) {
-       my $bc = $dbh->quote($branchcode);
-       $query = "Select * from branches where branchcode = $bc";
+        $data = GetBranchInfo($branchcode);
+        $data = $data->[0];
+
+        # get the old printer of the branch
+        $oldprinter = $data->{'branchprinter'} || '';
+        _branch_to_template($data, $innertemplate);
     }
-    else {$query = "Select * from branches";}
-    my $sth = $dbh->prepare($query);
-    $sth->execute;
-    my @results;
-    while (my $data = $sth->fetchrow_hashref) { 
-       my $tmp = $data->{'branchcode'}; my $brc = $dbh->quote($tmp);
-       $query = "select categorycode from branchrelations where branchcode = $brc";
-       my $nsth = $dbh->prepare($query);
-       $nsth->execute;
-       my @cats = ();
-       while (my ($cat) = $nsth->fetchrow_array) {
-           push(@cats, $cat);
-       }
-       $nsth->finish;
-       $data->{'categories'} = \@cats;
-       push(@results, $data);
+
+    foreach my $thisprinter ( keys %$printers ) {
+        push @printerloop, {
+            value         => $thisprinter,
+            selected      => ( $oldprinter eq $printers->{$thisprinter} ),
+            branchprinter => $printers->{$thisprinter}->{'printqueue'},
+        };
     }
-    $sth->finish;
-    return \@results;
-}
 
-sub getcategoryinfo {
-# returns a reference to an array of hashes containing branches,
-    my ($catcode) = @_;
-    my $dbh = C4::Context->dbh;
-    my $query;
-    if ($catcode) {
-       my $cc = $dbh->quote($catcode);
-       $query = "select * from branchcategories where categorycode = $cc";
-    } else {
-       $query = "Select * from branchcategories";
+    $innertemplate->param( printerloop => \@printerloop );
+    # make the checkboxes.....
+    #
+    # We export a "categoryloop" array to the template, each element of which
+    # contains separate 'categoryname', 'categorycode', 'codedescription', and
+    # 'checked' fields. The $checked field is either empty or 1'
+
+    my $catinfo = GetBranchCategory();
+    my @categoryloop = ();
+    foreach my $cat (@$catinfo) {
+        my $checked;
+        my $tmp     = quotemeta( $cat->{'categorycode'} );
+        if ( grep { /^$tmp$/ } @{ $data->{'categories'} } ) {
+            $checked = 1;
+        }
+        push @categoryloop, {
+            categoryname    => $cat->{'categoryname'},
+            categorycode    => $cat->{'categorycode'},
+            categorytype    => $cat->{'categorytype'},
+            codedescription => $cat->{'codedescription'},
+            checked         => $checked,
+        };
     }
-    my $sth = $dbh->prepare($query);
-    $sth->execute;
-    my @results;
-    while (my $data = $sth->fetchrow_hashref) { 
-       push(@results, $data);
+    $innertemplate->param( categoryloop => \@categoryloop );
+
+    for my $obsolete ( 'categoryname', 'categorycode', 'codedescription' ) {
+        $innertemplate->param(
+            $obsolete => 'Your template is out of date (bug 130)' );
     }
-    $sth->finish;
-    return \@results;
 }
 
-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 $tmp;
-    $tmp = $data->{'branchcode'}; $query.= $dbh->quote($tmp).",";
-    $tmp = $data->{'branchname'}; $query.= $dbh->quote($tmp).",";
-    $tmp = $data->{'branchaddress1'}; $query.= $dbh->quote($tmp).",";
-    $tmp = $data->{'branchaddress2'}; $query.= $dbh->quote($tmp).",";
-    $tmp = $data->{'branchaddress3'}; $query.= $dbh->quote($tmp).",";
-    $tmp = $data->{'branchphone'}; $query.= $dbh->quote($tmp).",";
-    $tmp = $data->{'branchfax'}; $query.= $dbh->quote($tmp).",";
-    $tmp = $data->{'branchemail'}; $query.= $dbh->quote($tmp).")";
-    my $sth=$dbh->prepare($query);
-    $sth->execute;
-    $sth->finish;
-# sort out the categories....
-    my @checkedcats;
-    my $cats = getcategoryinfo();
-    foreach my $cat (@$cats) {
-       my $code = $cat->{'categorycode'};
-       if ($data->{$code}) {
-           push(@checkedcats, $code);
-       }
+sub editcatform {
+
+    # prepares the edit form...
+    my ($categorycode,$innertemplate) = @_;
+    # warn "cat : $categorycode";
+       my @cats;
+    my $data;
+       if ($categorycode) {
+        my $data = GetBranchCategory($categorycode);
+        $data = $data->[0];
+        $innertemplate->param(
+            categorycode    => $data->{'categorycode'},
+            categoryname    => $data->{'categoryname'},
+            codedescription => $data->{'codedescription'},
+               );
     }
-    my $branchcode = $data->{'branchcode'};
-    my $branch = getbranchinfo($branchcode);
-    $branch = $branch->[0];
-    my $branchcats = $branch->{'categories'};
-    my @addcats;
-    my @removecats;
-    foreach my $bcat (@$branchcats) {
-       unless (grep {/^$bcat$/} @checkedcats) {
-           push(@removecats, $bcat);
+       for my $ctype (GetCategoryTypes()) {
+               push @cats , { type => $ctype , selected => ($data->{'categorytype'} and $data->{'categorytype'} eq $ctype) };
        }
+    $innertemplate->param(categorytype => \@cats);
+}
+
+sub branchinfotable {
+
+# makes the html for a table of branch info from reference to an array of hashs.
+
+    my ($branchcode,$innertemplate) = @_;
+    my $branchinfo = $branchcode ? GetBranchInfo($branchcode) : GetBranchInfo();
+    my @loop_data = ();
+    foreach my $branch (@$branchinfo) {
+        #
+        # We export the following fields to the template. These are not
+        # pre-composed as a single "address" field because the template
+        # might (and should) escape what is exported here. (See bug 180)
+        #
+        # - branch_name     (Note: not "branchname")
+        # - branch_code     (Note: not "branchcode")
+        # - address         (containing a static error message)
+        # - branchaddress1 \
+        # - branchaddress2  |
+        # - branchaddress3  | comprising the old "address" field
+        # - branchzip       |
+        # - branchcity      |
+        # - branchcountry   |
+        # - branchphone     |
+        # - branchfax       |
+        # - branchemail    /
+        # - branchurl      /
+        # - address-empty-p (1 if no address information, 0 otherwise)
+        # - categories      (containing a static error message)
+        # - category_list   (loop containing "categoryname")
+        # - no-categories-p (1 if no categories set, 0 otherwise)
+        # - value
+        #
+        my %row = ();
+
+        # Handle address fields separately
+        my $address_empty_p = 1;
+        for my $field (
+            'branchaddress1', 'branchaddress2',
+            'branchaddress3', 'branchzip',
+            'branchcity', 'branchstate', 'branchcountry',
+            'branchphone', 'branchfax',
+            'branchemail', 'branchurl',
+            'branchip',       'branchprinter', 'branchnotes'
+          )
+        {
+            $row{$field} = $branch->{$field};
+            $address_empty_p = 0 if ( $branch->{$field} );
+        }
+        $row{'address-empty-p'} = $address_empty_p;
+
+        # Handle categories
+        my $no_categories_p = 1;
+        my @categories;
+        foreach my $cat ( @{ $branch->{'categories'} } ) {
+            my ($catinfo) = @{ GetBranchCategory($cat) };
+            push @categories, { 'categoryname' => $catinfo->{'categoryname'} };
+            $no_categories_p = 0;
+        }
+
+        $row{'category_list'}   = \@categories;
+        $row{'no-categories-p'} = $no_categories_p;
+        $row{'branch_name'} = $branch->{'branchname'};
+        $row{'branch_code'} = $branch->{'branchcode'};
+        $row{'value'}       = $branch->{'branchcode'};
+
+        push @loop_data, \%row;
     }
-    foreach my $ccat (@checkedcats){
-       unless (grep {/^$ccat$/} @$branchcats) {
-           push(@addcats, $ccat);
+    my @branchcategories = ();
+       for my $ctype ( GetCategoryTypes() ) {
+       my $catinfo = GetBranchCategories(undef,$ctype);
+       my @categories;
+               foreach my $cat (@$catinfo) {
+            push @categories, {
+                categoryname    => $cat->{'categoryname'},
+                categorycode    => $cat->{'categorycode'},
+                codedescription => $cat->{'codedescription'},
+                categorytype    => $cat->{'categorytype'},
+            };
+       }
+        push @branchcategories, { categorytype => $ctype , $ctype => 1 , catloop => ( @categories ? \@categories : undef) };
        }
-    }  
-    # 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('$branchcode', '$cat')";
-       my $sth = $dbh->prepare($query);
-       $sth->execute;
-       $sth->finish;
-    }
-    foreach my $cat (@removecats) {
-       my $query = "delete from branchrelations where branchcode='$branchcode' and categorycode='$cat'";
-       my $sth = $dbh->prepare($query);
-       $sth->execute;
-       $sth->finish;
-    }
-}
+    $innertemplate->param(
+        branches         => \@loop_data,
+        branchcategories => \@branchcategories
+    );
 
-sub deletebranch {
-# delete branch...
-    my ($branchcode) = @_;
-    my $query = "delete from branches where branchcode = '$branchcode'";
-    my $dbh = C4::Context->dbh;
-    my $sth=$dbh->prepare($query);
-    $sth->execute;
-    $sth->finish;
 }
 
-sub checkdatabasefor {
-# check to see if the branchcode is being used in the database somewhere....
-    my ($branchcode) = @_;
-    my $dbh = C4::Context->dbh;
-    my $sth=$dbh->prepare("select count(*) from items where holdingbranch='$branchcode' or homebranch='$branchcode'");
-    $sth->execute;
-    my ($total) = $sth->fetchrow_array;
-    $sth->finish;
-    my $message;
-    if ($total) {
-       $message = "Branch cannot be deleted because there are $total items using that branch.";
-    } 
-    return $message;
+sub _branch_to_template {
+    my ($data, $template) = @_;
+    $template->param( 
+         branchcode     => $data->{'branchcode'},
+         branch_name    => $data->{'branchname'},
+         branchaddress1 => $data->{'branchaddress1'},
+         branchaddress2 => $data->{'branchaddress2'},
+         branchaddress3 => $data->{'branchaddress3'},
+         branchzip      => $data->{'branchzip'},
+         branchcity     => $data->{'branchcity'},
+         branchstate    => $data->{'branchstate'},
+         branchcountry  => $data->{'branchcountry'},
+         branchphone    => $data->{'branchphone'},
+         branchfax      => $data->{'branchfax'},
+         branchemail    => $data->{'branchemail'},
+         branchurl      => $data->{'branchurl'},
+         branchip       => $data->{'branchip'},
+         branchnotes    => $data->{'branchnotes'}, 
+    );
 }
 
 output_html_with_http_headers $input, $cookie, $template->output;
+
+# Local Variables:
+# tab-width: 8
+# End: