This is not done yet! We need that for modules (at least) ASAP
Created attachment 96202 [details] [review] Bug 24217: use Modern::Perl for modules We absolutely need that for modules. Test plan: git grep 2505 **/*.pm should not return any results And let's see later what needs to be fixed.
RMaints, I do not think you should backport this.
I think we missed a few ;) I used `grep -IRiL "Modern::Perl" **/*.pm` to check and came up with this list after your patch C4/AuthoritiesMarc.pm C4/Auth.pm C4/Auth_with_cas.pm C4/Barcodes.pm C4/Bookseller.pm C4/Boolean.pm C4/Breeding.pm C4/Calendar.pm C4/Charset.pm C4/ClassSortRoutine.pm C4/ClassSource.pm C4/Creators.pm C4/Debug.pm C4/HoldsQueue.pm C4/HTML5Media.pm C4/Images.pm C4/ImportBatch.pm C4/ImportExportFramework.pm C4/ItemCirculationAlertPreference.pm C4/Labels.pm C4/Languages.pm C4/Linker.pm C4/Log.pm C4/Message.pm C4/Patroncards.pm C4/Scrubber.pm C4/Service.pm C4/ShelfBrowser.pm C4/SMS.pm C4/Stats.pm C4/Tags.pm C4/Templates.pm C4/TmplToken.pm C4/TTParser.pm Koha/Cache.pm Koha/Calendar.pm Koha/Edifact.pm Koha/EDI.pm Koha/Schema.pm Koha/SuggestionEngine.pm OpenILS/QueryParser.pm
Created attachment 96204 [details] [review] Bug 24217: use Modern::Perl for modules when strict is missing We absolutely need that for modules. We are enabling strict by using Modern::Perl. Note that other modules from C4 use strict and warnings, instead of Modern::Perl Test plan: git grep 2505 **/*.pm should not return any results And let's see later what needs to be fixed.
Created attachment 96205 [details] [review] Bug 24217: use Modern::Perl for modules We absolutely need that for modules. Test plan: git grep 2505 **/*.pm should not return any results And let's see later what needs to be fixed. Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Created attachment 96206 [details] [review] Bug 24217: use Modern::Perl for modules when strict is missing We absolutely need that for modules. We are enabling strict by using Modern::Perl. Note that other modules from C4 use strict and warnings, instead of Modern::Perl Test plan: git grep 2505 **/*.pm should not return any results And let's see later what needs to be fixed. Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Agreed, Signing off
Created attachment 96312 [details] [review] Bug 24217: use Modern::Perl for modules when strict is missing We absolutely need that for modules. We are enabling strict by using Modern::Perl. Note that other modules from C4 use strict and warnings, instead of Modern::Perl Test plan: git grep 2505 **/*.pm should not return any results And let's see later what needs to be fixed. Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Nice work everyone! Pushed to master for 20.05
See bug 24361 for follow-ups.
It broke the translator script, see bug 24365.
not backported to 19.11.x