Trying to get property &#39;email&#39; of non-object

Level 13

show me your full code

OP

Level 1

Thanks for replying , updated it

Level 70

Try now-

Mail::to($user->email)->queur(new BlogPublished($blogByUser, $user));

OP

Level 1

still the same error, thanks for replying

Level 70

What is your error?

OP

Level 1

Level 31

What does normalizeRecipient function do? Don't you forget to return from that function? To use "return" keyword?

Level 70

I think it should be public property instead of protected.

In BlogPublished.php-

public $blog;
public $user;

Check more: https://laravel.com/docs/6.x/mail#view-data

OP

Level 1

made them public , still the same issue

Level 70

Dump your user data and show the result.

OP

Level 1

this when i dump all:

this is when i dump only $user->email:

Level 31

Try Mail::to($user) instead of Mail::to($user->email)

Level 70

Can you share your full BlogPublished.php code? Don't post image.

Use 3 tilde(`) sign before and after your code.

OP

Level 1

tried but still the same error , the error has been showing in the Mail\Mailable,php file on line:584

OP

Level 1

@tisuchi

i just made them back to protected

<?php

namespace App\Mail;

use App\Blog;
use App\User;
use Illuminate\Bus\Queueable;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Mail\Mailable;
use Illuminate\Queue\SerializesModels;

class BlogPublished extends Mailable implements ShouldQueue{
    use Queueable, SerializesModels;

    protected $blog;
    protected $user;

    public function __construct(Blog $blog, User $user)
    {
        $this->blog = $blog;
        $this->user = $user;
    }

    public function build()
    {
        $this->from(env('MAIL_FROM_ADDRESS'), env('MAIL_FROM_NAME'))
        ->subject('A new Blog has been published')
        ->view('emails.newblog')
        ->with([
            'user' => $this->user,
            'blog' => $this->blog,
        ]);

        return $this;
    }
}

Level 70

@shirshadebnath Honestly I cannot any issue so far. Can you clear your cache and try.

If still same, can you try this in your store method in the controller-

$users = User::get();

foreach($users as $user){
    echo $user->email . "<br/>";
}

dd('');

I just want to see that what's returning there.

OP

Level 1

it is returning this:

public function store(Request $request){
        //validate
        $rules = [
            'title' => ['required', 'min:10', 'max:160'],
            'content' => ['required', 'min:102'],
        ];

        $this->validate($request, $rules);
        $input = $request->all();
        //meta-stuff
        $input['slug'] =uniqid() . Str::slug($request->title, '-');
        $input['meta_title'] = Str::limit($request->title, 55);
        $input['meta_description'] = Str::limit($request->content, 155);
        //image-upload
        if ($file = $request->file('featured_image')) {
            $name = uniqid() . $file->getClientOriginalName();
            $name = strtolower(str_replace(' ', '-', $name));
            $file->move(public_path('/image/featured_image/'), $name);    
            $input['featured_image'] = $name;
        }

        //$blog = Blog::create($input);
        $blogByUser = $request->user()->blogs()->create($input);
        //sync with categories
        if ($request->category_id) {
            $blogByUser->category()->sync($request->category_id);
        }
        //mail
        $users = User::all();
        foreach($users as $user){
            echo $user->email . "<br/>";
        }
        
        dd('');

        Session::flash('blog_created_message', 'Blog Created!');

        return redirect('/blogs');
    }

Level 31

Couldn't you add dd($address, $name, $property) on 578th line of Mailable.php and try to send mail again?

Level 70

Seems good to me. Can I see your email environment variables? Do you set up everything properly?

OP

Level 1

OP

Level 1

here is the mail.php

OP

Level 1

@Martal

Output:

Code:

Level 31

Hmm, data seems correct.

Please, remove this dd and add dump($recipient) on 582th line.

OP

Level 1

@martal

Output:

Code:

Level 31

@shirshadebnath I see, that first $recipient is correct. But what about others? Please, change dd($recipient) to dump($recipient).

OP

Level 1

@martal

i did dump($recipient) but first it showed the output for a second and then redirected to error page and gave me the same error. therefore to show the output i got i dd($recipient)

thanks for patience till now

Level 31

Oh. But you can log it. Try logger('RECIPIENT', [$recipient]); and explore storage/logs/laravel.log

OP

Level 1

@martal

laravel did not create logs folder under storage

Level 31

May be you turned it off?

logs

OP