[Koha-devel] Low hanging fruit: Removing un-needed use statements in C4 namespace

Barton Chittenden barton at bywatersolutions.com
Tue Apr 17 16:10:25 CEST 2018


Just to see what happens, I enabled warnings in C4/Reserves.pm. All of the
warnings were of the form

Subroutine * redefined at C4/Reserves.pm line *.

This is happening because of circular dependencies, e.g.

    C4/Reserves.pm

uses

    C4::Members
    C4::Circulation

Those modules in turn use C4::Reserves. None of the functions used in these
modules is actually called in C4::Reserves, but removing the calls to use
C4::Members and use C4::Circulation from C4/Reserves.pm isn't enough,
apparently there's a deeper circular dependency there.

I suspect that there are a number of unnecessary 'use' statements peppered
throughout the C4 namespace; cleaning those up might fix the problem
entirely, or at least allow us to resolve any remaining circular
dependencies easily.

--Barton
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.koha-community.org/pipermail/koha-devel/attachments/20180417/1e18ce22/attachment.html>


More information about the Koha-devel mailing list