Bug 36792: Limit POSIX imports
A Marcel's QA patch to Bug 36552 added use POSIX; in two spots. In https://metacpan.org/pod/POSIX we read: CAVEATS Everything is exported by default (with a handful of exceptions). This is an unfortunate backwards compatibility feature and its use is strongly discouraged. You should either prevent the exporting (by saying use POSIX ();, as usual) and then use fully qualified names (e.g. POSIX::SEEK_END), or give an explicit import list. If you do neither and opt for the default (as in use POSIX;), you will import hundreds and hundreds of symbols into your namespace. This patch fixes this. No test plan. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de> (cherry picked from commit8d9ccd6fc3
) Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com> (cherry picked from commitca023b334c
) Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
This commit is contained in:
parent
a7959a7d30
commit
822d7f43ab
2 changed files with 2 additions and 2 deletions
|
@ -21,7 +21,7 @@
|
|||
use Modern::Perl;
|
||||
|
||||
use CGI qw ( -utf8 );
|
||||
use POSIX;
|
||||
use POSIX qw( strftime );
|
||||
use C4::Auth qw( get_template_and_user );
|
||||
use C4::Output qw( output_html_with_http_headers );
|
||||
use C4::AuthoritiesMarc qw( AddAuthority ModAuthority GetAuthority GetTagsLabels GetAuthMARCFromKohaField FindDuplicateAuthority );
|
||||
|
|
|
@ -22,7 +22,7 @@
|
|||
use Modern::Perl;
|
||||
|
||||
use CGI;
|
||||
use POSIX;
|
||||
use POSIX qw( strftime );
|
||||
use C4::Output qw( output_html_with_http_headers );
|
||||
use C4::Auth qw( get_template_and_user haspermission );
|
||||
use C4::Biblio qw(
|
||||
|
|
Loading…
Reference in a new issue