App\Controller\ConversationsController

Complexity

Total Complexity

94

Size/Duplication

Total Lines	1176
Duplicated Lines	13.1 %

Coupling/Cohesion

Components	1
Dependencies	16

Importance

Changes

0

18 Methods

Rating	Name	Duplication	Size	Complexity
C	create()	0	141	11
B	view()	0	82	2
B	inviteMember()	0	24	3
A	quote()	0	55	3
B	messageEdit()	0	53	6
C	kick()	0	82	7
A	initialize()	0	6	1
B	beforeFilter()	0	12	5
B	index()	0	32	1
A	go()	48	48	3
B	getEditMessage()	8	56	5
C	action()	55	110	12
C	reply()	5	101	11
B	edit()	14	49	6
C	invite()	5	95	11
A	leave()	9	47	3
A	search()	10	50	3
A	maintenance()	0	3	1

<?php
namespace App\Controller;

use App\Event\Notifications;
use Cake\Core\Configure;
use Cake\Database\Expression\QueryExpression;
use Cake\Event\Event;
use Cake\I18n\Time;
use Cake\Network\Exception\NotFoundException;
use Cake\Routing\Router;

class ConversationsController extends AppController
{

    /**
     * Initialization hook method.
     *
     * @return void
     */
    public function initialize()
    {
        parent::initialize();

        $this->loadComponent('RequestHandler');
    }

    /**
     * BeforeFilter handle.
     *
     * @param Event $event The beforeFilter event that was fired.
     *
     * @return void
     */
    public function beforeFilter(Event $event)
    {
        parent::beforeFilter($event);

        $this->Auth->deny();

        if (Configure::read('Conversations.enabled') === false && $this->request->action != 'maintenance') {
            $this->redirect(['action' => 'maintenance']);
        } elseif (Configure::read('Conversations.enabled') === true && $this->request->action == 'maintenance') {
            $this->redirect(['action' => 'index']);
        }
    }

