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

--- Comment #18 from David Cook <dc...@prosentient.com.au> ---
(In reply to David Cook from comment #16)
> (In reply to Andrii Nugged from comment #15)
> > in:
> > 
> > ```
> > for my $plugin (@plugins) {
> >     my $tasks = $plugin->background_tasks;
> > ```
> > 
> > probably we should have:
> > 
> > ```
> > for my $plugin (@plugins) {
> >     next unless $plugin->can('background_tasks');
> >     my $tasks = $plugin->background_tasks;
> > ```
> > 
> > if plugin does not have background_tasks method.
> 
> Strictly speaking, that shouldn't be necessary since only plugins that have
> registered a 'background_tasks' at install time will be in @plugins.

That is, @plugins wasn't being properly populated. But Tad has fixed it in the
other bug report

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to