[Koha-bugs] [Bug 25131] Web installer broken if enable_plugin is set

[bugzilla-daemon at bugs.koha-community.org]

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

--- Comment #16 from Jonathan Druart <jonathan.druart at bugs.koha-community.org> ---
(In reply to Tomás Cohen Arazi from comment #15)
> Created attachment 103015 [details] [review]
> Bug 25131: (QA follow-up) Move test for needs_install outside the plugin
> 
> This follow-up patch moves the check outside the register routine. The
> Mojo plugin won't be loaded unless there's no need to install.
> 
> As the original tests rely on V1.pm to be able to load the plugin, they
> are still valid, and they should still pass as there's no behaviour
> change.
> 
> Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>

The startup method seems to be a better place, thanks Tomás!

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