Merge lp:~hirt/ocb-addons/6.1_documents_fix_unlink_files into lp:ocb-addons/6.1
Proposed by
Etienne Hirt
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 6816 | ||||
Proposed branch: | lp:~hirt/ocb-addons/6.1_documents_fix_unlink_files | ||||
Merge into: | lp:ocb-addons/6.1 | ||||
Diff against target: |
43 lines (+18/-8) 1 file modified
document/document.py (+18/-8) |
||||
To merge this branch: | bzr merge lp:~hirt/ocb-addons/6.1_documents_fix_unlink_files | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Stefan Rijnhart (Opener) | Approve | ||
Pedro Manuel Baeza | Approve | ||
Review via email:
|
Description of the change
Fix to avoid deleting files that still are referenced from documents. This fix is for 6.1 only, because in 7.0 the document module is redesigned completely.
To post a comment you must log in.
Hi, Etienne, thank you very much for this patch. Indeed, is a needed change, but as you can see in the bug, the patch proposed by OpenERP is not the same as the want you have proposed, so if in the future OpenERP merge their patch into official branch, we will get into conflicts. Is there any problem with the OpenERP patch?
I have also target the bug to our branches to control the status. Please see how it's done for future references on other patches that you propose.
Regards.