    /**
     * Display all conversations for the user.
     *
     * @return void
     */
    public function index()
    {
        $this->ConversationsUsers = $this->loadModel('ConversationsUsers');

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __set, maybe consider adding a @property annotation.

        $this->paginate = [
            'maxLimit' => Configure::read('Conversations.conversations_per_page')
        ];
        $conversations = $this->ConversationsUsers

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'Users',
                'Conversations',
                'Conversations.Users' => function ($q) {
                    return $q->find('medium');
                },
                'Conversations.LastMessage',
                'Conversations.LastMessageUser'
            ])
            ->where([
                'ConversationsUsers.user_id' => $this->Auth->user('id'),
                'Conversations.conversation_open <>' => 2
            ])
            ->order([
                'ConversationsUsers.is_read' => 'ASC',
                'ConversationsUsers.is_star' => 'DESC',
                'Conversations.last_message_date' => 'DESC',
            ]);

        $conversations = $this->paginate($conversations);

        $this->set(compact('conversations'));
    }

    /**
     * Function to do an action when a user select a/many conversations.
     *
     * Action list :
     *      star : Conversations Important
     *      normal : Make normal conversation
     *      exit : Exit conversations
     *
     * @return void
     *
     * @throws NotFoundException
     */
    public function action()
    {
        if (!$this->request->is('ajax')) {
            throw new NotFoundException();
        }

        $actionAllowed = ['star', 'normal', 'exit'];

        if (!array_key_exists('action', $this->request->getParsedBody()) || !in_array($this->request->getData('action'), $actionAllowed)) {

This code seems to be duplicated across your project.

            $json = [];
            $json['error'] = '1';
            $json['message'] = __d('conversations', 'Action unknown.');

            $this->set(compact('json'));
            $this->set('_serialize', 'json');

            return;
        }

        if (!array_key_exists('conversations', $this->request->getParsedBody())) {

This code seems to be duplicated across your project.

            $json = [];
            $json['error'] = '1';
            $json['message'] = __d('conversations', 'You have not chosen any conversations.');

            $this->set(compact('json'));
            $this->set('_serialize', 'json');

            return;
        }

        $this->loadModel('ConversationsUsers');

        $action = $this->request->getData('action');
        $array = $this->request->getData('conversations');

        switch ($action) {
            case "star":

This code seems to be duplicated across your project.

                foreach ($array as $conversationId) {

The expression $array of type null|string|array is not guaranteed to be traversable. How about adding an additional type check?

                    $this->ConversationsUsers->updateAll(

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                        ['is_star' => 1],
                        [
                            'conversation_id' => $conversationId,
                            'user_id' => $this->Auth->user('id')
                        ]
                    );
                }

                $json['message'] = __d('conversations', 'Your conversation(s) has been Stared.');

$json was never initialized. Although not strictly required by PHP, it is generally a good practice to add $json = array(); before regardless.

                $json['error'] = '0';
                $json['redirect'] = Router::url(['action' => 'index']);

                $this->set(compact('json'));

                break;

            case "normal":

This code seems to be duplicated across your project.

                foreach ($array as $conversationId) {

The expression $array of type null|string|array is not guaranteed to be traversable. How about adding an additional type check?

                    $this->ConversationsUsers->updateAll(

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                        ['is_star' => 0],
                        [
                            'conversation_id' => $conversationId,
                            'user_id' => $this->Auth->user('id')
                        ]
                    );
                }

                $json['message'] = __d('conversations', 'Your conversation(s) has been set normal.');

$json was never initialized. Although not strictly required by PHP, it is generally a good practice to add $json = array(); before regardless.

                $json['error'] = '0';
                $json['redirect'] = Router::url(['action' => 'index']);

                $this->set(compact('json'));
                break;

            case "exit":
                foreach ($array as $conversationId) {

The expression $array of type null|string|array is not guaranteed to be traversable. How about adding an additional type check?

                    $user = $this->ConversationsUsers

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                        ->find()
                        ->contain([
                            'Conversations'
                        ])
                        ->where([
                            'ConversationsUsers.user_id' => $this->Auth->user('id'),
                            'ConversationsUsers.conversation_id' => $conversationId
                        ])
                        ->first();

                    //Check if the user is the owner of the conversation.
                    if ($user->conversation->user_id == $this->Auth->user('id')) {
                        $conversation = $this->Conversations->get($conversationId);

The property Conversations does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                        $conversation->conversation_open = 2;
                        $this->Conversations->save($conversation);

The property Conversations does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                    } else {
                        $this->ConversationsUsers->delete($user);

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                        $conversation = $this->Conversations->get($conversationId);

The property Conversations does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                        $conversation->recipient_count = $user->conversation->recipient_count - 1;
                        $this->Conversations->save($conversation);

The property Conversations does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                    }
                }

                $json['message'] = __d('conversations', 'You have left your conversation(s) successfully.');

$json was never initialized. Although not strictly required by PHP, it is generally a good practice to add $json = array(); before regardless.

                $json['error'] = '0';
                $json['redirect'] = Router::url(['action' => 'index']);

                $this->set(compact('json'));
                break;
        }

        $this->set('_serialize', 'json');
    }

    /**
     * Create a new conversation.
     *
     * @return void
     */
    public function create()
    {
        $this->loadModel('Conversations');
        $conversation = $this->Conversations->newEntity($this->request->getParsedBody(), ['validate' => 'create']);

The property Conversations does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

        if ($this->request->is('post')) {
            $users = str_replace(",", "", trim(strtolower($this->request->getData('users'))));
            $users = explode(" ", $users);

            //Check max users.
            if (!(count($users) <= Configure::read('Conversations.max_users_per_conversation'))) {
                $this->Flash->error(__d('conversations', 'You cannot invite more than {0} user(s) in this conversation.', Configure::read('Conversations.max_users_per_conversation')));
                $this->set(compact('conversation'));

                return;
            }

            $userMiniCount = false;
            $this->loadModel('Users');

            //We check if at least one user in all list exist.
            foreach ($users as $user) {
                $userCheck = $this->Users

The property Users does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                    ->find()
                    ->where([
                        'LOWER(Users.username)' => $user
                    ])
                    ->first();

                //If an user exist and if the user is not the own of the conversation.
                if ($userCheck && $userCheck->id != $this->Auth->user('id')) {
                    $userMiniCount = true;
                    break;
                }
            }

            if ($userMiniCount === false) {
                $this->Flash->error(__d('conversations', 'Please enter at least one valid recipient.'));
                $this->set(compact('conversation'));

                return;
            }

            $conversation->user_id = $this->Auth->user('id');
            $conversation->reply_count = 1;
            $conversation->recipient_count = 1;
            $conversation->last_message_user_id = $this->Auth->user('id');
            $conversation->last_message_date = new Time();

            if ($conversation = $this->Conversations->save($conversation)) {

The property Conversations does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                $this->loadModel('ConversationsMessages');
                $this->loadModel('ConversationsUsers');

                $data = [];
                $data['message'] = $this->request->getData('message');
                $data['conversation_id'] = $conversation->id;
                $data['user_id'] = $this->Auth->user('id');

                $entity = $this->ConversationsMessages->newEntity($data);

The property ConversationsMessages does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                $message = $this->ConversationsMessages->save($entity);

The property ConversationsMessages does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                $data = [];
                $data['conversation_id'] = $conversation->id;
                $data['user_id'] = $this->Auth->user('id');

                $entity = $this->ConversationsUsers->newEntity($data);

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                $user = $this->ConversationsUsers->save($entity);

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                $this->Conversations->updateAll(

The property Conversations does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                    [
                        'first_message_id' => $message->id,
                        'last_message_id' => $message->id
                    ],
                    [
                        'id' => $conversation->id
                    ]
                );

                //Save all invited users.
                foreach ($users as $user) {
                    $userExist = $this->Users

The property Users does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                        ->find()
                        ->where([
                            'LOWER(Users.username)' => $user
                        ])
                        ->first();

                    //If the user exist.
                    if (is_null($userExist)) {
                        break;
                    }

                    //Check if the user is not already in the conversation.
                    $conversUserCheck = $this->ConversationsUsers

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                        ->find()
                        ->where([
                            'ConversationsUsers.conversation_id' => $conversation->id,
                            'ConversationsUsers.user_id' => $userExist->id
                        ])
                        ->first();

                    if (!is_null($conversUserCheck)) {
                        break;
                    }

                    $data = [];
                    $data['conversation_id'] = $conversation->id;
                    $data['user_id'] = $userExist->id;

                    $entity = $this->ConversationsUsers->newEntity($data);

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                    $user = $this->ConversationsUsers->save($entity);

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                    $expression = new QueryExpression('recipient_count = recipient_count + 1');
                    $this->Conversations->updateAll(

The property Conversations does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                        [$expression],
                        [
                            'id' => $conversation->id
                        ]
                    );

                    //Notifications Event.
                    $this->eventManager()->attach(new Notifications());

new \App\Event\Notifications() is of type object<App\Event\Notifications>, but the function expects a callable.

The method Cake\Event\EventManager::attach() has been deprecated with message: 3.0.0 Use on() instead.

                    $event = new Event('Model.Notifications.dispatch', $this, [
                        'sender_id' => $this->Auth->user('id'),
                        'conversation_id' => $conversation->id,
                        'type' => 'conversation.reply'
                    ]);
                    $this->eventManager()->dispatch($event);
                }

                $this->Flash->success(__d('conversations', 'Your conversation has been created successfully !'));
                $this->redirect([
                    '_name' => 'conversations-view',
                    'slug' => $conversation->title,
                    'id' => $conversation->id
                ]);
            }
        }

        $this->set(compact('conversation'));
    }

    /**
     * Display a conversation.
     *
     * @return void|\Cake\Network\Response
     */
    public function view()
    {
        $this->loadModel('ConversationsUsers');

        $conversation = $this->ConversationsUsers

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'Users',
                'Conversations',
                'Conversations.LastMessage',
                'Conversations.LastMessageUser'
            ])
            ->where([
                'ConversationsUsers.conversation_id' => $this->request->id,
                'ConversationsUsers.user_id' => $this->Auth->user('id'),
                'Conversations.conversation_open <>' => 2
            ])
            ->first();

        if (is_null($conversation)) {
            $this->Flash->error(__d('conversations', "This conversation doesn't exist or has been deleted."));

            return $this->redirect(['action' => 'index']);
        }

        $this->loadModel('ConversationsMessages');

        $this->paginate = [
            'maxLimit' => Configure::read('Conversations.messages_per_page')
        ];

        $messages = $this->ConversationsMessages

