#!/usr/bin/perl
+# NOTE: Use standard 8-space tabs for this file (indents are 4 spaces)
+
+#require '/u/acli/lib/cvs.pl';#DEBUG
+#open(DEBUG,'>/tmp/koha.debug');
+
+# 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
+#
+# 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.....
use strict;
use CGI;
+use C4::Auth;
+use C4::Context;
use C4::Output;
-use C4::Database;
+use C4::Interface::CGI::Output;
+use HTML::Template;
+use C4::Koha;
# Fixed variables
my $linecolor1='#ffffcc';
#######################################################################################
# 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');
-# header
-print $input->header;
-
-# start the page and read in includes
-print startpage();
-print startmenu('admin');
-
+my ($template, $borrowernumber, $cookie)
+ = get_template_and_user({template_name => "admin/branches.tmpl",
+ query => $input,
+ type => "intranet",
+ authnotrequired => 0,
+ flagsrequired => {parameters => 1, management => 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.
- print heading("Branches: Add Branch");
- print editbranchform();
+ # 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.
- print heading("Branches: Edit Branch");
- print editbranchform($branchcode);
-
+ # 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");
- } else {
- setbranchinfo($params);
- default ("Branch record changed for branch: $params->{'branchname'}");
- }
-
+ # confirm settings change...
+ my $params = $input->Vars;
+ unless ($params->{'branchcode'} && $params->{'branchname'}) {
+ $template->param(else => 1);
+ default ("MESSAGE1");
+ } else {
+ setbranchinfo($params);
+ $template->param(else => 1);
+ default ("MESSAGE2");
+ }
} elsif ($op eq 'delete') {
-# if the user has pressed the "delete branch" button.
- my $message = checkdatabasefor($branchcode);
- if ($message) {
- default($message);
- } else {
- print deleteconfirm($branchcode);
- }
-
+ # if the user has pressed the "delete branch" button.
+ my $message = checkdatabasefor($branchcode);
+ if ($message) {
+ $template->param(else => 1);
+ default($message);
+ } else {
+ $template->param(branchname => $branchname);
+ $template->param(delete_confirm => 1);
+ $template->param(branchcode => $branchcode);
+ }
} elsif ($op eq 'delete_confirmed') {
-# actually delete branch and return to the main screen....
- deletebranch($branchcode);
- default("The branch with code $branchcode has been deleted.");
+ # actually delete branch and return to the main screen....
+ deletebranch($branchcode);
+ $template->param(else => 1);
+ default("MESSAGE3");
+} 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);
+} elsif ($op eq 'addcategory_validate') {
+ # confirm settings change...
+ my $params = $input->Vars;
+ unless ($params->{'categorycode'} && $params->{'categoryname'}) {
+ $template->param(else => 1);
+ default ("MESSAGE4");
+ } else {
+ setcategoryinfo($params);
+ $template->param(else => 1);
+ default ("MESSAGE5");
+ }
+} elsif ($op eq 'delete_category') {
+ # if the user has pressed the "delete branch" button.
+ my $message = checkcategorycode($categorycode);
+ if ($message) {
+ $template->param(else => 1);
+ default($message);
+ } else {
+ $template->param(delete_category => 1);
+ $template->param(categorycode => $categorycode);
+ }
+} elsif ($op eq 'categorydelete_confirmed') {
+ # actually delete branch and return to the main screen....
+ deletecategory($categorycode);
+ $template->param(else => 1);
+ default("MESSAGE6");
} else {
-# if no operation has been set...
- default();
+ # if no operation has been set...
+ default();
}
-print endmenu('admin');
-print endpage();
######################################################################################################
#
# html output functions....
sub default {
- my ($message) = @_;
- print heading("Branches");
- print "<font color='red'>$message</font>";
- print "<form action='$script_name' method=post><input type='hidden' name='op' value='add'><input type=submit value='Add New Branch'></form>";
- print branchinfotable();
- print branchcategoriestable();
+ my ($message) = @_;
+ heading("Branches");
+ $template->param('heading-branches-p' => 1);
+ $template->param("$message" => 1);
+ $template->param(action => $script_name);
+ branchinfotable();
}
+# FIXME: this function should not exist; otherwise headings are untranslatable
sub heading {
- my ($head) = @_;
- return "<FONT SIZE=6><em>$head</em></FONT><br>";
+ my ($head) = @_;
+ $template->param(head => $head);
}
sub editbranchform {
-# prepares the edit form...
- my ($branchcode) = @_;
- my $data;
- if ($branchcode) {
- $data = getbranchinfo($branchcode);
- $data = $data->[0];
- }
-# make the checkboxs.....
+ # 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(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.....
+ #
+ # 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 '' or 'checked'
+ # (see bug 130)
+ #
my $catinfo = getcategoryinfo();
my $catcheckbox;
+# print DEBUG "catinfo=".cvs($catinfo)."\n";
+ my @categoryloop = ();
foreach my $cat (@$catinfo) {
my $checked = "";
- my $tmp = $cat->{'categorycode'};
+ my $tmp = quotemeta($cat->{'categorycode'});
if (grep {/^$tmp$/} @{$data->{'categories'}}) {
- $checked = "CHECKED";
+ $checked = "checked=\"checked\"";
+ }
+ push @categoryloop, {
+ categoryname => $cat->{'categoryname'},
+ categorycode => $cat->{'categorycode'},
+ codedescription => $cat->{'codedescription'},
+ checked => $checked,
+ };
+ }
+ $template->param(categoryloop => \@categoryloop);
+
+ # {{{ Leave this here until bug 130 is completely resolved in the templates
+ for my $obsolete ('categoryname', 'categorycode', 'codedescription') {
+ $template->param($obsolete => 'Your template is out of date (bug 130)');
}
- $catcheckbox .= <<EOF;
-<tr><td>$cat->{'categoryname'}</td>
-<td><INPUT TYPE="checkbox" NAME="$cat->{'categorycode'}" VALUE="1" $checked>$cat->{'codedescription'}</td></tr>
-EOF
+ # }}}
+}
+
+sub editcatform {
+ # prepares the edit form...
+ my ($categorycode) = @_;
+ warn "cat : $categorycode";
+ my $data;
+ if ($categorycode) {
+ $data = getcategoryinfo($categorycode);
+ $data = $data->[0];
+ $template->param(categorycode => $data->{'categorycode'});
+ $template->param(categoryname => $data->{'categoryname'});
+ $template->param(codedescription => $data->{'codedescription'});
}
- my $form = <<EOF;
-<form action='$script_name' name=Aform method=post>
-<input type=hidden name=op value='add_validate'>
-<table>
-<tr><td>Branch code</td><td><input type=text name=branchcode size=5 maxlength=5 value='$data->{'branchcode'}'></td></tr>
-<tr><td>Name</td><td><input type=text name=branchname size=40 maxlength=80 value='$data->{'branchname'}'> </td></tr>
-$catcheckbox
-<tr><td>Address</td><td><input type=text name=branchaddress1 value='$data->{'branchaddress1'}'></td></tr>
-<tr><td> </td><td><input type=text name=branchaddress2 value='$data->{'branchaddress2'}'></td></tr>
-<tr><td> </td><td><input type=text name=branchaddress3 value='$data->{'branchaddress3'}'></td></tr>
-<tr><td>Phone</td><td><input type=text name=branchphone value='$data->{'branchphone'}'></td></tr>
-<tr><td>Fax</td><td><input type=text name=branchfax value='$data->{'branchfax'}'></td></tr>
-<tr><td>E-mail</td><td><input type=text name=branchemail value='$data->{'branchemail'}'></td></tr>
-<tr><td> </td><td><input type=submit value='Submit'></td></tr>
-</table>
-</form>
-EOF
- return $form;
}
sub deleteconfirm {
-# message to print if the
+# message to print if the
my ($branchcode) = @_;
- my $output = <<EOF;
-Confirm delete:
-<form action='$script_name' method=post><input type='hidden' name='op' value='delete_confirmed'>
-<input type='hidden' name='branchcode' value=$branchcode>
-<input type=submit value=YES></form>
-<form action='$script_name' method=post><input type='hidden' name='op' value=''>
-<input type=submit value=NO></form>
-EOF
- return $output;
}
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();
- }
- my $table = <<EOF;
-<table border='1' cellpadding='5' cellspacing='0' width='550'>
-<tr> <th colspan='5' align='left' bgcolor='#99cc33' background=$backgroundimage>
-<font size='5'><b>Branches</b></font></th> </tr>
-<tr bgcolor='#889999'>
-<td width='175'><b>Name</b></td>
-<td width='25'><b>Code</b></td>
-<td width='175'><b>Address</b></td>
-<td width='175'><b>Categories</b></td>
-<td width='50'><b> </b></td>
-</tr>
-EOF
-
- my $color;
- 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>";
+ my ($branchcode) = @_;
+ my $branchinfo;
+ if ($branchcode) {
+ $branchinfo = getbranchinfo($branchcode);
+ } else {
+ $branchinfo = getbranchinfo();
}
- $categories = '(no categories set)' unless ($categories);
- $table .= <<EOF;
-<tr bgcolor='$color'>
- <td align='left' valign='top'>$branch->{'branchname'}</td>
- <td align='left' valign='top'>$branch->{'branchcode'}</td>
- <td align='left' valign='top'>$address</td>
- <td align='left' valign='top'>$categories</td>
- <td align='left' valign='top'>
-<form action='$script_name' method=post>
-<input type='hidden' name='op' value='edit'>
-<input type='hidden' name='branchcode' value='$branch->{'branchcode'}'>
-<input type=submit value=Edit>
-</form>
-<form action='$script_name' method=post>
-<input type='hidden' name='branchcode' value='$branch->{'branchcode'}'>
-<input type='hidden' name='op' value='delete'><input type=submit value=Delete>
-</form></td>
-</tr>
-EOF
- }
- $table .= "</table><br>";
- return $table;
+ my $toggle;
+ my $i;
+ my @loop_data =();
+ foreach my $branch (@$branchinfo) {
+ ($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
+ # might (and should) escape what is exported here. (See bug 180)
+ #
+ # - color
+ # - branch_name (Note: not "branchname")
+ # - branch_code (Note: not "branchcode")
+ # - address (containing a static error message)
+ # - branchaddress1 \
+ # - branchaddress2 |
+ # - branchaddress3 | comprising the old "address" field
+ # - branchphone |
+ # - branchfax |
+ # - branchemail /
+ # - 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
+ # - action
+ #
+ my %row = ();
+
+ # Handle address fields separately
+ my $address_empty_p = 1;
+ for my $field ('branchaddress1', 'branchaddress2', 'branchaddress3',
+ 'branchphone', 'branchfax', 'branchemail', 'branchip', 'branchprinter') {
+ $row{$field} = $branch->{$field};
+ if ( $branch->{$field} ) {
+ $address_empty_p = 0;
+ }
+ }
+ $row{'address-empty-p'} = $address_empty_p;
+ # {{{ Leave this here until bug 180 is completely resolved in templates
+ $row{'address'} = 'Your template is out of date (see bug 180)';
+ # }}}
+
+ # Handle categories
+ my $no_categories_p = 1;
+ my @categories = '';
+ foreach my $cat (@{$branch->{'categories'}}) {
+ my ($catinfo) = @{getcategoryinfo($cat)};
+ push @categories, {'categoryname' => $catinfo->{'categoryname'}};
+ $no_categories_p = 0;
+ }
+ # {{{ Leave this here until bug 180 is completely resolved in templates
+ $row{'categories'} = 'Your template is out of date (see bug 180)';
+ # }}}
+ $row{'category_list'} = \@categories;
+ $row{'no-categories-p'} = $no_categories_p;
+
+ # Handle all other fields
+ $row{'branch_name'} = $branch->{'branchname'};
+ $row{'branch_code'} = $branch->{'branchcode'};
+ $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);
+
}
+# FIXME logic seems wrong
sub branchcategoriestable {
#Needs to be implemented...
my $categoryinfo = getcategoryinfo();
- my $table = <<EOF;
-<table border='1' cellpadding='5' cellspacing='0'>
-<tr> <th colspan='5' align='left' bgcolor='#99cc33' background=$backgroundimage>
-<font size='5'><b>Branches Categories</b></font></th> </tr>
-<tr bgcolor='#889999'>
-<td width='175'><b>Name</b></td>
-<td width='25'><b>Code</b></td>
-<td width='200'><b>Description</b></td>
-</tr>
-EOF
-my $color;
+ my $color;
foreach my $cat (@$categoryinfo) {
($color eq $linecolor1) ? ($color=$linecolor2) : ($color=$linecolor1);
- $table .= <<EOF;
-<tr bgcolor='$color'>
- <td align='left' valign='top'>$cat->{'categoryname'}</td>
- <td align='left' valign='top'>$cat->{'categorycode'}</td>
- <td align='left' valign='top'>$cat->{'codedescription'}</td>
-</tr>
-EOF
+ $template->param(color => $color);
+ $template->param(categoryname => $cat->{'categoryname'});
+ $template->param(categorycode => $cat->{'categorycode'});
+ $template->param(codedescription => $cat->{'codedescription'});
}
- $table .= "</table>";
- return $table;
}
######################################################################################################
# returns a reference to an array of hashes containing branches,
my ($branchcode) = @_;
- my $dbh = &C4Connect;
- my $query;
+ my $dbh = C4::Context->dbh;
+ my $sth;
if ($branchcode) {
- my $bc = $dbh->quote($branchcode);
- $query = "Select * from branches where branchcode = $bc";
+ $sth = $dbh->prepare("Select * from branches where branchcode = ? order by branchcode");
+ $sth->execute($branchcode);
+ } else {
+ $sth = $dbh->prepare("Select * from branches order by branchcode");
+ $sth->execute();
}
- 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;
+ while (my $data = $sth->fetchrow_hashref) {
+ my $nsth = $dbh->prepare("select categorycode from branchrelations where branchcode = ?");
+ $nsth->execute($data->{'branchcode'});;
my @cats = ();
while (my ($cat) = $nsth->fetchrow_array) {
push(@cats, $cat);
push(@results, $data);
}
$sth->finish;
- $dbh->disconnect;
return \@results;
}
+# FIXME This doesn't belong here; it should be moved into a module
sub getcategoryinfo {
# returns a reference to an array of hashes containing branches,
- my ($catcode) = @_;
- my $dbh = &C4Connect;
- my $query;
- if ($catcode) {
- my $cc = $dbh->quote($catcode);
- $query = "select * from branchcategories where categorycode = $cc";
- } else {
- $query = "Select * from branchcategories";
- }
- my $sth = $dbh->prepare($query);
- $sth->execute;
- my @results;
- while (my $data = $sth->fetchrow_hashref) {
- push(@results, $data);
- }
- $sth->finish;
- $dbh->disconnect;
- return \@results;
+ my ($catcode) = @_;
+ my $dbh = C4::Context->dbh;
+ my $sth;
+ # print DEBUG "getcategoryinfo: entry: catcode=".cvs($catcode)."\n";
+ if ($catcode) {
+ $sth = $dbh->prepare("select * from branchcategories where categorycode = ?");
+ $sth->execute($catcode);
+ } else {
+ $sth = $dbh->prepare("Select * from branchcategories");
+ $sth->execute();
+ }
+ my @results;
+ while (my $data = $sth->fetchrow_hashref) {
+ push(@results, $data);
+ }
+ $sth->finish;
+ # print DEBUG "getcategoryinfo: exit: returning ".cvs(\@results)."\n";
+ return \@results;
}
+# FIXME This doesn't belong here; it should be moved into a module
sub setbranchinfo {
# sets the data from the editbranch form, and writes to the database...
- my ($data) = @_;
- my $dbh=&C4Connect;
- 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;
- $dbh->disconnect;
-# sort out the categories....
- my @checkedcats;
- my $cats = getcategoryinfo();
- foreach my $cat (@$cats) {
- my $code = $cat->{'categorycode'};
- if ($data->{$code}) {
- push(@checkedcats, $code);
+ my ($data) = @_;
+ my $dbh = C4::Context->dbh;
+ 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->{'branchip'},$data->{'branchprinter'});
+
+ $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);
+ }
}
- }
- 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);
+ my $branchcode =uc($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);
+ }
}
- }
- foreach my $ccat (@checkedcats){
- unless (grep {/^$ccat$/} @$branchcats) {
- push(@addcats, $ccat);
+ foreach my $ccat (@checkedcats){
+ unless (grep {/^$ccat$/} @$branchcats) {
+ push(@addcats, $ccat);
+ }
+ }
+ foreach my $cat (@addcats) {
+ my $sth = $dbh->prepare("insert into branchrelations (branchcode, categorycode) values(?, ?)");
+ $sth->execute($branchcode, $cat);
+ $sth->finish;
+ }
+ foreach my $cat (@removecats) {
+ my $sth = $dbh->prepare("delete from branchrelations where branchcode=? and categorycode=?");
+ $sth->execute($branchcode, $cat);
+ $sth->finish;
}
- }
- my $dbh=&C4Connect;
- 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;
- }
- $dbh->disconnect;
}
sub deletebranch {
# delete branch...
my ($branchcode) = @_;
- my $query = "delete from branches where branchcode = '$branchcode'";
- my $dbh=&C4Connect;
- my $sth=$dbh->prepare($query);
- $sth->execute;
+ my $dbh = C4::Context->dbh;
+ my $sth=$dbh->prepare("delete from branches where branchcode = ?");
+ $sth->execute($branchcode);
+ $sth->finish;
+}
+
+sub setcategoryinfo {
+# sets the data from the editbranch form, and writes to the database...
+ my ($data) = @_;
+ my $dbh = C4::Context->dbh;
+ my $sth=$dbh->prepare("replace branchcategories (categorycode,categoryname,codedescription) values (?,?,?)");
+ $sth->execute(uc($data->{'categorycode'}), $data->{'categoryname'},$data->{'codedescription'});
+
+ $sth->finish;
+}
+sub deletecategory {
+# delete branch...
+ my ($categorycode) = @_;
+ my $dbh = C4::Context->dbh;
+ my $sth=$dbh->prepare("delete from branchcategories where categorycode = ?");
+ $sth->execute($categorycode);
$sth->finish;
- $dbh->disconnect;
}
sub checkdatabasefor {
# check to see if the branchcode is being used in the database somewhere....
my ($branchcode) = @_;
- my $dbh = &C4Connect;
- my $sth=$dbh->prepare("select count(*) from items where holdingbranch='$branchcode' or homebranch='$branchcode'");
- $sth->execute;
+ my $dbh = C4::Context->dbh;
+ my $sth=$dbh->prepare("select count(*) from items where holdingbranch=? or homebranch=?");
+ $sth->execute($branchcode, $branchcode);
my ($total) = $sth->fetchrow_array;
$sth->finish;
- $dbh->disconnect;
my $message;
if ($total) {
- $message = "Branch cannot be deleted because there are $total items using that branch.";
- }
+ # FIXME: need to be replaced by an exported boolean parameter
+ $message = "MESSAGE7";
+ }
return $message;
}
-
+sub checkcategorycode {
+# check to see if the branchcode is being used in the database somewhere....
+ my ($categorycode) = @_;
+ my $dbh = C4::Context->dbh;
+ my $sth=$dbh->prepare("select count(*) from branchrelations where categorycode=?");
+ $sth->execute($categorycode);
+ my ($total) = $sth->fetchrow_array;
+ $sth->finish;
+ my $message;
+ if ($total) {
+ # FIXME: need to be replaced by an exported boolean parameter
+ $message = "Category cannot be deleted because there are $total branches using that category.";
+ }
+ return $message;
+}
+$template->param(intranetcolorstylesheet => C4::Context->preference("intranetcolorstylesheet"),
+ intranetstylesheet => C4::Context->preference("intranetstylesheet"),
+ IntranetNav => C4::Context->preference("IntranetNav"),
+ );
+output_html_with_http_headers $input, $cookie, $template->output;
+
+# Local Variables:
+# tab-width: 8
+# End:
projects / koha_gimpoz / blobdiff