Bug 30941: (QA follow-up) Import methods explicitly
authorTomas Cohen Arazi <tomascohen@theke.io>
Fri, 19 Aug 2022 12:06:57 +0000 (09:06 -0300)
committerTomas Cohen Arazi <tomascohen@theke.io>
Fri, 19 Aug 2022 12:49:36 +0000 (09:49 -0300)
By explicitly importing the Koha::I18N methods, I get rid of this error:

kohadev-koha@kohadevbox:/kohadevbox/koha$ qa -c 2 --run-tests
testing 2 commit(s) (applied to 9b2bf04 '09 Bug 30507: Value builder for field')

Processing files before patches
|========================>| 12 / 12 (100.00%)

An error occurred : Inconsistent hierarchy during C3 merge of class 'Koha::I18N':
current merge results [
Koha::I18N,
]
merging failed on 'Exporter' at /kohadevbox/qa-test-tools/koha-qa.pl line 112.

This happens because the Koha::I18N library is exporting all its methods
explicitly and, by design, the __ methods used in value builders cannot
determine which of the imported methods to use i.e. if Koha::I118N::__
or Koha::CodeList::Unimarc::MediumOfPerformance:__

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Koha/CodeList/Unimarc/MediumOfPerformance.pm

index 12ba763..e839ec0 100644 (file)
@@ -18,7 +18,7 @@ package Koha::CodeList::Unimarc::MediumOfPerformance;
 use Modern::Perl;
 use utf8;
 
-use Koha::I18N;
+use Koha::I18N qw( N__ N__p );
 
 sub voices {
     return {