Details
-
Improvement
-
Resolution: Obsolete
-
Medium
-
4.3.0rc1
-
None
Description
Mail transport setting in notification.ini surely needs refactoring
Firstly default setting points to non-existing transport
[TransportSettings] DefaultTransport=mail
instead of
[TransportSettings] DefaultTransport=ezmail
Also all kernel code directly specifies used transport,
$transport = eZNotificationTransport::instance( 'ezmail' );
instead of
$transport = eZNotificationTransport::instance();
which make 'DefaultTransport' setting useless.
/kernel/classes/notification/handler/ezcollaborationnotification/ezcollaborationnotificationhandler.php
/kernel/classes/notification/handler/ezgeneraldigest/ezgeneraldigesthandler.php
/kernel/classes/notification/handler/ezsubtree/ezsubtreehandler.php