]> git.koha-community.org Git - koha.git/commit
Bug 21289: Fix "isa" bug during partner request
authorAndrew Isherwood <andrew.isherwood@ptfs-europe.com>
Thu, 30 Aug 2018 09:52:43 +0000 (10:52 +0100)
committerFridolin Somers <fridolin.somers@biblibre.com>
Thu, 11 Oct 2018 12:45:14 +0000 (14:45 +0200)
commitd951f0906b4edf0543be89b657e9f7bf24b9fa90
tree7d2fc0805d97dfe05d87c7b3ab4a24dbddff523c
parentb60e0ca2885817446d6649e4250bbf6af7fa75d4
Bug 21289: Fix "isa" bug during partner request

We cannot call 'handle_commit_maybe' inside a 'try' block.
handle_commit_maybe redirects the client then calls 'exit', this is
interpreted as an error and the 'catch' block is erroneously called.

This patch moves the calling of 'handle_commit_maybe' outside the try
block, it will only be reached if everything inside the try block
suceeds.

To test:
1) Enable ILL and have the FreeForm backend available
2) Create a patron category to hold ILL "partners" to whom requests can
be sent. The category can be called anything, you should note the code
you assign
3) Create a patron that belongs to your new category, the patron must
have a primary email defined.
4) In your <interlibrary_loans> block in koha-conf.xml, ensure you have
a <partner_code> element, it should contain the code you assigned in
step 2
5) Create an ILL request using the FreeForm backend
6) Once the request is created, select the "Place request with partners"
button
7) Select your "partner" from the "Select partner libraries" box
8) Click "Send email"
9) TEST: Observe no errors are displayed in the UI

Signed-off-by: Barry Cannon <bc@interleaf.ie>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
(cherry picked from commit fca5a1cb3dba5d6561eaf5f30dfd4d996a4042cc)
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
(cherry picked from commit 6bf50445b151ef94e6100da482e5826458a5136b)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
ill/ill-requests.pl