NotificationMailer constructor parameters changed in Flarum 1.5, override broken
Closed this issue · 2 comments
clarkwinkelmann commented
SettingsRepositoryInterface $settings
was added as a third constructor parameter to NotificationMailer
in Flarum 1.5.
public function __construct(Mailer $mailer, TranslatorInterface $translator, SettingsRepositoryInterface $settings)
It throws an error because we are not providing the correct number of parameters in our extended class:
pretty-mail/src/Overrides/NotificationMailer.php
Lines 50 to 59 in 43c8c21
Change made in flarum/framework#3525
betalk-ir commented
Hi, Any progress to fix?
I just waiting for this extension to upgrade to 1.5!
My forum just send alot mails to users and don't want to send email without our custom template.
clarkwinkelmann commented
Fixed in version 1.1.1