Instead of figuring out the user dir in this class, make the user pass it in as a dependency. This would eliminate the dependency on ElggFile. It's always a win when we can eliminate dependencies on classes that are dependent on the DB.