chamilo / chamilo-lms

main/inc/lib/image.lib.php

Braces +17 added lines, -6 removed lines

@@ -48,7 +48,8 @@ 
                     0
                 );
             }
-        } else { // height > $width
+        } else {
+// height > $width
             if ($height >= $max_size_for_picture) {
                 // scale width
                 $new_width = round($width * ($max_size_for_picture / $height));
@@ -193,7 +194,9 @@ 
      */
     public function set_image_wrapper()
     {
-        if ($this->debug) error_log('Image::set_image_wrapper loaded');
+        if ($this->debug) {
+            error_log('Image::set_image_wrapper loaded');
+        }
         try {
             if (file_exists($this->path)) {
                 $this->image = new Imagick($this->path);
@@ -202,10 +205,14 @@ 
                     $this->fill_image_info(); //Fills height, width and type
                 }
             } else {
-                if ($this->debug) error_log('Image::image does not exist');
+                if ($this->debug) {
+                    error_log('Image::image does not exist');
+                }
             }
         } catch (ImagickException $e) {
-            if ($this->debug) error_log($e->getMessage());
+            if ($this->debug) {
+                error_log($e->getMessage());
+            }
         }
     }
 
@@ -218,7 +225,9 @@ 
 
         if (in_array($this->type, $this->allowed_extensions)) {
             $this->image_validated = true;
-            if ($this->debug) error_log('image_validated true');
+            if ($this->debug) {
+                error_log('image_validated true');
+            }
         }
     }
 
