]> git.koha-community.org Git - koha.git/commit
Bug 13990: ILS-DI LookupPatron requires ID Type
authorJon Knight <J.P.Knight@lboro.ac.uk>
Tue, 7 Nov 2017 17:27:56 +0000 (17:27 +0000)
committerNick Clemens <nick@bywatersolutions.com>
Fri, 2 Feb 2018 15:04:08 +0000 (15:04 +0000)
commite83f5c807d717e447a52353af1067c6c241e952a
tree7aa3930ed5a06852535f75ac4f7a3d8ed511df5e
parent68f595c1c7fd51c528a6029bf3d45715c12f769b
Bug 13990: ILS-DI LookupPatron requires ID Type

Patch provides a constrained set of borrower attributes to try in turn if the
ID type is not specified.  Also added "email" to the POD documented list of
id types seeing as its quite a useful one for integration developers.

Test plan:

1) without the patch applied, turn on ils-di interface in your admin sysprefs
   and then try accessing the URL:

   https://your-server/cgi-bin/koha/ilsdi.pl?service=LookupPatron&id=Mabel

   replacing "your-server" your Koha dev test domain name (localhost:8080 if
   you're using a kohadevbox VM) and "Mabel" with the first name of a patron
   (Mabel is in the test database already if you're using that).

   You should get back an error message in the XML.

2) Apply the patch and repeat. This time you should get an <id> element in
   the XML with Mabel's patron ID in it.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
C4/ILSDI/Services.pm
koha-tmpl/opac-tmpl/bootstrap/en/modules/ilsdi.tt
opac/ilsdi.pl