Chamilo\CoreBundle\Controller\ChatController

Complexity

Total Complexity

13

Size/Duplication

Total Lines	124
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
C	ajaxAction()	0	94	12
A	indexAction()	0	16	1

<?php

/* For licensing terms, see /license.txt */

namespace Chamilo\CoreBundle\Controller;

use Chamilo\CoreBundle\Repository\ResourceNodeRepository;
use Chamilo\CourseBundle\Controller\CourseControllerInterface;
use Chamilo\CourseBundle\Controller\CourseControllerTrait;
use Chamilo\CourseBundle\Repository\CChatConversationRepository;
use Event;
use Symfony\Component\HttpFoundation\JsonResponse;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\Routing\Annotation\Route;

/**
 * Class ChatController.
 */
class ChatController extends AbstractResourceController implements CourseControllerInterface
{
    use CourseControllerTrait;

    /**
     * @Route("/resources/chat/", name="chat_home", options={"expose"=true})
     */
    public function indexAction(Request $request): Response
    {
        Event::event_access_tool(TOOL_CHAT);

The method event_access_tool() does not exist on Event.

        $logInfo = [
            'tool' => TOOL_CHAT,
            'action' => 'start',
            'action_details' => 'start-chat',
        ];
        Event::registerLog($logInfo);

The method registerLog() does not exist on Event.

        return $this->render(
            '@ChamiloTheme/Chat/chat.html.twig',
            [
                'restrict_to_coach' => api_get_configuration_value('course_chat_restrict_to_coach'),
                'user' => api_get_user_info(),
            ]
        );
    }

    /**
     * @Route("/resources/chat/conversations/", name="chat_ajax", options={"expose"=true})
     */
    public function ajaxAction(Request $request, ResourceNodeRepository $repo): Response
    {
        if (!api_protect_course_script(false)) {
            exit;

In this branch, the function will implicitly return null which is incompatible with the type-hinted return Symfony\Component\HttpFoundation\Response. Consider adding a return statement or allowing null as return value.

Using exit here is not recommended.

        }

        /** @var CChatConversationRepository $resourceRepo */
        $resourceRepo = $this->getRepository('chat', 'conversations');

        $courseId = api_get_course_int_id();
        $userId = api_get_user_id();
        $sessionId = api_get_session_id();
        $groupId = api_get_group_id();
        $json = ['status' => false];
        $parentResourceNode = $this->getParentResourceNode($request);

        $courseChatUtils = new \CourseChatUtils(
            $courseId,
            $userId,
            $sessionId,
            $groupId,
            $parentResourceNode,
            $resourceRepo
        );

        $action = $request->get('action');

        switch ($action) {
            case 'chat_logout':
                $logInfo = [
                    'tool' => TOOL_CHAT,
                    'action' => 'exit',
                    'action_details' => 'exit-chat',
                ];
                Event::registerLog($logInfo);

                break;
            case 'track':
                $courseChatUtils->keepUserAsConnected();
                $courseChatUtils->disconnectInactiveUsers();

                $friend = isset($_REQUEST['friend']) ? (int) $_REQUEST['friend'] : 0;
                //$filePath = $courseChatUtils->getFileName(true, $friend);
                //$newFileSize = file_exists($filePath) ? filesize($filePath) : 0;
                //$oldFileSize = isset($_GET['size']) ? (int) $_GET['size'] : -1;
                $newUsersOnline = $courseChatUtils->countUsersOnline();
                $oldUsersOnline = isset($_GET['users_online']) ? (int) $_GET['users_online'] : 0;

                $json = [
                    'status' => true,
                    'data' => [
                        //'oldFileSize' => file_exists($filePath) ? filesize($filePath) : 0,
                        'oldFileSize' => false,
                        'history' => $courseChatUtils->readMessages(false, $friend),
                        'usersOnline' => $newUsersOnline,
                        'userList' => $newUsersOnline != $oldUsersOnline ? $courseChatUtils->listUsersOnline() : null,
                        'currentFriend' => $friend,
                    ],
                ];

                break;
            case 'preview':
                $json = [
                    'status' => true,
                    'data' => [
                        'message' => $courseChatUtils->prepareMessage($_REQUEST['message']),
                    ],
                ];

                break;
            case 'reset':
                $friend = isset($_REQUEST['friend']) ? (int) $_REQUEST['friend'] : 0;

                $json = [
                    'status' => true,
                    'data' => $courseChatUtils->readMessages(true, $friend),
                ];

                break;
            case 'write':
                $friend = isset($_REQUEST['friend']) ? (int) $_REQUEST['friend'] : 0;
                $status = $courseChatUtils->saveMessage($_REQUEST['message'], $friend);

                $json = [
                    'status' => $status,
                    'data' => [
                        'writed' => $status,
                    ],
                ];

                break;
        }

        return new JsonResponse($json);
    }
}