Seongbae\Discuss\Notifications\NewReplyNotification

Complexity

Total Complexity

4

Size/Duplication

Total Lines	59
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	via()	0	4	1
A	toMail()	0	9	1
A	toArray()	0	6	1

<?php

namespace Seongbae\Discuss\Notifications;

use Illuminate\Bus\Queueable;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Notifications\Messages\MailMessage;
use Illuminate\Notifications\Notification;
use Illuminate\Support\Facades\Log;
use App\Mail\TeamMemberAdded;
use Helper;

class NewReplyNotification extends Notification
{
    use Queueable;

    private $user;
    private $reply;
    private $msg;

    /**
     * Create a new notification instance.
     *
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct($reply, $msg=null)
    {
        $this->reply = $reply;
        $this->msg = $msg;
    }

    /**
     * Get the notification's delivery channels.
     *
     * @param  mixed  $notifiable
     * @return array
     */
    public function via($notifiable)

The parameter $notifiable is not used and could be removed.

    {
        return ['mail'];
    }

    /**
     * Get the mail representation of the notification.
     *
     * @param  mixed  $notifiable
     * @return \Illuminate\Notifications\Messages\MailMessage
     */
    public function toMail($notifiable)

The parameter $notifiable is not used and could be removed.

    {
        $url = route('discuss.show', ['channel'=>$this->reply->thread->channel->slug, 'thread'=>$this->reply->thread]);

        return (new MailMessage)
            ->subject($this->msg)
            ->markdown('discuss::emails.newreply',['reply'=>$this->reply,'thread'=>$this->reply->thread, 'url'=>$url]);

    }

    /**
     * Get the array representation of the notification.
     *
     * @param  mixed  $notifiable
     * @return array
     */
    public function toArray($notifiable)

The parameter $notifiable is not used and could be removed.

    {
        return [

        ];
    }
}