chamilo / chamilo-lms

plugin/buycourses/src/paymentsetup.php

Spacing +8 added lines, -8 removed lines

@@ -28,7 +28,7 @@ 
             Display::return_message(get_lang('ItemRemoved'), 'success')

         );

 

-        header('Location: ' . api_get_self());

+        header('Location: '.api_get_self());

         exit;

     }

 }

@@ -46,7 +46,7 @@ 
         Display::return_message(get_lang('Saved'), 'success')

     );

 

-    header('Location:' . api_get_self());

+    header('Location:'.api_get_self());

     exit;

 }

 

@@ -78,7 +78,7 @@ 
     }

 }

 

-$currencyForm->addTextarea('terms_and_conditions', [ get_lang('TermsAndConditions'), $plugin->get_lang('WriteHereTheTermsAndConditionsOfYourECommerce') ], '');

+$currencyForm->addTextarea('terms_and_conditions', [get_lang('TermsAndConditions'), $plugin->get_lang('WriteHereTheTermsAndConditionsOfYourECommerce')], '');

 $currencyForm->addButtonSave(get_lang('Save'));

 $currencyForm->setDefaults($plugin->getGlobalParameters());

 

@@ -95,7 +95,7 @@ 
         Display::return_message(get_lang('Saved'), 'success')

     );

 

-    header('Location:' . api_get_self());

+    header('Location:'.api_get_self());

     exit;

 }

 

@@ -134,7 +134,7 @@ 
         Display::return_message(get_lang('Saved'), 'success')

     );

 

-    header('Location:' . api_get_self());

+    header('Location:'.api_get_self());

     exit;

 }

 

@@ -160,7 +160,7 @@ 
         Display::return_message(get_lang('Saved'), 'success')

     );

 

-    header('Location:' . api_get_self());

+    header('Location:'.api_get_self());

     exit;

 }

 

@@ -199,7 +199,7 @@ 
         Display::return_message(get_lang('Saved'), 'success')

     );

 

-    header('Location:' . api_get_self());

+    header('Location:'.api_get_self());

     exit;

 }

 

@@ -221,7 +221,7 @@ 
 

 // breadcrumbs

 $interbreadcrumb[] = [

-    'url' => api_get_path(WEB_PLUGIN_PATH) . 'buycourses/index.php',

+    'url' => api_get_path(WEB_PLUGIN_PATH).'buycourses/index.php',

     'name' => $plugin->get_lang('plugin_title')

 ];

 

plugin/buycourses/src/process_confirm.php

Spacing +5 added lines, -5 removed lines

@@ -48,8 +48,8 @@ 
             $sale['price'],

             $currency['iso_code'],

             'paypal',

-            api_get_path(WEB_PLUGIN_PATH) . 'buycourses/src/success.php',

-            api_get_path(WEB_PLUGIN_PATH) . 'buycourses/src/error.php',

+            api_get_path(WEB_PLUGIN_PATH).'buycourses/src/success.php',

+            api_get_path(WEB_PLUGIN_PATH).'buycourses/src/error.php',

             $extra

         );

 

@@ -94,7 +94,7 @@ 
 

                 unset($_SESSION['bc_sale_id']);

 

-                header('Location: ' . api_get_path(WEB_PLUGIN_PATH) . 'buycourses/index.php');

+                header('Location: '.api_get_path(WEB_PLUGIN_PATH).'buycourses/index.php');

                 exit;

             }

 

@@ -131,7 +131,7 @@ 
             );

 

             unset($_SESSION['bc_sale_id']);

-            header('Location: ' . api_get_path(WEB_PLUGIN_PATH) . 'buycourses/src/course_catalog.php');

+            header('Location: '.api_get_path(WEB_PLUGIN_PATH).'buycourses/src/course_catalog.php');

             exit;

         }

 

@@ -203,7 +203,7 @@ 
                     )

                 );

 

-                header('Location: ' . api_get_path(WEB_PLUGIN_PATH) . 'buycourses/index.php');

+                header('Location: '.api_get_path(WEB_PLUGIN_PATH).'buycourses/index.php');

                 exit;

             }

         }

plugin/buycourses/src/service_process_confirm.php

Spacing +6 added lines, -6 removed lines

@@ -42,8 +42,8 @@ 
         // purchase-specific fields are ignored. This little condition handle this fact.

         $itemPrice = $serviceSale['price'];

 

-        $returnUrl = api_get_path(WEB_PLUGIN_PATH) . 'buycourses/src/service_success.php';

-        $cancelUrl = api_get_path(WEB_PLUGIN_PATH) . 'buycourses/src/service_error.php';

+        $returnUrl = api_get_path(WEB_PLUGIN_PATH).'buycourses/src/service_success.php';

+        $cancelUrl = api_get_path(WEB_PLUGIN_PATH).'buycourses/src/service_error.php';

 

         // The extra params for handle the hard job, this var is VERY IMPORTANT !!

         $extra = '';

@@ -75,7 +75,7 @@ 
 

             $plugin->cancelServiceSale(intval($serviceSale['id']));

 

-            header('Location: '. api_get_path(WEB_PLUGIN_PATH) . 'buycourses/src/service_catalog.php');

+            header('Location: '.api_get_path(WEB_PLUGIN_PATH).'buycourses/src/service_catalog.php');

             exit;

         }

 

@@ -112,7 +112,7 @@ 
                 Display::addFlash(

                     Display::return_message($plugin->get_lang('OrderCancelled'), 'error', false)

                 );

-                header('Location: '. api_get_path(WEB_PLUGIN_PATH) . 'buycourses/src/service_catalog.php');

+                header('Location: '.api_get_path(WEB_PLUGIN_PATH).'buycourses/src/service_catalog.php');

                 exit;

             }

 

@@ -151,7 +151,7 @@ 
             );

 

             unset($_SESSION['bc_service_sale_id']);

-            header('Location: ' . api_get_path(WEB_PLUGIN_PATH) . 'buycourses/src/service_catalog.php');

+            header('Location: '.api_get_path(WEB_PLUGIN_PATH).'buycourses/src/service_catalog.php');

             exit;

         }

 

@@ -201,7 +201,7 @@ 
                     )

                 );

 

-                header('Location: ' . api_get_path(WEB_PLUGIN_PATH) . 'buycourses/index.php');

+                header('Location: '.api_get_path(WEB_PLUGIN_PATH).'buycourses/index.php');

                 exit;

             }

         }

plugin/buycourses/src/buy_course_plugin.class.php

Spacing +27 added lines, -27 removed lines

