Bug 6142

Summary: CanBookBeReserved function is redefined in C4/ILSDI/Utility.pm
Product: Koha Reporter: Alex Arnaud <alex.arnaud>
Component: OPACAssignee: Stéphane Delaune <stephane.delaune>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: normal    
Priority: PATCH-Sent (DO NOT USE) CC: chris, f.demians, katrin.fischer, koha.sekjal, m.de.rooy, mjr, paul.poulain, stephane.delaune
Version: 3.6   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Delete sub CanBookBeReserved
followup(delete reference to sub on @EXPORT) : PATCH 1/2 (original)
followup(delete reference to sub on @EXPORT) : PATCH 2/2 (followup)
Bug #6142 - Delete sub CanBookBeReserved and delete function's reference on @EXPORT
Bug #6142 - Delete sub CanBookBeReserved and delete function's reference on @EXPORT
Bug #6142 - Delete sub CanBookBeReserved and delete function's reference on @EXPORT

Description Alex Arnaud 2011-04-08 12:00:41 UTC

    
Comment 1 Alex Arnaud 2011-04-08 12:03:23 UTC Comment hidden (obsolete)
Comment 2 Stéphane Delaune 2011-07-08 09:50:34 UTC Comment hidden (obsolete)
Comment 3 Stéphane Delaune 2011-07-08 09:51:41 UTC Comment hidden (obsolete)
Comment 4 Owen Leonard 2011-08-11 18:23:19 UTC
Which of these attachments is to be tested? Is the first one obsolete? Please squash the valid patches if there are more than one.
Comment 5 MJ Ray (software.coop) 2011-10-22 17:37:33 UTC Comment hidden (obsolete)
Comment 6 Paul Poulain 2011-10-25 15:07:32 UTC
bug affected to Koha 3.6. Entries must be attached to rel_3_8 only when the patch is pushed here (and if it's an ENH)
Comment 7 MJ Ray (software.coop) 2011-10-26 23:48:06 UTC
You what? Surely this should be applied to master (3.8 now) and then pulled back to 3.6?
Comment 8 Ian Walls 2011-11-19 16:00:37 UTC
Can we confirm that C4/ILSDI/Services.pm correctly pulls in C4::Reserves::CanBookBeReserved?  It's not in the explicit import on C4/ILSDI/Services.pm line 29 (the use statement for C4/Reserves).
Comment 9 Marcel de Rooy 2011-12-15 14:32:26 UTC
Question seems unanswered. Changing status to reflect pending status.
Comment 10 MJ Ray (software.coop) 2011-12-15 19:14:02 UTC Comment hidden (obsolete)
Comment 11 Chris Cormack 2011-12-19 07:12:38 UTC
Created attachment 6859 [details] [review]
Bug #6142 - Delete sub CanBookBeReserved and delete function's reference on @EXPORT

Original-Author:    Alex Arnaud <alex.arnaud@biblibre.com>, Stéphane Delaune <stephane.delaune@biblibre.com>
Original-Signed-off-by: MJ Ray <mjr@phonecoop.coop>

Correct use statement bug found by Ian Walls

Author: MJ Ray <mjr@phonecoop.coop>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Comment 12 Ian Walls 2011-12-21 23:04:45 UTC
Removes subroutine from C4/ILSDI/Utility.pm in favor of importing from C4/Reserves.pm

Marking Passed QA
Comment 13 Paul Poulain 2011-12-27 17:39:04 UTC
patch pushed, please test