[Koha-bugs] [Bug 30939] remove_unused_authorities.pl is broken

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Tue Jun 28 14:33:23 CEST 2022


https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30939

--- Comment #6 from Tomás Cohen Arazi <tomascohen at gmail.com> ---
(In reply to Fridolin Somers from comment #5)
> How could be search for thoses ?
> 
> I try :
> git grep 'use C4::.*;' | egrep -v '^(C4|Koha|t)/' | egrep -v 'qw ?(\(|/)' |
> grep -v Context
> 
> If found for example :
> tools/batch_delete_records.pl:use C4::AuthoritiesMarc;
> 
> Is it a correct match ?

Nope. Because the script is using fully-qualified method names (e.g.
C4::AuthoritiesMarc::GetAuthority( $record_id ) ) so not importing the function
names into the package scope is not a real issue. We need to check that
C4::AuthoritiesMarc defines a method that is used in a script.

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list