Uploaded image for project: 'eZ Publish / Platform'
  1. eZ Publish / Platform
  2. EZP-16543

Default Mail Transport can't be overriden in notification.ini

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Medium
    • Resolution: Obsolete
    • Affects Version/s: 4.3.0rc1
    • Fix Version/s: Future
    • Component/s: Notifications
    • Labels:
      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

        Attachments

          Activity

            People

            Assignee:
            unknown unknown
            Reporter:
            ply Yuriy Pilgun
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Dates

              Created:
              Updated: