Bug 20934: Fix display of old checkouts in the checkout history page
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Wed, 13 Jun 2018 16:26:43 +0000 (13:26 -0300)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Wed, 18 Jul 2018 12:34:59 +0000 (13:34 +0100)
commit517494e0e51281607e101f63f02a3972179d7896
tree4e76ff420a29b9f3cdb18fbaf7259acb85ace315
parentca4a096e03179d8ececd19e6bc6bd75d0eaf6631
Bug 20934: Fix display of old checkouts in the checkout history page

Again a regression caused by
  commit fa54100dffe092e606f79b15692eedaf78f42e45
  Bug 18403: Use patron-title.inc when hidepatronname is used [SPECIFIC for issuehistory]

GetBiblioIssues does a union all with issues and old_issues, so we
should old_issues as well.
To make the join on the items table we need to define the item and
patron methods. For consistency the relationships have been redefined
(item instead of itemnumber, borrower instead of borrowernumber) in the
DBIx::Class definition.

This is not perfect but I think the best way to provide an easy to
backport patch.

It highlights that we need improvements in this area.

Signed-off-by: Maryse Simard <maryse.simard@inlibro.com>
The patch works as expected.

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
(cherry picked from commit 117e3a18f5b7c24e6c9fb7eabb78ab1b5320cd14)
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Koha/Old/Checkout.pm
Koha/Schema/Result/OldIssue.pm
catalogue/issuehistory.pl
koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/issuehistory.tt