There is currently no way to delete unused invoices (for example, invoices created by mistake), and there really should be, since errors and absent-mindedness can result in numerous empty invoices over the course of years.
Created attachment 18590 [details] [review] Bug 10390: Add ability to delete empty invoices There is currently no way to delete unused invoices (for example, invoices created by mistake), and there really should be, since errors and absent-mindedness can result in numerous empty invoices over the course of years. To test: 1) Apply patch. 2) Create three invoices in the Acquisitions module. For one of them, receive at least one item. For the other two, do not receive any items. 3) View one of the invoices that does not have any items on it. 4) Try to delete it. This should succeed. 5) View the invoice that has an item. There should not be any option to delete it. 6) Do an invoice search that brings up the other invoice with no items on it. Try to delete it from the results page. This should succeed. 7) Run the unit test: > prove t/Acquisition/Invoice.t 8) Sign off.
Created attachment 18596 [details] [review] [SIGNED-OFF] Bug 10390: Add ability to delete empty invoices There is currently no way to delete unused invoices (for example, invoices created by mistake), and there really should be, since errors and absent-mindedness can result in numerous empty invoices over the course of years. Signed-off-by: Srdjan <srdjan@catalyst.net.nz>
It's not abig deal, it's empty invoices, but maybe making a delete button and separating Delete link a bit on the list page? Alternatively a confirm pop-up? Not fussed if the answer is no.
(In reply to comment #3) > It's not abig deal, it's empty invoices, but maybe making a delete button > and separating Delete link a bit on the list page? Alternatively a confirm > pop-up? Not fussed if the answer is no. If you have any thoughts about how it would be best to do that, I'd be happy to make that change. The reason I did not do that initially is that I wanted to keep the user interface consistent and streamlined, and I was concerned that adding an entire additional step would make users' lives more difficult without really providing them with any protection.
Sure, that was just an idea, I'd actually go for adding an extra nbsp; or something before delete. But as I say, if you wish to leave it like it is now no big deal, I'll sign it off.
Created attachment 19036 [details] [review] [PASSED QA] Bug 10390: Add ability to delete empty invoices There is currently no way to delete unused invoices (for example, invoices created by mistake), and there really should be, since errors and absent-mindedness can result in numerous empty invoices over the course of years. To test: 1) Apply patch. 2) Create three invoices in the Acquisitions module. For one of them, receive at least one item. For the other two, do not receive any items. 3) View one of the invoices that does not have any items on it. 4) Try to delete it. This should succeed. 5) View the invoice that has an item. There should not be any option to delete it. 6) Do an invoice search that brings up the other invoice with no items on it. Try to delete it from the results page. This should succeed. 7) Run the unit test: > prove t/Acquisition/Invoice.t 8) Sign off. Signed-off-by: Srdjan <srdjan@catalyst.net.nz> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> All tests and QA script pass. I also did another test: I cancelled all receipts from an existing invoice and then could successfully delete it in the last step.
Pushed to master. Thanks, Jared!