Add mail service

[pi-engine]

Add mail service

[voltan]

Other needed option . use and manage mail template ( html and text )

[taiwen]

Implemented in commit 369a3f3

[voltan]

I will test it in contact module. Thanks

[voltan]

In Mail service -> message function, setTo ( or addTo ) method missing. And it good if add $message->setEncoding($charset);

[taiwen]

The service->message() returns a Zend\Message object and it already has the setTo/setEncoding methods.

[voltan]

I am sorry, I use this method and set other options after return object
thanks

[taiwen]

Are we okay to close the issue? @voltan @MarcoXoops

[voltan]

In localhost ( fedora : php 5.4.14 mysql 5.5.30 apache 2.4 But I am not sure my system mail service work will or not )I have problem whit : mimeMessage function, this part :

        $part = function ($content) use ($this)
        {
            if (is_string($content)) {
                $content = new Mime\Part($content);
            }
            if (is_array($content)) {
                list($data, $type) = $content;
                $content = $this->mimePart($data, $type);
            }
            return $content;
        };

White screen whiteout error. But I don't check on live server .I will check tomorrow

[taiwen]

I guess it is caused by using _$this_ directly in anonymous function.

[voltan]

For test I changed

    public function mimeMessage($parts = array())
    {
        $message = new Mime\Message;
        $part = function ($content) use ($this)
        {
            if (is_string($content)) {
                $content = new Mime\Part($content);
            }
            if (is_array($content)) {
                list($data, $type) = $content;
                $content = $this->mimePart($data, $type);
            }
            return $content;
        };
        $parts = (array) $parts;
        foreach ($parts as $content) {
            $message->addPart($part($content));
        }

        return $message;
    }

To

    public function mimeMessage($parts = array())
    {
        $message = new Mime\Message;
        $part = $this->mimeMessagePart($content);
        $parts = (array) $parts;
        foreach ($parts as $content) {
            $message->addPart($part($content));
        }

        return $message;
    }

function mimeMessagePart($content)
        {
            if (is_string($content)) {
                $content = new Mime\Part($content);
            }
            if (is_array($content)) {
                list($data, $type) = $content;
                $content = $this->mimePart($data, $type);
            }
            return $content;
        };

And it worked for me

[taiwen]

See updates in commit 77448fe

[voltan]

Let me time to check

[taiwen]

Ok, I am also going to do test with my linode stack.

[Marc-pi]

hello,
about See var/config/service.mail.php
what is default value? phpmail?
there is no mail method setting anymore, can you explain how it works now?

[taiwen]

Default transport is sendmail through PHP mail function.
sendmail settings are configured in php.ini
Options for other transports can be set in service.mail.php, but only options defined by Zend/Mail are accepted.

[voltan]

I test it, on server send mail system work will and without problem, Just it have problem in submitted mail, mail body set as text file and attached to the email, ( I send an example mail for you )

        // Set subject and body
        $subject = sprintf(__('Contact to %s , Subject : %s'), $values['department_title'], $values['subject']);
        $body = $values['message'];
        // Set message
        $message = Pi::service('mail')->message($subject, $body, 'text/html');
        $message->addTo($values['department_email'], $values['department_title']);
        $message->setEncoding("UTF-8");
        // Send mail
        $transport = Pi::service('mail')->transport();
        $transport->send($message);

I test whit template too, it use template very will , but again send body as attach file and I don't know how I can set information on template

[taiwen]

Mail service improved in commit 45d7947
BTW, I can not reproduce the attachment issue.