@@ -110,7 +110,7 @@ 
             return false;

         }

 

-        require_once api_get_path(SYS_PLUGIN_PATH) . 'buycourses/database.php';

+        require_once api_get_path(SYS_PLUGIN_PATH).'buycourses/database.php';

     }

 

     /**

@@ -158,10 +158,10 @@ 
             $item = $this->getItemByProduct(intval($productId), $productType);

             $return['html'] = '<div class="buycourses-price">';

             if ($item) {

-                $return['html'] .= '<span class="label label-primary"><b>'. $item['iso_code'] .' ' . $item['price'] . '</b></span>';

+                $return['html'] .= '<span class="label label-primary"><b>'.$item['iso_code'].' '.$item['price'].'</b></span>';

                 $return['verificator'] = true;

             } else {

-                $return['html'] .= '<span class="label label-primary"><b>'. $this->get_lang('Free'). '</b></span>';

+                $return['html'] .= '<span class="label label-primary"><b>'.$this->get_lang('Free').'</b></span>';

                 $return['verificator'] = false;

             }

             $return['html'] .= '</div>';

@@ -179,15 +179,15 @@ 
      * @return string $html

      */

     public function returnBuyCourseButton($productId, $productType) {

-        $url = api_get_path(WEB_PLUGIN_PATH) .

-            'buycourses/src/process.php?i=' .

-            intval($productId) .

-            '&t=' .

+        $url = api_get_path(WEB_PLUGIN_PATH).

+            'buycourses/src/process.php?i='.

+            intval($productId).

+            '&t='.

             $productType

         ;

 

-        $html = ' <a class="btn btn-success btn-sm" title="' . $this->get_lang('Buy') . '" href="' . $url . '">' .

-            Display::returnFontAwesomeIcon('fa fa-shopping-cart') . '</a>';

+        $html = ' <a class="btn btn-success btn-sm" title="'.$this->get_lang('Buy').'" href="'.$url.'">'.

+            Display::returnFontAwesomeIcon('fa fa-shopping-cart').'</a>';

 

         return $html;

     }

@@ -733,7 +733,7 @@ 
         );

 

         if (!empty($sessionImage)) {

-            $sessionInfo['image'] = api_get_path(WEB_UPLOAD_PATH) . $sessionImage['value'];

+            $sessionInfo['image'] = api_get_path(WEB_UPLOAD_PATH).$sessionImage['value'];

         }

 

         $sessionCourses = $session->getCourses();

@@ -1189,7 +1189,7 @@ 
 

         $str = '';

 

-        srand((double)microtime() * 1000000);

+        srand((double) microtime() * 1000000);

 

         for ($i = 0; $i < $length; $i++) {

             $numbers = rand(0, strlen($salt) - 1);

@@ -1536,8 +1536,8 @@ 
      */

     public function getPayouts($status = self::PAYOUT_STATUS_PENDING, $payoutId = false, $userId = false)

     {

-        $condition = ($payoutId) ? 'AND p.id = '. intval($payoutId) : '';

-        $condition2 = ($userId) ? ' AND p.user_id = ' . intval($userId) : '';

+        $condition = ($payoutId) ? 'AND p.id = '.intval($payoutId) : '';

+        $condition2 = ($userId) ? ' AND p.user_id = '.intval($userId) : '';

         $typeResult = ($condition) ? 'first' : 'all';

         $payoutsTable = Database::get_main_table(BuyCoursesPlugin::TABLE_PAYPAL_PAYOUTS);

         $saleTable = Database::get_main_table(BuyCoursesPlugin::TABLE_SALE);

@@ -1564,14 +1564,14 @@ 
             INNER JOIN $saleTable s ON s.id = p.sale_id

             INNER JOIN $currencyTable c ON s.currency_id = c.id

             LEFT JOIN  $extraFieldValues efv ON p.user_id = efv.item_id 

-            AND field_id = " . intval($paypalExtraField['id']) . "

+            AND field_id = ".intval($paypalExtraField['id'])."

         ";

 

         $payouts = Database::select(

             "p.* , u.firstname, u.lastname, efv.value as paypal_account, s.reference as sale_reference, s.price as item_price, c.iso_code",

             "$payoutsTable p $innerJoins",

             [

-                'where' => ['p.status = ? '.$condition . ' ' .$condition2 => $status]

+                'where' => ['p.status = ? '.$condition.' '.$condition2 => $status]

             ],

             $typeResult

         );

@@ -1635,7 +1635,7 @@ 
         $platformCommission = $this->getPlatformCommission();

 

         $sale = $this->getSale($saleId);

-        $teachersCommission = number_format((floatval($sale['price']) * intval($platformCommission['commission']))/100, 2);

+        $teachersCommission = number_format((floatval($sale['price']) * intval($platformCommission['commission'])) / 100, 2);

 

 

         $beneficiaries = $this->getBeneficiariesBySale($saleId);

@@ -1647,7 +1647,7 @@ 
                     'payout_date' => getdate(),

                     'sale_id' => intval($saleId),

                     'user_id' => $beneficiary['user_id'],

-                    'commission' => number_format((floatval($teachersCommission) * intval($beneficiary['commissions']))/100, 2),

+                    'commission' => number_format((floatval($teachersCommission) * intval($beneficiary['commissions'])) / 100, 2),

                     'status' => self::PAYOUT_STATUS_PENDING

                 ]

             );

