chamilo / chamilo-lms

main/admin/subscribe_user2course.php

Braces +8 added lines, -2 removed lines

@@ -317,7 +317,10 @@ 
     <td width="40%" align="center">
      <select name="UserList[]" multiple="multiple" size="20" style="width:300px;">
     <?php foreach ($db_users as $user) { ?>
-          <option value="<?php echo $user['user_id']; ?>" <?php if (in_array($user['user_id'], $users)) echo 'selected="selected"'; ?>>
+          <option value="<?php echo $user['user_id']; ?>" <?php if (in_array($user['user_id'], $users)) {
+    echo 'selected="selected"';
+}
+?>>
       <?php
         $userName = $user['lastname'].' '.$user['firstname'].' ('.$user['username'].')';
         if ($showOfficialCode) {
@@ -338,7 +341,10 @@ 
    <td width="40%" align="center">
     <select name="CourseList[]" multiple="multiple" size="20" style="width:300px;">
     <?php foreach ($db_courses as $course) { ?>
-         <option value="<?php echo $course['code']; ?>" <?php if (in_array($course['code'], $courses)) echo 'selected="selected"'; ?>>
+         <option value="<?php echo $course['code']; ?>" <?php if (in_array($course['code'], $courses)) {
+    echo 'selected="selected"';
+}
+?>>
              <?php echo '('.$course['visual_code'].') '.$course['title']; ?>
          </option>
     <?php } ?>

Indentation +4 added lines, -4 removed lines

@@ -299,7 +299,7 @@ 
       <option value="">--</option>
       <?php
         echo Display :: get_alphabet_options($first_letter_user);
-      ?>
+        ?>
      </select>
     </td>
     <td width="20%">&nbsp;</td>
@@ -312,8 +312,8 @@ 
         aria-label="<?php echo get_lang('FirstLetterCourse'); ?>">
       <option value="">--</option>
       <?php
-      echo Display :: get_alphabet_options($first_letter_course);
-      ?>
+        echo Display :: get_alphabet_options($first_letter_course);
+        ?>
      </select>
     </td>
    </tr>
@@ -329,7 +329,7 @@ 
             $userName = $officialCode.$userName;
         }
         echo $userName;
-      ?>
+        ?>
           </option>
     <?php } ?>
     </select>

main/social/profile.php

Indentation +1 added lines, -1 removed lines

@@ -595,7 +595,7 @@
                                 .' width="40px">';
                     $userInfo = api_get_user_info($user_invitation_id);
                     $invitations .= '<a href="'.api_get_path(WEB_PATH).'main/social/profile.php?u='.$user_invitation_id.'">'
