for testing and deploying your application
for finding and fixing issues
for empowering human code reviews
<?php
namespace App\Notifications;
use App\Models\TechTip;
use App\Models\TechTipComment;
use App\Models\User;
use Illuminate\Bus\Queueable;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Notifications\Messages\MailMessage;
use Illuminate\Notifications\Notification;
class NewTechTipCommentNotification extends Notification
{
use Queueable;
protected $comment;
protected $user;
protected $tip;
/**
* Create a new notification instance
*/
public function __construct(TechTipComment $comment, User $user)
$this->comment = $comment;
$this->user = $user;
$this->tip = TechTip::find($comment->tip_id);
}
* Get the notification's delivery channels
public function via($notifiable)
$notifiable
If this is a false-positive, you can also ignore this issue in your code via the ignore-unused annotation
ignore-unused
public function via(/** @scrutinizer ignore-unused */ $notifiable)
This check looks for parameters that have been defined for a function or method, but which are not used in the method body.
return ['mail'];
* Get the mail representation of the notification
public function toMail($notifiable)
public function toMail(/** @scrutinizer ignore-unused */ $notifiable)
return (new MailMessage)
->greeting('Hello')
->line('User '.$this->user->full_name.' has commented on Tech Tip - .'.$this->tip->subject)
->line('The comment is: ')
->line($this->comment->comment)
->action('Click Here to review the comment', url(route('tech-tips.show', $this->tip->slug)));
* Get the array representation of the notification.
*
* @param mixed $notifiable
* @return array
// public function toArray($notifiable)
// {
// return [
// //
// ];
// }
butcherman /
Tech_Bench
Loading history...
This check looks for parameters that have been defined for a function or method, but which are not used in the method body.