When using the KOCT plugin for offline circulation, checkout notices are created correctly, but checkin notices are no longer created. To test: - Check out an item via KOCT - Send to Koha and process it there - Verify that the checkout is on the account and a checkout notice has been generated - Check in the item via KOCT - Send to Koha and process it there - Verify that the item is no longer on the account but NO checkin notice has been generated This is quite a problem for libraries using the offline circulation plugin as they rely on the emails to inform the patrons.
Are you sure it is a regression? CHECKIN notices are not generated because ProcessOfflineReturn does not call AddReturn but MarkIssueReturned instead (why??)
We got this report from libraries updated from 16.11 to 17.11 - I think it might just have been broken for too long already.
Jonathan, do you think the fix would be to use AddReturn? I haven't checked yet, but I think the only issue I could imagine is that you need to manipulate the time of return. But we can do that in the GUI, so I assume it should be possible.
(In reply to Katrin Fischer from comment #3) > Jonathan, do you think the fix would be to use AddReturn? > > I haven't checked yet, but I think the only issue I could imagine is that > you need to manipulate the time of return. But we can do that in the GUI, so > I assume it should be possible. We should investigate the side effects, for instance AddReturn will test if the item can be checked in, whereas MarkIssueReturned will (almost only) move the entry from issues to old_issues.
(In reply to Jonathan Druart from comment #4) > (In reply to Katrin Fischer from comment #3) > > Jonathan, do you think the fix would be to use AddReturn? > > > > I haven't checked yet, but I think the only issue I could imagine is that > > you need to manipulate the time of return. But we can do that in the GUI, so > > I assume it should be possible. > > We should investigate the side effects, for instance AddReturn will test if > the item can be checked in, whereas MarkIssueReturned will (almost only) > move the entry from issues to old_issues. I think for the offline circulation it's important that the item is _always_ returned, because it's in the library already, so you have to deal with it somehow. Maybe something like a "force" parameter?
*** Bug 25977 has been marked as a duplicate of this bug. ***