Summary: | Fix Modules::RequireFilenameMatchesPackage Perl Critic Violations | ||
---|---|---|---|
Product: | Koha | Reporter: | David Cook <dcook> |
Component: | Architecture, internals, and plumbing | Assignee: | David Cook <dcook> |
Status: | CLOSED INVALID | QA Contact: | Testopia <testopia> |
Severity: | minor | ||
Priority: | P5 - low | CC: | tomascohen |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26673 | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Attachments: |
Bug 26683: Decompose C4::Barcodes::ValueBuilder to make Perl critic happy
Bug 26683: Decompose C4::Barcodes::ValueBuilder to make Perl critic happy Bug 26683: Decompose Koha::Z3950Responder::RPN into Net::Z3950::RPN::* packages Bug 26683: Load localized Net::Z3950::RPN::* modules |
Description
David Cook
2020-10-14 23:02:12 UTC
Created attachment 111657 [details] [review] Bug 26683: Decompose C4::Barcodes::ValueBuilder to make Perl critic happy Created attachment 111666 [details] [review] Bug 26683: Decompose C4::Barcodes::ValueBuilder to make Perl critic happy Created attachment 111667 [details] [review] Bug 26683: Decompose Koha::Z3950Responder::RPN into Net::Z3950::RPN::* packages This patch reverts the usage of Net::Z3950::SimpleServer's samples/render-search.pl example, and instead creates modified packages which will be loaded before the standard ones due to these being located higher up in the @INC path. C4::Barcodes::ValueBuilder test plan: 1. Iterate through the "autoBarcode" options 2. For each autoBarcode option, try generating a barcode at http://localhost:8081/cgi-bin/koha/cataloguing/additem.pl?biblionumber=29&searchid=scs_1602720764348 Created attachment 111669 [details] [review] Bug 26683: Load localized Net::Z3950::RPN::* modules Koha::Z3950Responder::RPN test plan: 0. Open 2 terminals Terminal 1: 1. koha-z3950-responder --stop kohadev 2. perl misc/z3950_responder.pl Terminal 2: 3. yaz-client 127.0.0.1:2100/biblios 4. #Note connection accepted 5. find @or @1=4 Bruce @1=4 Damian 6. #Note Number of hits: 9 7. #Iterate through results using "show" and notice the 245 has either Bruce or Damian in it Note that this just makes sure the koha-z3950-responder is still running. It doesn't actually test that the Net::Z3950::RPN::* modules are actually doing anything it seems. Ere Maijala might be the best person to verify this, since he's the one who added Koha::Z3950Responder::RPN in Bug 13937 for "Add support for SRU and Elasticsearch" Before I added the 3rd patch "Load localized Net::Z3950::RPN::* modules", the t/db_dependent/Koha/Z3950Responder/GenericSession.t test was failing, but now it's passing. -- (It's a shame we don't have a "lib" directory that contains all our Perl modules, but that would require renaming too many files, so I don't see that ever happening...) I don't think this is needed thanks to https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112814 |