@@ -234,7 +243,9 @@ 
     //@todo implement border logic case for Imagick
     public function resize($thumbw, $thumbh, $border, $specific_size = false)
     {
-        if (!$this->image_validated) return false;
+        if (!$this->image_validated) {
+            return false;
+        }
 
         if ($specific_size) {
             $width = $thumbw;

main/upload/upload.document.php

Braces +3 added lines, -2 removed lines

@@ -64,9 +64,10 @@
         $new_comment = isset($_POST['comment']) ? Database::escape_string(trim($_POST['comment'])) : '';
         $new_title = isset($_POST['title']) ? Database::escape_string(trim($_POST['title'])) : '';
 
-        if ($new_path && ($new_comment || $new_title))
-        if (($docid = DocumentManager::get_document_id($_course, $new_path))) {
+        if ($new_path && ($new_comment || $new_title)) {
+                if (($docid = DocumentManager::get_document_id($_course, $new_path))) {
             $table_document = Database::get_course_table(TABLE_DOCUMENT);
+        }
             $ct = '';
             if ($new_comment) {
                 $ct .= ", comment='$new_comment'";

main/install/install.lib.php

Braces +36 added lines, -18 removed lines

@@ -70,7 +70,8 @@ 
     $returnFailure = 'No',
     $optional = false,
     $enabledTerm = ''
-) {
+)
+{
     if (extension_loaded($extensionName)) {
         if (!empty($enabledTerm)) {
             $isEnabled = ini_get($enabledTerm);
@@ -109,7 +110,8 @@ 
     $recommendedValue,
     $returnSuccess = false,
     $returnFailure = false
-) {
+)
+{
     $currentPhpValue = getPhpSetting($phpSetting);
     if ($currentPhpValue == $recommendedValue) {
         return Display::label($currentPhpValue.' '.$returnSuccess, 'success');
@@ -563,7 +565,8 @@ 
     $dbPassForm,
     $dbNameForm,
     $dbPortForm = 3306
-) {
+)
+{
     $dbParams = array(
         'driver' => 'pdo_mysql',
         'host' => $dbHostForm,
@@ -610,7 +613,8 @@ 
 function display_language_selection_box(
     $name = 'language_list',
     $default_language = 'english'
-) {
+)
+{
     // Reading language list.
     $language_list = get_language_folder_list();
 
@@ -712,7 +716,8 @@ 
     $badUpdatePath,
     $updatePath = '',
     $update_from_version_8 = array()
-) {
+)
+{
     global $_setting;
     echo '<div class="RequirementHeading"><h2>'.display_step_sequence().get_lang('Requirements')."</h2></div>";
     echo '<div class="RequirementText">';
@@ -1032,8 +1037,7 @@ 
                 <?php echo get_lang('Error'); ?>!<br />
                 Chamilo <?php echo implode('|', $update_from_version_8).' '.get_lang('HasNotBeenFoundInThatDir'); ?>.
             </div>
-        <?php }
-        else {
+        <?php } else {
             echo '<br />';
         }
         ?>
@@ -1153,7 +1157,10 @@ 
         <button type="submit" name="step1" class="btn btn-default" onclick="javascript: window.location='index.php'; return false;" value="<?php echo get_lang('Previous'); ?>" >
             <em class="fa fa-backward"> </em> <?php echo get_lang('Previous'); ?>
         </button>
-        <button type="submit" name="step2_install" class="btn btn-success" value="<?php echo get_lang("NewInstallation"); ?>" <?php if ($error) echo 'disabled="disabled"'; ?> >
+        <button type="submit" name="step2_install" class="btn btn-success" value="<?php echo get_lang("NewInstallation"); ?>" <?php if ($error) {
+    echo 'disabled="disabled"';
+}
+?> >
             <em class="fa fa-forward"> </em> <?php echo get_lang('NewInstallation'); ?>
         </button>
         <input type="hidden" name="is_executable" id="is_executable" value="-" />
@@ -1368,7 +1375,8 @@ 
     $extra_notice,
     $displayWhenUpdate = true,
     $tr_attribute = ''
-) {
+)
+{
     //echo "<tr ".$tr_attribute.">";
     echo "<label class='col-sm-4'>$parameterName</label>";
 
@@ -1409,7 +1417,8 @@ 
     $dbNameForm,
     $dbPortForm = 3306,
     $installationProfile = ''
-) {
+)
+{
     if ($installType == 'update') {
         global $_configuration;
         $dbHostForm = $_configuration['db_host'];
@@ -1532,9 +1541,12 @@ 
             Database port: <strong><?php echo $manager->getConnection()->getPort(); ?></strong><br/>
             Database driver: <strong><?php echo $manager->getConnection()->getDriver()->getName(); ?></strong><br/>
         </div>
-    <?php else: ?>
+    <?php else {
+    : ?>
         <div id="db_status" class="alert alert-danger">
-            <p><?php echo get_lang('FailedConectionDatabase'); ?></strong></p>
+            <p><?php echo get_lang('FailedConectionDatabase');
+}
+?></strong></p>
             <code><?php echo $database_exists_text ?></code>
         </div>
     <?php endif; ?>
@@ -1594,7 +1606,8 @@ 
     $formFieldName,
     $parameterValue,
     $displayWhenUpdate = 'true'
-) {
+)
+{
     $html = '<div class="form-group">';
     $html .= '<label class="col-sm-6 control-label">'.$parameterName.'</label>';
     if ($installType == INSTALL_TYPE_UPDATE && $displayWhenUpdate) {
@@ -1641,7 +1654,8 @@ 
     $allowSelfRegProf,
     $loginForm,
     $passForm
-) {
+)
+{
     if ($installType != 'update' && empty($languageForm)) {
         $languageForm = $_SESSION['install_language'];
     }
@@ -1678,7 +1692,8 @@ 
     if ($installType == 'update') {
         $html .= '<input type="hidden" name="languageForm" value="'.api_htmlentities($languageForm, ENT_QUOTES).'" />'.$languageForm;
 
-    } else { // new installation
+    } else {
+// new installation
         $html .= '<div class="col-sm-6">';
         $html .= display_language_selection_box('languageForm', $languageForm);
         $html .= '</div>';
@@ -1951,7 +1966,8 @@ 
     $course_dir,
     $course_attempt_name,
     $file = 'test.php'
-) {
+)
+{
     $output = false;
     //Write in file
     $file_name = $course_dir.'/'.$file;
@@ -2061,7 +2077,8 @@ 
     $allowRegistration,
     $allowTeacherSelfRegistration,
     $installationProfile = ''
-) {
+)
+{
     $allowTeacherSelfRegistration = $allowTeacherSelfRegistration ? 'true' : 'false';
 
     // Use PHP 5.3 to avoid issue with weird peripherical auto-installers like travis-ci
@@ -2759,7 +2776,8 @@ 
     $allowSelfReg,
     $allowSelfRegProf,
     $installationProfile = ''
-) {
+)
+{
     $sysPath = !empty($sysPath) ? $sysPath : api_get_path(SYS_PATH);
 
     $connection = $manager->getConnection();

main/inc/lib/login.lib.php

Braces +22 added lines, -11 removed lines

@@ -80,7 +80,8 @@ 
     {
         $email_subject = "[".api_get_setting('siteName')."] ".get_lang('LoginRequest'); // SUBJECT
 
-        if ($by_username) { // Show only for lost password
+        if ($by_username) {
+// Show only for lost password
             $user_account_list = self::get_user_account_list($user, false, $by_username); // BODY
             $email_to = $user['email'];
         } else {
@@ -284,7 +285,8 @@ 
         global $is_allowedCreateCourse;
         global $_user;
 
-        if (isset($reset) && $reset) {    // session data refresh requested
+        if (isset($reset) && $reset) {
+// session data refresh requested
             unset($_SESSION['_user']['uidReset']);
             $is_platformAdmin = false;
             $is_allowedCreateCourse = false;
@@ -335,14 +337,16 @@ 
                     header('location:'.api_get_path(WEB_PATH));
                     //exit("WARNING UNDEFINED UID !! ");
                 }
-            } else { // no uid => logout or Anonymous
+            } else {
+// no uid => logout or Anonymous
                 Session::erase('_user');
                 Session::erase('_uid');
             }
 
             Session::write('is_platformAdmin', $is_platformAdmin);
             Session::write('is_allowedCreateCourse', $is_allowedCreateCourse);
-        } else { // continue with the previous values
+        } else {
+// continue with the previous values
             $_user = $_SESSION['_user'];
             $is_platformAdmin = $_SESSION['is_platformAdmin'];
             $is_allowedCreateCourse = $_SESSION['is_allowedCreateCourse'];
@@ -481,7 +485,8 @@ 
             }
         } else {
             // Continue with the previous values
-            if (empty($_SESSION['_course']) or empty($_SESSION['_cid'])) { //no previous values...
+            if (empty($_SESSION['_course']) or empty($_SESSION['_cid'])) {
+//no previous values...
                 $_cid = -1; //set default values that will be caracteristic of being unset
                 $_course = -1;
             } else {
@@ -654,7 +659,8 @@ 
                         $is_courseAdmin = true;
                     }
                 }
-            } else { // keys missing => not anymore in the course - user relation
+            } else {
+// keys missing => not anymore in the course - user relation
                 // course
                 $is_courseMember = false;
                 $is_courseAdmin = false;
@@ -746,12 +752,15 @@ 
         global $_course;
         global $_gid;
 
-        if ($reset) { // session data refresh requested
-            if ($group_id && $_cid && !empty($_course['real_id'])) { // have keys to search data
+        if ($reset) {
+// session data refresh requested
+            if ($group_id && $_cid && !empty($_course['real_id'])) {
+// have keys to search data
                 $group_table = Database::get_course_table(TABLE_GROUP);
                 $sql = "SELECT * FROM $group_table WHERE c_id = ".$_course['real_id']." AND id = '$group_id'";
                 $result = Database::query($sql);
-                if (Database::num_rows($result) > 0) { // This group has recorded status related to this course
+                if (Database::num_rows($result) > 0) {
+// This group has recorded status related to this course
                     $gpData = Database::fetch_array($result);
                     $_gid = $gpData ['id'];
                     Session::write('_gid', $_gid);
@@ -762,9 +771,11 @@ 
                 // Keys missing => not anymore in the group - course relation
                 Session::erase('_gid');
             }
-        } elseif (isset($_SESSION['_gid'])) { // continue with the previous values
+        } elseif (isset($_SESSION['_gid'])) {
+// continue with the previous values
             $_gid = $_SESSION ['_gid'];
-        } else { //if no previous value, assign caracteristic undefined value
+        } else {
+//if no previous value, assign caracteristic undefined value
             $_gid = -1;
         }
 

main/inc/lib/baker.lib.php

Braces +9 added lines, -4 removed lines

@@ -97,8 +97,9 @@ 
         // Read the magic bytes and verify
         $retval = substr($png, 0, 8);
         $ipos = 8;
-        if ($retval != "\x89PNG\x0d\x0a\x1a\x0a")
-            throw new Exception('Is not a valid PNG image');
+        if ($retval != "\x89PNG\x0d\x0a\x1a\x0a") {
+                    throw new Exception('Is not a valid PNG image');
+        }
         // Loop through the chunks. Byte 0-3 is length, Byte 4-7 is type
         $chunkHeader = substr($png, $ipos, 8);
         $ipos = $ipos + 8;
@@ -110,13 +111,17 @@ 
                 $data = substr($png, $ipos, $chunk['size']);
                 $sections = explode("\0", $data);
                 print_r($sections);
-                if ($sections[0] == $key) $skip = true;
+                if ($sections[0] == $key) {
+                    $skip = true;
+                }
             }
             // Extract the data and the CRC
             $data = substr($png, $ipos, $chunk['size'] + 4);
             $ipos = $ipos + $chunk['size'] + 4;
             // Add in the header, data, and CRC
-            if (!$skip) $retval = $retval.$chunkHeader.$data;
+            if (!$skip) {
+                $retval = $retval.$chunkHeader.$data;
+            }
             // Read next chunk header
             $chunkHeader = substr($png, $ipos, 8);
             $ipos = $ipos + 8;

main/user/resume_session.php

Braces +7 added lines, -1 removed lines

@@ -189,12 +189,18 @@
             <?php echo api_ucfirst(get_lang('SessionVisibility')) ?> :
         </td>
         <td>
-            <?php if ($session['visibility'] == 1) echo get_lang('ReadOnly'); elseif ($session['visibility'] == 2) echo get_lang('Visible'); elseif ($session['visibility'] == 3) echo api_ucfirst(get_lang('Invisible'))  ?>
+            <?php if ($session['visibility'] == 1) {
+    echo get_lang('ReadOnly');
+} elseif ($session['visibility'] == 2) {
+    echo get_lang('Visible');
+} elseif ($session['visibility'] == 3) {
+    echo api_ucfirst(get_lang('Invisible'))  ?>
         </td>
     </tr>
     <?php
 
     $multiple_url_is_on = api_get_multiple_access_url();
+}
     if ($multiple_url_is_on) {
         echo '<tr><td>';
         echo 'URL';

main/inc/lib/online.inc.php

Braces +4 added lines, -2 removed lines

@@ -214,7 +214,8 @@ 
     $direction = null,
     $time_limit = null,
     $friends = false
-) {
+)
+{
     // Time limit in seconds?
     if (empty($time_limit)) {
         $time_limit = api_get_setting('time_limit_whosonline');
@@ -442,7 +443,8 @@ 
     $uid,
     $time_limit,
     $coursecode = null
-) {
+)
+{
     if (empty($coursecode)) {
         return false;
     }

main/social/profile.php

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++;
             }

main/survey/fillsurvey.php

Braces +6 added lines, -3 removed lines

@@ -1153,7 +1153,8 @@ 
                 var_Dump($questions);
             }
         }
-    } else { // In case it's another type than 0 or 1
+    } else {
+// In case it's another type than 0 or 1
         echo get_lang('ErrorSurveyTypeUnknown');
     }
 }
@@ -1247,7 +1248,8 @@ 
         // The normal survey as always but with the form profile
         if (isset($_GET['show'])) {
             $numberofpages = count($paged_questions);
-            if (($show < $numberofpages) || !$_GET['show']) { //$show = $_GET['show'] + 1
+            if (($show < $numberofpages) || !$_GET['show']) {
+//$show = $_GET['show'] + 1
                 if ($show == 0) {
                     $form->addButton(
                         'next_survey_page',
@@ -1275,7 +1277,8 @@ 
             }
         }
     }
-} elseif ($survey_data['survey_type'] === '1') { //conditional/personality-test type survey
+} elseif ($survey_data['survey_type'] === '1') {
+//conditional/personality-test type survey
     if (isset($_GET['show']) || isset($_POST['personality'])) {
         $numberofpages = count($paged_questions);
         if (!empty($paged_questions_sec) && count($paged_questions_sec) > 0) {