realpath(): Passing null to parameter #1 ($path) of type string is deprecated in /var/www/html/sites/all/modules/contrib/htmlmail/htmlmail.tpl.php line 110

I ran an audit on the whole module and found two additional files which need patching, so have created a new version.

Thank you for the patches!

Why does $_SERVER['SERVER_SOFTWARE'] need to be cast to string?

Is this already fixed in the D8 version?

I think that there are cases where the $_SERVER['SERVER_SOFTWARE'] is not defined and the patch is trying to "fix" that by casting to a string. Proper fix should be isset (see for example here, where the similar fix was made in D7 core: #2356055: Notice in includes/mail.inc define → ). But as I have mentioned in #4, I am not entirely sure that the fix to $_SERVER['SERVER_SOFTWARE'] is in scope of this issue.

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