chamilo / chamilo-lms

main/lp/storageapi.php

Switch Indentation +46 added lines, -46 removed lines

@@ -15,52 +15,52 @@
 }
 
 switch ($_REQUEST['action']) {
-    case "get":
-        print storage_get($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco'], $_REQUEST['svkey']);
-        break;
-    case "set":
-        if (storage_can_set($_REQUEST['svuser'])) {
-            print storage_set($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco'], $_REQUEST['svkey'], $_REQUEST['svvalue']);
-        }
-        break;
-    case "getall":
-        print storage_getall($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco']);
-        break;
-    case "stackpush":
-        if (storage_can_set($_REQUEST['svuser'])) {
-            print storage_stack_push($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco'], $_REQUEST['svkey'], $_REQUEST['svvalue']);
-        }
-        break;
-    case "stackpop":
-        if (storage_can_set($_REQUEST['svuser'])) {
-            print storage_stack_pop($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco'], $_REQUEST['svkey']);
-        }
-        break;
-    case "stacklength":
-        print storage_stack_length($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco'], $_REQUEST['svkey']);
-        break;
-    case "stackclear":
-        if (storage_can_set($_REQUEST['svuser'])) {
-            print storage_stack_clear($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco'], $_REQUEST['svkey']);
-        }
-        break;
-    case "stackgetall":
-        if (storage_can_set($_REQUEST['svuser']))
-            print storage_stack_getall($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco'], $_REQUEST['svkey']);
-        break;
-    case "getposition":
-        print storage_get_position($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco'], $_REQUEST['svkey'], $_REQUEST['svasc']);
-        break;
-    case "getleaders":
-        print storage_get_leaders($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco'], $_REQUEST['svkey'], $_REQUEST['svasc'], $_REQUEST['svlength']);
-        break;
-    case "usersgetall":
-// security issue
-        print "NOT allowed, security issue, see sources";
-//		print storage_get_all_users();
-        break;
-    default:
-        // Do nothing
+        case "get":
+            print storage_get($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco'], $_REQUEST['svkey']);
+            break;
+        case "set":
+            if (storage_can_set($_REQUEST['svuser'])) {
+                print storage_set($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco'], $_REQUEST['svkey'], $_REQUEST['svvalue']);
+            }
+            break;
+        case "getall":
+            print storage_getall($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco']);
+            break;
+        case "stackpush":
+            if (storage_can_set($_REQUEST['svuser'])) {
+                print storage_stack_push($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco'], $_REQUEST['svkey'], $_REQUEST['svvalue']);
+            }
+            break;
+        case "stackpop":
+            if (storage_can_set($_REQUEST['svuser'])) {
+                print storage_stack_pop($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco'], $_REQUEST['svkey']);
+            }
+            break;
+        case "stacklength":
+            print storage_stack_length($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco'], $_REQUEST['svkey']);
+            break;
+        case "stackclear":
+            if (storage_can_set($_REQUEST['svuser'])) {
+                print storage_stack_clear($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco'], $_REQUEST['svkey']);
+            }
+            break;
+        case "stackgetall":
+            if (storage_can_set($_REQUEST['svuser']))
+                print storage_stack_getall($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco'], $_REQUEST['svkey']);
+            break;
+        case "getposition":
+            print storage_get_position($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco'], $_REQUEST['svkey'], $_REQUEST['svasc']);
+            break;
+        case "getleaders":
+            print storage_get_leaders($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco'], $_REQUEST['svkey'], $_REQUEST['svasc'], $_REQUEST['svlength']);
+            break;
+        case "usersgetall":
+    // security issue
+            print "NOT allowed, security issue, see sources";
+    //		print storage_get_all_users();
+            break;
+        default:
+            // Do nothing
 }
 
 function storage_can_set($sv_user) {

main/upload/upload.php

Switch Indentation +10 added lines, -10 removed lines

@@ -31,14 +31,14 @@
  * Redirect to the correct script to handle this type of upload
  */
 switch ($toolFromSession) {
-    case TOOL_LEARNPATH:
-        require 'upload.scorm.php';
-        break;
-    //the following cases need to be distinguished later on
-    case TOOL_DROPBOX:
-    case TOOL_STUDENTPUBLICATION:
-    case TOOL_DOCUMENT:
-    default:
-        require 'upload.document.php';
-        break;
+        case TOOL_LEARNPATH:
+            require 'upload.scorm.php';
+            break;
+        //the following cases need to be distinguished later on
+        case TOOL_DROPBOX:
+        case TOOL_STUDENTPUBLICATION:
+        case TOOL_DOCUMENT:
+        default:
+            require 'upload.document.php';
+            break;
 }

main/user/user_export.php

Switch Indentation +11 added lines, -11 removed lines

@@ -149,15 +149,15 @@
 }
 
 switch ($file_type) {
-    case 'xml':
-        Export::arrayToXml($data, $filename, 'Contact', 'Contacts');
-        exit;
-        break;
-    case 'csv':
-        Export::arrayToCsv($data, $filename);
-        exit;
-    case 'xls':
-        Export::arrayToXls($data, $filename);
-        exit;
-        break;
+        case 'xml':
+            Export::arrayToXml($data, $filename, 'Contact', 'Contacts');
+            exit;
+            break;
+        case 'csv':
+            Export::arrayToCsv($data, $filename);
+            exit;
+        case 'xls':
+            Export::arrayToXls($data, $filename);
+            exit;
+            break;
 }
\ No newline at end of file

main/ticket/tickets.php

Switch Indentation +41 added lines, -41 removed lines

@@ -72,48 +72,48 @@
 }
 
 switch ($action) {
-    case 'alert':
-        if (!$isAdmin && isset($_GET['ticket_id'])) {
-            TicketManager::send_alert($_GET['ticket_id'], $user_id);
-        }
-        break;
-    case 'export':
-        $data = array(
-            array(
-                '#',
-                get_lang('Date'),
-                get_lang('LastUpdate'),
-                get_lang('Category'),
-                get_lang('User'),
-                get_lang('Program'),
-                get_lang('AssignedTo'),
-                get_lang('Status'),
-                get_lang('Description')
-            )
-        );
-        $datos = $table->get_clean_html();
-        foreach ($datos as $ticket) {
-            $ticket[0] = substr(strip_tags($ticket[0]), 0, 12);
-            $ticket_rem = array(
-                utf8_decode(strip_tags($ticket[0])),
-                utf8_decode(api_html_entity_decode($ticket[1])),
-                utf8_decode(strip_tags($ticket[2])),
-                utf8_decode(strip_tags($ticket[3])),
-                utf8_decode(strip_tags($ticket[4])),
-                utf8_decode(strip_tags($ticket[5])),
-                utf8_decode(strip_tags($ticket[6])),
-                utf8_decode(strip_tags($ticket[7]))
+        case 'alert':
+            if (!$isAdmin && isset($_GET['ticket_id'])) {
+                TicketManager::send_alert($_GET['ticket_id'], $user_id);
+            }
+            break;
+        case 'export':
+            $data = array(
+                array(
+                    '#',
+                    get_lang('Date'),
+                    get_lang('LastUpdate'),
+                    get_lang('Category'),
+                    get_lang('User'),
+                    get_lang('Program'),
+                    get_lang('AssignedTo'),
+                    get_lang('Status'),
+                    get_lang('Description')
+                )
             );
-            $data[] = $ticket_rem;
-        }
-        Export::arrayToXls($data, get_lang('Tickets'));
-        exit;
-        break;
-    case 'close_tickets':
-        TicketManager::close_old_tickets();
-        break;
-    default:
-        break;
+            $datos = $table->get_clean_html();
+            foreach ($datos as $ticket) {
+                $ticket[0] = substr(strip_tags($ticket[0]), 0, 12);
+                $ticket_rem = array(
+                    utf8_decode(strip_tags($ticket[0])),
+                    utf8_decode(api_html_entity_decode($ticket[1])),
+                    utf8_decode(strip_tags($ticket[2])),
+                    utf8_decode(strip_tags($ticket[3])),
+                    utf8_decode(strip_tags($ticket[4])),
+                    utf8_decode(strip_tags($ticket[5])),
+                    utf8_decode(strip_tags($ticket[6])),
+                    utf8_decode(strip_tags($ticket[7]))
+                );
+                $data[] = $ticket_rem;
+            }
+            Export::arrayToXls($data, get_lang('Tickets'));
+            exit;
+            break;
+        case 'close_tickets':
+            TicketManager::close_old_tickets();
+            break;
+        default:
+            break;
 }
 
 if (empty($projectId)) {

main/session/session_course_user_list.php

Switch Indentation +22 added lines, -22 removed lines

@@ -62,30 +62,30 @@
 }
 
 switch ($action) {
-    case 'delete':
-        if (is_array($idChecked) && count($idChecked) > 0) {
-            array_map('intval', $idChecked);
-            $idChecked = implode(',', $idChecked);
-        }
-        if (!empty($idChecked)) {
-            $sql = "DELETE FROM $tbl_session_rel_course_rel_user
+        case 'delete':
+            if (is_array($idChecked) && count($idChecked) > 0) {
+                array_map('intval', $idChecked);
+                $idChecked = implode(',', $idChecked);
+            }
+            if (!empty($idChecked)) {
+                $sql = "DELETE FROM $tbl_session_rel_course_rel_user
                     WHERE session_id='$id_session' AND c_id='".$courseId."' AND user_id IN($idChecked)";
-            $result = Database::query($sql);
-            $nbr_affected_rows = Database::affected_rows($result);
-            $sql = "UPDATE $tbl_session_rel_course SET nbr_users=nbr_users-$nbr_affected_rows
+                $result = Database::query($sql);
+                $nbr_affected_rows = Database::affected_rows($result);
+                $sql = "UPDATE $tbl_session_rel_course SET nbr_users=nbr_users-$nbr_affected_rows
                     WHERE session_id='$id_session' AND c_id='".$courseId."'";
-            Database::query($sql);
-        }
-        header('Location: '.api_get_self()
-            .'?id_session='.$id_session.'&course_code='.urlencode($course_code).'&sort='.$sort);
-        exit();
-        break;
-    case 'add':
-        SessionManager::subscribe_users_to_session_course($idChecked, $id_session, $course_code);
-        header('Location: '.api_get_self()
-            .'?id_session='.$id_session.'&course_code='.urlencode($course_code).'&sort='.$sort);
-        exit;
-        break;
+                Database::query($sql);
+            }
+            header('Location: '.api_get_self()
+                .'?id_session='.$id_session.'&course_code='.urlencode($course_code).'&sort='.$sort);
+            exit();
+            break;
+        case 'add':
+            SessionManager::subscribe_users_to_session_course($idChecked, $id_session, $course_code);
+            header('Location: '.api_get_self()
+                .'?id_session='.$id_session.'&course_code='.urlencode($course_code).'&sort='.$sort);
+            exit;
+            break;
 }
 
 

main/session/scheduled_announcement.php

Switch Indentation +52 added lines, -52 removed lines

@@ -43,26 +43,26 @@ 
 }
 
 switch ($action) {
-    case 'run':
-        $messagesSent = $object->sendPendingMessages();
+        case 'run':
+            $messagesSent = $object->sendPendingMessages();
+
+            Display::addFlash(
+                Display::return_message(
+                    get_lang('MessageSent').': '.$messagesSent,
+                    'confirmation'
+                )
+            );
+            $content = $object->getGrid($sessionId);
 
-        Display::addFlash(
-            Display::return_message(
-                get_lang('MessageSent').': '.$messagesSent,
-                'confirmation'
-            )
-        );
-        $content = $object->getGrid($sessionId);
+            break;
+        case 'add':
+            $url  = api_get_self().'?action='.Security::remove_XSS($_GET['action']).'&session_id='.$sessionId;
+            $form = $object->returnForm($url, 'add', $sessionInfo);
 
-        break;
-    case 'add':
-        $url  = api_get_self().'?action='.Security::remove_XSS($_GET['action']).'&session_id='.$sessionId;
-        $form = $object->returnForm($url, 'add', $sessionInfo);
-
-        // The validation or display
-        if ($form->validate()) {
-            $values = $form->getSubmitValues();
-            switch ($values['type']) {
+            // The validation or display
+            if ($form->validate()) {
+                $values = $form->getSubmitValues();
+                switch ($values['type']) {
                 case 'base_date':
                     $numberDays = (int) $values['days'];
                     switch ($values['base_date']) {
@@ -73,20 +73,20 @@ 
                             $baseDate = new DateTime($sessionInfo['access_end_date']);
                             break;
                     }
-                    $interval = new DateInterval('P'.$numberDays.'D');
-                    switch ($values['moment_type']) {
+                        $interval = new DateInterval('P'.$numberDays.'D');
+                        switch ($values['moment_type']) {
                         case 'after':
                             $newDate = $baseDate->add($interval);
                             break;
                         case 'before':
                             $newDate = $baseDate->sub($interval);
                             break;
-                    }
+                        }
                     $values['date'] = $newDate->format('Y-m-d h:i:s');
                     break;
-                case 'specific_date':
-                    $values['date'] = api_get_utc_datetime($values['date']);
-                    break;
+                    case 'specific_date':
+                        $values['date'] = api_get_utc_datetime($values['date']);
+                        break;
             }
 
             $res = $object->save($values);
@@ -110,34 +110,34 @@ 
             $content .= $form->returnForm();
         }
         break;
-    case 'edit':
-        // Action handling: Editing
-        $url  = api_get_self().'?action='.Security::remove_XSS($_GET['action']).'&id='.intval($_GET['id']).'&session_id='.$sessionId;
-        $form = $object->returnSimpleForm($url, 'edit', $sessionInfo);
-        if ($form->validate()) {
-            $values = $form->getSubmitValues();
-            $values['id'] = $id;
-            $values['sent'] = isset($values['sent']) ? 1 : '';
-            $values['date'] = api_get_utc_datetime($values['date']);
-            $res = $object->update($values);
-
-            Display::addFlash(Display::return_message(
-                get_lang('Updated'),
-                'confirmation'
-            ));
-        }
-        $item = $object->get($id);
-        $item['date'] = api_get_local_time($item['date']);
-        $form->setDefaults($item);
-        $content = $form->returnForm();
-        break;
-    case 'delete':
-        $object->delete($_GET['id']);
-        $content = $object->getGrid($sessionId);
-        break;
-    default:
-        $content = $object->getGrid($sessionId);
-        break;
+        case 'edit':
+            // Action handling: Editing
+            $url  = api_get_self().'?action='.Security::remove_XSS($_GET['action']).'&id='.intval($_GET['id']).'&session_id='.$sessionId;
+            $form = $object->returnSimpleForm($url, 'edit', $sessionInfo);
+            if ($form->validate()) {
+                $values = $form->getSubmitValues();
+                $values['id'] = $id;
+                $values['sent'] = isset($values['sent']) ? 1 : '';
+                $values['date'] = api_get_utc_datetime($values['date']);
+                $res = $object->update($values);
+
+                Display::addFlash(Display::return_message(
+                    get_lang('Updated'),
+                    'confirmation'
+                ));
+            }
+            $item = $object->get($id);
+            $item['date'] = api_get_local_time($item['date']);
+            $form->setDefaults($item);
+            $content = $form->returnForm();
+            break;
+        case 'delete':
+            $object->delete($_GET['id']);
+            $content = $object->getGrid($sessionId);
+            break;
+        default:
+            $content = $object->getGrid($sessionId);
+            break;
 }
 
 $url = api_get_path(WEB_AJAX_PATH).'model.ajax.php?a=get_programmed_announcements&session_id='.$sessionId;

main/attendance/index.php

Switch Indentation +103 added lines, -103 removed lines

@@ -228,113 +228,113 @@
 }
 
 switch ($action) {
-    case 'attendance_list':
-        $attendanceController->attendance_list();
-        break;
-    case 'attendance_add':
-        if ($allowToEdit) {
-            $attendanceController->attendance_add();
-        } else {
-            api_not_allowed(true);
-        }
-        break;
-    case 'attendance_edit':
-        if ($allowToEdit) {
-            $attendanceController->attendance_edit($attendance_id);
-        } else {
-            api_not_allowed(true);
-        }
-        break;
-    case 'attendance_delete':
-        if ($allowToEdit) {
-            $attendanceController->attendance_delete($attendance_id);
-        } else {
-            api_not_allowed(true);
-        }
-        break;
-    case 'attendance_set_invisible':
-        if ($allowToEdit) {
-            $attendanceController->attendanceSetInvisible($attendance_id);
-        } else {
-            api_not_allowed(true);
-        }
-        break;
-    case 'attendance_set_visible':
-        if ($allowToEdit) {
-            $attendanceController->attendanceSetVisible($attendance_id);
-        } else {
-            api_not_allowed(true);
-        }
-        break;
-    /*case 'attendance_restore':
+        case 'attendance_list':
+            $attendanceController->attendance_list();
+            break;
+        case 'attendance_add':
+            if ($allowToEdit) {
+                $attendanceController->attendance_add();
+            } else {
+                api_not_allowed(true);
+            }
+            break;
+        case 'attendance_edit':
+            if ($allowToEdit) {
+                $attendanceController->attendance_edit($attendance_id);
+            } else {
+                api_not_allowed(true);
+            }
+            break;
+        case 'attendance_delete':
+            if ($allowToEdit) {
+                $attendanceController->attendance_delete($attendance_id);
+            } else {
+                api_not_allowed(true);
+            }
+            break;
+        case 'attendance_set_invisible':
+            if ($allowToEdit) {
+                $attendanceController->attendanceSetInvisible($attendance_id);
+            } else {
+                api_not_allowed(true);
+            }
+            break;
+        case 'attendance_set_visible':
+            if ($allowToEdit) {
+                $attendanceController->attendanceSetVisible($attendance_id);
+            } else {
+                api_not_allowed(true);
+            }
+            break;
+        /*case 'attendance_restore':
         if ($allowToEdit) {
             $attendanceController->attendance_restore($attendance_id);
         } else {
             api_not_allowed(true);
         }
         break;*/
-    case 'attendance_sheet_list':
-        $attendanceController->attendance_sheet(
-            $action,
-            $attendance_id,
-            $student_id,
-            true
-        );
-        break;
-    case 'attendance_sheet_list_no_edit':
-        $attendanceController->attendance_sheet(
-            $action,
-            $attendance_id,
-            $student_id,
-            false
-        );
-        break;
-    case 'attendance_sheet_export_to_pdf':
-        $attendanceController->attendance_sheet_export_to_pdf(
-            $action,
-            $attendance_id,
-            $student_id,
-            $course_id
-        );
-        break;
-    case 'attendance_sheet_add':
-        if ($allowToEdit) {
-            $attendanceController->attendance_sheet($action, $attendance_id);
-        } else {
-            api_not_allowed(true);
-        }
-        break;
-    case 'lock_attendance':
-    case 'unlock_attendance':
-        if ($allowToEdit) {
-            $attendanceController->lock_attendance($action, $attendance_id);
-        } else {
-            api_not_allowed(true);
-        }
-        break;
-    case 'calendar_add':
-        //no break
-    case 'calendar_edit':
-        //no break
-    case 'calendar_all_delete':
-        //no break
-    case 'calendar_delete':
-        if (!$allowToEdit) {
-            api_not_allowed(true);
-        }
-        //no break
-    case 'calendar_list':
-        $attendanceController->attendance_calendar(
-            $action,
-            $attendance_id,
-            $calendar_id
-        );
-        break;
-    case 'calendar_logins':
-        if (api_is_course_admin() || api_is_drh()) {
-            $attendanceController->getAttendanceBaseInLogin(false, true);
-        }
-        break;
-    default:
-        $attendanceController->attendance_list();
+        case 'attendance_sheet_list':
+            $attendanceController->attendance_sheet(
+                $action,
+                $attendance_id,
+                $student_id,
+                true
+            );
+            break;
+        case 'attendance_sheet_list_no_edit':
+            $attendanceController->attendance_sheet(
+                $action,
+                $attendance_id,
+                $student_id,
+                false
+            );
+            break;
+        case 'attendance_sheet_export_to_pdf':
+            $attendanceController->attendance_sheet_export_to_pdf(
+                $action,
+                $attendance_id,
+                $student_id,
+                $course_id
+            );
+            break;
+        case 'attendance_sheet_add':
+            if ($allowToEdit) {
+                $attendanceController->attendance_sheet($action, $attendance_id);
+            } else {
+                api_not_allowed(true);
+            }
+            break;
+        case 'lock_attendance':
+        case 'unlock_attendance':
+            if ($allowToEdit) {
+                $attendanceController->lock_attendance($action, $attendance_id);
+            } else {
+                api_not_allowed(true);
+            }
+            break;
+        case 'calendar_add':
+            //no break
+        case 'calendar_edit':
+            //no break
+        case 'calendar_all_delete':
+            //no break
+        case 'calendar_delete':
+            if (!$allowToEdit) {
+                api_not_allowed(true);
+            }
+            //no break
+        case 'calendar_list':
+            $attendanceController->attendance_calendar(
+                $action,
+                $attendance_id,
+                $calendar_id
+            );
+            break;
+        case 'calendar_logins':
+            if (api_is_course_admin() || api_is_drh()) {
+                $attendanceController->getAttendanceBaseInLogin(false, true);
+            }
+            break;
+        default:
+            $attendanceController->attendance_list();
 }

main/inc/ajax/chat.ajax.php

Switch Indentation +70 added lines, -70 removed lines

@@ -47,77 +47,77 @@
 }
 
 switch ($action) {
-    case 'chatheartbeat':
-        $chat->heartbeat();
-        break;
-    case 'closechat':
-        $chat->close();
-        break;
-    case 'sendchat':
-        $chat->send(api_get_user_id(), $toUserId, $message);
-        break;
-    case 'startchatsession':
-        $chat->startSession();
-        break;
-    case 'get_previous_messages':
-        $userId = isset($_REQUEST['user_id']) ? $_REQUEST['user_id'] : null;
-        $visibleMessages = isset($_REQUEST['visible_messages']) ? $_REQUEST['visible_messages'] : null;
-        if (empty($userId)) {
-            return '';
-        }
-        $items = $chat->getPreviousMessages(
-            $userId,
-            api_get_user_id(),
-            $visibleMessages
-        );
-        echo json_encode($items);
-        exit;
-        break;
-    case 'set_status':
-        $status = isset($_REQUEST['status']) ? intval($_REQUEST['status']) : 0;
-        $chat->setUserStatus($status);
-        break;
-    case 'create_room':
-        $room = VideoChat::getChatRoomByUsers(api_get_user_id(), $toUserId);
-
-        if ($room === false) {
-            $createdRoom = VideoChat::createRoom(api_get_user_id(), $toUserId);
-
-            if ($createdRoom === false) {
-                echo Display::return_message(
-                    get_lang('ChatRoomNotCreated'),
-                    'error'
-                );
-                break;
+        case 'chatheartbeat':
+            $chat->heartbeat();
+            break;
+        case 'closechat':
+            $chat->close();
+            break;
+        case 'sendchat':
+            $chat->send(api_get_user_id(), $toUserId, $message);
+            break;
+        case 'startchatsession':
+            $chat->startSession();
+            break;
+        case 'get_previous_messages':
+            $userId = isset($_REQUEST['user_id']) ? $_REQUEST['user_id'] : null;
+            $visibleMessages = isset($_REQUEST['visible_messages']) ? $_REQUEST['visible_messages'] : null;
+            if (empty($userId)) {
+                return '';
             }
-
+            $items = $chat->getPreviousMessages(
+                $userId,
+                api_get_user_id(),
+                $visibleMessages
+            );
+            echo json_encode($items);
+            exit;
+            break;
+        case 'set_status':
+            $status = isset($_REQUEST['status']) ? intval($_REQUEST['status']) : 0;
+            $chat->setUserStatus($status);
+            break;
+        case 'create_room':
             $room = VideoChat::getChatRoomByUsers(api_get_user_id(), $toUserId);
-        }
-
-        $videoChatUrl = api_get_path(WEB_LIBRARY_JS_PATH)."chat/video.php?room={$room['id']}";
-        $videoChatLink = Display::url(
-            Display::returnFontAwesomeIcon('video-camera').get_lang('StartVideoChat'),
-            $videoChatUrl
-        );
-
-        $chat->send(
-            api_get_user_id(),
-            $toUserId,
-            $videoChatLink,
-            false,
-            false
-        );
-
-        echo Display::tag('p', $videoChatLink, ['class' => 'lead']);
-        break;
-    case 'notify_not_support':
-        $chat->send(
-            api_get_user_id(),
-            $toUserId,
-            get_lang('TheXUserBrowserDoesNotSupportWebRTC')
-        );
-        break;
-    default:
-        echo '';
+
+            if ($room === false) {
+                $createdRoom = VideoChat::createRoom(api_get_user_id(), $toUserId);
+
+                if ($createdRoom === false) {
+                    echo Display::return_message(
+                        get_lang('ChatRoomNotCreated'),
+                        'error'
+                    );
+                    break;
+                }
+
+                $room = VideoChat::getChatRoomByUsers(api_get_user_id(), $toUserId);
+            }
+
+            $videoChatUrl = api_get_path(WEB_LIBRARY_JS_PATH)."chat/video.php?room={$room['id']}";
+            $videoChatLink = Display::url(
+                Display::returnFontAwesomeIcon('video-camera').get_lang('StartVideoChat'),
+                $videoChatUrl
+            );
+
+            $chat->send(
+                api_get_user_id(),
+                $toUserId,
+                $videoChatLink,
+                false,
+                false
+            );
+
+            echo Display::tag('p', $videoChatLink, ['class' => 'lead']);
+            break;
+        case 'notify_not_support':
+            $chat->send(
+                api_get_user_id(),
+                $toUserId,
+                get_lang('TheXUserBrowserDoesNotSupportWebRTC')
+            );
+            break;
+        default:
+            echo '';
 }
 exit;

main/auth/courses.php

Switch Indentation +128 added lines, -128 removed lines

@@ -186,29 +186,29 @@ 
     }
 }
 switch ($action) {
-    case 'subscribe_user_with_password':
-        $courses_controller->subscribe_user(
-            isset($_POST['subscribe_user_with_password']) ? $_POST['subscribe_user_with_password'] : '',
-            $searchTerm,
-            isset($_POST['category_code']) ? $_POST['category_code'] : ''
-        );
-        break;
-    case 'createcoursecategory':
-        $courses_controller->categories_list($action);
-        break;
-    case 'deletecoursecategory':
-        $courses_controller->courses_list($action);
-        break;
-    case 'sortmycourses':
-        $courses_controller->courses_list($action);
-        break;
-    case 'subscribe':
-        if (!$user_can_view_page) {
-            api_not_allowed(true);
-        }
-        header('Location: '.api_get_self());
-        exit;
-        /* if (!CoursesAndSessionsCatalog::is(CATALOG_SESSIONS)) {
+        case 'subscribe_user_with_password':
+            $courses_controller->subscribe_user(
+                isset($_POST['subscribe_user_with_password']) ? $_POST['subscribe_user_with_password'] : '',
+                $searchTerm,
+                isset($_POST['category_code']) ? $_POST['category_code'] : ''
+            );
+            break;
+        case 'createcoursecategory':
+            $courses_controller->categories_list($action);
+            break;
+        case 'deletecoursecategory':
+            $courses_controller->courses_list($action);
+            break;
+        case 'sortmycourses':
+            $courses_controller->courses_list($action);
+            break;
+        case 'subscribe':
+            if (!$user_can_view_page) {
+                api_not_allowed(true);
+            }
+            header('Location: '.api_get_self());
+            exit;
+            /* if (!CoursesAndSessionsCatalog::is(CATALOG_SESSIONS)) {
             $courses_controller->courses_categories(
                 $action,
                 $categoryCode,
@@ -221,121 +221,121 @@ 
             header('Location: ' . api_get_self());
             exit;
         }*/
-        break;
-    case 'display_random_courses':
-        if (!$user_can_view_page) {
-            api_not_allowed(true);
-        }
-
-        $courses_controller->courses_categories($action);
-        break;
-    case 'display_courses':
-        if (!$user_can_view_page) {
-            api_not_allowed(true);
-        }
-
-        $courses_controller->courses_categories(
-            $action,
-            $categoryCode,
-            null,
-            null,
-            null,
-            $limit
-        );
-        break;
-    case 'display_sessions':
-        if (!$user_can_view_page) {
-            api_not_allowed(true);
-        }
-
-        $courses_controller->sessionsList($action, $nameTools, $limit);
-        break;
-    case 'subscribe_to_session':
-        if (!$user_can_view_page) {
-            api_not_allowed(true);
-        }
-
-        $userId = api_get_user_id();
-        $confirmed = isset($_GET['confirm']);
-        $sessionId = intval($_GET['session_id']);
-
-        if (empty($userId)) {
-            api_not_allowed();
-            exit;
-        }
-
-        if (!$confirmed) {
-            $template = new Template(null, false, false, false, false, false);
-            $template->assign('session_id', $sessionId);
-            $layout = $template->get_template('auth/confirm_session_subscription.tpl');
-            echo $template->fetch($layout);
-            exit;
-        }
+            break;
+        case 'display_random_courses':
+            if (!$user_can_view_page) {
+                api_not_allowed(true);
+            }
 
-        $registrationAllowed = api_get_setting('catalog_allow_session_auto_subscription');
-        if ($registrationAllowed === 'true') {
-            $entityManager = Database::getManager();
-            $repository = $entityManager->getRepository('ChamiloCoreBundle:SequenceResource');
+            $courses_controller->courses_categories($action);
+            break;
+        case 'display_courses':
+            if (!$user_can_view_page) {
+                api_not_allowed(true);
+            }
 
-            $sequences = $repository->getRequirements(
-                $sessionId,
-                SequenceResource::SESSION_TYPE
+            $courses_controller->courses_categories(
+                $action,
+                $categoryCode,
+                null,
+                null,
+                null,
+                $limit
             );
+            break;
+        case 'display_sessions':
+            if (!$user_can_view_page) {
+                api_not_allowed(true);
+            }
+
+            $courses_controller->sessionsList($action, $nameTools, $limit);
+            break;
+        case 'subscribe_to_session':
+            if (!$user_can_view_page) {
+                api_not_allowed(true);
+            }
+
+            $userId = api_get_user_id();
+            $confirmed = isset($_GET['confirm']);
+            $sessionId = intval($_GET['session_id']);
+
+            if (empty($userId)) {
+                api_not_allowed();
+                exit;
+            }
+
+            if (!$confirmed) {
+                $template = new Template(null, false, false, false, false, false);
+                $template->assign('session_id', $sessionId);
+                $layout = $template->get_template('auth/confirm_session_subscription.tpl');
+                echo $template->fetch($layout);
+                exit;
+            }
 
-            if (count($sequences) > 0) {
-                $requirementsData = SequenceResourceManager::checkRequirementsForUser(
-                    $sequences,
-                    SequenceResource::SESSION_TYPE,
-                    $userId
+            $registrationAllowed = api_get_setting('catalog_allow_session_auto_subscription');
+            if ($registrationAllowed === 'true') {
+                $entityManager = Database::getManager();
+                $repository = $entityManager->getRepository('ChamiloCoreBundle:SequenceResource');
+
+                $sequences = $repository->getRequirements(
+                    $sessionId,
+                    SequenceResource::SESSION_TYPE
                 );
 
-                $continueWithSubscription = SequenceResourceManager::checkSequenceAreCompleted($requirementsData);
+                if (count($sequences) > 0) {
+                    $requirementsData = SequenceResourceManager::checkRequirementsForUser(
+                        $sequences,
+                        SequenceResource::SESSION_TYPE,
+                        $userId
+                    );
+
+                    $continueWithSubscription = SequenceResourceManager::checkSequenceAreCompleted($requirementsData);
 
-                if (!$continueWithSubscription) {
-                    header('Location: '.api_get_path(WEB_CODE_PATH).'auth/courses.php');
-                    exit;
+                    if (!$continueWithSubscription) {
+                        header('Location: '.api_get_path(WEB_CODE_PATH).'auth/courses.php');
+                        exit;
+                    }
                 }
-            }
 
-            SessionManager::subscribe_users_to_session(
-                $_GET['session_id'],
-                array($userId),
-                SESSION_VISIBLE_READ_ONLY,
-                false
-            );
+                SessionManager::subscribe_users_to_session(
+                    $_GET['session_id'],
+                    array($userId),
+                    SESSION_VISIBLE_READ_ONLY,
+                    false
+                );
 
-            $coursesList = SessionManager::get_course_list_by_session_id($_GET['session_id']);
-            $count = count($coursesList);
-            $url = '';
-
-            if ($count <= 0) {
-                // no course in session -> return to catalog
-                $url = api_get_path(WEB_CODE_PATH).'auth/courses.php';
-            } elseif ($count == 1) {
-                // only one course, so redirect directly to this course
-                foreach ($coursesList as $course) {
-                    $url = api_get_path(WEB_COURSE_PATH).$course['directory'].'/index.php?id_session='.intval($_GET['session_id']);
+                $coursesList = SessionManager::get_course_list_by_session_id($_GET['session_id']);
+                $count = count($coursesList);
+                $url = '';
+
+                if ($count <= 0) {
+                    // no course in session -> return to catalog
+                    $url = api_get_path(WEB_CODE_PATH).'auth/courses.php';
+                } elseif ($count == 1) {
+                    // only one course, so redirect directly to this course
+                    foreach ($coursesList as $course) {
+                        $url = api_get_path(WEB_COURSE_PATH).$course['directory'].'/index.php?id_session='.intval($_GET['session_id']);
+                    }
+                } else {
+                    $url = api_get_path(WEB_CODE_PATH).'session/index.php?session_id='.intval($_GET['session_id']);
                 }
-            } else {
-                $url = api_get_path(WEB_CODE_PATH).'session/index.php?session_id='.intval($_GET['session_id']);
+                header('Location: '.$url);
+                exit;
             }
-            header('Location: '.$url);
-            exit;
-        }
-        //else show error message?
-        break;
-    case 'search_tag':
-        if (!$user_can_view_page) {
-            api_not_allowed(true);
-        }
-
-        $courses_controller->sessionsListByCoursesTag($limit);
-        break;
-    case 'search_session':
-        if (!$user_can_view_page) {
-            api_not_allowed(true);
-        }
-
-        $courses_controller->sessionListBySearch($limit);
-        break;
+            //else show error message?
+            break;
+        case 'search_tag':
+            if (!$user_can_view_page) {
+                api_not_allowed(true);
+            }
+
+            $courses_controller->sessionsListByCoursesTag($limit);
+            break;
+        case 'search_session':
+            if (!$user_can_view_page) {
+                api_not_allowed(true);
+            }
+
+            $courses_controller->sessionListBySearch($limit);
+            break;
 }