The property ConversationsMessages does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'Users' => function ($q) {
                    return $q->find('full')->formatResults(function ($users) {
                        return $users->map(function ($user) {
                            $user->online = $this->SessionsActivity->getOnlineStatus($user);

The property SessionsActivity does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                            return $user;
                        });
                    });
                },
                'Users.Groups',
                'LastEditUsers' => function ($q) {
                    return $q->find('short');
                },
            ])
            ->where([
                'ConversationsMessages.conversation_id' => $this->request->id
            ])
            ->order([
                'ConversationsMessages.created' => 'ASC'
            ]);

        $messages = $this->paginate($messages);

        //Update "is_read" for the current user.
        $user = $this->ConversationsUsers->get($conversation->id);

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

        $user->is_read = 1;
        $this->ConversationsUsers->save($user);

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

        //Current user.
        $this->loadModel('Users');
        $currentUser = $this->Users

The property Users does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'Groups' => function ($q) {
                    return $q->select(['id', 'is_staff']);
                }
            ])
            ->where([
                'Users.id' => $this->Auth->user('id')
            ])
            ->select(['id', 'group_id'])
            ->first();

        //Build the newEntity for the comment form.
        $messageForm = $this->ConversationsMessages->newEntity();

The property ConversationsMessages does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->set(compact('conversation', 'messages', 'currentUser', 'messageForm'));
    }

    /**
     * Action to search some users when adding an user in a conversation.
     *
     * @return void
     *
     * @throws \Cake\Network\Exception\NotFoundException
     */
    public function inviteMember()
    {
        if (!$this->request->is('ajax')) {
            throw new NotFoundException();
        }
        $keyword = strtolower($this->request->getQuery('query'));

        $this->loadModel('Users');
        $users = $this->Users

The property Users does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->where(function ($q) use ($keyword) {
                    return $q
                        ->like('LOWER(Users.username)', "%$keyword%");
            })
            ->limit(12)
            ->toArray();

        foreach ($users as $user) {
            $json[] = h($user->username);

$json was never initialized. Although not strictly required by PHP, it is generally a good practice to add $json = array(); before regardless.

        }

        $this->set(compact('json'));
        $this->set('_serialize', 'json');
    }

    /**
     * Quote a post.
     *
     * @throws \Cake\Network\Exception\NotFoundException
     *
     * @return mixed
     */
    public function quote()
    {
        if (!$this->request->is('ajax')) {
            throw new NotFoundException();
        }

        $this->loadModel('ConversationsMessages');

        $message = $this->ConversationsMessages

The property ConversationsMessages does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->where([
                'ConversationsMessages.id' => $this->request->id
            ])
            ->contain([
                'Users' => function ($q) {
                        return $q->find('short');
                }
            ])
            ->first();

        $json = [];

        if (!is_null($message)) {
            $message->toArray();

            $url = Router::url(['action' => 'go', $message->id]);
            $text = __d('conversations', 'has said :');

            //Build the quote.
            $json['message'] = <<<EOT
<div>
    <div>
        <a href="{$url}">
            <strong>{$message->user->full_name} {$text}</strong>
        </a>
    </div>
    <blockquote>
        $message->message
    </blockquote>
</div><p>&nbsp;</p><p>&nbsp;</p>
EOT;

            $json['error'] = false;

            $this->set(compact('json'));
        } else {
            $json['post'] = __d('conversations', "This message doesn't exist.");
            $json['error'] = true;

            $this->set(compact('json'));
        }

        //Send response in JSON.
        $this->set('_serialize', 'json');
    }

    /**
     * Get the form to edit a message.
     *
     * @throws \Cake\Network\Exception\NotFoundException When it's not an AJAX request.
     *
     * @return void
     */
    public function getEditMessage()
    {
        if (!$this->request->is('ajax')) {
            throw new NotFoundException();
        }

        $this->loadModel('ConversationsMessages');
        $this->viewBuilder()->layout(false);

false is of type boolean, but the function expects a string|null.

The method Cake\View\ViewBuilder::layout() has been deprecated with message: 3.4.0 Use setLayout()/getLayout() instead.

        $message = $this->ConversationsMessages

The property ConversationsMessages does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->where([
                'ConversationsMessages.id' => $this->request->getData('id')
            ])
            ->first();

        $json = [
            'error' => false,
            'errorMessage' => ''
        ];

        if (is_null($message)) {

This code seems to be duplicated across your project.

            $json['error'] = true;
            $json['errorMessage'] = __d('conversations', "This message doesn't exist or has been deleted !");

            $this->set(compact('json'));

            return;
        }

        //Current user.
        $this->loadModel('Users');
        $currentUser = $this->Users

The property Users does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'Groups' => function ($q) {
                    return $q->select(['id', 'is_staff']);
                }
            ])
            ->where([
                'Users.id' => $this->Auth->user('id')
            ])
            ->select(['id', 'group_id'])
            ->first();

        if ($message->user_id != $this->Auth->user('id') && !$currentUser->group->is_staff) {
            $json['error'] = true;
            $json['errorMessage'] = __d('conversations', "You don't have the authorization to edit this message !");

            $this->set(compact('json'));

            return;
        }

        $this->set(compact('json', 'message'));
    }

    /**
     * Edit a message.
     *
     * @param int $id Id of the message.
     *
     * @return \Cake\Network\Response
     */
    public function messageEdit($id = null)
    {
        if (!$this->request->is(['post', 'put'])) {
            throw new NotFoundException();
        }

        $this->loadModel('ConversationsMessages');

        $message = $this->ConversationsMessages

The property ConversationsMessages does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->where([
                'ConversationsMessages.id' => $id
            ])
            ->first();

        if (is_null($message)) {
            $this->Flash->error(__d('conversations', "This post doesn't exist or has been deleted !"));

            return $this->redirect($this->referer());

It seems like $this->referer() targeting Cake\Controller\Controller::referer() can also be of type object<Cake\Http\ServerRequest>; however, Cake\Controller\Controller::redirect() does only seem to accept string|array, maybe add an additional type check?

        }

        //Current user.
        $this->loadModel('Users');
        $currentUser = $this->Users

The property Users does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'Groups' => function ($q) {
                    return $q->select(['id', 'is_staff']);
                }
            ])
            ->where([
                'Users.id' => $this->Auth->user('id')
            ])
            ->select(['id', 'group_id'])
            ->first();

        if ($message->user_id != $this->Auth->user('id') && !$currentUser->group->is_staff) {
            $this->Flash->error(__d('conversations', "You don't have the authorization to edit this post !"));

            return $this->redirect($this->referer());

It seems like $this->referer() targeting Cake\Controller\Controller::referer() can also be of type object<Cake\Http\ServerRequest>; however, Cake\Controller\Controller::redirect() does only seem to accept string|array, maybe add an additional type check?

        }

        $this->ConversationsMessages->patchEntity($message, $this->request->getParsedBody());

