Bug 17600: Standardize our EXPORT_OK

[koha-ffzg.git] / tools / holidays.pl

diff --git a/tools/holidays.pl b/tools/holidays.pl
index 2aee9ba..ddf45db 100755 (executable)
--- a/tools/holidays.pl
+++ b/tools/holidays.pl
@@ -16,19 +16,17 @@
 # along with Koha; if not, see <http://www.gnu.org/licenses>.
 
 #####Sets holiday periods for each branch. Datedues will be extended if branch is closed -TG
-use strict;
-use warnings;
+use Modern::Perl;
 
 use CGI qw ( -utf8 );
 
-use C4::Auth;
-use C4::Output;
+use C4::Auth qw( get_template_and_user );
+use C4::Output qw( output_html_with_http_headers );
 
-use C4::Branch; # GetBranches
 use C4::Calendar;
-use Koha::DateUtils;
+use Koha::DateUtils qw( dt_from_string output_pref );
 
-my $input = new CGI;
+my $input = CGI->new;
 
 my $dbh = C4::Context->dbh();
 # Get the template to use
@@ -36,9 +34,7 @@ my ($template, $loggedinuser, $cookie)
     = get_template_and_user({template_name => "tools/holidays.tt",
                              type => "intranet",
                              query => $input,
-                             authnotrequired => 0,
                              flagsrequired => {tools => 'edit_calendar'},
-                             debug => 1,
                            });
 
 # calendardate - date passed in url for human readability (syspref)
@@ -51,29 +47,6 @@ my $keydate = output_pref( { dt => $calendarinput_dt, dateonly => 1, dateformat
 $keydate =~ s/-/\//g;
 
 my $branch= $input->param('branch') || C4::Context->userenv->{'branch'};
-# Set all the branches.
-my $onlymine =
-  (      C4::Context->preference('IndependentBranches')
-      && C4::Context->userenv
-      && !C4::Context->IsSuperLibrarian()
-      && C4::Context->userenv->{branch} ? 1 : 0 );
-if ( $onlymine ) { 
-    $branch = C4::Context->userenv->{'branch'};
-}
-my $branches   = GetBranches($onlymine);
-my @branchloop;
-for my $thisbranch (
-    sort { $branches->{$a}->{branchname} cmp $branches->{$b}->{branchname} }
-    keys %{$branches} ) {
-    push @branchloop,
-      { value      => $thisbranch,
-        selected   => $thisbranch eq $branch,
-        branchname => $branches->{$thisbranch}->{'branchname'},
-      };
-}
-
-# branches calculated - put branch codes in a single string so they can be passed in a form
-my $branchcodes = join '|', keys %{$branches};
 
 # Get all the holidays
 
@@ -145,13 +118,11 @@ foreach my $yearMonthDay (keys %$single_holidays) {
 
 $template->param(
     WEEK_DAYS_LOOP           => \@week_days,
-    branchloop               => \@branchloop,
     HOLIDAYS_LOOP            => \@holidays,
     EXCEPTION_HOLIDAYS_LOOP  => \@exception_holidays,
     DAY_MONTH_HOLIDAYS_LOOP  => \@day_month_holidays,
     calendardate             => $calendardate,
     keydate                  => $keydate,
-    branchcodes              => $branchcodes,
     branch                   => $branch,
 );