Balazs Horvath wrote:

Hi,

*attachment type* the newly introduced extension based mime-ing does not work, an old code fragment got back...: https://svn.roundcube.net/trunk/roundcubemail/program/steps/mail/attachments... instead of 'mimetype' => rc_mime_content_type($filepath, $_FILES['_attachments']['type'][$i]) should be this 'mimetype' => rc_mime_content_type($filepath, $_FILES['_attachments']['name'][$i], $_FILES['_attachments']['type'][$i])

Fixed in r2724.

*undocumented plugin hook* http://trac.roundcube.net/wiki/Plugin_Hooks $attachment = $RCMAIL->plugins->exec_hook('upload_attachment', $attachment);

Yes and there are others like save_attachment, get_attachment, remove_attachment, display_attachment and cleanup_attachments

These hooks are used for core plugins (filesystem_attachments and database_attachments) which are required to handle the storage of attachment files. However the upload_attachment hook could be of interest for other plugins too (e.g. for virus checking) and I've now added it to the documentation.

Thanks a lot for reporting!

~Thomas _______________________________________________ List info: http://lists.roundcube.net/dev/