chamilo / chamilo-lms

main/admin/user_add.php

Braces +2 added lines, -1 removed lines

@@ -376,7 +376,8 @@
 
         $extra = array();
         foreach ($user as $key => $value) {
-            if (substr($key, 0, 6) == 'extra_') { //an extra field
+            if (substr($key, 0, 6) == 'extra_') {
+//an extra field
                 $extra[substr($key, 6)] = $value;
             }
         }

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 } ?>

plugin/sepe/src/sepe_plugin.class.php

Braces +14 added lines, -14 removed lines

@@ -6,8 +6,8 @@ 
  * @author Jose Angel Ruiz    <jaruiz@nosolored.com>

  * @author Julio Montoya <gugli100@gmail.com>

  */

-class SepePlugin extends Plugin

-{

+class SepePlugin extends Plugin
+{
     const TABLE_SEPE_CENTER = 'plugin_sepe_center';

     const TABLE_SEPE_ACTIONS = 'plugin_sepe_actions';

     const TABLE_SEPE_SPECIALTY = 'plugin_sepe_specialty';

@@ -30,14 +30,14 @@ 
      *

      * @return StaticPlugin

      */

-    public static function create()

-    {

+    public static function create()
+    {
         static $result = null;

         return $result ? $result : $result = new self();

     }

 

-    protected function __construct()

-    {

+    protected function __construct()
+    {
         parent::__construct(

             '2.0', 

             '

@@ -51,8 +51,8 @@ 
     /**

      * This method creates the tables required to this plugin

      */

-    function install()

-    {

+    function install()
+    {
         $tablesToBeCompared = array(

             self::TABLE_SEPE_CENTER,

             self::TABLE_SEPE_ACTIONS,

@@ -76,7 +76,7 @@ 
         $sm = $cn->getSchemaManager();

         $tables = $sm->tablesExist($tablesToBeCompared);

 

-        if ($tables) {

+        if ($tables) {
             return false;

         }

 

@@ -86,8 +86,8 @@ 
     /**

      * This method drops the plugin tables

      */

-    function uninstall()

-    {

+    function uninstall()
+    {
         $tablesToBeDeleted = array(

             self::TABLE_SEPE_CENTER,

             self::TABLE_SEPE_SPECIALTY_CLASSROOM,

@@ -107,7 +107,7 @@ 
             self::TABLE_SEPE_LOG

         );

 

-        foreach ($tablesToBeDeleted as $tableToBeDeleted) {

+        foreach ($tablesToBeDeleted as $tableToBeDeleted) {
             $table = Database::get_main_table($tableToBeDeleted);

             $sql = "DROP TABLE IF EXISTS $table";

             Database::query($sql);

@@ -115,8 +115,8 @@ 
         $this->manageTab(false);

     }

 

-    function update()

-    {

+    function update()
+    {
         $oldTableCenters = 'plugin_sepe_centros';

         $oldTableTutorsCompany = 'plugin_sepe_tutors_empresa';

         $oldTableCompetence = 'plugin_sepe_competencia_docente';

plugin/sepe/src/index.sepe.php

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/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/diagnoser.lib.php

Braces +12 added lines, -8 removed lines

@@ -232,8 +232,9 @@ 
         $array[] = $this->build_setting($status, '[INI]', 'display_errors', 'http://www.php.net/manual/en/ini.core.php#ini.display_errors', $setting, $req_setting, 'on_off', get_lang('DisplayErrorsInfo'));
 
         $setting = ini_get('default_charset');
-        if ($setting == '')
-            $setting = null;
+        if ($setting == '') {
+                    $setting = null;
+        }
         $req_setting = null;
         $status = $setting == $req_setting ? self::STATUS_OK : self::STATUS_ERROR;
         $array[] = $this->build_setting($status, '[INI]', 'default_charset', 'http://www.php.net/manual/en/ini.core.php#ini.default-charset', $setting, $req_setting, null, get_lang('DefaultCharsetInfo'));
@@ -251,22 +252,25 @@ 
         $setting = ini_get('memory_limit');
         $req_setting = '>= '.REQUIRED_MIN_MEMORY_LIMIT.'M';
         $status = self::STATUS_ERROR;
-        if ((float) $setting >= REQUIRED_MIN_MEMORY_LIMIT)
-            $status = self::STATUS_OK;
+        if ((float) $setting >= REQUIRED_MIN_MEMORY_LIMIT) {
+                    $status = self::STATUS_OK;
+        }
         $array[] = $this->build_setting($status, '[INI]', 'memory_limit', 'http://www.php.net/manual/en/ini.core.php#ini.memory-limit', $setting, $req_setting, null, get_lang('MemoryLimitInfo'));
 
         $setting = ini_get('post_max_size');
         $req_setting = '>= '.REQUIRED_MIN_POST_MAX_SIZE.'M';
         $status = self::STATUS_ERROR;
-        if ((float) $setting >= REQUIRED_MIN_POST_MAX_SIZE)
-            $status = self::STATUS_OK;
+        if ((float) $setting >= REQUIRED_MIN_POST_MAX_SIZE) {
+                    $status = self::STATUS_OK;
+        }
         $array[] = $this->build_setting($status, '[INI]', 'post_max_size', 'http://www.php.net/manual/en/ini.core.php#ini.post-max-size', $setting, $req_setting, null, get_lang('PostMaxSizeInfo'));
 
         $setting = ini_get('upload_max_filesize');
         $req_setting = '>= '.REQUIRED_MIN_UPLOAD_MAX_FILESIZE.'M';
         $status = self::STATUS_ERROR;
-        if ((float) $setting >= REQUIRED_MIN_UPLOAD_MAX_FILESIZE)
-            $status = self::STATUS_OK;
+        if ((float) $setting >= REQUIRED_MIN_UPLOAD_MAX_FILESIZE) {
+                    $status = self::STATUS_OK;
+        }
         $array[] = $this->build_setting($status, '[INI]', 'upload_max_filesize', 'http://www.php.net/manual/en/ini.core.php#ini.upload_max_filesize', $setting, $req_setting, null, get_lang('UploadMaxFilesizeInfo'));
 
         $setting = ini_get('variables_order');

main/inc/lib/specific_fields_manager.lib.php

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(

main/auth/external_login/functions.inc.php

Braces +2 added lines, -1 removed lines

@@ -24,7 +24,8 @@
     $sql = "SELECT * from $table where username='".Database::escape_string($login)."'";
     $result = Database::query($sql);
 
-    if (Database::num_rows($result) == 0) { //false password
+    if (Database::num_rows($result) == 0) {
+//false password
         return false;
     }
     $user_info = Database::fetch_assoc($result);