Thomas Bruederli wrote:
To get best of both worlds, we could create another hook like 'open_filew' which can be called before 'save_attachment' and will return a file pointer, in case the plugin works with files. The 'save_attachment' hook will then pass the file pointer in the 'data' field. If the 'open_filew' hook doesn't return a file pointer, we keep the current way of passing the file contents as 'data'.
What do you think about this?
Second option, since we have attachments handler plugin in requirements, is to use config variable for that.
I don't understand. What do you want to configure about this?
Because both plugins could handle both methods of file handling. So, we don't need another plugin, but just an option.