[Koha-bugs] [Bug 15135] Remove Warning Subroutine HasOverdues redefined

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Mon Jan 11 11:39:27 CET 2016


http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15135

--- Comment #18 from Marcel de Rooy <m.de.rooy at rijksmuseum.nl> ---
(In reply to Marc Véron from comment #17)
> (In reply to Marcel de Rooy from comment #16)
> > If I understood correctly, the problem is that we now have
> > Borrowers::HasOverdues and Members::HasOverdues and similar for IsDebarred.
> > Looking at the Borrowers plugin, not importing Overdues from Members there,
> > resolves the warning for now and is consistent with the IsDebarred case. So
> > I agree with Jonathan.
> > 
> > Marc: Could you adjust your patch?
> > 
> > Too bad btw that TT forces us to create a stub for each Koha routine we want
> > to call in the templates. (Exception for OO modules as below)
> > 
> > From the TT docs:
> > ===
> > If a regular Perl module supports an object-oriented interface and a new()
> > constructor then it can be loaded and instantiated automatically like:
> > [% USE file = IO.File('/tmp/mydata') %]
> > ===
> 
> "...not importing Overdues from Members there..."
> Sorry, I lost track here (Ik ben the kluts kwijt). Wasn't that what I did in
> my very first patch?

No you removed the use completely there.
You only should say: use Module qw();
In that case you do not import any routines in the current namespace.

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


More information about the Koha-bugs mailing list