Hi there,

thanks for the plugin. This is rather a question. I am experiencing issues using serialize type with Postgres, so I'll shift my implementation to json.

Were there any security concerns or drawback using json? What is the motivation behind using serialize per default?

This issue addresses deprecation messages introduced in CakePHP 4.3.

Additionally, the fixture management is now handled by migrations.

A pull request will follow to have these deprecation messages solved.

The database has from_name and from_email fields, and I need those set to get

set.

Is there a way to set these using enqueue?

Notice : Undefined index: template [ROOT/vendor/lorenzo/cakephp-email-queue/src/Model/Table/EmailQueueTable.php, line 61]

tag: 3.3.1 (compatible with cake 3)

solved in version 4
tag: 4.0.0 (compatible with cake 4)

could you fix and do new tag for cake 3 ?
no base branch for it and I could not open a PR

Why you do not use migration instead of using the shema to create the table?

Is this is a better way to do or is it just that migration are not available in the plugins?

If create email in table email_queue in column send_at is sam date as column created.
after send is set column sent to 1 bad column send_at not changed.

Just noticed that I can not use Html Helper or Url Helper with full base links.
The domain, reverts to localhost

These lines appear to have the effect of "if the specified layout is default, then the specified configName, template, and layout will be ignored when the email is being sent and replaced with values provided in the command line or defined in SenderShell::getOptionParser()".

I'm trying to use this plugin and I'm not able to use any template file or config value other than default while the layout is default.

Can you confirm that this is a bug, or is this the expected behavior?

modified col should be change to null or delete it from col list
mysql 5.7 can not receive 0000-00-00 00:00:00
https://github.com/lorenzo/cakephp-email-queue/blob/master/config/Schema/schema.sql#L20 and ...

Hello,

There is no option/functionality to set reply-to. In various company they use one mail id to send mails.
So what basic idea is generally if some one get an email from sender, the concern person i.e receiver can reply to the sender and vice-versa.

Now using this email queue functionality which you have given there is no facility to set reply-to. So how to do this.

Kindly help.

It would be handy to retrieve the EmailQueue.id when adding to the queue is successful.

I've just discovered that exchange will consider as spam an email without a messageId.

Is there a reason why you set it to false in SenderShell.php?
Line 122
my proposal is to set it to true (or not set at all to use the default)

                $sent = $email
                    ->setTo($e->email)
                    ->setSubject($e->subject)
                    ->setEmailFormat($e->format)
                    ->addHeaders($headers)
                    ->setViewVars($viewVars)
                    ->setMessageId(true)
                    ->setReturnPath($email->getFrom());

See the results of the recipt

                              X-Spam-Flag: NO
                              
                              X-Spam-Score: -2.512
                              
                              X-Spam-Level:
                              
                              X-Spam-Status: No, score=-2.512 tagged_above=-9999 required=5

               tests=[ALL_TRUSTED=-1, BAYES_00=-1.9, DKIM_SIGNED=0.1,

               DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1,

               HTML_MESSAGE=0.001, MIME_HTML_ONLY=0.1, **MISSING_MID=0.497**,

               SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01, URIBL_BLOCKED=0.001]

               autolearn=ham autolearn_force=no

Dear Lorenzo.
Why we have to use

'send_at' => new FrozenTime('now', 'UTC'),

instead of

'send_at' => new FrozenTime('now'),

I think we should unified default timezone configured in bootstrap.php

Great plugin! I really appreciate your works. Thank you.

I am using CakePHP 4.

When making a test from my app, sending email from person A to person B, both have the same email address (of mine), I received 2 identical emails.

And I notice that the time EmailQueue.sender takes to send 1 email is longer than the time I send the same email template from my App controller action (send directly). When I use this direct approach, I only get 1 email instead of 2.

I think EmailQueue send 2 separate emails. Instead of adding recipient to To, it actually create new email.

Is there any way to configure the maximum retry count via bootstrap.php or wherever?
I would like to request that this library should have an option to configure the maximum count.

@lorenzo

Is there a way to set the language using i18n of the email when adding to the queue? Or is the solution to just create language specific templates?

How can i add attachments to emails that need to be send ?

Thank you

This plugin looks like it will do exactly what I need, but there is no license specified. In order to use it at my work this needs to be made explicit (MIT license ideally).

How about changing text type to longtext type?

https://github.com/cakephp/cakephp/blob/13ab4d788ce553056938b60b3039197b8edd9993/tests/TestCase/Database/Schema/MysqlSchemaTest.php#L143-L144

My Configurations:

$ php --version 
PHP 5.6.29 (cli) (built: Dec  9 2016 07:03:56) 

$ bin/cake --version
3.4.5

While checking preview got these errors:

[Exception: Transport config "Debug" is missing. in [/my/project/path/vendor/cakephp/cakephp/src/Mailer/Email.php, line 1525]

2017-04-14 12:21:58 Error: [InvalidArgumentException] Transport config "Debug" is missing. in /my/project/path/vendor/cakephp/cakephp/src/Mailer/Email.php on line 1525

After going through code, I found that it is expected to create a Email config named 'Default' to preview emails. It could have saved some time if it was mentioned in the documentation. Much better would be to make it configurable - so that developer can change it to whatever without editing the code.

When I created 'Debug' transport, I got following error:

PHP Fatal error:  Cannot use object of type stdClass as array in /my/project/path/vendor/lorenzo/cakephp-email-queue/src/Shell/previewShell.php on line 71

Also noted that it is sending emails in preview mode. It would have been really helpful if sending emails was optional. Sometimes we don't want to send mails, but view only the content is being formatted properly.

So in short:

• Remove hardcoded word 'Debug', make it user configurable
• Fix use of array for Object
• Sending emails in preview should be optional

Thanks for this useful plugin!
Is there a way to set cc and bcc?

Hello,

Do you have any plans on upgrading this plugin to work with Cakephp 3.6?
Thanks!

Pleas create timestamp where email is realy send
Pleas create function for console, where delete sended records.
Thenks

When I run

composer require lorenzo/cakephp-email-queue

I get a old version of the plugin. Says it's Installing lorenzo/cakephp-email-queue (3.1.3). Which I guess is the latest numbered version. Is current master ready to bump to 3.1.4?

Jose,
would you consider adding a user defined callback function in SenderShell::main() for template_vars - before send() is called?

I use Emojione, and in db I store icons in so called 'short' format i.e. : thumbs : : zap : : computer :,
but email requires icons in Unicode format.
I need to convert them between db read and send().

Do you think this would be useful also for others?

I can try to implement it and create pull request.
If so, please give me a hint where to store the user function? Should I add it to the OptionParser?