@@ -2229,17 +2229,17 @@ 
      */

     public function getPath($var)

     {

-        $pluginPath = api_get_path(WEB_PLUGIN_PATH) . 'buycourses/';

+        $pluginPath = api_get_path(WEB_PLUGIN_PATH).'buycourses/';

         $paths = [

-            'SERVICE_IMAGES' => $pluginPath . 'uploads/services/images/',

-            'SRC' => $pluginPath . 'src/',

-            'VIEW' => $pluginPath . 'view/',

-            'UPLOADS' => $pluginPath . 'uploads/',

-            'LANGUAGES' => $pluginPath . 'lang/',

-            'RESOURCES' => $pluginPath . 'resources/',

-            'RESOURCES_IMG' => $pluginPath . 'resources/img/',

-            'RESOURCES_CSS' => $pluginPath . 'resources/css/',

-            'RESOURCES_JS' => $pluginPath . 'resources/js/',

+            'SERVICE_IMAGES' => $pluginPath.'uploads/services/images/',

+            'SRC' => $pluginPath.'src/',

+            'VIEW' => $pluginPath.'view/',

+            'UPLOADS' => $pluginPath.'uploads/',

+            'LANGUAGES' => $pluginPath.'lang/',

+            'RESOURCES' => $pluginPath.'resources/',

+            'RESOURCES_IMG' => $pluginPath.'resources/img/',

+            'RESOURCES_CSS' => $pluginPath.'resources/css/',

+            'RESOURCES_JS' => $pluginPath.'resources/js/',

         ];

 

         return $paths[$var];

plugin/buycourses/src/buycourses.ajax.php

Spacing +25 added lines, -25 removed lines

@@ -58,20 +58,20 @@ 
 
         $userInfo = api_get_user_info($sale['user_id']);
 
-        $html = '<h2>' . $sale['product_name'] .'</h2>';
+        $html = '<h2>'.$sale['product_name'].'</h2>';
         $html .= '<div class="row">';
         $html .= '<div class="col-sm-6 col-md-6">';
         $html .= '<ul>';
-        $html .= '<li><b>'. $plugin->get_lang('OrderPrice') . ':</b> '. $sale['price'] . '</li>';
-        $html .= '<li><b>'. $plugin->get_lang('CurrencyType') . ':</b> '. $currency['iso_code'] . '</li>';
-        $html .= '<li><b>'. $plugin->get_lang('ProductType') . ':</b> '. $productType . '</li>';
-        $html .= '<li><b>'. $plugin->get_lang('OrderDate') . ':</b> '. api_format_date($sale['date'], DATE_TIME_FORMAT_LONG_24H) . '</li>';
-        $html .= '<li><b>'. $plugin->get_lang('Buyer') . ':</b> '. $userInfo['complete_name'] . '</li>';
-        $html .= '<li><b>'. $plugin->get_lang('PaymentMethods') . ':</b> '. $paymentType . '</li>';
+        $html .= '<li><b>'.$plugin->get_lang('OrderPrice').':</b> '.$sale['price'].'</li>';
+        $html .= '<li><b>'.$plugin->get_lang('CurrencyType').':</b> '.$currency['iso_code'].'</li>';
+        $html .= '<li><b>'.$plugin->get_lang('ProductType').':</b> '.$productType.'</li>';
+        $html .= '<li><b>'.$plugin->get_lang('OrderDate').':</b> '.api_format_date($sale['date'], DATE_TIME_FORMAT_LONG_24H).'</li>';
+        $html .= '<li><b>'.$plugin->get_lang('Buyer').':</b> '.$userInfo['complete_name'].'</li>';
+        $html .= '<li><b>'.$plugin->get_lang('PaymentMethods').':</b> '.$paymentType.'</li>';
         $html .= '</ul>';
         $html .= '</div>';
         $html .= '<div class="col-sm-6 col-md-6">';
-        $html .= '<img class="thumbnail" src="'. $productImage .'" >';
+        $html .= '<img class="thumbnail" src="'.$productImage.'" >';
         $html .= '</div>';
         $html .= '</div>';
 
@@ -119,9 +119,9 @@ 
         $html = '<div class="row">'
         . '<p>'
             . '<ul>'
-                . '<li>'. get_plugin_lang("PayoutsTotalCompleted", "BuyCoursesPlugin") .' <b>'. $stats['completed_count'] .'</b> - '. get_plugin_lang("TotalAmount", "BuyCoursesPlugin") .' <b>'. $stats['completed_total_amount'] .' '. $currency['iso_code'] . '</b></li>'
-                . '<li>'. get_plugin_lang("PayoutsTotalPending", "BuyCoursesPlugin") .' <b>'. $stats['pending_count'] .'</b> - '. get_plugin_lang("TotalAmount", "BuyCoursesPlugin") .' <b>'. $stats['pending_total_amount'] .' '. $currency['iso_code'] . '</b></li>'
-                . '<li>'. get_plugin_lang("PayoutsTotalCanceled", "BuyCoursesPlugin") .' <b>'. $stats['canceled_count'] .'</b> - '. get_plugin_lang("TotalAmount", "BuyCoursesPlugin") .' <b>'. $stats['canceled_total_amount'] .' '. $currency['iso_code'] . '</b></li>'
+                . '<li>'.get_plugin_lang("PayoutsTotalCompleted", "BuyCoursesPlugin").' <b>'.$stats['completed_count'].'</b> - '.get_plugin_lang("TotalAmount", "BuyCoursesPlugin").' <b>'.$stats['completed_total_amount'].' '.$currency['iso_code'].'</b></li>'
+                . '<li>'.get_plugin_lang("PayoutsTotalPending", "BuyCoursesPlugin").' <b>'.$stats['pending_count'].'</b> - '.get_plugin_lang("TotalAmount", "BuyCoursesPlugin").' <b>'.$stats['pending_total_amount'].' '.$currency['iso_code'].'</b></li>'
+                . '<li>'.get_plugin_lang("PayoutsTotalCanceled", "BuyCoursesPlugin").' <b>'.$stats['canceled_count'].'</b> - '.get_plugin_lang("TotalAmount", "BuyCoursesPlugin").' <b>'.$stats['canceled_total_amount'].' '.$currency['iso_code'].'</b></li>'
             . '</ul>'
         . '</p>';
         $html .= '</div>';
@@ -161,15 +161,15 @@ 
 
         $isoCode = $currentCurrency['iso_code'];
 
-        $html .= '<p>'. get_plugin_lang("VerifyTotalAmountToProceedPayout", "BuyCoursesPlugin") .'</p>';
+        $html .= '<p>'.get_plugin_lang("VerifyTotalAmountToProceedPayout", "BuyCoursesPlugin").'</p>';
         $html .= ''
         . '<p>'
             . '<ul>'
-                . '<li>'. get_plugin_lang("TotalAcounts", "BuyCoursesPlugin") .' <b>'. $totalAccounts .'</b></li>'
-                . '<li>'. get_plugin_lang("TotalPayout", "BuyCoursesPlugin") .' <b>'. $isoCode .' '. $totalPayout .'</b></li>'
+                . '<li>'.get_plugin_lang("TotalAcounts", "BuyCoursesPlugin").' <b>'.$totalAccounts.'</b></li>'
+                . '<li>'.get_plugin_lang("TotalPayout", "BuyCoursesPlugin").' <b>'.$isoCode.' '.$totalPayout.'</b></li>'
             . '</ul>'
         . '</p>';
-        $html .= '<p>'. get_plugin_lang("CautionThisProcessCantBeCanceled", "BuyCoursesPlugin") .'</p>';
+        $html .= '<p>'.get_plugin_lang("CautionThisProcessCantBeCanceled", "BuyCoursesPlugin").'</p>';
         $html .= '<br /><br />';
         $html .= '<div id="spinner" class="text-center"></div>';
 
@@ -223,7 +223,7 @@ 
 
         } else {
 
-            echo Display::return_message('<b>'.$result['L_SEVERITYCODE0'].' '.$result['L_ERRORCODE0'].'</b> - '.$result['L_SHORTMESSAGE0'].'<br /><ul><li>'. $result['L_LONGMESSAGE0'].'</li></ul>', 'error', false);
+            echo Display::return_message('<b>'.$result['L_SEVERITYCODE0'].' '.$result['L_ERRORCODE0'].'</b> - '.$result['L_SHORTMESSAGE0'].'<br /><ul><li>'.$result['L_LONGMESSAGE0'].'</li></ul>', 'error', false);
 
         }
 
