Comment 14 for bug 659540

now (March 23rd. 2011) it's point 3.8. and page 5
********* quote ***********
Unveränderbarkeit
(20)
Eine Buchung darf nicht in einer Weise verändert werden, dass der ursprüngliche
Inhalt nicht mehr feststellbar ist. Daher sind spätere Veränderungen ausschließlich
so vorzunehmen, dass sowohl der ursprüngliche Inhalt als auch die Tatsache, dass
Veränderungen vorgenommen wurden, für einen sachverständigen Dritten in
angemessener Zeit nachvollziehbar sind. Ab dem technischen Buchungszeitpunkt
darf eine Buchung bei Verwendung der regulären Anwendungsfunktionalität nur
mehr über eine Stornobuchung rückgängig gemacht werden. Änderungen vor dem
technischen Buchungszeitpunkt sind hiervon nicht betroffen.
********* end quote ***********
I hope this is clear enough

Hence - account_move with state "Posted" most not be modified.

my point is - we should distinguish between cancel and delete

cancel - must be done using reversal postings - and this function is needed.
delete - must not be done/used. (especially not if partners are involved)
may be once the audit module works and traces deletion of account_move_lines it can be considered as a technically valid option. (but not accounting)

the module name suggests that it cancels, but it deletes posted moves

I also acknowledge, that it might be desirable to alter some less important fields in invoices, but this opens doors to fraud.

Because of this I also recommend to implement rules not to allow deletion of attached invoices, pickings, sales order, purchase order etc. once these are confirmed.