Bug 25175

Summary: Don't die on broken plugins
Product: Koha Reporter: Tomás Cohen Arazi (tcohen) <tomascohen>
Component: Plugin architectureAssignee: Bugs List <koha-bugs>
Status: CLOSED WONTFIX QA Contact: Testopia <testopia>
Severity: major    
Priority: P5 - low    
Version: Main   
Hardware: All   
OS: All   
GIT URL: Initiative type: ---
Sponsorship status: --- Comma delimited list of Sponsors:
Crowdfunding goal: 0 Crowdfunding committed: 0
Crowdfunding contact: Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:

Description Tomás Cohen Arazi (tcohen) 2020-04-17 11:31:06 UTC

    
Comment 1 Tomás Cohen Arazi (tcohen) 2020-04-17 11:32:40 UTC
If a plugin doesn't compile, Plack currently explodes:

1. Add use Tomas; to a sample plugin and restart plack
=> FAIL: It explodes
2. Add die "algo" to the plugin's neW() method and rstart plack
=> FAIL: It explodes
Comment 2 Tomás Cohen Arazi (tcohen) 2020-04-17 12:09:08 UTC
I will roll back on this one, as putting die in ->new is extreme. And missing libs and syntax errors are already cough by the call to 'load'.