@@ -312,7 +312,7 @@ 
 
             echo json_encode($cargo);
 
-        } catch(Exception $e) {
+        } catch (Exception $e) {
 
             $cargo = json_decode($e->getMessage(), true);
 
@@ -411,7 +411,7 @@ 
 
             echo json_encode($cargo);
 
-        } catch(Exception $e) {
+        } catch (Exception $e) {
 
             $cargo = json_decode($e->getMessage(), true);
 
@@ -451,9 +451,9 @@ 
             break;
         }
 
-        $ajaxCallFile = $plugin->getPath('SRC') . 'buycourses.ajax.php';
+        $ajaxCallFile = $plugin->getPath('SRC').'buycourses.ajax.php';
 
-        $serviceImg = $plugin->getPath('SERVICE_IMAGES') . $serviceSale['service']['image'];
+        $serviceImg = $plugin->getPath('SERVICE_IMAGES').$serviceSale['service']['image'];
         $html = "<img class='img-responsive text-center' src='$serviceImg'>";
         $html .= "<br />";
         $html .= "<legend>{$plugin->get_lang('ServiceInformation')}</legend>";
@@ -473,7 +473,7 @@ 
         }
         $html .= "<li><b>{$plugin->get_lang('AppliesTo')}:</b> $nodeType</li> ";
         $html .= "<li><b>{$plugin->get_lang('Price')}:</b> {$serviceSale['service']['price']} {$serviceSale['currency']}</li> ";
-        $duration = $serviceSale['service']['duration_days'] . ' ' . $plugin->get_lang('Days');
+        $duration = $serviceSale['service']['duration_days'].' '.$plugin->get_lang('Days');
         $html .= "</ul>";
         $html .= "<legend>{$plugin->get_lang('SaleInfo')}</legend>";
         $html .= "<ul>";
@@ -555,10 +555,10 @@ 
                 'success'
             );
         } else {
-            $html .= Display::return_message('Error - ' . $plugin->get_lang('ErrorContactPlatformAdmin'), 'error');
+            $html .= Display::return_message('Error - '.$plugin->get_lang('ErrorContactPlatformAdmin'), 'error');
         }
 
-        $html .= "<a id='finish-button' class='btn btn-primary'>" . $plugin->get_lang('ClickHereToFinish') . "</a>";
+        $html .= "<a id='finish-button' class='btn btn-primary'>".$plugin->get_lang('ClickHereToFinish')."</a>";
         $html .= "</div>";
         $html .= "<script>";
         $html .= "$('#finish-button').click(function() {";
@@ -584,10 +584,10 @@ 
                 'warning'
             );
         } else {
-            $html .= Display::return_message('Error - ' . $plugin->get_lang('ErrorContactPlatformAdmin'), 'error');
+            $html .= Display::return_message('Error - '.$plugin->get_lang('ErrorContactPlatformAdmin'), 'error');
         }
 
-        $html .= "<a id='finish-button' class='btn btn-primary'>" . $plugin->get_lang('ClickHereToFinish') . "</a>";
+        $html .= "<a id='finish-button' class='btn btn-primary'>".$plugin->get_lang('ClickHereToFinish')."</a>";
         $html .= "</div>";
         $html .= "<script>";
         $html .= "$('#finish-button').click(function() {";

main/inc/lib/agenda.lib.php

Spacing +2 added lines, -2 removed lines

@@ -3151,12 +3151,12 @@
                     $end->format('e')
                 );
                 $title = api_convert_encoding(
-                    (string)$event->summary,
+                    (string) $event->summary,
                     $charset,
                     'UTF-8'
                 );
                 $description = api_convert_encoding(
-                    (string)$event->description,
+                    (string) $event->description,
                     $charset,
                     'UTF-8'
                 );

src/Chamilo/CoreBundle/Component/Utils/ChamiloApi.php

Spacing +9 added lines, -9 removed lines

@@ -73,14 +73,14 @@ 
         $accessUrlId = api_get_current_access_url_id();
         $customLogoPath = "themes/$theme/images/header-logo-custom$accessUrlId.png";
 
