chamilo / chamilo-lms

main/exercise/tests_category.php

Spacing +8 added lines, -8 removed lines

@@ -197,7 +197,7 @@ 
 {
     $action = Security::remove_XSS($action);
     // initiate the object
-    $form = new FormValidator('note', 'post', api_get_self() . '?action=' . $action.'&'.api_get_cidreq());
+    $form = new FormValidator('note', 'post', api_get_self().'?action='.$action.'&'.api_get_cidreq());
     // Setting the form elements
     $form->addElement('header', get_lang('AddACategory'));
     $form->addElement('text', 'category_name', get_lang('CategoryName'), array('size' => '95'));
@@ -237,23 +237,23 @@ 
 function displayActionBar()
 {
     echo '<div class="actions">';
-    echo '<a href="'.api_get_path(WEB_CODE_PATH).'exercise/exercise.php?' . api_get_cidreq() . '">' .
-            Display::return_icon('back.png', get_lang('GoBackToQuestionList'), '', ICON_SIZE_MEDIUM) . '</a>';
+    echo '<a href="'.api_get_path(WEB_CODE_PATH).'exercise/exercise.php?'.api_get_cidreq().'">'.
+            Display::return_icon('back.png', get_lang('GoBackToQuestionList'), '', ICON_SIZE_MEDIUM).'</a>';
 
-    echo '<a href="' . api_get_self() . '?action=addcategory&'.api_get_cidreq().'">' .
-        Display::return_icon('question_category.gif', get_lang('AddACategory')) . '</a>';
+    echo '<a href="'.api_get_self().'?action=addcategory&'.api_get_cidreq().'">'.
+        Display::return_icon('question_category.gif', get_lang('AddACategory')).'</a>';
 
     echo Display::url(
         Display::return_icon('export_csv.png', get_lang('ExportAsCSV'), [], ICON_SIZE_MEDIUM),
-        api_get_self() . '?action=export_category&'.api_get_cidreq()
+        api_get_self().'?action=export_category&'.api_get_cidreq()
     );
 
     echo Display::url(
         Display::return_icon('import_csv.png', get_lang('ImportAsCSV'), [], ICON_SIZE_MEDIUM),
-        api_get_self() . '?action=import_category&'.api_get_cidreq()
+        api_get_self().'?action=import_category&'.api_get_cidreq()
     );
 
     echo '</div>';
     echo "<br/>";
-    echo "<fieldset><legend>" . get_lang('QuestionCategory') . "</legend></fieldset>";
+    echo "<fieldset><legend>".get_lang('QuestionCategory')."</legend></fieldset>";
 }

main/gradebook/lib/fe/dataform.class.php

Spacing +9 added lines, -9 removed lines

@@ -41,13 +41,13 @@ 
 
     protected function build_pdf_export_form()
     {
-        $renderer =& $this->defaultRenderer();
+        $renderer = & $this->defaultRenderer();
         $renderer->setCustomElementTemplate('<span>{element}</span>');
         $this->addElement('header', get_lang('ChooseOrientation'));
         $this->addElement('radio', 'orientation', null, get_lang('Portrait'), 'portrait');
         $this->addElement('radio', 'orientation', null, get_lang('Landscape'), 'landscape');
         $this->addButtonExport(get_lang('Export'));
-        $this->setDefaults(array (
+        $this->setDefaults(array(
             'orientation' => 'portrait'
         ));
     }
@@ -59,12 +59,12 @@ 
         $this->addElement('radio', 'file_type', null, 'XML (Extensible Markup Language)', 'xml');
         $this->addElement('radio', 'file_type', null, 'PDF (Portable Document Format)', 'pdf');
         $this->addButtonExport(get_lang('Export'));
-        $this->setDefaults(array (
+        $this->setDefaults(array(
             'file_type' => 'csv'
         ));
     }
 
-    protected function build_export_form_option($show_pdf=true)
+    protected function build_export_form_option($show_pdf = true)
     {
         $this->addElement('header', get_lang('ChooseFormat'));
         $this->addElement('radio', 'file_type', get_lang('OutputFileType'), 'CSV (Comma-Separated Values)', 'csv');
@@ -78,7 +78,7 @@ 
             array('disabled')
         );
         $this->addButtonExport(get_lang('Export'));
-        $this->setDefaults(array (
+        $this->setDefaults(array(
             'file_type' => 'csv'
         ));
     }
@@ -87,14 +87,14 @@ 
     {
         $this->addElement('hidden', 'formSent');
         $this->addElement('header', get_lang('ImportFileLocation'));
-        $this->addElement('file', 'import_file',get_lang('Location'));
-        $allowed_file_types = array (
+        $this->addElement('file', 'import_file', get_lang('Location'));
+        $allowed_file_types = array(
             'xml',
             'csv'
         );
         //$this->addRule('file', get_lang('InvalidExtension') . ' (' . implode(',', $allowed_file_types) . ')', 'filetype', $allowed_file_types);
-        $this->addElement('radio', 'file_type', get_lang('FileType'), 'CSV (<a href="docs/example_csv.html" target="_blank">' . get_lang('ExampleCSVFile') . '</a>)', 'csv');
-        $this->addElement('radio', 'file_type', null, 'XML (<a href="docs/example_xml.html" target="_blank">' . get_lang('ExampleXMLFile') . '</a>)', 'xml');
+        $this->addElement('radio', 'file_type', get_lang('FileType'), 'CSV (<a href="docs/example_csv.html" target="_blank">'.get_lang('ExampleCSVFile').'</a>)', 'csv');
+        $this->addElement('radio', 'file_type', null, 'XML (<a href="docs/example_xml.html" target="_blank">'.get_lang('ExampleXMLFile').'</a>)', 'xml');
         $this->addElement('checkbox', 'overwrite', null, get_lang('OverwriteScores'));
         $this->addElement('checkbox', 'ignoreerrors', null, get_lang('IgnoreErrors'));
         $this->addButtonImport(get_lang('Ok'));

main/lp/lp_view.php

Spacing +23 added lines, -23 removed lines

@@ -20,7 +20,7 @@ 
 $this_section = SECTION_COURSES;
 
 if ($lp_controller_touched != 1) {
-    header('location: lp_controller.php?action=view&item_id=' . intval($_REQUEST['item_id']));
+    header('location: lp_controller.php?action=view&item_id='.intval($_REQUEST['item_id']));
     exit;
 }
 
@@ -118,7 +118,7 @@ 
 //Impress js
 if ($_SESSION['oLP']->mode == 'impress') {
     $lp_id = $_SESSION['oLP']->get_id();
-    $url = api_get_path(WEB_CODE_PATH) . "lp/lp_impress.php?lp_id=$lp_id&" . api_get_cidreq();
+    $url = api_get_path(WEB_CODE_PATH)."lp/lp_impress.php?lp_id=$lp_id&".api_get_cidreq();
     header("Location: $url");
     exit;
 }
@@ -136,15 +136,15 @@ 
 
 // additional APIs
 $htmlHeadXtra[] = '<script>
-chamilo_courseCode = "' . $course_code . '";
+chamilo_courseCode = "' . $course_code.'";
 </script>';
 // Document API
 $htmlHeadXtra[] = '<script src="js/documentapi.js" type="text/javascript" language="javascript"></script>';
 // Storage API
 $htmlHeadXtra[] = '<script>
-var sv_user = \'' . api_get_user_id() . '\';
+var sv_user = \'' . api_get_user_id().'\';
 var sv_course = chamilo_courseCode;
-var sv_sco = \'' . $lp_id . '\';
+var sv_sco = \'' . $lp_id.'\';
 </script>'; // FIXME fetch sco and userid from a more reliable source directly in sotrageapi.js
 $htmlHeadXtra[] = '<script type="text/javascript" src="js/storageapi.js"></script>';
 
@@ -216,7 +216,7 @@ 
         case 3:
             // aicc
             $_SESSION['oLP']->stop_previous_item(); // save status manually if asset
-            $htmlHeadXtra[] = '<script src="' . $_SESSION['oLP']->get_js_lib().'" type="text/javascript" language="javascript"></script>';
+            $htmlHeadXtra[] = '<script src="'.$_SESSION['oLP']->get_js_lib().'" type="text/javascript" language="javascript"></script>';
             $preReqCheck = $_SESSION['oLP']->prerequisites_match($lp_item_id);
             if ($preReqCheck === true) {
                 $src = $_SESSION['oLP']->get_link(
@@ -238,10 +238,10 @@ 
 // Update status, total_time from lp_item_view table when you finish the exercises in learning path.
 
 if ($debug) {
-    error_log('$type_quiz: ' . $type_quiz);
-    error_log('$_REQUEST[exeId]: ' . intval($_REQUEST['exeId']));
-    error_log('$lp_id: ' . $lp_id);
-    error_log('$_GET[lp_item_id]: ' . intval($_GET['lp_item_id']));
+    error_log('$type_quiz: '.$type_quiz);
+    error_log('$_REQUEST[exeId]: '.intval($_REQUEST['exeId']));
+    error_log('$lp_id: '.$lp_id);
+    error_log('$_GET[lp_item_id]: '.intval($_GET['lp_item_id']));
 }
 
 if (
@@ -264,7 +264,7 @@ 
         $safe_item_id == strval(intval($safe_item_id))
     ) {
         $sql = 'SELECT start_date, exe_date, exe_result, exe_weighting, exe_exo_id
-                FROM ' . $TBL_TRACK_EXERCICES . '
+                FROM ' . $TBL_TRACK_EXERCICES.'
                 WHERE exe_id = ' . $safe_exe_id;
         $res = Database::query($sql);
         $row_dates = Database::fetch_array($res);
@@ -278,14 +278,14 @@ 
 
         $sql = "UPDATE $TBL_LP_ITEM SET
                     max_score = '$max_score'
-                WHERE c_id = $course_id AND id = '" . $safe_item_id . "'";
+                WHERE c_id = $course_id AND id = '".$safe_item_id."'";
         Database::query($sql);
 
         $sql = "SELECT id FROM $TBL_LP_ITEM_VIEW
                 WHERE
                     c_id = $course_id AND
                     lp_item_id = '$safe_item_id' AND
-                    lp_view_id = '" . $_SESSION['oLP']->lp_view_id . "'
+                    lp_view_id = '".$_SESSION['oLP']->lp_view_id."'
                 ORDER BY id DESC
                 LIMIT 1";
         $res_last_attempt = Database::query($sql);
@@ -314,7 +314,7 @@ 
                         status = '$status',
                         score = $score,
                         total_time = $mytime
-                    WHERE id='" . $lp_item_view_id . "' AND c_id = $course_id ";
+                    WHERE id='".$lp_item_view_id."' AND c_id = $course_id ";
 
             if ($debug) {
                 error_log($sql);
@@ -324,17 +324,17 @@ 
 
             $sql = "UPDATE $TBL_TRACK_EXERCICES SET
                         orig_lp_item_view_id = $lp_item_view_id
-                    WHERE exe_id = " . $safe_exe_id;
+                    WHERE exe_id = ".$safe_exe_id;
             Database::query($sql);
         }
     }
     if (intval($_GET['fb_type']) > 0) {
         $src = 'blank.php?msg=exerciseFinished';
     } else {
-        $src = api_get_path(WEB_CODE_PATH) . 'exercise/result.php?origin=learnpath&id=' . $safe_exe_id.'&'.api_get_cidreq();
+        $src = api_get_path(WEB_CODE_PATH).'exercise/result.php?origin=learnpath&id='.$safe_exe_id.'&'.api_get_cidreq();
 
         if ($debug) {
-            error_log('Calling URL: ' . $src);
+            error_log('Calling URL: '.$src);
         }
     }
     $autostart = 'false';
@@ -392,8 +392,8 @@ 
 $tbl_lp_item = Database::get_course_table(TABLE_LP_ITEM);
 $show_audioplayer = false;
 // Getting all the information about the item.
-$sql = "SELECT audio FROM " . $tbl_lp_item . "
-        WHERE c_id = $course_id AND lp_id = '" . $_SESSION['oLP']->lp_id . "'";
+$sql = "SELECT audio FROM ".$tbl_lp_item."
+        WHERE c_id = $course_id AND lp_id = '".$_SESSION['oLP']->lp_id."'";
 $res_media = Database::query($sql);
 
 if (Database::num_rows($res_media) > 0) {
@@ -414,7 +414,7 @@ 
         'name' => get_lang('LearningPaths')
     );
     $interbreadcrumb[] = array(
-        'url' => api_get_self() . "?action=add_item&type=step&lp_id={$_SESSION['oLP']->lp_id}&isStudentView=false&".api_get_cidreq(true, true, 'course'),
+        'url' => api_get_self()."?action=add_item&type=step&lp_id={$_SESSION['oLP']->lp_id}&isStudentView=false&".api_get_cidreq(true, true, 'course'),
         'name' => $_SESSION['oLP']->get_name()
     );
 
@@ -426,12 +426,12 @@ 
 
 // Return to course home.
 if ($is_allowed_to_edit) {
-    $buttonHomeUrl = 'lp_controller.php?' . api_get_cidreq(true, true, 'course') . '&' . http_build_query([
+    $buttonHomeUrl = 'lp_controller.php?'.api_get_cidreq(true, true, 'course').'&'.http_build_query([
         'isStudentView' => 'false',
         'action' => 'return_to_course_homepage'
     ]);
 } else {
-    $buttonHomeUrl = 'lp_controller.php?' . api_get_cidreq(true, true, 'course') . '&' . http_build_query([
+    $buttonHomeUrl = 'lp_controller.php?'.api_get_cidreq(true, true, 'course').'&'.http_build_query([
         'action' => 'return_to_course_homepage'
     ]);
 }
@@ -533,7 +533,7 @@ 
 
 $template->assign('lp_author', $_SESSION['oLP']->get_author());
 $template->assign('lp_mode', $_SESSION['oLP']->mode);
-$template->assign('lp_title_scorm',$_SESSION['oLP']->name);
+$template->assign('lp_title_scorm', $_SESSION['oLP']->name);
 $template->assign(
     'lp_html_toc',
     $_SESSION['oLP']->get_html_toc($get_toc_list)

main/ticket/tickets.php

Spacing +15 added lines, -15 removed lines

@@ -19,7 +19,7 @@ 
     $.ajax({
         contentType: "application/x-www-form-urlencoded",
         beforeSend: function(object) {
-        $("div#"+div_course).html("<img src=\'' . $webLibPath . 'javascript/indicator.gif\' />"); },
+        $("div#"+div_course).html("<img src=\'' . $webLibPath.'javascript/indicator.gif\' />"); },
         type: "POST",
         url: "ticket_assign_log.php",
         data: "ticket_id="+ticket_id,
@@ -42,10 +42,10 @@ 
 function display_advanced_search_form() {
     if ($("#advanced_search_form").css("display") == "none") {
         $("#advanced_search_form").css("display","block");
-        $("#img_plus_and_minus").html(\'&nbsp;' . Display::return_icon('div_hide.gif', get_lang('Hide'), array('style' => 'vertical-align:middle')) . '&nbsp;' . get_lang('AdvancedSearch') . '\');
+        $("#img_plus_and_minus").html(\'&nbsp;' . Display::return_icon('div_hide.gif', get_lang('Hide'), array('style' => 'vertical-align:middle')).'&nbsp;'.get_lang('AdvancedSearch').'\');
     } else {
         $("#advanced_search_form").css("display","none");
-        $("#img_plus_and_minus").html(\'&nbsp;' . Display::return_icon('div_show.gif', get_lang('Show'), array('style' => 'vertical-align:middle')) . '&nbsp;' . get_lang('AdvancedSearch') . '\');
+        $("#img_plus_and_minus").html(\'&nbsp;' . Display::return_icon('div_show.gif', get_lang('Show'), array('style' => 'vertical-align:middle')).'&nbsp;'.get_lang('AdvancedSearch').'\');
     }
 }
 </script>';
@@ -219,21 +219,21 @@ 
         null,
         '<a href="javascript://" class = "advanced-parameters" onclick="display_advanced_search_form();">'
         . '<span id="img_plus_and_minus">&nbsp;'
-        . Display::return_icon('div_show.gif', get_lang('Show')) . ' '
-        . get_lang('AdvancedSearch') . '</span></a>'
+        . Display::return_icon('div_show.gif', get_lang('Show')).' '
+        . get_lang('AdvancedSearch').'</span></a>'
     );
 
     echo '<div class="actions" >';
     if (api_is_platform_admin()) {
-        echo '<span class="left">' .
-                '<a href="' . api_get_path(WEB_CODE_PATH) . 'ticket/new_ticket.php?project_id='.$projectId.'">' .
-                    Display::return_icon('add.png', get_lang('Add'), '', ICON_SIZE_MEDIUM) . '</a>' .
-                '<a href="' . api_get_self() . '?action=export' . $get_parameter . $get_parameter2 . '&project_id='.$projectId.'">' .
-                    Display::return_icon('export_excel.png', get_lang('Export'), '', ICON_SIZE_MEDIUM) . '</a>';
+        echo '<span class="left">'.
+                '<a href="'.api_get_path(WEB_CODE_PATH).'ticket/new_ticket.php?project_id='.$projectId.'">'.
+                    Display::return_icon('add.png', get_lang('Add'), '', ICON_SIZE_MEDIUM).'</a>'.
+                '<a href="'.api_get_self().'?action=export'.$get_parameter.$get_parameter2.'&project_id='.$projectId.'">'.
+                    Display::return_icon('export_excel.png', get_lang('Export'), '', ICON_SIZE_MEDIUM).'</a>';
 
         echo Display::url(
             Display::return_icon('settings.png', get_lang('Categories')),
-            api_get_path(WEB_CODE_PATH) . 'ticket/settings.php'
+            api_get_path(WEB_CODE_PATH).'ticket/settings.php'
         );
         echo '</span>';
     }
@@ -287,8 +287,8 @@ 
 } else {
     if (api_get_setting('ticket_allow_student_add') === 'true') {
         echo '<div class="actions" >';
-        echo '<a href="' . api_get_path(WEB_CODE_PATH) . 'ticket/new_ticket.php?project_id='.$projectId.'">' .
-                Display::return_icon('add.png', get_lang('Add'), '', '32') .
+        echo '<a href="'.api_get_path(WEB_CODE_PATH).'ticket/new_ticket.php?project_id='.$projectId.'">'.
+                Display::return_icon('add.png', get_lang('Add'), '', '32').
              '</a>';
         echo '</div>';
     }
@@ -320,8 +320,8 @@ 
     $table->set_header(6, get_lang('AssignedTo'), true);
     $table->set_header(7, get_lang('Message'), true);
 } else {
-    echo '<center><h1>' . get_lang('MyTickets') . '</h1></center>';
-    echo '<center><p>' . get_lang('TicketMsgWelcome') . '</p></center>';
+    echo '<center><h1>'.get_lang('MyTickets').'</h1></center>';
+    echo '<center><p>'.get_lang('TicketMsgWelcome').'</p></center>';
     $table->set_header(0, '#', true);
     $table->set_header(1, get_lang('Status'), false);
     $table->set_header(2, get_lang('Date'), true);

main/inc/lib/internationalization.lib.php

Spacing +5 added lines, -5 removed lines

@@ -633,10 +633,10 @@ 
         $date_format = str_replace(
             array('%A', '%a', '%B', '%b'),
             array(
-                $translated['days_long'][(int)strftime('%w', $time)],
-                $translated['days_short'][(int)strftime('%w', $time)],
-                $translated['months_long'][(int)strftime('%m', $time) - 1],
-                $translated['months_short'][(int)strftime('%m', $time) - 1],
+                $translated['days_long'][(int) strftime('%w', $time)],
+                $translated['days_short'][(int) strftime('%w', $time)],
+                $translated['months_long'][(int) strftime('%m', $time) - 1],
+                $translated['months_short'][(int) strftime('%m', $time) - 1],
             ),
             $date_format
         );
@@ -1583,7 +1583,7 @@ 
  * @return string					Returns the encoding identificator modified in suitable for comparison way.
  */
 function api_refine_encoding_id($encoding) {
-    if (is_array($encoding)){
+    if (is_array($encoding)) {
         return array_map('api_refine_encoding_id', $encoding);
     }
     return strtoupper(str_replace('_', '-', $encoding));

main/ticket/new_ticket.php

Spacing +17 added lines, -17 removed lines

@@ -8,7 +8,7 @@ 
 require_once __DIR__.'/../inc/global.inc.php';
 
 if (!api_is_platform_admin() && api_get_setting('ticket_allow_student_add') != 'true') {
-    header('location:' . api_get_path(WEB_CODE_PATH).'ticket/tickets.php');
+    header('location:'.api_get_path(WEB_CODE_PATH).'ticket/tickets.php');
     exit;
 }
 
@@ -84,15 +84,15 @@ 
     var selected = document.getElementById("category_id").selectedIndex;
     var id = document.getElementById("category_id").options[selected].value;
     if(parseInt(course_required[id]) == 1 && document.getElementById("course_id").value == 0) {
-        alert("' . addslashes(get_lang("ValidCourse")) . '");
+        alert("' . addslashes(get_lang("ValidCourse")).'");
         return false;
     } else if(id != "CUR" && parseInt(course_required[id]) != 1  && !re.test(document.getElementById("personal_email").value)) {
         if (document.getElementById("personal_email").value != "") {
-            alert("' . addslashes(get_lang("PleaseEnterValidEmail")) . '");
+            alert("' . addslashes(get_lang("PleaseEnterValidEmail")).'");
             return false;
         }
     } else if(fckEditor1val == "") {
-        alert("' . addslashes(get_lang("Filled")) . '");
+        alert("' . addslashes(get_lang("Filled")).'");
         return false;
     }
 }
@@ -135,7 +135,7 @@ 
     });
 
     img_remove = $("<img/>", {
-        src: "' . Display::returnIconPath('delete.png') . '"
+        src: "' . Display::returnIconPath('delete.png').'"
     });
 
     new_filepath_id = $("#filepath_" + counter_image);
@@ -167,10 +167,10 @@ 
 
 $types = TicketManager::get_all_tickets_categories($projectId, 'category.name ASC');
 $htmlHeadXtra[] = '<script language="javascript">
-    var projects = ' . js_array($types, 'projects', 'project_id') . '
-    var course_required = ' . js_array($types, 'course_required', 'course_required') . '
-    var other_area = ' . js_array($types, 'other_area', 'other_area') . '
-    var email = ' . js_array($types, 'email', 'email') .
+    var projects = ' . js_array($types, 'projects', 'project_id').'
+    var course_required = ' . js_array($types, 'course_required', 'course_required').'
+    var other_area = ' . js_array($types, 'other_area', 'other_area').'
+    var email = ' . js_array($types, 'email', 'email').
 '</script>';
 
 /**
@@ -179,7 +179,7 @@ 
  */
 function js_str($s)
 {
-    return '"' . addcslashes($s, "\0..\37\"\\") . '"';
+    return '"'.addcslashes($s, "\0..\37\"\\").'"';
 }
 
 /**
@@ -192,7 +192,7 @@ 
 {
     $return = "new Array(); ";
     foreach ($array as $value) {
-        $return .= $name . "['" . $value['category_id'] . "'] ='" . $value[$key] . "'; ";
+        $return .= $name."['".$value['category_id']."'] ='".$value[$key]."'; ";
     }
 
     return $return;
@@ -380,7 +380,7 @@ 
     $form->addElement(
         'text',
         'phone',
-        get_lang('Phone') . ' (' . get_lang('Optional') . ')',
+        get_lang('Phone').' ('.get_lang('Optional').')',
         array(
             'id' => 'phone'
         )
@@ -419,7 +419,7 @@ 
 
     $form->addLabel('',
         '<span id="link-more-attach">
-         <span class="btn btn-success" onclick="return add_image_form()">' . get_lang('AddOneMoreFile') . '</span>
+         <span class="btn btn-success" onclick="return add_image_form()">' . get_lang('AddOneMoreFile').'</span>
          </span>
          ('.sprintf(get_lang('MaximunFileSizeX'), format_file_size(api_get_setting('message_max_upload_filesize'))).')
     ');
@@ -453,7 +453,7 @@ 
     $category_id = $_POST['category_id'];
     $content = $_POST['content'];
     if ($_POST['phone'] != '') {
-        $content .= '<p style="color:red">&nbsp;' . get_lang('Phone') . ': ' . Security::remove_XSS($_POST['phone']). '</p>';
+        $content .= '<p style="color:red">&nbsp;'.get_lang('Phone').': '.Security::remove_XSS($_POST['phone']).'</p>';
     }
     $course_id = isset($_POST['course_id']) ? $_POST['course_id'] : '';
     $sessionId = isset($_POST['session_id']) ? $_POST['session_id'] : '';
@@ -485,7 +485,7 @@ 
         $status,
         $user_id
     )) {
-        header('Location:' . api_get_path(WEB_CODE_PATH).'ticket/tickets.php');
+        header('Location:'.api_get_path(WEB_CODE_PATH).'ticket/tickets.php');
         exit;
     } else {
         Display::display_header(get_lang('ComposeMessage'));
@@ -504,7 +504,7 @@ 
     $sql = "SELECT COUNT(u.user_id) AS total_number_of_items FROM $user_table u";
     if ((api_is_platform_admin() || api_is_session_admin()) && api_get_multiple_access_url()) {
         $access_url_rel_user_table = Database :: get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
-        $sql.= " INNER JOIN $access_url_rel_user_table url_rel_user ON (u.user_id=url_rel_user.user_id)";
+        $sql .= " INNER JOIN $access_url_rel_user_table url_rel_user ON (u.user_id=url_rel_user.user_id)";
     }
     if (isset($_GET['keyword'])) {
         $keyword = Database::escape_string(trim($_GET['keyword']));
@@ -581,7 +581,7 @@ 
     while ($user = Database::fetch_row($res)) {
         $user_id = $user[0];
         $userPicture = UserManager::getUserPicture($user_id);
-        $photo = '<img src="' . $userPicture. '" alt="' . api_get_person_name($user[2], $user[3]) . '" title="' . api_get_person_name($user[2], $user[3]) . '" />';
+        $photo = '<img src="'.$userPicture.'" alt="'.api_get_person_name($user[2], $user[3]).'" title="'.api_get_person_name($user[2], $user[3]).'" />';
         $users[] = array(
             $photo,
             $user_id,

main/messages/new_message.php

Spacing +23 added lines, -23 removed lines

@@ -24,7 +24,7 @@ 
 $nameTools = api_xml_http_response_encode(get_lang('Messages'));
 /*	Constants and variables */
 
-$htmlHeadXtra[]='
+$htmlHeadXtra[] = '
 <script>
 function validate(form, list) {
 	if(list.selectedIndex<0) {
@@ -82,7 +82,7 @@ 
               FROM $table_message
 			  WHERE user_receiver_id = ".intval($receiver_id)." AND id='".intval($message_id)."';";
 	$result = Database::query($query);
-	$row = Database::fetch_array($result,'ASSOC');
+	$row = Database::fetch_array($result, 'ASSOC');
 	if (!isset($row['user_sender_id'])) {
 		$html = get_lang('InvalidMessageId');
 
@@ -95,7 +95,7 @@ 
     return $html;
 }
 
-function show_compose_to_user ($receiver_id)
+function show_compose_to_user($receiver_id)
 {
     $userInfo = api_get_user_info($receiver_id);
 	$html = get_lang('To').':&nbsp;<strong>'.$userInfo['complete_name'].'</strong>';
@@ -109,7 +109,7 @@ 
 function manage_form($default, $select_from_user_list = null, $sent_to = null)
 {
     $group_id = isset($_REQUEST['group_id']) ? intval($_REQUEST['group_id']) : null;
-    $message_id = isset($_GET['message_id'])  ?  intval($_GET['message_id']) : null;
+    $message_id = isset($_GET['message_id']) ? intval($_GET['message_id']) : null;
     $param_f = isset($_GET['f']) && $_GET['f'] == 'social' ? 'social' : null;
 
     $form = new FormValidator(
@@ -133,7 +133,7 @@ 
             );
             $form->addRule('id_text_name', get_lang('ThisFieldIsRequired'), 'required');
             $form->addElement('html', '<div id="id_div_search" style="padding:0px" class="message-select-box" >&nbsp;</div>');
-            $form->addElement('hidden','user_list', 0, array('id'=>'user_list'));
+            $form->addElement('hidden', 'user_list', 0, array('id'=>'user_list'));
         } else {
             if (!empty($sent_to)) {
                 $form->addLabel(get_lang('SendMessageTo'), $sent_to);
@@ -147,7 +147,7 @@ 
                     array(),
                     [
                         'multiple' => 'multiple',
-                        'url' => api_get_path(WEB_AJAX_PATH) . 'message.ajax.php?a=find_users'
+                        'url' => api_get_path(WEB_AJAX_PATH).'message.ajax.php?a=find_users'
                     ]
                 );
             } else {
@@ -211,7 +211,7 @@ 
 
         $form->addLabel(
             '',
-            '<span id="link-more-attach"><a href="javascript://" onclick="return add_image_form()">'.get_lang('AddOneMoreFile').'</a></span>&nbsp;('.sprintf(get_lang('MaximunFileSizeX'),format_file_size(api_get_setting('message_max_upload_filesize'))).')'
+            '<span id="link-more-attach"><a href="javascript://" onclick="return add_image_form()">'.get_lang('AddOneMoreFile').'</a></span>&nbsp;('.sprintf(get_lang('MaximunFileSizeX'), format_file_size(api_get_setting('message_max_upload_filesize'))).')'
         );
     }
 
@@ -233,7 +233,7 @@ 
             $content = $default['content'];
             $group_id = isset($default['group_id']) ? $default['group_id'] : null;
             $parent_id = isset($default['parent_id']) ? $default['parent_id'] : null;
-            if (is_array($user_list) && count($user_list)> 0) {
+            if (is_array($user_list) && count($user_list) > 0) {
                 //all is well, send the message
                 foreach ($user_list as $userId) {
                     $res = MessageManager::send_message(
@@ -248,7 +248,7 @@ 
                     if ($res) {
                         $userInfo = api_get_user_info($userId);
                         Display::addFlash(Display::return_message(
-                            get_lang('MessageSentTo') ."&nbsp;<b>" .$userInfo['complete_name'] ."</b>",
+                            get_lang('MessageSentTo')."&nbsp;<b>".$userInfo['complete_name']."</b>",
                             'confirmation',
                             false
                         ));
@@ -291,9 +291,9 @@ 
 if ($group_id != 0) {
 	$social_right_content .= '<div class=actions>';
 	$social_right_content .= '<a href="'.api_get_path(WEB_PATH).'main/social/group_view.php?id='.$group_id.'">'.
-		Display::return_icon('back.png',api_xml_http_response_encode(get_lang('ComposeMessage'))).'</a>';
+		Display::return_icon('back.png', api_xml_http_response_encode(get_lang('ComposeMessage'))).'</a>';
 	$social_right_content .= '<a href="'.api_get_path(WEB_PATH).'main/messages/new_message.php?group_id='.$group_id.'">'.
-		Display::return_icon('message_new.png',api_xml_http_response_encode(get_lang('ComposeMessage'))).'</a>';
+		Display::return_icon('message_new.png', api_xml_http_response_encode(get_lang('ComposeMessage'))).'</a>';
 	$social_right_content .= '</div>';
 } else {
 	if ($socialToolIsActive) {
@@ -305,11 +305,11 @@ 
 		}
 		if (api_get_setting('allow_message_tool') === 'true') {
 			$social_right_content .= '<a href="'.api_get_path(WEB_PATH).'main/messages/new_message.php">'.
-                Display::return_icon('message_new.png',get_lang('ComposeMessage')).'</a>';
+                Display::return_icon('message_new.png', get_lang('ComposeMessage')).'</a>';
 			$social_right_content .= '<a href="'.api_get_path(WEB_PATH).'main/messages/inbox.php">'.
-                Display::return_icon('inbox.png',get_lang('Inbox')).'</a>';
+                Display::return_icon('inbox.png', get_lang('Inbox')).'</a>';
             $social_right_content .= '<a href="'.api_get_path(WEB_PATH).'main/messages/outbox.php">'.
-                Display::return_icon('outbox.png',get_lang('Outbox')).'</a>';
+                Display::return_icon('outbox.png', get_lang('Outbox')).'</a>';
 		}
 		$social_right_content .= '</div>';
 	}
@@ -323,21 +323,21 @@ 
     $social_right_content .= '<div class="row">';
     $social_right_content .= '<div class="col-md-12">';
     $social_right_content .= '<div class="actions">';
-    $social_right_content .=  '<a href="'.api_get_path(WEB_PATH).'main/messages/inbox.php?f=social">'.
+    $social_right_content .= '<a href="'.api_get_path(WEB_PATH).'main/messages/inbox.php?f=social">'.
         Display::return_icon('back.png', get_lang('Back'), array(), 32).'</a>';
-    $social_right_content .=  '</div>';
-    $social_right_content .=  '</div>';
+    $social_right_content .= '</div>';
+    $social_right_content .= '</div>';
     $social_right_content .= '<div class="col-md-12">';
 }
 
 // MAIN CONTENT
 if (!isset($_POST['compose'])) {
-    if(isset($_GET['re_id'])) {
+    if (isset($_GET['re_id'])) {
         $social_right_content .= show_compose_reply_to_message(
             $_GET['re_id'],
             api_get_user_id()
         );
-    } elseif(isset($_GET['send_to_user'])) {
+    } elseif (isset($_GET['send_to_user'])) {
         $social_right_content .= show_compose_to_user($_GET['send_to_user']);
     } else {
         $social_right_content .= show_compose_to_any(api_get_user_id());
@@ -348,7 +348,7 @@ 
         $restrict = true;
     } elseif (isset($_POST['group_id'])) {
         $restrict = true;
-    } elseif(isset($_POST['hidden_user'])) {
+    } elseif (isset($_POST['hidden_user'])) {
         $restrict = true;
     }
 
@@ -371,13 +371,13 @@ 
             }
             $social_right_content .= manage_form($default);
         } else {
-            $social_right_content .= Display::return_message(get_lang('ErrorSendingMessage'),'error');
+            $social_right_content .= Display::return_message(get_lang('ErrorSendingMessage'), 'error');
         }
     }
 }
 if (api_get_setting('allow_social_tool') === 'true') {
-    $social_right_content .=  '</div>';
-    $social_right_content .=  '</div>';
+    $social_right_content .= '</div>';
+    $social_right_content .= '</div>';
 }
 
 $tpl = new Template(get_lang('ComposeMessage'));

main/mySpace/users.php

Spacing +8 added lines, -8 removed lines

@@ -20,14 +20,14 @@ 
 
 $this_section = SECTION_TRACKING;
 
-$interbreadcrumb[] = array ("url" => "index.php", "name" => get_lang('MySpace'));
+$interbreadcrumb[] = array("url" => "index.php", "name" => get_lang('MySpace'));
 
 if (isset($_GET["user_id"]) && $_GET["user_id"] != "" && !isset($_GET["type"])) {
-    $interbreadcrumb[] = array ("url" => "teachers.php", "name" => get_lang('Teachers'));
+    $interbreadcrumb[] = array("url" => "teachers.php", "name" => get_lang('Teachers'));
 }
 
-if (isset($_GET["user_id"]) && $_GET["user_id"]!="" && isset($_GET["type"]) && $_GET["type"] == "coach") {
-    $interbreadcrumb[] = array ("url" => "coaches.php", "name" => get_lang('Tutors'));
+if (isset($_GET["user_id"]) && $_GET["user_id"] != "" && isset($_GET["type"]) && $_GET["type"] == "coach") {
+    $interbreadcrumb[] = array("url" => "coaches.php", "name" => get_lang('Tutors'));
 }
 
 function get_count_users()
@@ -163,7 +163,7 @@ 
             $detailsLink = '<a href="myStudents.php?student='.$student_id.'&id_coach='.$coach_id.'&id_session='.$sessionId.'">
 				            '.Display::return_icon('2rightarrow.png', get_lang('Details')).'</a>';
         } else {
-            $detailsLink =  '<a href="myStudents.php?student='.$student_id.'">
+            $detailsLink = '<a href="myStudents.php?student='.$student_id.'">
 				            '.Display::return_icon('2rightarrow.png', get_lang('Details')).'</a>';
         }
         $row[] = $detailsLink;
@@ -184,7 +184,7 @@ 
 
 if (api_is_drh()) {
     $menu_items = array(
-        Display::url(Display::return_icon('stats.png', get_lang('MyStats'), '', ICON_SIZE_MEDIUM), api_get_path(WEB_CODE_PATH)."auth/my_progress.php" ),
+        Display::url(Display::return_icon('stats.png', get_lang('MyStats'), '', ICON_SIZE_MEDIUM), api_get_path(WEB_CODE_PATH)."auth/my_progress.php"),
         Display::url(Display::return_icon('user_na.png', get_lang('Students'), array(), ICON_SIZE_MEDIUM), '#'),
         Display::url(Display::return_icon('teacher.png', get_lang('Trainers'), array(), ICON_SIZE_MEDIUM), 'teachers.php'),
         Display::url(Display::return_icon('course.png', get_lang('Courses'), array(), ICON_SIZE_MEDIUM), 'course.php'),
@@ -240,14 +240,14 @@ 
 
 if ($export_csv) {
     if ($is_western_name_order) {
-        $csv_header[] = array (
+        $csv_header[] = array(
             get_lang('FirstName'),
             get_lang('LastName'),
             get_lang('FirstLogin'),
             get_lang('LastConnexion')
         );
     } else {
-        $csv_header[] = array (
+        $csv_header[] = array(
             get_lang('LastName'),
             get_lang('FirstName'),
             get_lang('FirstLogin'),

main/forum/newthread.php

Spacing +3 added lines, -3 removed lines

@@ -54,12 +54,12 @@ 
 
 /* Breadcrumbs */
 
-if (isset($_SESSION['gradebook'])){
+if (isset($_SESSION['gradebook'])) {
     $gradebook = Security::remove_XSS($_SESSION['gradebook']);
 }
 
 if (!empty($gradebook) && $gradebook == 'view') {
-    $interbreadcrumb[] = array (
+    $interbreadcrumb[] = array(
         'url' => '../gradebook/'.Security::remove_XSS($_SESSION['gradebook_dest']),
         'name' => get_lang('ToolGradebook')
     );
@@ -117,7 +117,7 @@ 
     $interbreadcrumb[] = array('url' => api_get_path(WEB_CODE_PATH).'group/group.php?'.$cidreq, 'name' => get_lang('Groups'));
     $interbreadcrumb[] = array('url' => api_get_path(WEB_CODE_PATH).'group/group_space.php?'.$cidreq, 'name' => get_lang('GroupSpace').' '.$groupProperties['name']);
     $interbreadcrumb[] = array('url' => api_get_path(WEB_CODE_PATH).'forum/viewforum.php?'.$cidreq.'&forum='.intval($_GET['forum']), 'name' => $current_forum['forum_title']);
-    $interbreadcrumb[] = array('url' => api_get_path(WEB_CODE_PATH).'forum/newthread.php?'.$cidreq.'&forum='.intval($_GET['forum']),'name' => get_lang('NewTopic'));
+    $interbreadcrumb[] = array('url' => api_get_path(WEB_CODE_PATH).'forum/newthread.php?'.$cidreq.'&forum='.intval($_GET['forum']), 'name' => get_lang('NewTopic'));
 } else {
     $interbreadcrumb[] = array('url' => api_get_path(WEB_CODE_PATH).'forum/index.php?'.$cidreq, 'name' => $nameTools);
     $interbreadcrumb[] = array('url' => api_get_path(WEB_CODE_PATH).'forum/viewforumcategory.php?'.$cidreq.'&forumcategory='.$current_forum_category['cat_id'], 'name' => $current_forum_category['cat_title']);