# 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.....
# 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 $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) {
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.
$template->param(else => 1);
default($message);
} else {
+ $template->param(branchname => $branchname);
$template->param(delete_confirm => 1);
$template->param(branchcode => $branchcode);
}
# 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.
# 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...
# html output functions....
sub default {
- my ($message) = @_;
+ my ($message) = @_ || "";
heading("Branches");
- $template->param(message => $message);
+ $template->param('heading-branches-p' => 1);
+ $template->param("$message" => 1);
$template->param(action => $script_name);
branchinfotable();
}
} else {
$branchinfo = getbranchinfo();
}
- my $color;
+ my $toggle;
+ my $i=0;
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
# 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();
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);
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;
}
projects / koha-ffzg.git / blobdiff