Mandrill-driver together with css-inliner

[juukie]

Hi! First of all thanks for creating and maintaining this package :-).

I've installed it together with therobfonz/laravel-mandrill-driver. Both packages use Laravel's auto-discovery feature.

The css-inliner package boots before the mandrill-driver package, but since the css-inliner needs the SwiftMailer it tries to build the mail (Mandrill) transport which is not registered in the mail manager yet.

$this->app->extend('mail.manager', function (MailManager $mailManager) {
    $mailManager->getSwiftMailer()->registerPlugin($this->app->make(CssInlinerPlugin::class));
    return $mailManager;
});

To make both packages work I tried to add them to composer.json's dont-discover array:

"dont-discover": [
    "fedeisas/laravel-mail-css-inliner",
    "therobfonz/laravel-mandrill-driver" 
]

and register their service providers manually via config/app.php:

/** Package Service Providers... */
\LaravelMandrill\MandrillServiceProvider::class,
\Fedeisas\LaravelMailCssInliner\LaravelMailCssInlinerServiceProvider::class,

but it did not solve the problem straight away. Only when I moved

$this->app->extend('mail.manager', function (MailManager $mailManager) {
    $mailManager->getSwiftMailer()->registerPlugin($this->app->make(CssInlinerPlugin::class));
    return $mailManager;
});

into LaravelMailCssInlinerServiceProvider::boot().

Since I shouldn't change the files in /vendor, I've added

$this->app->register(\Fedeisas\LaravelMailCssInliner\LaravelMailCssInlinerServiceProvider::class);

to AppServiceProvider::boot().
So the LaravelMailCssInlinerServiceProvider will be registered after the laravel-mandrill-driver.

Both packages used to work together in Laravel 6. Laravel 7's changes on the mailing system seems to be the problem. Not sure if the solution to this issue would be to just move the extending of the mail.manager into boot() since therobfonz/laravel-mandrill-driver is auto-discovered after fedeisas/laravel-mail-css-inliner.

[lbeauvisage]

i Have the same problem with vemcogroup/laravel-sparkpost-driver

with this error

InvalidArgumentException
Unsupported mail transport [sparkpost].

[Gimcrack]

I had the same issue with the sparkpost driver but was able to work around it by not auto-discovering laravel-mail-css-inliner.

// composer.json
...
"extra": {
  "laravel": {
    "dont-discover": [
      "fedeisas/laravel-mail-css-inliner"
    ]
  }
},

// app.config
...
'providers' => [
  ...
  Fedeisas\LaravelMailCssInliner\LaravelMailCssInlinerServiceProvider::class,
  ...
]

Don't forget to composer dump-autoload after making the change.

[fedeisas]

#90 should fix this.