-                                 .api_get_person_name($userInfo['firstname'], $userInfo['lastname']).'</a>';
+                                    .api_get_person_name($userInfo['firstname'], $userInfo['lastname']).'</a>';
 
                     $invitations .= '<div class="pull-right">';
                     $invitations .= Display::toolbarButton(

Braces +2 added lines, -1 removed lines

@@ -538,7 +538,8 @@
         $i = 1;
 
         foreach ($list as $key => $value) {
-            if (empty($value[2])) { //if out of any session
+            if (empty($value[2])) {
+//if out of any session
                 $my_courses .= $value[1];
                 $i++;
             }

plugin/sepe/src/sepe.lib.php

Indentation +1 added lines, -1 removed lines

@@ -192,7 +192,7 @@
 function listTutorType($condition)

 {

     global $tableTutorCompany;

-       $sql = "SELECT * FROM $tableTutorCompany WHERE ".$condition." ORDER BY alias ASC, document_number ASC;";

+        $sql = "SELECT * FROM $tableTutorCompany WHERE ".$condition." ORDER BY alias ASC, document_number ASC;";

     $res = Database::query($sql);

     $aux = array();

     while ($row = Database::fetch_assoc($res)) {

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -360,7 +360,7 @@ 
 {

     global $tableSepeParticipants;

     global $tableUser;

-    $sql = "SELECT $tableSepeParticipants.id AS id, document_type, document_number, document_letter, firstname, lastname

+    $sql = "select $tableSepeParticipants.id AS id, document_type, document_number, document_letter, firstname, lastname

             FROM $tableSepeParticipants LEFT JOIN $tableUser ON $tableSepeParticipants.platform_user_id=$tableUser.user_id

             WHERE action_id = $actionId";

     $res = Database::query($sql);

@@ -442,7 +442,7 @@ 
 {

     global $tableSepeActions;

     global $tableSepeCourseActions;

-    $sql = "SELECT $tableSepeCourseActions.*, course.title AS title, $tableSepeActions.action_origin AS action_origin, $tableSepeActions.action_code AS action_code 

+    $sql = "select $tableSepeCourseActions.*, course.title AS title, $tableSepeActions.action_origin AS action_origin, $tableSepeActions.action_code AS action_code 

             FROM $tableSepeCourseActions, course, $tableSepeActions 

             WHERE $tableSepeCourseActions.course_id=course.id 

             AND $tableSepeActions.id=$tableSepeCourseActions.action_id";

Braces +105 added lines, -105 removed lines

@@ -25,12 +25,12 @@ 
 $tableSepeLogParticipant = Database::get_main_table(SepePlugin::TABLE_SEPE_LOG_PARTICIPANT);

 $tableSepeLogChangeParticipant = Database::get_main_table(SepePlugin::TABLE_SEPE_LOG_MOD_PARTICIPANT);

 

-function getInfoIdentificationData()

-{

+function getInfoIdentificationData()
+{
     global $tableSepeCenter;

     $sql = "SELECT * FROM $tableSepeCenter;";

     $res = Database::query($sql);

-    if (Database::num_rows($res) > 0) {

+    if (Database::num_rows($res) > 0) {
         $row = Database::fetch_assoc($res);

         $row['center_origin'] = Security::remove_XSS(stripslashes($row['center_origin']));

         $row['center_code'] = Security::remove_XSS(stripslashes($row['center_code']));

@@ -39,26 +39,26 @@ 
         $row['tracking_url'] = Security::remove_XSS(stripslashes($row['tracking_url']));

         $row['phone'] = Security::remove_XSS(stripslashes($row['phone']));

         $row['mail'] = Security::remove_XSS(stripslashes($row['mail']));

-    } else {

+    } else {
         $row = false;

     }

     return $row;

 }

 

-function checkIdentificationData()

-{

+function checkIdentificationData()
+{
     global $tableSepeCenter;

     $sql = "SELECT 1 FROM $tableSepeCenter;";

     $result = Database::query($sql);

-    if (Database::affected_rows($result) > 0) {

+    if (Database::affected_rows($result) > 0) {
         return true;

-    } else {

+    } else {
         return false;

     }

 }

 

-function getActionId($courseId)

-{

+function getActionId($courseId)
+{
     global $tableSepeCourseActions;

     $sql = "SELECT action_id FROM $tableSepeCourseActions WHERE course_id = $courseId";

     $rs = Database::query($sql);

@@ -66,16 +66,16 @@ 
     return $aux['action_id'];

 }

 

-function getCourse($actionId)

-{

+function getCourse($actionId)
+{
     global $tableSepeCourseActions;

     $sql = "SELECT course_id FROM $tableSepeCourseActions WHERE action_id = $actionId";

     $rs = Database::query($sql);

     $aux = Database::fetch_assoc($rs);

     return $aux['course_id'];

 }

-function getCourseCode($actionId)

-{

+function getCourseCode($actionId)
+{
     global $tableCourse;

     $courseId = getCourse($actionId);

     $sql = "SELECT code FROM $tableCourse WHERE id = $courseId";    

@@ -84,13 +84,13 @@ 
     return $aux['code'];

 }

 

-function getActionInfo($id)

-{

+function getActionInfo($id)
+{
     global $tableSepeActions;

     $sql = "SELECT * FROM $tableSepeActions WHERE id = $id";

     $res = Database::query($sql);

     $row = false;

-    if (Database::num_rows($res) > 0) {

+    if (Database::num_rows($res) > 0) {
         $row['action_origin'] = Security::remove_XSS(stripslashes($row['action_origin']));

         $row['action_code'] = Security::remove_XSS(stripslashes($row['action_code']));

         $row['situation'] = Security::remove_XSS(stripslashes($row['situation']));

@@ -109,13 +109,13 @@ 
     return $row;

 }

 

-function getSpecialtActionInfo($specialtyId)

-{

+function getSpecialtActionInfo($specialtyId)
+{
     global $tableSepeSpecialty;

     $sql = "SELECT * FROM $tableSepeSpecialty WHERE id = $specialtyId";

     $res = Database::query($sql);

     $row = false;

-    if (Database::num_rows($res) > 0) {

+    if (Database::num_rows($res) > 0) {
         $row['specialty_origin'] = Security::remove_XSS(stripslashes($row['specialty_origin']));

         $row['professional_area'] = Security::remove_XSS(stripslashes($row['professional_area']));

         $row['specialty_code'] = Security::remove_XSS(stripslashes($row['specialty_code']));

@@ -127,8 +127,8 @@ 
     return $row;

 }

 

-function getInfoSpecialtyClassroom($classroomId)

-{

+function getInfoSpecialtyClassroom($classroomId)
+{
     global $tableSepeSpecialtyClassroom;

     global $tableCenters;

     $sql = "SELECT a.*, center_origin, center_code 

@@ -136,7 +136,7 @@ 
             WHERE a.id = $classroomId";

     $res = Database::query($sql);

     $row = false;

-    if (Database::num_rows($res) > 0) {

+    if (Database::num_rows($res) > 0) {
         $row['center_origin'] = Security::remove_XSS(stripslashes($row['center_origin']));

         $row['center_code'] = Security::remove_XSS(stripslashes($row['center_code']));

         $row = Database::fetch_assoc($res);

@@ -144,57 +144,57 @@ 
     return $row;

 }

 

-function getInfoSpecialtyTutorial($tutorialId)

-{

+function getInfoSpecialtyTutorial($tutorialId)
+{
     global $tableSepeParticipantsSpecialtyTutorials;

     $sql = "SELECT * FROM $tableSepeParticipantsSpecialtyTutorials WHERE id = $tutorialId";

     $res = Database::query($sql);

     $aux = array();

-    if (Database::num_rows($res) > 0) {

+    if (Database::num_rows($res) > 0) {
         $row = Database::fetch_assoc($res);

-    } else {

+    } else {
         $row = false;

     }

     return $row;

 }

 

-function list_tutor($specialtyId)

-{

+function list_tutor($specialtyId)
+{
     global $tableSepeSpecialtyTutors;

     $sql = "SELECT * FROM $tableSepeSpecialtyTutors WHERE specialty_id = $specialtyId";

     $res = Database::query($sql);

-    if (Database::num_rows($res) > 0) {

+    if (Database::num_rows($res) > 0) {
         $row = Database::fetch_assoc($res);

-    } else {

+    } else {
         $row = false;

     }

     return $row;

 }

 

-function getCentersList() 

-{

+function getCentersList()
+{
     global $tableCenters;

     $sql = "SELECT * FROM $tableCenters;";

     $res = Database::query($sql);

     $aux = array();

-    while ($row = Database::fetch_assoc($res)) {

+    while ($row = Database::fetch_assoc($res)) {
         $aux[] = $row;

     }

     return $aux;

 }

 

-function listTutorType($condition)

-{

+function listTutorType($condition)
+{
     global $tableTutorCompany;

        $sql = "SELECT * FROM $tableTutorCompany WHERE ".$condition." ORDER BY alias ASC, document_number ASC;";

     $res = Database::query($sql);

     $aux = array();

-    while ($row = Database::fetch_assoc($res)) {

+    while ($row = Database::fetch_assoc($res)) {
         $tmp = array();

         $tmp['id'] = $row['id'];

-        if (trim($row['alias']) != '') {

+        if (trim($row['alias']) != '') {
             $tmp['alias'] = $row['alias'].' - '.$row['document_type'].' '.$row['document_number'].' '.$row['document_letter'];    

-        } else {

+        } else {
             $tmp['alias'] = $row['document_type'].' '.$row['document_number'].' '.$row['document_letter'];    

         }

         $aux[] = $tmp;

@@ -202,28 +202,28 @@ 
     return $aux;

 }

 

-function getTutorsSpecialty($specialtyId)

-{

+function getTutorsSpecialty($specialtyId)
+{
     global $tableSepeSpecialtyTutors;

     global $tableSepeTutors;

     global $tableUser;

     $sql = "SELECT tutor_id FROM $tableSepeSpecialtyTutors WHERE specialty_id = $specialtyId";

     $rs = Database::query($sql);

     $tutorsList = array();

-    while ($tmp = Database::fetch_assoc($rs)) {

+    while ($tmp = Database::fetch_assoc($rs)) {
         $tutorsList[] = $tmp['tutor_id'];

     }

     $sql = "SELECT a.*, b.firstname AS firstname, b.lastname AS lastname 

             FROM $tableSepeTutors AS a LEFT JOIN $tableUser AS b ON a.platform_user_id=b.user_id;";

     $res = Database::query($sql);

     $aux = array();

-    while ($row = Database::fetch_assoc($res)) {

-        if (!in_array($row['id'], $tutorsList)) {

+    while ($row = Database::fetch_assoc($res)) {
+        if (!in_array($row['id'], $tutorsList)) {
             $tutor = array();

             $tutor['id'] = $row['id'];

-            if (trim($row['firstname']) != '' || trim($row['lastname']) != '') {

+            if (trim($row['firstname']) != '' || trim($row['lastname']) != '') {
                 $tutor['data'] = $row['firstname'].' '.$row['lastname'].' ('.$row['document_type'].' '.$row['document_number'].' '.$row['document_letter'].' )';    

-            } else {

+            } else {
                 $tutor['data'] = $row['document_type'].' '.$row['document_number'].' '.$row['document_letter'];    

             }

             $aux[] = $tutor;

@@ -232,8 +232,8 @@ 
     return $aux;

 }

 

-function getInfoSpecialtyTutor($tutorId)

-{

+function getInfoSpecialtyTutor($tutorId)
+{
     global $tableSepeSpecialtyTutors;

     global $tableSepeTutors;

     $sql = "SELECT a.*,platform_user_id,document_type, document_number,document_letter 

@@ -242,32 +242,32 @@ 
             WHERE a.id = $tutorId;";

     $res = Database::query($sql);

     $aux = array();

-    if (Database::num_rows($res) > 0) {

+    if (Database::num_rows($res) > 0) {
         $row['tutor_accreditation'] = Security::remove_XSS(stripslashes($row['tutor_accreditation']));

         $row['teaching_competence'] = Security::remove_XSS(stripslashes($row['teaching_competence']));

         $row['training_teleforming'] = Security::remove_XSS(stripslashes($row['training_teleforming']));

         $row = Database::fetch_assoc($res);

-    } else {

+    } else {
         $row = false;

     }

     return $row;

 }

 

-function freeTeacherList($teacherList, $specialtyId, $platform_user_id)

-{

+function freeTeacherList($teacherList, $specialtyId, $platform_user_id)
+{
     global $tableSepeSpecialtyTutors;

     global $tableSepeTutors;

     $sql = "SELECT tutor_id FROM $tableSepeSpecialtyTutors WHERE specialty_id = $specialtyId";

     $rs = Database::query($sql);

-    if (Database::num_rows($rs) > 0) {

-        while ($aux = Database::fetch_assoc($rs)) {

+    if (Database::num_rows($rs) > 0) {
+        while ($aux = Database::fetch_assoc($rs)) {
             $sql = "SELECT platform_user_id FROM $tableSepeTutors WHERE id='".$aux['tutor_id']."';";

             $res = Database::query($sql);

-            if (Database::num_rows($res) > 0) {

+            if (Database::num_rows($res) > 0) {
                 $tmp = Database::fetch_assoc($res);

-                if ($tmp['platform_user_id'] != 0 && $tmp['platform_user_id'] != $platform_user_id) {

-                    foreach ($teacherList as $key => $value) {

-                        if ($value['id'] == $tmp['platform_user_id']) {

+                if ($tmp['platform_user_id'] != 0 && $tmp['platform_user_id'] != $platform_user_id) {
+                    foreach ($teacherList as $key => $value) {
+                        if ($value['id'] == $tmp['platform_user_id']) {
                             unset($teacherList[$key]);

                             break;

                         }

@@ -279,13 +279,13 @@ 
     return $teacherList;

 }

 

-function getInfoParticipantAction($participantId)

-{

+function getInfoParticipantAction($participantId)
+{
     global $tableSepeParticipants;

     $sql = "SELECT * FROM $tableSepeParticipants WHERE id = $participantId";

     $res = Database::query($sql);

     $aux = array();

-    if (Database::num_rows($res) > 0) {

+    if (Database::num_rows($res) > 0) {
         $row = Database::fetch_assoc($res);

         $result = [];

         $result['id'] = $row[''];

@@ -299,14 +299,14 @@ 
         $result['key_competence'] = Security::remove_XSS(stripslashes($row['key_competence']));

         $result['contract_id'] = Security::remove_XSS(stripslashes($row['contract_id']));

         $result['company_fiscal_number'] = Security::remove_XSS(stripslashes($row['company_fiscal_number']));

-    } else {

+    } else {
         $result = false;

     }

     return $result;

 }

 

-function getParticipantId($id)

-{

+function getParticipantId($id)
+{
     global $tableSepeParticipantsSpecialty;

     $sql = "SELECT participant_id FROM  $tableSepeParticipantsSpecialty WHERE id = $id";

     $rs = Database::query($sql);

@@ -314,13 +314,13 @@ 
     return $aux['participant_id'];

 }

 

-function getInfoSpecialtyParticipant($specialtyId)

-{

+function getInfoSpecialtyParticipant($specialtyId)
+{
     global $tableSepeParticipantsSpecialty;

     $sql = "SELECT * FROM $tableSepeParticipantsSpecialty WHERE id = $specialtyId";

     $res = Database::query($sql);

     $aux = array();

-    if (Database::num_rows($res) > 0) {

+    if (Database::num_rows($res) > 0) {
         $row = Database::fetch_assoc($res);

         $row['specialty_origin'] = Security::remove_XSS(stripslashes($row['specialty_origin']));

         $row['professional_area'] = Security::remove_XSS(stripslashes($row['professional_area']));

@@ -330,28 +330,28 @@ 
         $row['final_result'] = Security::remove_XSS(stripslashes($row['final_result']));

         $row['final_qualification'] = Security::remove_XSS(stripslashes($row['final_qualification']));

         $row['final_score'] = Security::remove_XSS(stripslashes($row['final_score']));

-    } else {

+    } else {
         $row = false;

     }

     return $row;

 }

 

-function specialtyList($actionId)

-{

+function specialtyList($actionId)
+{
     global $tableSepeSpecialty;

     $sql = "SELECT id, specialty_origin, professional_area, specialty_code

             FROM $tableSepeSpecialty

             WHERE action_id = $actionId";

     $res = Database::query($sql);

     $aux = array();

-    while ($row = Database::fetch_assoc($res)) {

+    while ($row = Database::fetch_assoc($res)) {
         $aux[] = $row;

     }

     return $aux;

 }

 

-function participantList($actionId)

-{

+function participantList($actionId)
+{
     global $tableSepeParticipants;

     global $tableUser;

     $sql = "SELECT $tableSepeParticipants.id AS id, document_type, document_number, document_letter, firstname, lastname

@@ -359,19 +359,19 @@ 
             WHERE action_id = $actionId";

     $res = Database::query($sql);

     $aux = array();

-    while ($row = Database::fetch_assoc($res)) {

+    while ($row = Database::fetch_assoc($res)) {
         $aux[] = $row;

     }

     return $aux;

 }

 

-function listParticipantSpecialty($participantId)

-{

+function listParticipantSpecialty($participantId)
+{
     global $tableSepeParticipantsSpecialty;

     $sql = "SELECT * FROM $tableSepeParticipantsSpecialty WHERE participant_id = $participantId";

     $res = Database::query($sql);

     $aux = array();

-    while ($row = Database::fetch_assoc($res)) {

+    while ($row = Database::fetch_assoc($res)) {
         $row['specialty_origin'] = Security::remove_XSS(stripslashes($row['specialty_origin']));

         $row['professional_area'] = Security::remove_XSS(stripslashes($row['professional_area']));

         $row['specialty_code'] = Security::remove_XSS(stripslashes($row['specialty_code']));

@@ -385,8 +385,8 @@ 
     return $aux;

 }

 

-function classroomList($specialtyId)

-{

+function classroomList($specialtyId)
+{
     global $tableSepeSpecialtyClassroom;

     global $tableCenters;

     $sql = "SELECT a.*, center_origin, center_code

@@ -394,14 +394,14 @@ 
             WHERE specialty_id = $specialtyId";

     $res = Database::query($sql);

     $aux = array();

-    while ($row = Database::fetch_assoc($res)) {

+    while ($row = Database::fetch_assoc($res)) {
         $aux[] = $row;

     }

     return $aux;

 }

 

-function tutorsList($specialtyId)

-{

+function tutorsList($specialtyId)
+{
     global $tableSepeSpecialtyTutors;

     global $tableSepeTutors;

     global $tableUser;

@@ -411,19 +411,19 @@ 
             LEFT JOIN $tableUser c ON b.platform_user_id=c.user_id 

             WHERE a.specialty_id = $specialtyId";

     $res = Database::query($sql);

-    while ($row = Database::fetch_assoc($res)) {

+    while ($row = Database::fetch_assoc($res)) {
         $aux[] = $row;

     }

     return $aux;

 }

 

-function getListSpecialtyTutorial($specialtyId)

-{

+function getListSpecialtyTutorial($specialtyId)
+{
     global $tableSepeParticipantsSpecialtyTutorials;

     $sql = "SELECT * FROM $tableSepeParticipantsSpecialtyTutorials WHERE participant_specialty_id = $specialtyId";

     $res = Database::query($sql);

     $aux = array();

-    while ($row = Database::fetch_assoc($res)) {

+    while ($row = Database::fetch_assoc($res)) {
         $row['tutor_accreditation'] = Security::remove_XSS(stripslashes($row['tutor_accreditation']));

         $row['teaching_competence'] = Security::remove_XSS(stripslashes($row['teaching_competence']));

         $row['training_teleforming'] = Security::remove_XSS(stripslashes($row['training_teleforming']));

@@ -432,8 +432,8 @@ 
     return $aux;

 }

 

-function listCourseAction()

-{

+function listCourseAction()
+{
     global $tableSepeActions;

     global $tableSepeCourseActions;

     $sql = "SELECT $tableSepeCourseActions.*, course.title AS title, $tableSepeActions.action_origin AS action_origin, $tableSepeActions.action_code AS action_code 

@@ -442,14 +442,14 @@ 
             AND $tableSepeActions.id=$tableSepeCourseActions.action_id";

     $res = Database::query($sql);

     $aux = array();

-    while ($row = Database::fetch_assoc($res)) {

+    while ($row = Database::fetch_assoc($res)) {
         $aux[] = $row;

     }

     return $aux;

 }

 

-function listCourseFree()

-{

+function listCourseFree()
+{
     global $tableCourse;

     global $tableSepeCourseActions;

     $sql = "SELECT id, title FROM $tableCourse

@@ -457,14 +457,14 @@ 
                 SELECT * FROM $tableSepeCourseActions WHERE $tableCourse.id = $tableSepeCourseActions.course_id)

             ;";

     $res = Database::query($sql);

-    while ($row = Database::fetch_assoc($res)) {

+    while ($row = Database::fetch_assoc($res)) {
         $aux[] = $row;

     }

     return $aux;

 }

 

-function listActionFree()

-{

+function listActionFree()
+{
     global $tableSepeActions;

     global $tableSepeCourseActions;

     $sql = "SELECT id, action_origin, action_code FROM $tableSepeActions

@@ -473,7 +473,7 @@ 
             ;";

     $res = Database::query($sql);

     $aux = array();

-    while ($row = Database::fetch_assoc($res)) {

+    while ($row = Database::fetch_assoc($res)) {
         $row['action_origin'] = Security::remove_XSS(stripslashes($row['action_origin']));

         $row['action_code'] = Security::remove_XSS(stripslashes($row['action_code']));

         $aux[] = $row;

@@ -481,8 +481,8 @@ 
     return $aux;

 }

 

-function getSpecialtyTutorId($specialtyId, $tutorId)

-{

+function getSpecialtyTutorId($specialtyId, $tutorId)
+{
     global $tableSepeSpecialtyTutors;

     $sql = "SELECT id 

             FROM $tableSepeSpecialtyTutors 

@@ -492,27 +492,27 @@ 
     return $row['id'];

 }

 

-function checkInsertNewLog($platformUserId, $actionId)

-{

+function checkInsertNewLog($platformUserId, $actionId)
+{
     global $tableSepeLogParticipant;

     $sql = "SELECT * FROM $tableSepeLogParticipant WHERE platform_user_id = $platformUserId AND action_id = $actionId";

     $res = Database::query($sql);

-    if (Database::num_rows($res) > 0) {

+    if (Database::num_rows($res) > 0) {
         return false;

-    } else {

+    } else {
         return true;

     }

 }

 

-function getUserPlatformFromParticipant($participantId)

-{

+function getUserPlatformFromParticipant($participantId)
+{
     global $tableSepeParticipants;

     $sql = "SELECT * FROM $tableSepeParticipants WHERE id = $participantId";

     $res = Database::query($sql);

     $row = Database::fetch_assoc($res);

-    if ($row['platform_user_id'] == 0 || $row['platform_user_id'] == '') {

+    if ($row['platform_user_id'] == 0 || $row['platform_user_id'] == '') {
         return false;

-    } else {

+    } else {
         return $row['platform_user_id'];

     }

 }

plugin/sepe/src/index.sepe.php

Indentation +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 if (api_is_platform_admin() && $enable) {

     echo '<div class="panel panel-default">';

         echo '<div class="panel-heading" role="tab">';

-           echo '<h4 class="panel-title">'.$title.'</h4>';

+            echo '<h4 class="panel-title">'.$title.'</h4>';

         echo '</div>';

         echo '<div class="panel-collapse collapse in" role="tabpanel">';

             echo '<div class="panel-body">';

Braces +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 $enable = $plugin->get('sepe_enable') == 'true';

 $title = $plugin->get_lang('AdministratorSepe');

 $pluginPath = api_get_path(WEB_PLUGIN_PATH).'sepe/src/';

-if (api_is_platform_admin() && $enable) {

+if (api_is_platform_admin() && $enable) {
     echo '<div class="panel panel-default">';

         echo '<div class="panel-heading" role="tab">';

            echo '<h4 class="panel-title">'.$title.'</h4>';

plugin/sepe/src/identification-data-edit.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
         $id = intval($_POST['id']);

         

         if (checkIdentificationData()) {

-            $sql = "UPDATE $tableSepeCenter SET 

+            $sql = "update $tableSepeCenter SET 

                         center_origin = '".$centerOrigin."', 

                         center_code = '".$centerCode."', 

                         center_name = '".$centerName."', 

Braces +12 added lines, -12 removed lines

@@ -9,9 +9,9 @@ 
 require_once '../config.php';

 $plugin = SepePlugin::create();

 

-if (!empty($_POST)) {

+if (!empty($_POST)) {
     $check = Security::check_token('post');

-    if ($check) {

+    if ($check) {
         $centerOrigin = Database::escape_string(trim($_POST['center_origin']));

         $centerCode = Database::escape_string(trim($_POST['center_code']));

         $centerName = Database::escape_string(trim($_POST['center_name']));

@@ -21,7 +21,7 @@ 
         $mail = Database::escape_string(trim($_POST['mail']));

         $id = intval($_POST['id']);

         

-        if (checkIdentificationData()) {

+        if (checkIdentificationData()) {
             $sql = "UPDATE $tableSepeCenter SET 

                         center_origin = '".$centerOrigin."', 

                         center_code = '".$centerCode."', 

@@ -31,7 +31,7 @@ 
                         phone = '".$phone."', 

                         mail = '".$mail."' 

                     WHERE id = $id";    

-        } else {

+        } else {
             $sql = "INSERT INTO $tableSepeCenter (

                         id, 

                         center_origin, 

@@ -53,33 +53,33 @@ 
                     );";

         }

         $res = Database::query($sql);

-        if (!$res) {

+        if (!$res) {
             $_SESSION['sepe_message_error'] = $plugin->get_lang('NoSaveChange');

-        } else {

+        } else {
             $_SESSION['sepe_message_info'] = $plugin->get_lang('SaveChange');

         }

         header("Location: identification-data.php");

-    } else {

+    } else {
         $_SESSION['sepe_message_error'] = $plugin->get_lang('ProblemToken');

         Security::clear_token();

         $token = Security::get_token();

     }

-} else {

+} else {
     $token = Security::get_token();

 }

 

-if (api_is_platform_admin()) {

+if (api_is_platform_admin()) {
     $interbreadcrumb[] = array("url" => "/plugin/sepe/src/sepe-administration-menu.php", "name" => $plugin->get_lang('MenuSepe'));

     $interbreadcrumb[] = array("url" => "identification-data.php", "name" => $plugin->get_lang('DataCenter'));

     $templateName = $plugin->get_lang('DataCenterEdit');

     $tpl = new Template($templateName);

     $info = getInfoIdentificationData();

     $tpl->assign('info', $info);

-    if (isset($_SESSION['sepe_message_info'])) {

+    if (isset($_SESSION['sepe_message_info'])) {
         $tpl->assign('message_info', $_SESSION['sepe_message_info']);

         unset($_SESSION['sepe_message_info']);

     }

-    if (isset($_SESSION['sepe_message_error'])) {

+    if (isset($_SESSION['sepe_message_error'])) {
         $tpl->assign('message_error', $_SESSION['sepe_message_error']);

         unset($_SESSION['sepe_message_error']);

     }

@@ -88,6 +88,6 @@ 
     $content = $tpl->fetch($listing_tpl);

     $tpl->assign('content', $content);

     $tpl->display_one_col_template();

-} else {

+} else {
     header('Location:'.api_get_path(WEB_PATH));

 }

plugin/sepe/admin.php

Braces +2 added lines, -2 removed lines

@@ -5,8 +5,8 @@
 $enable = $plugin->get('sepe_enable') == 'true';

 $pluginPath = api_get_path(WEB_PLUGIN_PATH).'sepe/src/sepe-administration-menu.php';

 

-if ($enable && api_is_platform_admin()) {

+if ($enable && api_is_platform_admin()) {
     header('Location:'.$pluginPath);

-} else {

+} else {
     header('Location: ../../index.php');

 }
\ No newline at end of file

main/exercise/admin.php

Braces +3 added lines, -2 removed lines

@@ -359,9 +359,10 @@
 
 if ($inATest) {
     echo '<div class="actions">';
-    if (isset($_GET['hotspotadmin']) || isset($_GET['newQuestion']) || isset($_GET['myid']))
-        echo '<a href="'.api_get_path(WEB_CODE_PATH).'exercise/admin.php?exerciseId='.$exerciseId.'&'.api_get_cidreq().'">'.
+    if (isset($_GET['hotspotadmin']) || isset($_GET['newQuestion']) || isset($_GET['myid'])) {
+            echo '<a href="'.api_get_path(WEB_CODE_PATH).'exercise/admin.php?exerciseId='.$exerciseId.'&'.api_get_cidreq().'">'.
             Display::return_icon('back.png', get_lang('GoBackToQuestionList'), '', ICON_SIZE_MEDIUM).'</a>';
+    }
 
     if (!isset($_GET['hotspotadmin']) && !isset($_GET['newQuestion']) && !isset($_GET['myid']) && !isset($_GET['editQuestion'])) {
         echo '<a href="'.api_get_path(WEB_CODE_PATH).'exercise/exercise.php?'.api_get_cidreq().'">'.

main/inc/lib/specific_fields_manager.lib.php

Doc Comments +3 added lines, -1 removed lines

@@ -100,7 +100,7 @@ 
 
 /**
  * @param array $conditions a list of condition (exemple : status=>STUDENT)
- * @param array $order_by a list of fields on which sort
+ * @param string[] $order_by a list of fields on which sort
  * @return array An array with all users of the platform.
  */
 function get_specific_field_values_list(
@@ -235,6 +235,7 @@ 
  * @param   string  Course code
  * @param   string  Tool ID
  * @param   int     Internal ID used in specific tool table
+ * @param string $tool_id
  */
 function delete_all_values_for_item($course_id, $tool_id, $ref_id) {
   $table_sf_values = Database::get_main_table(TABLE_MAIN_SPECIFIC_FIELD_VALUES);
@@ -248,6 +249,7 @@ 
  * Defaults to the first letter of the name, otherwise iterate through available
  * letters
  * @param   string  Name
+ * @param string $name
  * @return  string  One-letter code, upper-case
  */
 function get_specific_field_code_from_name($name) {

Indentation +4 added lines, -4 removed lines

@@ -237,10 +237,10 @@
  * @param   int     Internal ID used in specific tool table
  */
 function delete_all_values_for_item($course_id, $tool_id, $ref_id) {
-  $table_sf_values = Database::get_main_table(TABLE_MAIN_SPECIFIC_FIELD_VALUES);
-  $sql = 'DELETE FROM %s WHERE course_code = \'%s\' AND tool_id = \'%s\' AND ref_id = %s';
-  $sql = sprintf($sql, $table_sf_values, $course_id, $tool_id, $ref_id);
-  Database::query($sql);
+    $table_sf_values = Database::get_main_table(TABLE_MAIN_SPECIFIC_FIELD_VALUES);
+    $sql = 'DELETE FROM %s WHERE course_code = \'%s\' AND tool_id = \'%s\' AND ref_id = %s';
+    $sql = sprintf($sql, $table_sf_values, $course_id, $tool_id, $ref_id);
+    Database::query($sql);
 }
 
 /**

Braces +12 added lines, -6 removed lines

@@ -106,7 +106,8 @@ 
 function get_specific_field_values_list(
     $conditions = array(),
     $order_by = array()
-) {
+)
+{
     $table_sfv = Database::get_main_table(TABLE_MAIN_SPECIFIC_FIELD_VALUES);
     $return_array = array();
     $sql = "SELECT * FROM $table_sfv";
@@ -153,7 +154,8 @@ 
     $course_code,
     $tool_id,
     $ref_id
-) {
+)
+{
     $table_sf = Database::get_main_table(TABLE_MAIN_SPECIFIC_FIELD);
     $table_sfv = Database::get_main_table(TABLE_MAIN_SPECIFIC_FIELD_VALUES);
     $sql = 'SELECT sfv.value FROM %s sf LEFT JOIN %s sfv ON sf.id = sfv.field_id'.
@@ -189,7 +191,8 @@ 
     $tool_id,
     $ref_id,
     $value
-) {
+)
+{
     $table_sf_values = Database::get_main_table(
         TABLE_MAIN_SPECIFIC_FIELD_VALUES
     );
@@ -222,7 +225,8 @@ 
  * @param string $tool_id tool id, from main.api.lib
  * @param int $ref_id intern id inside specific tool table
  */
-function delete_all_specific_field_value($course_id, $id_specific_field, $tool_id, $ref_id) {
+function delete_all_specific_field_value($course_id, $id_specific_field, $tool_id, $ref_id)
+{
     $table_sf_values = Database::get_main_table(TABLE_MAIN_SPECIFIC_FIELD_VALUES);
     $sql = 'DELETE FROM %s WHERE course_code = \'%s\' AND tool_id = \'%s\' AND ref_id = %s AND field_id = %s';
     $sql = sprintf($sql, $table_sf_values, $course_id, $tool_id, $ref_id, $id_specific_field);
@@ -236,7 +240,8 @@ 
  * @param   string  Tool ID
  * @param   int     Internal ID used in specific tool table
  */
-function delete_all_values_for_item($course_id, $tool_id, $ref_id) {
+function delete_all_values_for_item($course_id, $tool_id, $ref_id)
+{
   $table_sf_values = Database::get_main_table(TABLE_MAIN_SPECIFIC_FIELD_VALUES);
   $sql = 'DELETE FROM %s WHERE course_code = \'%s\' AND tool_id = \'%s\' AND ref_id = %s';
   $sql = sprintf($sql, $table_sf_values, $course_id, $tool_id, $ref_id);
@@ -250,7 +255,8 @@ 
  * @param   string  Name
  * @return  string  One-letter code, upper-case
  */
-function get_specific_field_code_from_name($name) {
+function get_specific_field_code_from_name($name)
+{
     // Z is used internally by Xapian
     // O & C already used by tool_id and course_id
     $list = array(

plugin/vchamilo/views/editinstance_form.php

Doc Comments +5 added lines, -5 removed lines

@@ -13,9 +13,9 @@ 
 
     /**
      * ChamiloForm constructor.
-     * @param $mode
-     * @param $returnurl
-     * @param $cancelurl
+     * @param string $mode
+     * @param string $returnurl
+     * @param string $cancelurl
      * @param $customdata
      */
     public function __construct($mode, $returnurl, $cancelurl, $customdata = [])
@@ -137,7 +137,7 @@ 
      * Check that form data is valid.
      * You should almost always use this, rather than {@see validate_defined_fields}
      *
-     * @return bool true if form data valid
+     * @return boolean|null true if form data valid
      */
     public function is_validated()
     {
@@ -166,7 +166,7 @@ 
      *                is NOT to validate the form when a no submit button has been pressed.
      *                pass true here to override this behaviour
      *
-     * @return bool true if form data valid
+     * @return boolean|null true if form data valid
      */
     public function validate_defined_fields($validateonnosubmit = false)
     {

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -248,7 +248,7 @@
         if (is_object($default_values)) {
             $default_values = (array) $default_values;
         }
-        $filter = $slashed ? 'stripslashes' : NULL;
+        $filter = $slashed ? 'stripslashes' : null;
         $this->_form->setDefaults($default_values, $filter);
     }