Koha-clms/C4/SIP/ILS
Kyle M Hall 0ebfff111d Bug 27204: Fix end boundary index, never return results for non-existant accountlines
There are two primary issues I've identified with requesting line items
for fees via SIP:

1) The end boundary is incorrect. For example, if send a request with a
BP ( starting item) of 1, and a BQ (end item) of 1, I should get just
the first item. Instead I will get two items

2) Our SIP server does not check bounds. For example, if I have 3 fines,
but I send a BP of 1 and a BQ of 5, I will get back 5 AVs, two of
them being "empty" because the patron only has 3 accountlines!

Test Plan:
1) Apply the unit test patch
2) prove t/db_dependent/SIP/Patron.t
3) Note the failures
4) Apply the second patch
5) prove t/db_dependent/SIP/Patron.t
6) All tests should pass!

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
(cherry picked from commit 0a5232412f)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
2021-02-05 16:40:55 +01:00
..
Transaction Bug 27196: Don't explode if item that is waiting if checked in by SIP 2020-12-30 15:15:49 +01:00
Item.pm Bug 12556: (QA follow-up) Fix QA Script failures 2020-10-14 14:50:07 +02:00
Item.pod bug 3435: SIP Checkin extension for 3M SmartChute - partial 2009-07-30 11:06:47 -04:00
Patron.pm Bug 27204: Fix end boundary index, never return results for non-existant accountlines 2021-02-05 16:40:55 +01:00
Patron.pod Bug 14383: C4: Fix some typos (mostly in comments and documentation) 2015-06-22 17:34:45 -03:00
Transaction.pm Bug 17680: C4::Circulation - Remove GetItemIssue, simple calls 2017-07-10 12:06:37 -03:00