I'm currently working on a SimpleTest suite which has just revealed the first bug: Deleting a node that contains the same file in multiple revisions does not delete the file. This is because our field_file_references() function doesn't take into account deleting an entire node, it assumes a deletion results in exactly one less reference. When deleting a node with several revisions though, that node may be responsible for multiple references.

Files: 
CommentFileSizeAuthor
filefield_delete_node.patch2 KBquicksketch

Comments

quicksketch’s picture

Status:Needs review» Fixed

Committed.

Status:Fixed» Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.