Support preset parameter in email action

[bezin]

Hi all,

thanks for this wonderful plugin. I have a minor thing that bugs me for a long time already:

We have a default email preset in all our projects, that removes the X-Mailer header to not expose the PHPMailer version.

We use your plugin to handle all our forms. Everytime we call the emailAction, we have to repeat the default email preset settings:

->emailAction([
    // […] All other email action related settings removed for brevity
    'beforeSend' => function ($mailer) {
        $mailer->XMailer = ' ';
    },
])

This PR adds the possibility to just pass along a preset parameter:

->emailAction([
    // […] All other email action related settings removed for brevity
    'preset' => 'default'
])

This way, the desired preset is just passed along to the Kirby email method and handled accordingly.

Would love to see that merged, because you can also globally define even more parameters like default sender and reply-to addresses and what not.

Thanks and cheers

[mzur]

Thanks! Do you see any way to add a test case for this?

[bezin]

Yes, of course 👍 I added two test cases and actually caught a bug that I had not anticipated in my application code I tested the changes with.

I had to refactor in order to allow loading required parameters from and to from the preset. We now load the options from the presets ourselves, rather than letting \Kirby\Cms\Email do it. I also throw an exception if you want to use an undefined preset. I reused \Kirby\Exception\NotFoundException as in the original Email class. Let me know if you would prefer a custom Uniform exception.

[mzur]

Looks good, thank you! Now if you could also extend the documentation with a description of the new option it would be perfect 👌 You should also mention that a preset can define "to" and "from", making these no longer required options in that case.

[mzur]

Perfect, thank you!