Created attachment 5226 [details] basket edit and delete options When you're in a basket you have edit and delete links to the right of each line item (this is good). The problem is that these options appear even if you have already received an item. I don't think we need to remove the edit option in this case (but maybe we should) but we certainly should remove the delete option. If I've received something already then money has been subtracted and deleting a line from a basket could cause major problems.
Nicole, is it still an issue? If you have received an order, for me that means the basket was closed and send to the vendor. If the basket is closed, you don't see the "delete" and "modify" links in the table. M. Saby
The basket does not have to be closed to receive items - it's good practice - but it's not required. So this is still an issue. Nicole
Hm, there are some things that make it more than best practice this days. For example if you want to claim items I think it's important that you have closed the basket (indicating it has been actually ordered). Also if the Acq history search gets pushed this will be get even more important.
Nicole, are you sure you can receive orders in receipt module when the basket is not closed? For me it would look like an issue... Mathieu
While I say it's required and you all think it's required - Koha does not require it. I have received many thing when the basket was still open. Also, you can reopen a basket and delete something you have received. So there are multiple ways to accidentally delete something you have received.
I admit being flexible can be a good thing, but sometimes there's too much way for doing things in Koha... If each library can have its own workflow, it is difficult to ensure code and feature consistency. OR if there is several way of doing a thing as important as placing an order, it must be syspref dependant. For me it is strange to have the ability to receive orders when basket is open. Because Koha is calculating late orders on the base of closing basket date (by the way, if basketgroups are used, the closing date of basketgroups should be use instead) I want to add a new column in the table home page for acquisitions, for orders in non closed baskets (because in my library, they are NOT sent, so they should be in a "prepared orders" column). I can make this syspref dependant of course... Regarding cancellation : maybe the problem is don't have a quick and easy way to cancel an order once the basket is closed. We need to go on Receipt page, create a invoice or reopen an existing one (even if there is no invoice in real life!), find the order and cancel it. For our colleagues, it is strange to go in "Receipt" page to cancel an order (like the 'Start' button on Windows to Shut down the computer), and strange to write an invoice number for that. An other issue : if you cancel an order before closing the basket, for us, it means the librarian changed his mind, and never send this order to the vendor. If we cancel AFER closing the basket, for us it means the vendor was not able to deliver the book. 2 very different things thats Koha manage the same way. So maybe the solution would be for example - to improve the way of cancelling ( 2 types of cancellation), and make it more obvious - to add a syspref for using the date of basket goup closing instead of baskets - to add a syspref for chosing if orders in open bakets could be received - to add a syspref for chosing if a basket not put in a basket group is to be considered closed or just in preparation (the case in my library) Sorry for these long and general remarks, they are not strictly related to the bug... Mathieu
(In reply to Nicole C. Engard from comment #5) > While I say it's required and you all think it's required - Koha does not > require it. I have received many thing when the basket was still open. > Also, you can reopen a basket and delete something you have received. So > there are multiple ways to accidentally delete something you have received. Apart from standing-order orders it is now required to close the basket in order to receive something from it. You can reopen the basket and cancel something, but it's more secure than it used to be. That said, I agree that cancelling an order should not be possible for received items. The way should be: undo receive > cancel order.
Once the received item is cancelled, the invoice is forever wrong. And it doesn’t just affect EDI invoices, rather all invoices, so a fix to this would be welcome. We are also wanting to review the opposite- whether it is possible for a cancelled item to be received, but we have to wait. (We are not ordering just now, and are having to set up a new vendor offering EDI, following the demise of our main book supplier.) Like… what does an incoming EDI invoice do when the order it is trying to receive has been cancelled? Does it force the receive? It shouldn’t be possible in a manual situation since once cancelled, there is no option for the user to select that order to receive- it would have disappeared from the receive shipment display.
Created attachment 111951 [details] [review] Bug 6819: Hide individual cancel order links when order line has been received This is to prevent mistakes - a received order should not be cancelled, but 'undo receive' needs to be done first for things to turn out correctly. To test: - Create a basket with at least 2 orders - Close basket - Receive one of your orders at least - Go back to the basket - Reopen basket - The cancel links should only show on the "not received" orders
Created attachment 111994 [details] [review] Bug 26515: Don't need to call safe_to_delete Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr>
Created attachment 111996 [details] [review] Bug 6819: Hide individual cancel order links when order line has been received This is to prevent mistakes - a received order should not be cancelled, but 'undo receive' needs to be done first for things to turn out correctly. To test: - Create a basket with at least 2 orders - Close basket - Receive one of your orders at least - Go back to the basket - Reopen basket - The cancel links should only show on the "not received" orders Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr>
Created attachment 112106 [details] [review] Bug 6819: Hide individual cancel order links when order line has been received This is to prevent mistakes - a received order should not be cancelled, but 'undo receive' needs to be done first for things to turn out correctly. To test: - Create a basket with at least 2 orders - Close basket - Receive one of your orders at least - Go back to the basket - Reopen basket - The cancel links should only show on the "not received" orders Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Nice improvement, no regresssions.. Passing QA
Pushed to master for 20.11, thanks to everybody involved!
doesnt apply cleanly to 20.05.x, no backport
*** Bug 28771 has been marked as a duplicate of this bug. ***