The property ConversationsMessages does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

        $message->last_edit_date = new Time();
        $message->last_edit_user_id = $this->Auth->user('id');
        $message->edit_count++;

        if ($this->ConversationsMessages->save($message)) {

The property ConversationsMessages does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

            $this->Flash->success(__d('conversations', 'This message has been edited successfully !'));
        }

        return $this->redirect(['action' => 'go', $message->id]);
    }

    /**
     * Redirect an user to a conversation, page and message.
     *
     * @param int $messageId Id of the message.
     *
     * @return \Cake\Network\Response
     */
    public function go($messageId = null)

This method seems to be duplicated in your project.

    {
        $this->loadModel('ConversationsMessages');

        $message = $this->ConversationsMessages

The property ConversationsMessages does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'Conversations'
            ])
            ->where([
                'ConversationsMessages.id' => $messageId
            ])
            ->first();

        if (is_null($message)) {
            $this->Flash->error(__d('conversations', "This message doesn't exist or has been deleted."));

            return $this->redirect(['controller' => 'conversations', 'action' => 'index']);
        }

        $message->toArray();

        //Count the number of messages before this message.
        $messagesBefore = $this->ConversationsMessages

The property ConversationsMessages does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->where([
                'ConversationsMessages.conversation_id' => $message->conversation_id,
                'ConversationsMessages.created <' => $message->created
            ])
            ->count();

        //Get the number of messages per page.
        $messagesPerPage = Configure::read('Conversations.messages_per_page');

        //Calculate the page.
        $page = ceil($messagesBefore / $messagesPerPage);

        $page = ($page > 1) ? $page : 1;

        //Redirect the user.
        return $this->redirect([
            '_name' => 'conversations-view',
            'slug' => $message->conversation->title,
            'id' => $message->conversation->id,
            '?' => ['page' => $page],
            '#' => 'message-' . $messageId
        ]);
    }

    /**
     * Function to kick an user from a conversation.
     *
     * @return void
     */
    public function kick()
    {
        if (!$this->request->is('ajax')) {
            throw new NotFoundException();
        }

        $this->loadModel('ConversationsUsers');

        $currentUser = $this->ConversationsUsers

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'Conversations',
                'Users' => function ($q) {
                    return $q->find('short');
                },
                'Users.Groups' => function ($q) {
                    return $q->select(['id', 'is_staff']);
                }
            ])
            ->where([
                'ConversationsUsers.user_id' => $this->Auth->user('id'),
                'ConversationsUsers.conversation_id' => $this->request->id
            ])
            ->first();

        //Check if the current user is the owner of this conversation or if he is not a staff member.
        if ($currentUser->user_id != $currentUser->conversation->user_id && !$currentUser->user->group->is_staff) {
            $json['message'] = __d('conversations', 'You cannot kick this user from this conversation.');

$json was never initialized. Although not strictly required by PHP, it is generally a good practice to add $json = array(); before regardless.

            $json['error'] = true;

            $this->set(compact('json'));
            $this->set('_serialize', 'json');

            return;
        }

        $user = $this->ConversationsUsers

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'Conversations',
                'Users' => function ($q) {
                    return $q->find('short');
                },
                'Users.Groups' => function ($q) {
                    return $q->select(['id', 'is_staff']);
                }
            ])
            ->where([
                'ConversationsUsers.user_id' => $this->request->user_id,
                'ConversationsUsers.conversation_id' => $this->request->id
            ])
            ->first();

        //Check if the user to kick is in the conversation and if he is not owner of this conversation and if he is not a staff member.
        if (is_null($user) || $this->request->user_id == $currentUser->conversation->user_id || $user->user->group->is_staff) {
            $json['message'] = __d('conversations', 'You cannot kick this user from this conversation.');

$json was never initialized. Although not strictly required by PHP, it is generally a good practice to add $json = array(); before regardless.

            $json['error'] = true;

            $this->set(compact('json'));
            $this->set('_serialize', 'json');

            return;
        }

        $this->ConversationsUsers->delete($user);

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

        $expression = new QueryExpression('recipient_count = recipient_count - 1');
        $this->Conversations->updateAll(

The property Conversations does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

            [$expression],
            [
                'id' => $this->request->id
            ]
        );

        $json['message'] = __d('conversations', 'This user has been kicked successfully.');

