rygos / rmarchiv

app/Http/Controllers/MessagesController.php

<?php

/*
 * rmarchiv.tk
 * (c) 2016-2017 by Marcel 'ryg' Hering
 */

namespace App\Http\Controllers;

use Carbon\Carbon;
use App\Models\User;
use Cmgmyr\Messenger\Models\Thread;
use Cmgmyr\Messenger\Models\Message;
use Illuminate\Support\Facades\Input;
use Illuminate\Support\Facades\Session;
use Cmgmyr\Messenger\Models\Participant;
use Illuminate\Database\Eloquent\ModelNotFoundException;

class MessagesController extends Controller
{
    public function index()
    {
        $currentUserId = \Auth::id();

        //Alle threads laden, abgesehen von gelöscht und archivierten empfängern
        //$threads = Thread::getAllLatest()->get();

58% of this comment could be valid code. Did you maybe forget this after debugging?

        // All threads that user is participating in
        $threads = Thread::forUser($currentUserId)->latest('updated_at')->paginate(25);

The method forUser() does not exist on Cmgmyr\Messenger\Models\Thread. Did you maybe mean scopeForUser()?

        // All threads that user is participating in, with new messages
        // $threads = Thread::forUserWithNewMessages($currentUserId)->latest('updated_at')->get();

62% of this comment could be valid code. Did you maybe forget this after debugging?

        $users = User::where('id', '!=', \Auth::id())->get();

        return view('messenger.index', compact('threads', 'currentUserId', 'users'));
    }

    public function create()
    {
        $users = User::where('id', '!=', \Auth::id())->get();

        return view('messenger.create', compact('users'));
    }

    public function store()
    {
        $input = Input::get();

Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space

        $thread = Thread::create(

The method create() does not exist on Cmgmyr\Messenger\Models\Thread. Did you maybe mean created()?

            [
                'subject' => $input['subject'],
            ]
        );
        // Message
        Message::create(

The method create() does not exist on Cmgmyr\Messenger\Models\Message. Did you maybe mean created()?

            [
                'thread_id' => $thread->id,
                'user_id'   => \Auth::user()->id,
                'body'      => $input['msg'],
            ]
        );
        // Sender
        Participant::create(

The method create() does not exist on Cmgmyr\Messenger\Models\Participant. Did you maybe mean created()?

            [
                'thread_id' => $thread->id,
                'user_id'   => \Auth::user()->id,
                'last_read' => new Carbon(),
            ]
        );
        // Recipients
        foreach ($input['recipients'] as $rec) {
            Participant::create(

The method create() does not exist on Cmgmyr\Messenger\Models\Participant. Did you maybe mean created()?

                [
                    'thread_id' => $thread->id,
                    'user_id'   => $rec,
                ]
            );
        }

        return redirect('messages');
    }

    public function show($id)
    {
        if (\Auth::check()) {
            try {
                $thread = Thread::findOrFail($id);
            } catch (ModelNotFoundException $e) {
                Session::flash('error_message', 'Thema mit der ID: '.$id.' konnte nicht gefunden werden.');

                return redirect('messages');
            }
            // show current user in list if not a current participant
            // $users = User::whereNotIn('id', $thread->participantsUserIds())->get();

61% of this comment could be valid code. Did you maybe forget this after debugging?

            // don't show the current user in list
            $userId = \Auth::user()->id;
            if ($thread->hasParticipant($userId)) {
                $users = User::whereNotIn('id', $thread->participantsUserIds($userId))->get();
                $thread->markAsRead($userId);
                $messages = $thread->messages()->paginate(25);

                if (! Input::get('page')) {
                    return redirect('messages/'.$id.'?page='.$messages->lastPage());
                } else {
                    return view('messenger.show', compact('thread', 'users', 'messages'));
                }
            }
            //Todo:View für Keine Berechtigung.

Comments for TODO tasks are often forgotten in the code; it might be better to use a dedicated issue tracker.

        }
    }

    public function update($id)
    {
        if (\Auth::check()) {
            try {
                $thread = Thread::findOrFail($id);
            } catch (ModelNotFoundException $e) {
                Session::flash('error_message', 'Thema mit der ID: '.$id.' konnte nicht gefunden werden.');

                return redirect('messages');
            }
            $thread->activateAllParticipants();
            // Message
            Message::create(

The method create() does not exist on Cmgmyr\Messenger\Models\Message. Did you maybe mean created()?

                [
                    'thread_id' => $thread->id,
                    'user_id'   => \Auth::id(),
                    'body'      => Input::get('msg'),
                ]
            );
            // Add replier as a participant
            $participant = Participant::firstOrCreate(

Equals sign not aligned with surrounding assignments; expected 12 spaces but found 1 space

                [
                    'thread_id' => $thread->id,
                    'user_id'   => \Auth::user()->id,
                ]
            );
            $participant->last_read = new Carbon();
            $participant->save();
            // Recipients
            if (Input::has('recipients')) {
                $thread->addParticipant(Input::get('recipients'));
            }

            return redirect('messages/'.$id);
        }

        //Todo:View für Keine Berechtigung

Comments for TODO tasks are often forgotten in the code; it might be better to use a dedicated issue tracker.

    }
}