Bug 21298 - Reopening a basket, the dates reset for date ordered and thus affect the late orders report
Summary: Reopening a basket, the dates reset for date ordered and thus affect the late...
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Documentation (show other bugs)
Version: 17.11
Hardware: Macintosh Mac OS
: P4 normal (vote)
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-08-31 00:56 UTC by arees
Modified: 2019-10-14 19:57 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Small patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Screenshots of reports before date change and after (221.83 KB, application/pdf)
2018-08-31 00:56 UTC, arees
Details

Note You need to log in before you can comment on or make changes to this bug.
Description arees 2018-08-31 00:56:12 UTC
Created attachment 78318 [details]
Screenshots of reports before date change and after

The late orders report allows you to track the length of time an order has been with a supplier. If you re-open a basket in acquisitions to, for example, cancel the order of an item that cannot be supplied, then the dates in the late order report reset to the day you re-opened the basket i.e. An order that is at 90 days will reset to 1 day late. 
This renders the late reporting facility very inaccurate and we cannot rely on this report to track our orders.
Comment 1 Bob Birchall 2018-09-14 23:25:06 UTC
This issue has been addressed with an update to the documentation. It seems there are two ways to delete an order. It is not recommended to re-open the basket to delete, precisely because re-closing the basket will update aqbasket.closeddate, thereby invalidating the Late Orders Report.
The preferred way to cancel an order is to receive, by creating a false invoice and cancelling from there. 
A new section 'Cancelling an order' has been added to the Acquisitions chapter of the manual. 
The commit has been merged, so closing this out now.