<div dir="ltr">I think the first step to fight with this problem is bug 17600 - Standardize the EXPORT

    
  </div><br><div class="gmail_quote"><div dir="ltr">On Tue, 17 Apr 2018 at 22:05 Barton Chittenden <<a href="mailto:barton@bywatersolutions.com">barton@bywatersolutions.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Continuing to fiddle around with this. I commented out all of the use statements in C4/Reserves.pm, then uncommented them one by one until the circular dependencies came back. There are two modules that create the dependencies in C4::Reserves:<div><br></div><div>C4::Accounts;</div><div>C4::Letters;<br><div><br></div><div><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 17, 2018 at 10:10 AM, Barton Chittenden <span dir="ltr"><<a href="mailto:barton@bywatersolutions.com" target="_blank">barton@bywatersolutions.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Just to see what happens, I enabled warnings in C4/Reserves.pm. All of the warnings were of the form</div><div><br></div><div>Subroutine * redefined at C4/Reserves.pm line *.</div><div><br></div><div>This is happening because of circular dependencies, e.g. </div><div><br></div><div>    C4/Reserves.pm</div><div><br></div><div>uses</div><div><br></div><div>    C4::Members</div><div>    C4::Circulation</div><div><br></div><div>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.</div><div><br></div><div>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.</div><span class="m_765373125345088983HOEnZb"><font color="#888888"><div><br></div><div>--Barton</div></font></span></div>
</blockquote></div><br></div>
_______________________________________________<br>
Koha-devel mailing list<br>
<a href="mailto:Koha-devel@lists.koha-community.org" target="_blank">Koha-devel@lists.koha-community.org</a><br>
<a href="http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel" rel="noreferrer" target="_blank">http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel</a><br>
website : <a href="http://www.koha-community.org/" rel="noreferrer" target="_blank">http://www.koha-community.org/</a><br>
git : <a href="http://git.koha-community.org/" rel="noreferrer" target="_blank">http://git.koha-community.org/</a><br>
bugs : <a href="http://bugs.koha-community.org/" rel="noreferrer" target="_blank">http://bugs.koha-community.org/</a></blockquote></div>