Trying to get property of non-object in Laravel Function - php

Question

I'm making a function that each time I add a comment to an RMA Ticket it sends an email to the user but I'm getting Trying to get property of non-object when I submit the comment
Below I pasted the code from the CommentController and the Email function
if you want more information just ask and i will post what i could
I'm getting the error on this code - $mailer->sendRmaTicketComments($comment->ticket->user, Auth::user(), $comment->ticket, $comment);
CommentController Code
<?php
namespace App\Http\Controllers\Rma;
use Auth;
use App\User;
use App\Rma\RmaTicket;
use App\Rma\RmaComments;
use App\Mailers\AppMailer;
use Illuminate\Http\Request;
use App\Http\Controllers\Controller;
class RmaCommentController extends Controller
{
public function postComment(Request $request, AppMailer $mailer)
{
$this->validate($request, [
'comment' => 'required',
'user_id' => 'required',
]);
$comment = RmaComments::create([
'rma_ticket_id' => $request->input('ticket_id'),
'user_id' => $request->input('user_id'),
'comment' => $request->input('comment'),
]);
$mailer->sendRmaTicketComments($comment->ticket->user, Auth::user(), $comment->ticket, $comment);
return redirect()->back()->with('successPanel', 'Thank you for your message please allow up to 48 hours for a response.');
}
}
Mail Function Code
<?php
namespace App\Mailers;
use App\Rma\RmaTicket;
use App\Rma\RmaStatus;
use App\SupportTickets\SupportTicket;
use App\SupportTickets\SupportTicketStatus;
use App\User;
use Illuminate\Contracts\Mail\Mailer;
use Illuminate\Support\Facades\Mail;
class AppMailer
{
protected $mailer;
/**
* Get the email address
*
* #var [type]
*/
protected $to;
/**
* Set the subject for the email
*
* #var [type]
*/
protected $subject;
/**
* Set the template for the email
*
* #var [type]
*/
protected $view;
/**
* Set the data for the email
*
* #var array
*/
protected $data = [];
public function __construct(Mailer $mailer)
{
$this->mailer = $mailer;
}
/**
* Send RMA Comments/Replies to RMA Owner
*
* #param User $ticketOwner
* #param User $user
* #param rmaTicket $rmaTicket
* #param Comment $comment
*
* #return method delivered()
*/
public function sendRmaTicketComments($ticketOwner, $user, RmaTicket $rmaTicket, $comment)
{
$this->to = $ticketOwner->email;
$this->subject = "RE[RMA Ticket # $rmaTicket->rma_ticket_id]";
$this->view = 'emails.rma.rma_comments';
$this->data = compact('ticketOwner', 'user', 'rmaTicket', 'comment');
return $this->deliver();
}
RmaComments Model
<?php
namespace App\Rma;
use Illuminate\Database\Eloquent\Model;
class RmaComments extends Model
{
protected $table = 'rma_comments';
protected $fillable = [
'rma_ticket_id', 'user_id', 'comment'
];
public function ticket()
{
return $this->belongsTo(RmaTicket::class);
}
public function user()
{
return $this->belongsTo(User::class);
}
}
RMA Model
<?php
namespace App\Rma;
use Illuminate\Database\Eloquent\Model;
class RmaTicket extends Model
{
protected $table = 'rma_tickets';
protected $fillable = [
'user_id', 'rma_ticket_id', 'first_name', 'last_name',
'email', 'fax', 'company', 'marketplace_id', 'returnTypeId',
'product_name', 'quantity', 'order_number', 'address_one',
'address_two', 'city', 'state', 'zip_code', 'sku', 'model_number', 'comment',
'status_id', 'rma_number', 'refund_number', 'return_tracking_number',
'replacement_number', 'return_label'
];
public function returnType()
{
return $this->belongsTo(RmaReturnType::class);
}
public function marketplace()
{
return $this->belongsTo(RmaMarketplace::class);
}
public function status()
{
return $this->belongsTo(RmaStatus::class);
}
public function comments()
{
return $this->hasMany(RmaComments::class);
}
public function user()
{
return $this->belongsTo(User::class);
}
}

Answer 1

Looking at your code and since you have null if you dd($comment->ticket) i believe it's the naming of your foreign keys.
your belongsTo:
public function ticket()
{
return $this->belongsTo(RmaTicket::class);
}
expects an ticket_id but you named it rma_ticket_id.
You could specify a different foreign key name in the second argument in the belongTo relationship
public function ticket()
{
return $this->belongsTo(RmaTicket::class, 'rma_ticket_id');
}
As per the docs:
. Eloquent determines the default foreign key name by examining the name of the relationship method and suffixing the method name with _id. However, if the foreign key on the Phone model is not user_id, you may pass a custom key name as the second argument to the belongsTo method: