[Bug 27483] New: Use Modern::Perl in other misc scripts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27483 Bug ID: 27483 Summary: Use Modern::Perl in other misc scripts Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: koha-bugs@lists.koha-community.org Reporter: aleisha@catalyst.net.nz QA Contact: testopia@bugs.koha-community.org use Modern::Perl in the place of use strict; use warnings; misc/batchCompareMARCvsFrameworks.pl misc/batchDeleteUnusedSubfields.pl misc/batchImportMARCWithBiblionumbers.pl misc/batchRebuildBiblioTables.pl misc/cronjobs/build_browser_and_cloud.pl misc/exportauth.pl misc/kohalib.pl misc/translator/tmpl_process3.pl -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27483 Aleisha Amohia <aleisha@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |2505 Keywords| |Academy Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=2505 [Bug 2505] Omnibus: Enable Perl warnings in all modules and scripts -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27483 Eden <eden.bacani@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Assignee|koha-bugs@lists.koha-commun |eden.bacani@gmail.com |ity.org | CC| |eden.bacani@gmail.com -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27483 Eden <eden.bacani@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27483 --- Comment #1 from Eden <eden.bacani@gmail.com> --- Created attachment 115409 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115409&action=edit Bug 27483: Replaces use strict; and use warnings; with Modern::Perl Test Plan 1. check on files below that use strict; and use warings; do not exist 2. check on files below that Modern::Perl exists misc/batchCompareMARCvsFrameworks.pl misc/batchDeleteUnusedSubfields.pl misc/batchImportMARCWithBiblionumbers.pl misc/batchRebuildBiblioTables.pl misc/cronjobs/build_browser_and_cloud.pl misc/exportauth.pl misc/kohalib.pl misc/translator/tmpl_process3.pl -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27483 Eden <eden.bacani@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |ASSIGNED --- Comment #2 from Eden <eden.bacani@gmail.com> --- Still working on this -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27483 Eden <eden.bacani@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27483 --- Comment #3 from Eden <eden.bacani@gmail.com> --- Created attachment 115458 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115458&action=edit Bug 27483: Deleted unnecessary use warnings comments Test Plan 1. check on files below that use warings comments does not exist 2. check on files below that Modern::Perl does not exist as it is unnecessary. 3. Check that use strict; still exists misc/batchCompareMARCvsFrameworks.pl misc/batchDeleteUnusedSubfields.pl misc/batchImportMARCWithBiblionumbers.pl misc/batchRebuildBiblioTables.pl misc/cronjobs/build_browser_and_cloud.pl misc/exportauth.pl misc/kohalib.pl misc/translator/tmpl_process3.pl -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27483 Eden <eden.bacani@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |ASSIGNED --- Comment #4 from Eden <eden.bacani@gmail.com> --- Still working on it -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27483 Eden <eden.bacani@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115409|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27483 Eden <eden.bacani@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27483 --- Comment #5 from Eden <eden.bacani@gmail.com> --- ready for testing -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27483 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA CC| |tomascohen@gmail.com --- Comment #6 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Hi, Eden. The idea is to introduce use Modern::Perl; Modern::Perl implies use strict; use warnings; which is desirable. The idea is you should replace use strict; #use warnings; for use Modern::Perl; and it will make some warnings show up. And those need to be fixed. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org