MailManagerReplacement::mail() doesn't always compose

You are right.

I suggest the fix is something like this:

if (!$send) {
  $email->addProcessor(function () { 
    throw new SkipMailException('Sending disabled'); 
  }, PHASE_POST_RENDER, 1000);
}

Thx for the quick reply. I'll test and come back to you later this week - if confirmation is needed ;-)

I have been playing a little bit with $email->addProcessor, but I don't seem to be able to add / call it at a valid phase - should be init (0) and I'm only able to do it the build phase (1) it seems. Could you give me some more pointers?

Yes I see what you mean. I don't have an immediate answer, I'll think about it😃

Thx for you time and effort. It seems the way forward is to avoid using MailManager (for Symfony Mailer) and rather implement support for Symfony Mailer using the new API.

I'm sorry I wish I'd been more positive. It's quite challenging maintaining this module with so many people having different requests, and sometimes I get stressed.

MailManagerReplacement is not perfect and I never intended or expected it to be perfect, but still it's being used successfully on 1000s of sites. In fact it's an extremely important part of this module, and no way can we remove it.

I found a challenge in the dialogue here that seemed to be insisting and expecting a level of perfection that in fact I never promised. As I said before, I'm willing to fix it if it's easy. If not then we have other options.