Bug 20047

Summary: Add Z3950Server Object and use it for getting server count
Product: Koha Reporter: Nick Clemens <nick>
Component: Architecture, internals, and plumbingAssignee: Nick Clemens <nick>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: katrin.fischer, martin.renvoize
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 19987    
Bug Blocks:    
Attachments: Bug 20047 - Add Koha::Z3950Server Oject and use it to get server count
Bug 20047 - Add Koha::Z3950Server Oject and use it to get server count
Bug 20047 - Add Koha::Z3950Server Oject and use it to get server count
Bug 20047 - Add Koha::Z3950Server Oject and use it to get server count
Bug 20047: (follow-up) Retrieve only biblio/authority servers as appropriate
Bug 20047: Add Koha::Z3950Server Oject and use it to get server count
Bug 20047: (follow-up) Retrieve only biblio/authority servers as appropriate

Description Nick Clemens 2018-01-19 15:56:14 UTC

    
Comment 1 Nick Clemens 2018-01-19 15:59:14 UTC
Created attachment 70756 [details] [review]
Bug 20047 - Add Koha::Z3950Server Oject and use it to get server count

To test:
1 - prove t/db_dependent/Koha/Z3950Servers.t
2 - Load cataloging and authority home pages
3 - Verify you can add form z395 on each page
4 - Delete all z3950 servers
5 - Verify option to add from Z3950 is removed on both pages
Comment 2 Owen Leonard 2018-01-22 19:45:49 UTC
The QA tool has a couple of complaints about this patch:

 FAIL   Koha/Z3950Server.pm
   OK     critic
   OK     forbidden patterns
   OK     git manipulation
   OK     pod
   FAIL   pod coverage
     POD coverage was greater before, try perl -MPod::Coverage=PackageName -e666
   OK     spelling
   OK     valid

 FAIL   Koha/Z3950Servers.pm
   OK     critic
   OK     forbidden patterns
   OK     git manipulation
   OK     pod
   FAIL   pod coverage
                POD is missing for 'object_class'
   OK     spelling
   OK     valid
Comment 3 Nick Clemens 2018-01-23 13:29:43 UTC
Created attachment 70829 [details] [review]
Bug 20047 - Add Koha::Z3950Server Oject and use it to get server count

To test:
1 - prove t/db_dependent/Koha/Z3950Servers.t
2 - Load cataloging and authority home pages
3 - Verify you can add form z395 on each page
4 - Delete all z3950 servers
5 - Viery option to add from Z3950 is removed on both pages
Comment 4 Nick Clemens 2018-01-23 13:30:21 UTC
(In reply to Owen Leonard from comment #2)
> The QA tool has a couple of complaints about this patch:

It still does, but ignore them, they are not relevant
Comment 5 Jesse Maseto 2018-03-12 13:17:39 UTC
Created attachment 72681 [details] [review]
Bug 20047 - Add Koha::Z3950Server Oject and use it to get server count

To test:
1 - prove t/db_dependent/Koha/Z3950Servers.t
2 - Load cataloging and authority home pages
3 - Verify you can add form z395 on each page
4 - Delete all z3950 servers
5 - Viery option to add from Z3950 is removed on both pages

Signed-off-by: Jesse Maseto <jesse@bywatersolutions.com>
Comment 6 Katrin Fischer 2018-04-16 06:04:59 UTC
Please rebase!
Comment 7 Nick Clemens 2018-04-17 12:06:17 UTC
Created attachment 74316 [details] [review]
Bug 20047 - Add Koha::Z3950Server Oject and use it to get server count

To test:
1 - prove t/db_dependent/Koha/Z3950Servers.t
2 - Load cataloging and authority home pages
3 - Verify you can add form z395 on each page
4 - Delete all z3950 servers
5 - Viery option to add from Z3950 is removed on both pages

Signed-off-by: Jesse Maseto <jesse@bywatersolutions.com>
Comment 8 Nick Clemens 2018-04-17 12:06:20 UTC
Created attachment 74317 [details] [review]
Bug 20047: (follow-up) Retrieve only biblio/authority servers as appropriate
Comment 9 Katrin Fischer 2018-04-18 06:13:57 UTC
Created attachment 74405 [details] [review]
Bug 20047: Add Koha::Z3950Server Oject and use it to get server count

To test:
1 - prove t/db_dependent/Koha/Z3950Servers.t
2 - Load cataloging and authority home pages
3 - Verify you can add form z395 on each page
4 - Delete all z3950 servers
5 - Viery option to add from Z3950 is removed on both pages

Signed-off-by: Jesse Maseto <jesse@bywatersolutions.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 10 Katrin Fischer 2018-04-18 06:14:14 UTC
Created attachment 74406 [details] [review]
Bug 20047: (follow-up) Retrieve only biblio/authority servers as appropriate

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 11 Jonathan Druart 2018-04-20 16:45:16 UTC
Pushed to master for 18.05, thanks to everybody involved!