Bug 27482

Summary: Use Modern::Perl in migration tools
Product: Koha Reporter: Aleisha Amohia <aleisha>
Component: Architecture, internals, and plumbingAssignee: Mazen Khallaf <mazen.i.khallaf>
Status: CLOSED INVALID QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: jonathan.druart, mazen.i.khallaf
Version: unspecifiedKeywords: Academy
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on:    
Bug Blocks: 2505    

Description Aleisha Amohia 2021-01-20 02:39:23 UTC
use Modern::Perl in the place of use strict; use warnings;

misc/migration_tools/22_to_30/missing090field.pl
misc/migration_tools/22_to_30/move_marc_to_authheader.pl
misc/migration_tools/22_to_30/rebuild_leader.pl
misc/migration_tools/22_to_30/rebuild_unimarc_100.pl
misc/migration_tools/buildCOUNTRY.pl
misc/migration_tools/buildEDITORS.pl
misc/migration_tools/buildLANG.pl
misc/migration_tools/create_analytical_rel.pl
misc/migration_tools/upgradeitems.pl
Comment 1 Jonathan Druart 2021-01-20 08:01:32 UTC
Hi Aleisha,

Those scripts will be very hard to test, and adding Modern::Perl may break them.
I think it's better to let them as it.
Comment 2 Aleisha Amohia 2021-01-20 19:29:16 UTC
No worries, I had just taken them from your list on bug 2505. will mark resolved.