Pass Variable to Markdown from Notification through Mailable

@heisabrutalo

I think you got the variable order wrong, try

return new EmailVerification($maildata, $verifyUrl, $notifiable);	

ty i tried it, but didnt help :( I also tried this and changed everything according:

return new EmailVerification($maildata);

@heisabrutalo

Your EmailVerification's __construct($maildata, $url, $user) takes 3 variables.

Your must call new EmailVerification(...) in exactly the same order.

What is the error you getting now?

@heisabrutalo

public function build()
{
    $maildata = $this->maildata;
    return $this->markdown( 'mail.emailVerification', compact('maildata') );
}	

ty, yeah i changed that. Same Error: Undefined variable: maildata (View: messsage.blade.php)

I dont get the point, the email is working fine without the variable :(

@heisabrutalo

public function build()
{
    // dd($this->maildata); // uncomment and show output if below does not work
    return $this->markdown('mail.emailVerification')
	->with('maildata', $this->maildata);
}

Same error :(

Also if i comment the inside of the whole build() function, it shows the same Error...

@heisabrutalo

show the full error please

Sorry.

Undefined variable: maildata (View: E:\laragon\www\xxx\resources\views\vendor\mail\html\message.blade.php)

@component('mail::header', ['maildata' => $maildata])

@heisabrutalo

We are jumping the gun, what do you see when

public function build()
{
    dd($this->maildata);
}

I am going crazy. Undefined variable: maildata ... Is it possible that the AppServiceProvider is something wrong?

public function boot()
    {
        Schema::defaultStringLength(191);

        view()->composer('*', function($view) {
            $view->with('user', Auth::user() );
        });

        // Override the email notification for verifying email
            VerifyEmail::toMailUsing(function ($notifiable){
                $verifyUrl = URL::temporarySignedRoute(
                    'verification.verify',
                    Carbon::now()->addMinutes(Config::get('auth.verification.expire', 60)),
                    ['id' => $notifiable->getKey()]
                );
                $maildata = [
                    'mainTitle' => 'TEST'
                ];
                return new EmailVerification($maildata);
            });

    }

maybe at the VerifyEmail::toMailUsing?

@heisabrutalo

Why your return new not fixed?

return new EmailVerification($maildata, $verifyUrl, $notifiable);

what do u mean by not fixed? Im sending only 1 Var.

@heisabrutalo

please... you are not making sense

 public function __construct($maildata, $url, $user)
{
   ...
}

@heisabrutalo

And can please show output of below... don't keep jumping the gun

public function build()
{
    dd($this->maildata);
}

I changed this long time ago.

dd is not working, it shows the same error. Undefined variable: maildata even if i comment out everything in this build method :( dont get it...

public $maildata;

public function __construct($maildata)
    {
        $this->maildata = $maildata;
    }

@heisabrutalo

How are you calling the mail sending code?

In the AppServiceProvider.php i override the VerifyEmail::toMailUsing Method

and i call the EmailVerification Mailalbe in the boot Method.

public function boot()
    {
        Schema::defaultStringLength(191);

        view()->composer('*', function($view) {
            $view->with('user', Auth::user() );
        });

        // Override the email notification for verifying email
            VerifyEmail::toMailUsing(function ($notifiable){
                $verifyUrl = URL::temporarySignedRoute(
                    'verification.verify',
                    Carbon::now()->addMinutes(Config::get('auth.verification.expire', 60)),
                    ['id' => $notifiable->getKey()]
                );
                $maildata = [
                    'mainTitle' => 'TEST'
                ];
                return new EmailVerification($maildata);
            });

    }

@heisabrutalo

1. boot() is only defining what to do, not triggering
2. What is the URL you are using to trigger the email verification? By creating an account?

ive seen that from a tutorial... with the AppServiceProvider.php

yes registering, abc.com/register is the url

@heisabrutalo

Comment out VerifyEmail:: in your boot() and test your EmailVerification first.

Create a totally new route and test it

Route::get('/testemail', function () {
    $maildata = [
        'mainTitle' => 'TEST'
    ];
    Mail::to('[email protected]')->send(new EmailVerification($maildata));
})

dd works

array:1 [▼
  "mainTitle" => "TEST"
]

so the mailable should be fine, right?

@heisabrutalo

Ok, now try this in boot()

VerifyEmail::toMailUsing(function ($notifiable){
    $verifyUrl = URL::temporarySignedRoute(
        'verification.verify',
        Carbon::now()->addMinutes(Config::get('auth.verification.expire', 60)),
        ['id' => $notifiable->getKey()]
    );
    $maildata = [
        'mainTitle' => 'TEST'
    ];

    return (new MailMessage)->markdown('mail.emailVerification')->with('maildata', $maildata);
});

Same Error :(

Undefined variable: maildata (View: E:\laragon\www\xxx\resources\views\vendor\mail\html\message.blade.php)

at: @component('mail::header', ['maildata' => $maildata])

@heisabrutalo Sent your data to the component in mail.emailVerification like

@component('mail::message', compact('maildata'))

Which version of Laravel?

"laravel/framework": "^7.0",

Guys, it works! ty

I made the mistake to change the var maildata in component in vendor/mail/html/message.blade.php and not in the Mailable template....

This was so confusing with the AppServiceProvider, so i dont need it anymore?

Guys, im getting crazy. Again i cannot pass a Variable from the Notification to the Mailable, what am i doing? Is it me or sometimes we cannot mix classes and mailables and notifications?

toMail function in the Notification

public function toMail($notifiable)
    {
        $verificationUrl = $this->verificationUrl($notifiable);
        /*
        return (new MailMessage)
            ->subject(Lang::get('user.emailVerificationTitle'))
            ->action(Lang::get('user.emailVerification'), $verificationUrl)
            ->line(Lang::get('user.emailVerificationNotRequested'));
        */

        $maildata = [
            'mainTitle' => __('mail.confirmation'),
            'url' => $verificationUrl
        ];

        return (new EmailVerification)
            ->subject(Lang::get('user.emailVerificationTitle'))
            ->locale(session()->get('locale'))
            ->to($notifiable->email)
            ->with(['maildata', $maildata]);
    }

Mailable, im passing it right?

class EmailVerification extends Mailable
{
    use Queueable, SerializesModels;

    public $maildata;

    /**
     * Create a new message instance.
     *
     * @return void
     */
    public function __construct($maildata)
    {
        $this->maildata = $maildata;
    }

    /**
     * Build the message.
     *
     * @return $this
     */
    public function build()
    {
        dd($this->maildata);
        return $this->markdown( 'mail.emailVerification');
    }
}

Error: Too few arguments to function App\Mail\EmailVerification::__construct(), 0 passed 1 expected.

How can i pass data from Notification to Mailable? Sounds like the created array in the Notification is empty, but why?

ty

@heisabrutalo

1. PAY ATTENTION to your __construct()
2. How many variables are there in __construct()? 1, 2 or 3?
3. When you create a NEW instance return (new EmailVerification($maildata)), number of variables MUST match

        return (new EmailVerification($maildata))
            ->subject(Lang::get('user.emailVerificationTitle'))
            ->locale(session()->get('locale'))
            ->to($notifiable->email);