Bug 10781

Summary: Remove C4::ILSDI::Utility
Product: Koha Reporter: Srdjan Jankovic <srdjan>
Component: OPACAssignee: Srdjan Jankovic <srdjan>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: minor    
Priority: P5 - low CC: fridolin.somers, gitbot, gmcharlt, m.de.rooy
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: bug_10781: Removed ILSDI::Utility
bug_10781: Removed ILSDI::Utility
bug_10781: Removed ILSDI::Utility
bug_10781: Removed ILSDI::Utility

Description Srdjan Jankovic 2013-08-26 00:09:23 UTC
C4::ILSDI::Utility is not really required. It contains only two routines:
* BorrowerExists() which is not used
* Availability(), used only in C4::ILSDI::Services

I propose we move Availability() to C4::ILSDI::Services and remove C4::ILSDI::Utility
Comment 1 Srdjan Jankovic 2013-08-26 00:23:15 UTC Comment hidden (obsolete)
Comment 2 Marcel de Rooy 2013-09-23 11:19:01 UTC
Applying: bug_10781: Removed ILSDI::Utility
/usr/share/koha/testclone/.git/rebase-apply/patch:43: tab in indent.
        my ($biblionumber, $status, $msg, $location) = _availability($id);
warning: 1 line adds whitespace errors.
Using index info to reconstruct a base tree...
<stdin>:43: tab in indent.
        my ($biblionumber, $status, $msg, $location) = _availability($id);
warning: 1 line applied after fixing whitespace errors.
Falling back to patching base and 3-way merge...
Removing t/ILSDI_Utility.t
CONFLICT (modify/delete): C4/ILSDI/Utility.pm deleted in bug_10781: Removed ILSDI::Utility and modified in HEAD. Version HEAD of C4/ILSDI/Utility.pm left in tree.
Auto-merging C4/ILSDI/Services.pm
CONFLICT (content): Merge conflict in C4/ILSDI/Services.pm
Failed to merge in the changes.
Comment 3 Srdjan Jankovic 2013-09-30 23:39:56 UTC Comment hidden (obsolete)
Comment 4 I'm just a bot 2013-10-01 18:44:14 UTC
Patch applied cleanly, go forth and signoff
Comment 5 Kyle M Hall 2013-10-25 13:41:30 UTC Comment hidden (obsolete)
Comment 6 Marcel de Rooy 2013-11-05 14:37:38 UTC
Created attachment 22725 [details] [review]
bug_10781: Removed ILSDI::Utility

Test:
This should be a noop. Regression testing required:
/cgi-bin/koha/ilsdi.pl functioanality, in particuler:
GetAvailability - ?service=Describe&verb=GetAvailability
AuthenticatePatron - ?service=Describe&verb=AuthenticatePatron

ILS-DI syspref must be turned on

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 7 Marcel de Rooy 2013-11-05 14:40:22 UTC
QA Comment:
Tested both functions with ilsdi.pl. Works fine. Code looks good to me. No complaints from qa tools.
Moving from enh to bugfix. No string changes. Still looks like a candidate for 3.14.

Passed QA
Comment 8 Galen Charlton 2013-12-10 01:29:31 UTC
Pushed to master.  Thanks, Srdjan!
Comment 9 Fridolin Somers 2013-12-24 09:39:03 UTC
Patch pushed to 3.14.x, will be in 3.14.2