Bug 38491

Summary: Update OAI servers code to current standards
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: CatalogingAssignee: Bugs List <koha-bugs>
Status: In Discussion --- QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: dcook, m.de.rooy, matthias.meusburger, victor
Version: Main   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: --- Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
Circulation function:
Bug Depends on: 35659    
Bug Blocks:    

Description Katrin Fischer 2024-11-20 10:35:47 UTC
admin/oai_servers.pl and .tt are really old style written, could have been nice to see them follow the current standard.

1. msg_added => should use a @messages array
2. add_form and op => Why 2 variables for the same purpose?
3. No try-catch!
4. type => $op eq 'add' ? lc $type : '' 
What is "type"? Is not used in template.