Exception when attaching a non-existing file

This looks good. I think the check in the embedded images method is probably not needed. Embedded images are validated before this step, and there's already a check in there for file_exists(). But, I don't think it hurts to leave it in, since I suppose there could be another type of error reading the file, and we don't want a fatal error in that case.

The only thing I added is a couple of log error messages when we skip a file. Since we're silently failing in this case, I think it's a good idea to throw an error in the logs. I do get a PHP warning in the logs from file_get_contents(), but I'm not sure if everyone would get those depending on their PHP configuration.

Thanks for this. Merging now.

I released version 1.0.1 with this fix.

Thanks!

Automatically closed - issue fixed for 2 weeks with no activity.