Bug 5957

Summary: COiNS in the OPAC search results should be optional
Product: Koha Reporter: Ian Walls <koha.sekjal>
Component: OPACAssignee: Ian Walls <koha.sekjal>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: minor    
Priority: PATCH-Sent (DO NOT USE) CC: chris, claire.hernandez, jcamins, magnus
Version: 3.4   
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:    
Bug Blocks: 5729    
Attachments: Proposed patch
Follow-up patch
1st signed off
2nd patch signed off

Description Ian Walls 2011-03-26 00:54:09 UTC
The use of COins / OpenURL / Z3988 in the OPAC search results should be optional, as adds a great deal of processing time per record returned, slowing search.  

This feature should be disabled by default, so libraries who know they want to use it (and accept the costs) can opt-in
Comment 1 Chris Cormack 2011-03-31 22:58:41 UTC
Created attachment 3598 [details] [review]
Proposed patch
Comment 2 Chris Cormack 2011-03-31 23:08:46 UTC
Pushed please test
Comment 3 Jared Camins-Esakov 2011-04-05 12:24:07 UTC
Created attachment 3678 [details] [review]
Follow-up patch

Follow-up patch changing an incorrect assignment operator to a comparison operator.
Comment 4 Magnus Enger 2011-04-05 12:52:26 UTC
*** Bug 6060 has been marked as a duplicate of this bug. ***
Comment 5 claire.hernandez@biblibre.com 2011-04-05 15:50:06 UTC
Created attachment 3709 [details] [review]
1st signed off
Comment 6 claire.hernandez@biblibre.com 2011-04-05 15:50:32 UTC
Created attachment 3710 [details] [review]
2nd patch signed off
Comment 7 claire.hernandez@biblibre.com 2011-04-05 15:51:36 UTC
This very little patch make sense for me, didn't do some functionnal test (only a return line and an "eq")
Comment 8 Chris Cormack 2011-04-07 04:09:22 UTC
Follow up patch pushed