From e612d9fa47d3dc9c2db258184dceb65d52bde0b0 Mon Sep 17 00:00:00 2001 From: Aleisha Amohia Date: Wed, 19 Dec 2018 20:32:32 +0000 Subject: [PATCH] Bug 20000: Use Modern::Perl in installer scripts To test: Check that we are using Modern::Perl when strict *and* warnings were used before, and commented lines and FIXMEs have been removed from the rest, out of files listed below. data/mysql/backfill_statistics.pl data/mysql/labels_upgrade.pl data/mysql/patroncards_upgrade.pl data/mysql/update22to30.pl data/mysql/updatedatabase.pl externalmodules.pl html-template-to-template-toolkit.pl install.pl Sponsored-by: Catalyst IT Signed-off-by: Maryse Simard --- installer/data/mysql/backfill_statistics.pl | 3 +-- installer/data/mysql/labels_upgrade.pl | 3 --- installer/data/mysql/patroncards_upgrade.pl | 3 --- installer/data/mysql/update22to30.pl | 1 - installer/data/mysql/updatedatabase.pl | 3 +-- installer/externalmodules.pl | 5 ++--- installer/html-template-to-template-toolkit.pl | 3 +-- installer/install.pl | 3 +-- 8 files changed, 6 insertions(+), 18 deletions(-) diff --git a/installer/data/mysql/backfill_statistics.pl b/installer/data/mysql/backfill_statistics.pl index d03b461..488255b 100755 --- a/installer/data/mysql/backfill_statistics.pl +++ b/installer/data/mysql/backfill_statistics.pl @@ -3,8 +3,7 @@ # Part of the Koha Library Software www.koha-community.org # Licensed under the GPL. -use strict; -use warnings; +use Modern::Perl; # CPAN modules use DBI; diff --git a/installer/data/mysql/labels_upgrade.pl b/installer/data/mysql/labels_upgrade.pl index 158a4ec..859a7d9 100755 --- a/installer/data/mysql/labels_upgrade.pl +++ b/installer/data/mysql/labels_upgrade.pl @@ -17,9 +17,6 @@ # You should have received a copy of the GNU General Public License # along with Koha; if not, see . -#use strict; -#use warnings; FIXME - Bug 2505 - use C4::Context; my $sth = C4::Context->dbh; diff --git a/installer/data/mysql/patroncards_upgrade.pl b/installer/data/mysql/patroncards_upgrade.pl index b47049b..fde702e 100755 --- a/installer/data/mysql/patroncards_upgrade.pl +++ b/installer/data/mysql/patroncards_upgrade.pl @@ -17,9 +17,6 @@ # You should have received a copy of the GNU General Public License # along with Koha; if not, see . -#use strict; -#use warnings; FIXME - Bug 2505 - use C4::Context; my $sth = C4::Context->dbh; diff --git a/installer/data/mysql/update22to30.pl b/installer/data/mysql/update22to30.pl index f10ace9..d38f080 100755 --- a/installer/data/mysql/update22to30.pl +++ b/installer/data/mysql/update22to30.pl @@ -12,7 +12,6 @@ # NOTE: If you do something more than once in here, make it table driven. use strict; -#use warnings; FIXME - Bug 2505 # CPAN modules use DBI; diff --git a/installer/data/mysql/updatedatabase.pl b/installer/data/mysql/updatedatabase.pl index 92c8fc1..a4e4339 100755 --- a/installer/data/mysql/updatedatabase.pl +++ b/installer/data/mysql/updatedatabase.pl @@ -27,8 +27,7 @@ # NOTE: Please keep the version in kohaversion.pl up-to-date! -use strict; -use warnings; +use Modern::Perl; use feature 'say'; diff --git a/installer/externalmodules.pl b/installer/externalmodules.pl index 42eb4b7..0d338ab 100755 --- a/installer/externalmodules.pl +++ b/installer/externalmodules.pl @@ -4,10 +4,9 @@ # It provides you not only the list of modules BUT ALSO the files that uses those modules. # utf8 or warnings or other lib use are not taken into account at the moment. - -use strict; -use warnings; +use Modern::Perl; use C4::Context; + my $dir=C4::Context->config('intranetdir'); qx(grep -r "^ *use" $dir | grep -v "C4\|strict\|vars" >/tmp/modulesKoha.log); $dir=C4::Context->config('opacdir'); diff --git a/installer/html-template-to-template-toolkit.pl b/installer/html-template-to-template-toolkit.pl index 9905abe..60cd951 100755 --- a/installer/html-template-to-template-toolkit.pl +++ b/installer/html-template-to-template-toolkit.pl @@ -1,7 +1,6 @@ #!/usr/bin/perl -use strict; -use warnings; +use Modern::Perl; use Carp; use Data::Dumper; diff --git a/installer/install.pl b/installer/install.pl index 459606d..bf9ccc4 100755 --- a/installer/install.pl +++ b/installer/install.pl @@ -17,8 +17,7 @@ # You should have received a copy of the GNU General Public License # along with Koha; if not, see . -use strict; -use warnings; +use Modern::Perl; use diagnostics; use C4::InstallAuth; -- 2.7.4