-        if (file_exists(api_get_path(SYS_PUBLIC_PATH) . "css/$customLogoPath")) {
-            return api_get_path(WEB_CSS_PATH) . $customLogoPath;
+        if (file_exists(api_get_path(SYS_PUBLIC_PATH)."css/$customLogoPath")) {
+            return api_get_path(WEB_CSS_PATH).$customLogoPath;
         }
 
         $originalLogoPath = "themes/$theme/images/header-logo.png";
 
-        if (file_exists(api_get_path(SYS_CSS_PATH) . $originalLogoPath)) {
-            return api_get_path(WEB_CSS_PATH) . $originalLogoPath;
+        if (file_exists(api_get_path(SYS_CSS_PATH).$originalLogoPath)) {
+            return api_get_path(WEB_CSS_PATH).$originalLogoPath;
         }
 
         return null;
@@ -100,10 +100,10 @@ 
         $siteName = api_get_setting('siteName');
 
         if ($logoPath === null) {
-            $headerLogo = \Display::url($siteName, api_get_path(WEB_PATH) . 'index.php');
+            $headerLogo = \Display::url($siteName, api_get_path(WEB_PATH).'index.php');
 
             if (!empty($institutionUrl) && !empty($institution)) {
-                $headerLogo .= ' - ' . \Display::url($institution, $institutionUrl);
+                $headerLogo .= ' - '.\Display::url($institution, $institutionUrl);
             }
 
             $courseInfo = api_get_course_info();
@@ -127,7 +127,7 @@ 
 
         $image = \Display::img($logoPath, $institution, $imageAttributes);
 
-        return \Display::url($image, api_get_path(WEB_PATH) . 'index.php');
+        return \Display::url($image, api_get_path(WEB_PATH).'index.php');
     }
 
     /**
@@ -139,9 +139,9 @@ 
     public static function stripGivenTags($string, $tags)
     {
         foreach ($tags as $tag) {
-            $string2 = preg_replace('#</' . $tag . '[^>]*>#i', ' ', $string);
+            $string2 = preg_replace('#</'.$tag.'[^>]*>#i', ' ', $string);
             if ($string2 != $string) {
-                $string = preg_replace('/<' . $tag . '[^>]*>/i', ' ', $string2);
+                $string = preg_replace('/<'.$tag.'[^>]*>/i', ' ', $string2);
             }
         }
         return $string;

main/inc/lib/system_announcements.lib.php

Spacing +14 added lines, -14 removed lines

@@ -22,14 +22,14 @@ 
         $tbl_announcement_group = Database :: get_main_table(TABLE_MAIN_SYSTEM_ANNOUNCEMENTS_GROUPS);
         $userGroup = new UserGroup();
 
-        $temp_user_groups = $userGroup->get_groups_by_user(api_get_user_id(),0);
+        $temp_user_groups = $userGroup->get_groups_by_user(api_get_user_id(), 0);
         $groups = array();
         foreach ($temp_user_groups as $user_group) {
             $groups = array_merge($groups, array($user_group['id']));
             $groups = array_merge($groups, $userGroup->get_parent_groups($user_group['id']));
         }
 
-        $groups_string = '('.implode($groups,',').')';
+        $groups_string = '('.implode($groups, ',').')';
         $now = api_get_utc_datetime();
         $sql = "SELECT *, DATE_FORMAT(date_start,'%d-%m-%Y %h:%i:%s') AS display_date
                 FROM  $db_table
@@ -104,13 +104,13 @@ 
      * @param string $user_id
      * @return string
      */
-    public static function display_all_announcements($visible, $id = -1, $start = 0,$user_id='')
+    public static function display_all_announcements($visible, $id = -1, $start = 0, $user_id = '')
     {
         $user_selected_language = api_get_interface_language();
         $start = intval($start);
         $userGroup = new UserGroup();
         $tbl_announcement_group = Database :: get_main_table(TABLE_MAIN_SYSTEM_ANNOUNCEMENTS_GROUPS);
-        $temp_user_groups = $userGroup->get_groups_by_user(api_get_user_id(),0);
+        $temp_user_groups = $userGroup->get_groups_by_user(api_get_user_id(), 0);
         $groups = array();
         foreach ($temp_user_groups as $user_group) {
             $groups = array_merge($groups, array($user_group['id']));
@@ -118,10 +118,10 @@ 
         }
 
         // Checks if tables exists to not break platform not updated
-        $groups_string = '('.implode($groups,',').')';
+        $groups_string = '('.implode($groups, ',').')';
 
         $db_table = Database :: get_main_table(TABLE_MAIN_SYSTEM_ANNOUNCEMENTS);
-        $now  = api_get_utc_datetime();
+        $now = api_get_utc_datetime();
 
         $sql = "SELECT * FROM ".$db_table."
                 WHERE
@@ -155,7 +155,7 @@ 
         if (!isset($_GET['start']) || $_GET['start'] == 0) {
             $sql .= " ORDER BY date_start DESC LIMIT ".$start.",20";
         } else {
-            $sql .= " ORDER BY date_start DESC LIMIT ".($start+1).",20";
+            $sql .= " ORDER BY date_start DESC LIMIT ".($start + 1).",20";
         }
         $announcements = Database::query($sql);
         $content = '';
@@ -204,10 +204,10 @@ 
      */
     public static function display_arrow($user_id)
     {
-        $start = (int)$_GET['start'];
+        $start = (int) $_GET['start'];
         $nb_announcement = SystemAnnouncementManager :: count_nb_announcement($start, $user_id);
-        $next = ((int)$_GET['start']+19);
-        $prev = ((int)$_GET['start']-19);
+        $next = ((int) $_GET['start'] + 19);
+        $prev = ((int) $_GET['start'] - 19);
         $content = '';
         if (!isset($_GET['start']) || $_GET['start'] == 0) {
             if ($nb_announcement > 20) {
@@ -567,7 +567,7 @@ 
                 $sendEmailTest
             );
         } else {
-            if ($send_mail==1) {
+            if ($send_mail == 1) {
                 SystemAnnouncementManager::send_system_announcement_by_email(
                     $title,
                     $content,
@@ -678,7 +678,7 @@ 
         $language = null,
         $sendEmailTest = false
     ) {
-        $content = str_replace(array('\r\n', '\n', '\r'),'', $content);
+        $content = str_replace(array('\r\n', '\n', '\r'), '', $content);
         $now = api_get_utc_datetime();
 
         if ($sendEmailTest) {
@@ -704,7 +704,7 @@ 
 					WHERE status = '5' ";
 		}
 
-		if ($teacher<> 0 && $student <> 0) {
+		if ($teacher <> 0 && $student <> 0) {
 			$sql = "SELECT DISTINCT u.user_id FROM $user_table u $url_condition 
 					WHERE 1 = 1 ";
 		}
@@ -737,7 +737,7 @@ 
 
         $message_sent = false;
 
-		while ($row = Database::fetch_array($result,'ASSOC')) {
+		while ($row = Database::fetch_array($result, 'ASSOC')) {
             MessageManager::send_message_simple($row['user_id'], $title, $content);
             $message_sent = true;
 		}

main/mySpace/myStudents.php

Spacing +66 added lines, -66 removed lines

@@ -87,52 +87,52 @@ 
 if (isset($_GET['details'])) {
     if ($origin === 'user_course') {
         if (empty($cidReq)) {
-            $interbreadcrumb[] = array (
-                "url" => api_get_path(WEB_COURSE_PATH) . $courseInfo['directory'],
+            $interbreadcrumb[] = array(
+                "url" => api_get_path(WEB_COURSE_PATH).$courseInfo['directory'],
                 'name' => $courseInfo['title']
             );
         }
-        $interbreadcrumb[] = array (
-            "url" => "../user/user.php?cidReq=" . $course_code,
+        $interbreadcrumb[] = array(
+            "url" => "../user/user.php?cidReq=".$course_code,
             "name" => get_lang("Users")
         );
     } else {
         if ($origin === 'tracking_course') {
-            $interbreadcrumb[] = array (
-                "url" => "../tracking/courseLog.php?cidReq=".$course_code.'&id_session=' . api_get_session_id(),
+            $interbreadcrumb[] = array(
+                "url" => "../tracking/courseLog.php?cidReq=".$course_code.'&id_session='.api_get_session_id(),
                 "name" => get_lang("Tracking")
             );
         } else {
             if ($origin === 'resume_session') {
-                $interbreadcrumb[] = array (
+                $interbreadcrumb[] = array(
                     'url' => "../session/session_list.php",
                     "name" => get_lang('SessionList')
                 );
-                $interbreadcrumb[] = array (
-                    'url' => "../session/resume_session.php?id_session=" . $sessionId,
+                $interbreadcrumb[] = array(
+                    'url' => "../session/resume_session.php?id_session=".$sessionId,
                     "name" => get_lang('SessionOverview')
                 );
             } else {
-                $interbreadcrumb[] = array (
-                    "url" => api_is_student_boss()?"#":"index.php",
+                $interbreadcrumb[] = array(
+                    "url" => api_is_student_boss() ? "#" : "index.php",
                     "name" => get_lang('MySpace')
                 );
                 if (isset($_GET['id_coach']) && intval($_GET['id_coach']) != 0) {
-                    $interbreadcrumb[] = array (
-                        "url" => "student.php?id_coach=" . Security :: remove_XSS($_GET['id_coach']),
+                    $interbreadcrumb[] = array(
+                        "url" => "student.php?id_coach=".Security :: remove_XSS($_GET['id_coach']),
                         "name" => get_lang("CoachStudents")
                     );
-                    $interbreadcrumb[] = array (
-                        "url" => "myStudents.php?student=" . $student_id. '&id_coach=' . Security :: remove_XSS($_GET['id_coach']),
+                    $interbreadcrumb[] = array(
+                        "url" => "myStudents.php?student=".$student_id.'&id_coach='.Security :: remove_XSS($_GET['id_coach']),
                         "name" => get_lang("StudentDetails")
                     );
                 } else {
-                    $interbreadcrumb[] = array (
+                    $interbreadcrumb[] = array(
                         "url" => "student.php",
                         "name" => get_lang("MyStudents")
                     );
-                    $interbreadcrumb[] = array (
-                        "url" => "myStudents.php?student=" . $student_id,
+                    $interbreadcrumb[] = array(
+                        "url" => "myStudents.php?student=".$student_id,
                         "name" => get_lang("StudentDetails")
                     );
                 }
@@ -142,7 +142,7 @@ 
     $nameTools = get_lang("DetailsStudentInCourse");
 } else {
     if ($origin == 'resume_session') {
-        $interbreadcrumb[] = array (
+        $interbreadcrumb[] = array(
             'url' => "../session/session_list.php",
             "name" => get_lang('SessionList')
         );
@@ -153,19 +153,19 @@ 
             );
         }
     } else {
-        $interbreadcrumb[] = array (
-            "url" => api_is_student_boss()?"#":"index.php",
+        $interbreadcrumb[] = array(
+            "url" => api_is_student_boss() ? "#" : "index.php",
             "name" => get_lang('MySpace')
         );
         if (isset($_GET['id_coach']) && intval($_GET['id_coach']) != 0) {
             if ($sessionId) {
                 $interbreadcrumb[] = array(
-                    "url" => "student.php?id_coach=" . Security :: remove_XSS($_GET['id_coach']) . "&id_session=" . $sessionId,
+                    "url" => "student.php?id_coach=".Security :: remove_XSS($_GET['id_coach'])."&id_session=".$sessionId,
                     "name" => get_lang("CoachStudents")
                 );
             } else {
                 $interbreadcrumb[] = array(
-                    "url" => "student.php?id_coach=" . Security :: remove_XSS($_GET['id_coach']),
+                    "url" => "student.php?id_coach=".Security :: remove_XSS($_GET['id_coach']),
                     "name" => get_lang("CoachStudents")
                 );
             }
@@ -350,28 +350,28 @@ 
     // Actions bar
     echo '<div class="actions">';
     echo '<a href="javascript: window.history.go(-1);">'.
-            Display::return_icon('back.png', get_lang('Back'),'',ICON_SIZE_MEDIUM).'</a>';
+            Display::return_icon('back.png', get_lang('Back'), '', ICON_SIZE_MEDIUM).'</a>';
 
     echo '<a href="javascript: void(0);" onclick="javascript: window.print();">'.
-            Display::return_icon('printer.png', get_lang('Print'),'',ICON_SIZE_MEDIUM).'</a>';
+            Display::return_icon('printer.png', get_lang('Print'), '', ICON_SIZE_MEDIUM).'</a>';
 
-    echo '<a href="' . api_get_self() . '?' . Security :: remove_XSS($_SERVER['QUERY_STRING']) . '&export=csv">'.
-            Display::return_icon('export_csv.png', get_lang('ExportAsCSV'),'',ICON_SIZE_MEDIUM).'</a> ';
+    echo '<a href="'.api_get_self().'?'.Security :: remove_XSS($_SERVER['QUERY_STRING']).'&export=csv">'.
+            Display::return_icon('export_csv.png', get_lang('ExportAsCSV'), '', ICON_SIZE_MEDIUM).'</a> ';
 
-    echo '<a href="' . api_get_self() . '?' . Security :: remove_XSS($_SERVER['QUERY_STRING']) . '&export=xls">'.
-    Display::return_icon('export_excel.png', get_lang('ExportAsXLS'),'',ICON_SIZE_MEDIUM).'</a> ';
+    echo '<a href="'.api_get_self().'?'.Security :: remove_XSS($_SERVER['QUERY_STRING']).'&export=xls">'.
+    Display::return_icon('export_excel.png', get_lang('ExportAsXLS'), '', ICON_SIZE_MEDIUM).'</a> ';
 
     if (!empty ($user_info['email'])) {
         $send_mail = '<a href="mailto:'.$user_info['email'].'">'.
-            Display :: return_icon('mail_send.png', get_lang('SendMail'),'',ICON_SIZE_MEDIUM).'</a>';
+            Display :: return_icon('mail_send.png', get_lang('SendMail'), '', ICON_SIZE_MEDIUM).'</a>';
     } else {
-        $send_mail = Display :: return_icon('mail_send_na.png', get_lang('SendMail'),'',ICON_SIZE_MEDIUM);
+        $send_mail = Display :: return_icon('mail_send_na.png', get_lang('SendMail'), '', ICON_SIZE_MEDIUM);
     }
     echo $send_mail;
     if (!empty($student_id) && !empty($course_code)) {
         // Only show link to connection details if course and student were defined in the URL
-        echo '<a href="access_details.php?student=' . $student_id . '&course=' . $course_code . '&origin=' . $origin. '&cidReq='.$course_code.'&id_session='.$sessionId.'">'.
-            Display :: return_icon('statistics.png', get_lang('AccessDetails'),'',ICON_SIZE_MEDIUM).'</a>';
+        echo '<a href="access_details.php?student='.$student_id.'&course='.$course_code.'&origin='.$origin.'&cidReq='.$course_code.'&id_session='.$sessionId.'">'.
+            Display :: return_icon('statistics.png', get_lang('AccessDetails'), '', ICON_SIZE_MEDIUM).'</a>';
     }
     if (api_can_login_as($student_id)) {
         echo '<a href="'.api_get_path(WEB_CODE_PATH).'admin/user_list.php?action=login_as&user_id='.$student_id.'&sec_token='.$token.'">'.
@@ -381,7 +381,7 @@ 
     if (api_is_platform_admin(false, true) || api_is_student_boss()) {
         echo Display::url(
             Display::return_icon('skill-badges.png', get_lang('AssignSkill'), null, ICON_SIZE_MEDIUM),
-            api_get_path(WEB_CODE_PATH) . 'badge/assign.php?' . http_build_query(['user' => $student_id])
+            api_get_path(WEB_CODE_PATH).'badge/assign.php?'.http_build_query(['user' => $student_id])
         );
     }
 
@@ -442,7 +442,7 @@ 
     $csv_content[] = array(
         get_lang('Information')
     );
-    $csv_content[] = array (
+    $csv_content[] = array(
         get_lang('Name'),
         get_lang('Email'),
         get_lang('Tel')
@@ -470,7 +470,7 @@ 
         strip_tags($first_connection_date),
         strip_tags($last_connection_date),
         $time_spent_on_the_course,
-        $avg_student_progress . '%',
+        $avg_student_progress.'%',
         $avg_student_score
     );
 
@@ -500,19 +500,19 @@ 
             </thead>
             <tbody>
             <tr>
-                <td><?php echo get_lang('Name') . ' : '.$user_info['complete_name']; ?></td>
+                <td><?php echo get_lang('Name').' : '.$user_info['complete_name']; ?></td>
             </tr>
             <tr>
-                <td><?php echo get_lang('Email') . ' : ';
+                <td><?php echo get_lang('Email').' : ';
                     if (!empty ($user_info['email'])) {
-                        echo '<a href="mailto:' . $user_info['email'] . '">' . $user_info['email'] . '</a>';
+                        echo '<a href="mailto:'.$user_info['email'].'">'.$user_info['email'].'</a>';
                     } else {
                         echo get_lang('NoEmail');
                     } ?>
                 </td>
             </tr>
             <tr>
-                <td> <?php echo get_lang('Tel') . ' : ';
+                <td> <?php echo get_lang('Tel').' : ';
                     if (!empty ($user_info['phone'])) {
                         echo $user_info['phone'];
                     } else {
@@ -522,7 +522,7 @@ 
                 </td>
             </tr>
             <tr>
-                <td> <?php echo get_lang('OfficialCode') . ' : ';
+                <td> <?php echo get_lang('OfficialCode').' : ';
                     if (!empty ($user_info['official_code'])) {
                         echo $user_info['official_code'];
                     } else {
@@ -532,7 +532,7 @@ 
                 </td>
             </tr>
             <tr>
-                <td><?php echo get_lang('OnLine') . ' : '.$online; ?> </td>
+                <td><?php echo get_lang('OnLine').' : '.$online; ?> </td>
             </tr>
             <?php
 
@@ -546,7 +546,7 @@ 
             if ($timezone !== null) {
                 ?>
                 <tr>
-                    <td> <?php echo get_lang('Timezone') . ' : '.$timezone; ?> </td>
+                    <td> <?php echo get_lang('Timezone').' : '.$timezone; ?> </td>
                 </tr>
             <?php
             }
@@ -693,9 +693,9 @@ 
                 }
 
                 if (!empty($access_start_date) && !empty($access_end_date)) {
-                    $date_session = get_lang('From') . ' ' . $access_start_date . ' ' . get_lang('Until') . ' ' . $access_end_date;
+                    $date_session = get_lang('From').' '.$access_start_date.' '.get_lang('Until').' '.$access_end_date;
                 }
-                $title = Display::return_icon('session.png', get_lang('Session'), array(), ICON_SIZE_SMALL).' '.$session_name.($date_session?' ('.$date_session.')':'');
+                $title = Display::return_icon('session.png', get_lang('Session'), array(), ICON_SIZE_SMALL).' '.$session_name.($date_session ? ' ('.$date_session.')' : '');
             }
 
             // Courses
@@ -757,16 +757,16 @@ 
                         $scoretotal = array();
                         if (isset($cats) && isset($cats[0])) {
                             if (!empty($sId)) {
-                                $scoretotal= $cats[0]->calc_score($student_id, null, $courseCodeItem, $sId);
+                                $scoretotal = $cats[0]->calc_score($student_id, null, $courseCodeItem, $sId);
                             } else {
-                                $scoretotal= $cats[0]->calc_score($student_id, null, $courseCodeItem);
+                                $scoretotal = $cats[0]->calc_score($student_id, null, $courseCodeItem);
                             }
                         }
 
                         $scoretotal_display = '0/0 (0%)';
                         if (!empty($scoretotal) && !empty($scoretotal[1])) {
                             $scoretotal_display =
-                                round($scoretotal[0], 1 ).'/'.
+                                round($scoretotal[0], 1).'/'.
                                 round($scoretotal[1], 1).
                                 ' ('.round(($scoretotal[0] / $scoretotal[1]) * 100, 2).' %)';
                         }
@@ -797,9 +797,9 @@ 
                         );
 
                         echo '<tr>
-                        <td ><a href="' .$courseInfoItem['course_public_url'] .'?id_session=' . $sId . '">'.
+                        <td ><a href="' .$courseInfoItem['course_public_url'].'?id_session='.$sId.'">'.
                             $courseInfoItem['title'].'</a></td>
-                        <td >'.$time_spent_on_course .'</td>
+                        <td >'.$time_spent_on_course.'</td>
                         <td >'.$progress.'</td>
                         <td >'.$score.'</td>
                         <td >'.$attendances_faults_avg.'</td>
@@ -875,7 +875,7 @@ 
                 <table class="table table-striped table-hover">
                 <thead>
                 <tr>
-                    <th><?php echo get_lang('LearningPath');?></th>
+                    <th><?php echo get_lang('LearningPath'); ?></th>
                     <th>
                         <?php
                         echo get_lang('Time').' ';
@@ -978,7 +978,7 @@ 
                     if (!empty($start_time)) {
                         $start_time = api_convert_and_format_date($start_time, DATE_TIME_FORMAT_LONG);
                     } else {
-                        $start_time =  '-';
+                        $start_time = '-';
                     }
 
                     if (!empty($total_time)) {
@@ -1015,8 +1015,8 @@ 
                     $csv_content[] = array(
                         api_html_entity_decode(stripslashes($lp_name), ENT_QUOTES, $charset),
                         api_time_to_hms($total_time),
-                        $score . '%',
-                        $score_latest . '%',
+                        $score.'%',
+                        $score_latest.'%',
                         $progress.'%',
                         $start_time
                     );
@@ -1054,7 +1054,7 @@ 
                     if ($any_result === true) {
                         $from = '';
                         if ($from_myspace) {
-                            $from ='&from=myspace';
+                            $from = '&from=myspace';
                         }
                         $link = Display::url(
                             Display::return_icon('2rightarrow.png', get_lang('Details')),
@@ -1067,14 +1067,14 @@ 
                         echo '<td>';
                         if ($any_result === true) {
                             echo '<a href="myStudents.php?action=reset_lp&sec_token='.$token.'&cidReq='.$course_code.'&course='.$course_code.'&details='.Security::remove_XSS($_GET['details']).'&origin='.$origin.'&lp_id='.$learnpath->getId().'&student='.$user_info['user_id'].'&details=true&id_session='.$sessionId.'">';
-                            echo Display::return_icon('clean.png', get_lang('Clean'),'',ICON_SIZE_SMALL).'</a>';
+                            echo Display::return_icon('clean.png', get_lang('Clean'), '', ICON_SIZE_SMALL).'</a>';
                             echo '</a>';
                         }
                         echo '</td>';
                         echo '</tr>';
                     }
                     $data_learnpath[$i][] = $lp_name;
-                    $data_learnpath[$i][] = $progress . '%';
+                    $data_learnpath[$i][] = $progress.'%';
                 }
                 ?>
                 </tbody>
@@ -1090,7 +1090,7 @@ 
         <thead>
         <tr>
             <th><?php echo get_lang('Exercises'); ?></th>
-            <th><?php echo get_lang('LearningPath');?></th>
+            <th><?php echo get_lang('LearningPath'); ?></th>
             <th><?php echo get_lang('AvgCourseScore').' '.Display :: return_icon('info3.gif', get_lang('AverageScore'), array('align' => 'absmiddle', 'hspace' => '3px')) ?></th>
             <th><?php echo get_lang('Attempts'); ?></th>
             <th><?php echo get_lang('LatestAttempt'); ?></th>
@@ -1116,7 +1116,7 @@ 
             'quiz.session_id'
         );
 
-        $sql = "SELECT quiz.title, id FROM " . $t_quiz . " AS quiz
+        $sql = "SELECT quiz.title, id FROM ".$t_quiz." AS quiz
                 WHERE
                     quiz.c_id = ".$courseInfo['real_id']." AND
                     active IN (0, 1)
@@ -1181,7 +1181,7 @@ 
                 echo '<td>';
 
                 if ($count_attempts > 0) {
-                    echo $score_percentage . '%';
+                    echo $score_percentage.'%';
                 } else {
                     echo '-';
                     $score_percentage = 0;
@@ -1191,7 +1191,7 @@ 
                 echo '<td>'.$count_attempts.'</td>';
                 echo '<td>';
 
-                $sql = 'SELECT exe_id FROM ' . $tbl_stats_exercices . '
+                $sql = 'SELECT exe_id FROM '.$tbl_stats_exercices.'
                          WHERE
                             exe_exo_id = "'.$exercise_id.'" AND
                             exe_user_id ="'.$student_id.'" AND
@@ -1204,7 +1204,7 @@ 
                 if (Database :: num_rows($result_last_attempt) > 0) {
                     $id_last_attempt = Database :: result($result_last_attempt, 0, 0);
                     if ($count_attempts > 0)
-                        echo '<a href="../exercise/exercise_show.php?id=' . $id_last_attempt . '&cidReq='.$course_code.'&session_id='.$sessionId.'&student='.$student_id.'&origin='.(empty($origin)?'tracking':$origin).'">
+                        echo '<a href="../exercise/exercise_show.php?id='.$id_last_attempt.'&cidReq='.$course_code.'&session_id='.$sessionId.'&student='.$student_id.'&origin='.(empty($origin) ? 'tracking' : $origin).'">
                         '.Display::return_icon('quiz.png').'
                      </a>';
                 }
@@ -1212,11 +1212,11 @@ 
 
                 echo '<td>';
                 $all_attempt_url = "../exercise/exercise_report.php?exerciseId=$exercise_id&cidReq=$course_code&filter_by_user=$student_id&id_session=$sessionId";
-                echo Display::url(Display::return_icon('test_results.png', get_lang('AllAttempts'), array(), ICON_SIZE_SMALL), $all_attempt_url );
+                echo Display::url(Display::return_icon('test_results.png', get_lang('AllAttempts'), array(), ICON_SIZE_SMALL), $all_attempt_url);
 
                 echo '</td></tr>';
                 $data_exercices[$i][] = $exercices['title'];
-                $data_exercices[$i][] = $score_percentage . '%';
+                $data_exercices[$i][] = $score_percentage.'%';
                 $data_exercices[$i][] = $count_attempts;
 
                 $csv_content[] = array(
@@ -1244,7 +1244,7 @@ 
             $survey_list = SurveyManager::get_surveys($course_code, $sessionId);
 
             $survey_data = array();
-            foreach($survey_list as $survey) {
+            foreach ($survey_list as $survey) {
                 $user_list = SurveyManager::get_people_who_filled_survey($survey['survey_id'], false, $courseInfo['real_id']);
                 $survey_done = Display::return_icon("accept_na.png", get_lang('NoAnswer'), array(), ICON_SIZE_SMALL);
                 if (in_array($student_id, $user_list)) {
@@ -1269,7 +1269,7 @@ 
                         $column = 0;
                         $table->setCellContents($row, $column, $data);
                         $class = 'class="row_odd"';
-                        if($row % 2) {
+                        if ($row % 2) {
                             $class = 'class="row_even"';
                         }
                         $table->setRowAttributes($row, $class, true);