$json was never initialized. Although not strictly required by PHP, it is generally a good practice to add $json = array(); before regardless.

        $json['error'] = false;
        $json['id'] = $this->request->user_id;

The property user_id does not seem to exist in Cake\Http\ServerRequest.

        $json['recipients'] = $currentUser->conversation->recipient_count - 1;

        $this->set(compact('json'));
        $this->set('_serialize', 'json');
    }

    /**
     * Reply to a conversation.
     *
     * @return void|\Cake\Network\Response
     */
    public function reply()
    {
        $this->loadModel('ConversationsMessages');

        if ($this->request->is('post')) {
            $conversation = $this->Conversations

The property Conversations does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                ->find()
                ->where(['Conversations.id' => $this->request->id])
                ->first();

            $this->loadModel('ConversationsUsers');
            $user = $this->ConversationsUsers

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                ->find()
                ->where([
                    'ConversationsUsers.conversation_id' => $this->request->id,
                    'ConversationsUsers.user_id' => $this->Auth->user('id')
                ])
                ->first();

            //Check if the conversation is found.
            if (is_null($user) || is_null($conversation) || $conversation->conversation_open == 2) {
                $this->Flash->error(__d('conversations', "This conversation doesn't exist or has been deleted !"));

                return $this->redirect($this->referer());

It seems like $this->referer() targeting Cake\Controller\Controller::referer() can also be of type object<Cake\Http\ServerRequest>; however, Cake\Controller\Controller::redirect() does only seem to accept string|array, maybe add an additional type check?

            }

            //Check if the conversation is open.
            if ($conversation->conversation_open == false) {

This code seems to be duplicated across your project.

                $this->Flash->error(__d('conversations', 'This conversation is closed, you cannot reply.'));

                return $this->redirect($this->referer());

It seems like $this->referer() targeting Cake\Controller\Controller::referer() can also be of type object<Cake\Http\ServerRequest>; however, Cake\Controller\Controller::redirect() does only seem to accept string|array, maybe add an additional type check?

            }

            //Build the newEntity for the post form.
            $this->request = $this->request
                ->withData('conversation.last_post_date', new Time())
                ->withData('conversation.last_post_user_id', $this->Auth->user('id'))
                ->withData('user_id', $this->Auth->user('id'))
                ->withData('conversation_id', $this->request->id);

            $message = $this->ConversationsMessages->newEntity($this->request->getParsedBody(), [

The property ConversationsMessages does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                'associated' => ['Conversations'],
                'validate' => 'create'
            ]);

            //Handle validation errors (Due to the redirect)
            if (!empty($message->errors())) {
                $this->Flash->conversationsReply('Validation errors', [
                    'key' => 'ConversationsReply',
                    'params' => [
                        'errors' => $message->errors()
                    ]
                ]);

                return $this->redirect($this->referer());

It seems like $this->referer() targeting Cake\Controller\Controller::referer() can also be of type object<Cake\Http\ServerRequest>; however, Cake\Controller\Controller::redirect() does only seem to accept string|array, maybe add an additional type check?

            }

            if ($message->conversation->isNew() === true) {
                $message->conversation->isNew(false);
            }

            $message->conversation->accessible('id', true);
            $message->conversation->id = $this->request->id;

The property id does not seem to exist in Cake\Http\ServerRequest.

            if ($message = $this->ConversationsMessages->save($message)) {

The property ConversationsMessages does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                //Update the last message id for the conversation.
                $this->loadModel('Conversations');
                $conversation = $this->Conversations->get($this->request->id);

The property Conversations does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                $conversation->last_message_id = $message->id;
                $conversation->last_message_date = new Time();
                $conversation->last_message_user_id = $this->Auth->user('id');
                $this->Conversations->save($conversation);

The property Conversations does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                //Notifications Event.
                $this->eventManager()->attach(new Notifications());

new \App\Event\Notifications() is of type object<App\Event\Notifications>, but the function expects a callable.

The method Cake\Event\EventManager::attach() has been deprecated with message: 3.0.0 Use on() instead.

                $event = new Event('Model.Notifications.dispatch', $this, [
                    'sender_id' => $this->Auth->user('id'),
                    'conversation_id' => $conversation->id,
                    'type' => 'conversation.reply'
                ]);
                $this->eventManager()->dispatch($event);

                $conversationOpen = !is_null($this->request->getData('conversation.conversation_open')) ? $this->request->getData('conversation.conversation_open') : true;

                if ($conversationOpen == false) {
                    $this->Flash->success(__d('conversations', 'Your reply has been posted successfully and the conversation has been closed !'));
                } else {
                    $this->Flash->success(__d('conversations', 'Your reply has been posted successfully !'));
                }

                //Redirect the user to the last page of the article.
                return $this->redirect([
                    'controller' => 'conversations',
                    'action' => 'go',
                    $message->id
                ]);
            }
        }

        $this->redirect($this->referer());

It seems like $this->referer() targeting Cake\Controller\Controller::referer() can also be of type object<Cake\Http\ServerRequest>; however, Cake\Controller\Controller::redirect() does only seem to accept string|array, maybe add an additional type check?

    }

    /**
     * Edit a conversation.
     *
     * @return \Cake\Network\Response
     */
    public function edit()
    {
        $this->loadModel('Conversations');

        if ($this->request->is('put')) {
            $conversation = $this->Conversations

The property Conversations does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                ->find()
                ->where([
                    'Conversations.id' => $this->request->id
                ])
                ->first();

            //Check if the conversation is found.
            if (is_null($conversation)) {

This code seems to be duplicated across your project.

                $this->Flash->error(__d('conversations', "This conversation doesn't exist or has been deleted !"));

                return $this->redirect($this->referer());

It seems like $this->referer() targeting Cake\Controller\Controller::referer() can also be of type object<Cake\Http\ServerRequest>; however, Cake\Controller\Controller::redirect() does only seem to accept string|array, maybe add an additional type check?

            }

            //Check if the user has the permission to edit it.
            if ($this->Auth->isAuthorized() === false) {

This code seems to be duplicated across your project.

                $this->Flash->error(__d('conversations', "You don't have the authorization to edit this conversation !"));

                return $this->redirect([
                    'controller' => 'conversations',
                    'action' => 'go',
                    $conversation->last_message_id
                ]);
            }

            $this->Conversations->patchEntity($conversation, $this->request->getParsedBody(), ['validate' => 'edit']);

The property Conversations does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

            if ($this->Conversations->save($conversation)) {

The property Conversations does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                if ($conversation->conversation_open == false) {
                    $this->Flash->success(__d('conversations', 'Your conversation has been edited and closed successfully !'));
                } else {
                    $this->Flash->success(__d('conversations', 'Your conversation has been edited successfully !'));
                }

                return $this->redirect([
                    'controller' => 'conversations',
                    'action' => 'go',
                    $conversation->last_message_id
                ]);
            }
        }

        $this->redirect($this->referer());

It seems like $this->referer() targeting Cake\Controller\Controller::referer() can also be of type object<Cake\Http\ServerRequest>; however, Cake\Controller\Controller::redirect() does only seem to accept string|array, maybe add an additional type check?

    }

    /**
     * Function to invite user(s) in a conversation.
     *
     * @return void|\Cake\Network\Response
     */
    public function invite()
    {
        $this->loadModel('ConversationsUsers');

        $conversation = $this->ConversationsUsers

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'Conversations',
                'Users',
                'Users.Groups'
            ])
            ->where([
                'ConversationsUsers.conversation_id' => $this->request->id,
                'ConversationsUsers.user_id' => $this->Auth->user('id')
            ])
            ->first();

        if (is_null($conversation) || $conversation->conversation->conversation_open != 1) {

This code seems to be duplicated across your project.

            $this->Flash->error(__d('conversations', 'This conversation is closed or has been deleted !'));

            return $this->redirect($this->referer());

It seems like $this->referer() targeting Cake\Controller\Controller::referer() can also be of type object<Cake\Http\ServerRequest>; however, Cake\Controller\Controller::redirect() does only seem to accept string|array, maybe add an additional type check?

        }

        if (!$conversation->conversation->open_invite && $conversation->conversation->user_id != $this->Auth->user('id') && !$conversation->user->group->is_staff) {
            $this->Flash->error(__d('conversations', "You don't have the authorization to invite in this conversation !"));

            return $this->redirect($this->referer());

It seems like $this->referer() targeting Cake\Controller\Controller::referer() can also be of type object<Cake\Http\ServerRequest>; however, Cake\Controller\Controller::redirect() does only seem to accept string|array, maybe add an additional type check?

        }

        if ($this->request->is(['post', 'put'])) {
            $users = str_replace(",", "", trim(strtolower($this->request->getData('users'))));
            $users = explode(" ", $users);

            $maxUsersCheck = $this->ConversationsUsers

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                ->find()
                ->where([
                    'ConversationsUsers.conversation_id' => $this->request->id
                ])
                ->count();

            //Check max users.
            if (count($users) + $maxUsersCheck >= Configure::read('Conversations.max_users_per_conversation')) {
                $this->Flash->error(__d('conversations', 'You cannot invite more than {0} user(s) in this conversation.', Configure::read('Conversations.max_users_per_conversation')));

                return $this->redirect($this->referer());

It seems like $this->referer() targeting Cake\Controller\Controller::referer() can also be of type object<Cake\Http\ServerRequest>; however, Cake\Controller\Controller::redirect() does only seem to accept string|array, maybe add an additional type check?

            }

            $this->loadModel('Users');
            $this->loadModel('Conversations');

            foreach ($users as $user) {
                $user = $this->Users

The property Users does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                    ->find()
                    ->where([
                        'LOWER(Users.username)' => $user
                    ])
                    ->first();

                if (!is_null($user)) {
                    $check = $this->ConversationsUsers

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                        ->find()
                        ->where([
                            'ConversationsUsers.conversation_id' => $this->request->id,
                            'ConversationsUsers.user_id' => $user->id
                        ])
                        ->first();

                    if (is_null($check)) {
                        $data = [];
                        $data['conversation_id'] = $this->request->id;

The property id does not seem to exist in Cake\Http\ServerRequest.

                        $data['user_id'] = $user->id;

                        $entity = $this->ConversationsUsers->newEntity($data);

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                        $this->ConversationsUsers->save($entity);

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                        $expression = new QueryExpression('recipient_count = recipient_count + 1');
                        $this->Conversations->updateAll(

The property Conversations does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                            [$expression],
                            [
                                'id' => $this->request->id
                            ]
                        );
                    }
                }
            }

            $this->Flash->success(__d('conversations', 'Your user(s) has been added successfully.'));
        }

        return $this->redirect([
            'controller' => 'conversations',
            'action' => 'go',
            $conversation->conversation->last_message_id
        ]);
    }

    /**
     * Function to leave a conversation.
     *
     * @return void|\Cake\Network\Response
     */
    public function leave()
    {
        $this->loadModel('ConversationsUsers');

        $user = $this->ConversationsUsers

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'Conversations'
            ])
            ->where([
                'ConversationsUsers.conversation_id' => $this->request->id,
                'ConversationsUsers.user_id' => $this->Auth->user('id')
            ])
            ->first();

        if (is_null($user)) {
            $this->Flash->error(__d('conversations', 'You are not in this conversation.'));

            return $this->redirect($this->referer());

It seems like $this->referer() targeting Cake\Controller\Controller::referer() can also be of type object<Cake\Http\ServerRequest>; however, Cake\Controller\Controller::redirect() does only seem to accept string|array, maybe add an additional type check?

        }

        if ($user->conversation->user_id != $this->Auth->user('id')) {
            $this->loadModel('Conversations');

            $this->ConversationsUsers->delete($user);

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

            $expression = new QueryExpression('recipient_count = recipient_count - 1');
            $this->Conversations->updateAll(

The property Conversations does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

                [$expression],
                [
                    'id' => $this->request->id
                ]
            );

            $this->Flash->success(__d('conversations', 'You have left the conversation successfully.'));

            return $this->redirect(['controller' => 'conversations', 'action' => 'index']);
        } else {

This code seems to be duplicated across your project.

            $this->Flash->error(__d('conversations', 'You can not leave your own conversation.'));

            return $this->redirect([
                'controller' => 'conversations',
                'action' => 'go',
                $user->conversation->last_message_id
            ]);
        }
    }

    /**
     * Search conversations.
     *
     * @return void
     */
    public function search()
    {
        $this->loadModel('ConversationsUsers');

        //Check the keyword to search. (For pagination)
        if (!empty($this->request->getData('search'))) {

This code seems to be duplicated across your project.

            $keyword = $this->request->getData('search');
            $this->request->session()->write('Search.Conversations.Keyword', $keyword);
        } else {
            if ($this->request->session()->read('Search.Conversations.Keyword')) {

The expression $this->request->session()->read('Search.Conversations.Keyword') of type string|null is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

                $keyword = $this->request->session()->read('Search.Conversations.Keyword');
            } else {
                $keyword = '';
            }
        }

        //Pagination
        $this->paginate = [
            'maxLimit' => Configure::read('Conversations.conversations_per_page')
        ];

        $conversations = $this->ConversationsUsers

The property ConversationsUsers does not exist on object<App\Controller\ConversationsController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'Users',
                'Conversations',
                'Conversations.Users' => function ($q) {
                    return $q->find('medium');
                },
                'Conversations.LastMessage',
                'Conversations.LastMessageUser'
            ])
            ->where([
                'ConversationsUsers.user_id' => $this->Auth->user('id'),
                'Conversations.conversation_open <>' => 2
            ])
            ->andWhere(function ($q) use ($keyword) {
                    return $q
                        ->like('Conversations.title', "%$keyword%");
            })
            ->order([
                'ConversationsUsers.is_read' => 'ASC',
                'ConversationsUsers.is_star' => 'DESC',
                'Conversations.last_message_date' => 'DESC',
            ]);

        $conversations = $this->paginate($conversations);

        $this->set(compact('conversations', 'keyword'));
    }

    /**
     * Action to render the maintenance page.
     *
     * @return void
     */
    public function maintenance()
    {
    }
}