While working on a project for a client that was a perfect fit for the file_entity module, I came across an issue I didn't care for. Implementing hook_file_presave, the file_entity module assigns the file type based upon the first part of the mime type. This is a good thing technically, but my client's needs had to trump that. Instead they wanted two file types; image and files.
