pass AbstractFile as part of FILE_DONE event

Rather than just passing the file id in the FILE_DONE event, pass to AbstractFile so listeners don't have to go to the db redundantly.  

I left in the old method that only pass the file id, but to my knowledge it is not used anywhere. Shoul it  be removed?