chamilo / chamilo-lms

plugin/sepe/src/sepe-administration-menu.php

Braces +2 added lines, -2 removed lines

@@ -15,7 +15,7 @@ 
 $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) {
     $htmlText = '';

     $htmlText .= '<div class="panel panel-default">';

         $htmlText .= '<div class="panel-heading" role="tab">';

@@ -56,6 +56,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/src/formative-actions-list.php

Braces +4 added lines, -4 removed lines

@@ -10,16 +10,16 @@ 
 

 $plugin = SepePlugin::create();

 

-if (api_is_platform_admin()) {

+if (api_is_platform_admin()) {
     $templateName = $plugin->get_lang('FormativesActionsList');

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

     $tpl = new Template($templateName);

     

-    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']);

     }

@@ -35,6 +35,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/src/formative-action.php

Braces +5 added lines, -5 removed lines

@@ -13,10 +13,10 @@ 
 $plugin = SepePlugin::create();

 $_cid = 0;

 

-if (api_is_platform_admin()) {

+if (api_is_platform_admin()) {
     $actionId = getActionId(intval($_GET['cid']));

     $info = getActionInfo($actionId);

-    if ($info === false) {

+    if ($info === false) {
         header("Location: formative-actions-list.php");    

     }

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

@@ -24,11 +24,11 @@ 
     $interbreadcrumb[] = array("url" => "formative-actions-list.php", "name" => $plugin->get_lang('FormativesActionsList'));

     $tpl = new Template($templateName);

     

-    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']);

     }

@@ -45,6 +45,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/database.php

Braces +7 added lines, -7 removed lines

@@ -8,7 +8,7 @@ 
 /**

  * Check if script can be called

  */

-if (!function_exists('api_get_path')) {

+if (!function_exists('api_get_path')) {
     die('This script must be loaded through the Chamilo plugin installer sequence');

 }

 

@@ -710,7 +710,7 @@ 
 

 $queries = $pluginSchema->toSql($platform);

 

-foreach ($queries as $query) {

+foreach ($queries as $query) {
     Database::query($query);

 }

 

@@ -725,7 +725,7 @@ 
     array(6, '06', 'Experiencia docente contrastada de al menos 600 horas de impartición de acciones formativas de formación profesional para el empleo o del sistema educativo en modalidad presencial, en los últimos diez años.')

 );

 

-foreach ($competences as $competence) {

+foreach ($competences as $competence) {
     Database::insert(

         $sepeTeachingCompetenceTable,

         array(

@@ -787,7 +787,7 @@ 
 $provinces = 'Albacete;Alicante/Alacant;Almería;Araba/Álava;Asturias;Ávila;Badajoz;Balears, Illes;Barcelona;Bizkaia;Burgos;Cáceres;Cádiz;Cantabria;Castellón/Castelló;Ciudad Real;Córdoba;Coruña, A;Cuenca;Gipuzkoa;Girona;Granada;Guadalajara;Huelva;Huesca;Jaén;León;Lleida;Lugo;Madrid;Málaga;Murcia;Navarra;Ourense;Palencia;Palmas, Las;Pontevedr;Rioja, La;Salamanca;Santa Cruz de Tenerife;Segovia;Sevilla;Soria;Tarragona;Teruel;Toledo;Valencia/Valéncia;Valladolid;Zamora;Zaragoza;Ceuta;Melilla';

 $list_provinces = explode(';', $provinces);

 $i = 1;

-foreach ($list_provinces as $value) {

+foreach ($list_provinces as $value) {
     $sql = "INSERT INTO extra_field_options (field_id, option_value, display_text, option_order) VALUES ('".$field_id."', '".$i."', '".$value."','".$i."');";

     Database::query($sql);

     $i++;

@@ -801,7 +801,7 @@ 
 $ccaa = ';Andalucía;Aragón;Asturias, Principado de;Balears, Illes;Canarias;Cantabria;Castilla y León;Castilla - La Mancha;Cataluña;Comunitat Valenciana;Extremadura;Galicia;Madrid, Comunidad de;Murcia, Región de;Navarra, Comunidad Foral de;País Vasco;Rioja, La;Ceuta;Melilla';

 $list_ccaa = explode(';', $ccaa);

 $i = 1;

-foreach ($list_ccaa as $value) {

+foreach ($list_ccaa as $value) {
     $sql = "INSERT INTO extra_field_options (field_id, option_value, display_text, option_order) VALUES ('".$field_id."', '".$i."', '".$value."','".$i."');";

     Database::query($sql);

     $i++;

@@ -814,7 +814,7 @@ 
 //$fieldoptions = ';Albacete;Alicante/Alacant;Almería;Araba/Álava;Asturias;Ávila;Badajoz;Balears, Illes;Barcelona;Bizkaia;Burgos;Cáceres;Cádiz;Cantabria;Castellón/Castelló;Ciudad Real;Córdoba;Coruña, A;Cuenca;Gipuzkoa;Girona;Granada;Guadalajara;Huelva;Huesca;Jaén;León;Lleida;Lugo;Madrid;Málaga;Murcia;Navarra;Ourense;Palencia;Palmas, Las;Pontevedr;Rioja, La;Salamanca;Santa Cruz de Tenerife;Segovia;Sevilla;Soria;Tarragona;Teruel;Toledo;Valencia/Valéncia;Valladolid;Zamora;Zaragoza;Ceuta;Melilla';

 $field_id = UserManager::create_extra_field($fieldlabel, $fieldtype, $fieldtitle, $fielddefault);

 $i = 1;

-foreach ($list_provinces as $value) {

+foreach ($list_provinces as $value) {
     $sql = "INSERT INTO extra_field_options (field_id, option_value, display_text, option_order) VALUES ('".$field_id."', '".$i."', '".$value."','".$i."');";

     Database::query($sql);

     $i++;

@@ -827,7 +827,7 @@ 
 //$fieldoptions = ';Andalucía;Aragón;Asturias, Principado de;Balears, Illes;Canarias;Cantabria;Castilla y León;Castilla - La Mancha;Cataluña;Comunitat Valenciana;Extremadura;Galicia;Madrid, Comunidad de;Murcia, Región de;Navarra, Comunidad Foral de;País Vasco;Rioja, La;Ceuta;Melilla';

 $field_id = UserManager::create_extra_field($fieldlabel, $fieldtype, $fieldtitle, $fielddefault);

 $i = 1;

-foreach ($list_ccaa as $value) {

+foreach ($list_ccaa as $value) {
     $sql = "INSERT INTO extra_field_options (field_id, option_value, display_text, option_order) VALUES ('".$field_id."', '".$i."', '".$value."','".$i."');";

     Database::query($sql);

     $i++;

main/exercise/hotspot_updatescore.inc.php

Braces +4 added lines, -2 removed lines

@@ -18,13 +18,15 @@
 $objExercise = Session::read('objExercise');
 $hotspotId = $_GET['hotspotId'];
 $exerciseId = $objExercise->selectId();
-if ($_GET['answerId'] == "0") { // click is NOT on a hotspot
+if ($_GET['answerId'] == "0") {
+// click is NOT on a hotspot
     $hit = 0;
     $answerId = $hotspotId;
 
     // remove from session
     unset($_SESSION['exerciseResult'][$questionId][$answerId]);
-} else { // user clicked ON a hotspot
+} else {
+// user clicked ON a hotspot
     $hit = 1;
     $answerId = $hotspotId;
 

main/inc/lib/tracking.lib.php

Braces +42 added lines, -18 removed lines

@@ -414,7 +414,8 @@ 
                                     Display::return_icon('visible.gif', get_lang('HideAttemptView')),
                                     api_get_self().'?action=stats&extend_id='.$my_item_id.'&fold_attempt_id='.$row['iv_id'].$url_suffix
                                 );
-                            } else { // Same case if fold_attempt_id is set, so not implemented explicitly.
+                            } else {
+// Same case if fold_attempt_id is set, so not implemented explicitly.
                                 // The extend button for this attempt has not been clicked.
                                 $extend_attempt_link = Display::url(
                                     Display::return_icon('invisible.gif', get_lang('ExtendAttemptView')),
@@ -2608,7 +2609,9 @@ 
         $getOnlyBestAttempt = false
     ) {
         $debug = false;
-        if ($debug) echo '<h1>Tracking::get_avg_student_score</h1>';
+        if ($debug) {
+            echo '<h1>Tracking::get_avg_student_score</h1>';
+        }
         $tbl_stats_exercices = Database::get_main_table(TABLE_STATISTIC_TRACK_E_EXERCISES);
         $tbl_stats_attempts = Database::get_main_table(TABLE_STATISTIC_TRACK_E_ATTEMPT);
         $course = api_get_course_info($course_code);
@@ -2794,7 +2797,9 @@ 
                             // Came from the lp_item_view
                             $max_score_item_view = $row_max_score['max_score_item_view'];
                             $score = $row_max_score['score'];
-                            if ($debug) echo '<h3>Item Type: '.$row_max_score['item_type'].'</h3>';
+                            if ($debug) {
+                                echo '<h3>Item Type: '.$row_max_score['item_type'].'</h3>';
+                            }
 
                             if ($row_max_score['item_type'] == 'sco') {
                                 /* Check if it is sco (easier to get max_score)
@@ -2845,18 +2850,24 @@ 
                                         LIMIT 1";
 
                                 $result_last_attempt = Database::query($sql);
-                                if ($debug) var_dump($sql);
+                                if ($debug) {
+                                    var_dump($sql);
+                                }
                                 $num = Database::num_rows($result_last_attempt);
                                 if ($num > 0) {
                                     $attemptResult = Database::fetch_array($result_last_attempt, 'ASSOC');
                                     $id_last_attempt = $attemptResult['exe_id'];
                                     // We overwrite the score with the best one not the one saved in the LP (latest)
                                     if ($getOnlyBestAttempt && $get_only_latest_attempt_results == false) {
-                                        if ($debug) echo "Following score comes from the track_exercise table not in the LP because the score is the best<br />";
+                                        if ($debug) {
+                                            echo "Following score comes from the track_exercise table not in the LP because the score is the best<br />";
+                                        }
                                         $score = $attemptResult['exe_result'];
                                     }
 
-                                    if ($debug) echo "Attempt id: $id_last_attempt with score $score<br />";
+                                    if ($debug) {
+                                        echo "Attempt id: $id_last_attempt with score $score<br />";
+                                    }
                                     // Within the last attempt number tracking, get the sum of
                                     // the max_scores of all questions that it was
                                     // made of (we need to make this call dynamic because of random questions selection)
@@ -2901,7 +2912,9 @@ 
                                         $count_items++;
                                     }
                                 }
-                                if ($debug) echo '$count_items: '.$count_items;
+                                if ($debug) {
+                                    echo '$count_items: '.$count_items;
+                                }
                             }
                         } //end for
 
@@ -2937,19 +2950,29 @@ 
                     }
                 }
 
-                if ($debug) echo '<h3>$lp_with_quiz '.$lp_with_quiz.' </h3>';
-                if ($debug) echo '<h3>Final return</h3>';
+                if ($debug) {
+                    echo '<h3>$lp_with_quiz '.$lp_with_quiz.' </h3>';
+                }
+                if ($debug) {
+                    echo '<h3>Final return</h3>';
+                }
 
                 if ($lp_with_quiz != 0) {
                     if (!$return_array) {
                         $score_of_scorm_calculate = round(($global_result / $lp_with_quiz), 2);
-                        if ($debug) var_dump($score_of_scorm_calculate);
+                        if ($debug) {
+                            var_dump($score_of_scorm_calculate);
+                        }
                         if (empty($lp_ids)) {
-                            if ($debug) echo '<h2>All lps fix: '.$score_of_scorm_calculate.'</h2>';
+                            if ($debug) {
+                                echo '<h2>All lps fix: '.$score_of_scorm_calculate.'</h2>';
+                            }
                         }
                         return $score_of_scorm_calculate;
                     } else {
-                        if ($debug) var_dump($global_result, $lp_with_quiz);
+                        if ($debug) {
+                            var_dump($global_result, $lp_with_quiz);
+                        }
                         return array($global_result, $lp_with_quiz);
                     }
                 } else {
@@ -3436,11 +3459,13 @@ 
 
         if (!empty ($id_session)) {
             $sql .= ' WHERE session_course.session_id='.$id_session;
-            if (api_is_multiple_url_enabled())
-            $sql .= ' AND access_url_id = '.$access_url_id;
+            if (api_is_multiple_url_enabled()) {
+                        $sql .= ' AND access_url_id = '.$access_url_id;
+            }
         } else {
-            if (api_is_multiple_url_enabled())
-            $sql .= ' WHERE access_url_id = '.$access_url_id;
+            if (api_is_multiple_url_enabled()) {
+                        $sql .= ' WHERE access_url_id = '.$access_url_id;
+            }
         }
 
         $result = Database::query($sql);
@@ -3554,8 +3579,7 @@ 
                 if (empty($session['access_start_date'])
                 ) {
                     $session['status'] = get_lang('SessionActive');
-                }
-                else {
+                } else {
                     $time_start = api_strtotime($session['access_start_date'], 'UTC');
                     $time_end = api_strtotime($session['access_end_date'], 'UTC');
                     if ($time_start < time() && time() < $time_end) {

plugin/sepe/install.php

Braces +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
  */

 require_once __DIR__.'/config.php';

 

-if (!api_is_platform_admin()) {

+if (!api_is_platform_admin()) {
     die ('You must have admin permissions to install plugins');

 }

 

plugin/sepe/src/identification-data.php

Braces +4 added lines, -4 removed lines

@@ -12,17 +12,17 @@ 
 $plugin = SepePlugin::create();

 $_cid = 0;

 

-if (api_is_platform_admin()) {

+if (api_is_platform_admin()) {
     $info = getInfoIdentificationData();

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

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

     $tpl = new Template($templateName);

 

-    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']);

     }

@@ -31,6 +31,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/src/configuration.php

Braces +4 added lines, -4 removed lines

@@ -13,17 +13,17 @@ 
 $plugin = SepePlugin::create();

 $_cid = 0;

 

-if (api_is_platform_admin()) {

+if (api_is_platform_admin()) {
     $tUser = Database::get_main_table(TABLE_MAIN_USER);

     $tApi = Database::get_main_table(TABLE_MAIN_USER_API_KEY);

     $login = 'SEPE';

     $sql = "SELECT a.api_key AS api FROM $tUser u, $tApi a WHERE u.username='".$login."' and u.user_id = a.user_id AND a.api_service = 'dokeos';";

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

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

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

         $info = $tmp['api'];

         

-    } else {

+    } else {
         $info = '';

     }

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

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

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

     $tpl->display_one_col_template();

-} else {

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

 }