It's not clear in the method name what it is doing. Generally, it's good to name a method as an action command. As a static factory, you shouldn't need to say "factory" but just try to describe the action as clearly as possible (with constraints on length, of course). Like createFromUpload maybe?