Transmissor\Http\Api\ChatController

Complexity

Total Complexity

4

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	history()	0	5	1
A	send()	0	8	1
A	index()	0	9	1

<?php

namespace Transmissor\Http\Api;

use Illuminate\Http\Request;

class ChatController extends Controller
{
    /**
     * Create a new controller 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()
    {
        $this->middleware('auth');
    }
    
    /**
     * Show the application dashboard.
     *
     * @return \Illuminate\Http\Response
     */
    public function index(Request $request)
    {
        $conversations = Chat::conversation(Chat::conversations()->conversation)
          ->for(auth()->user())

The method user does only exist in Illuminate\Contracts\Auth\Guard, but not in Illuminate\Contracts\Auth\Factory.

          ->get()
          ->toArray()['data'];
        $conversations = array_pluck($conversations, 'id');
        return view('home', compact('conversations'));
    }

    public function history($toId)
    {
        $user = Auth::getToken();
        $conversation = Chat::conversations()->between($user, User::find($toId));

$conversation is not used, you could remove the assignment.

    }

    public function send($toId)
    {
        $user = Auth::getToken();
        $message = Chat::message('Hello')

$message is not used, you could remove the assignment.

            ->from($user)
            ->to($conversation)

The variable $conversation does not exist. Did you forget to declare it?

            ->send();
    }
}