chamilo / chamilo-lms

main/inc/lib/text.lib.php

Braces +31 added lines, -20 removed lines

@@ -351,8 +351,9 @@ 
     }
 
     $url = esc_url($url);
-    if ( empty($url) )
-        return $matches[0];
+    if ( empty($url) ) {
+            return $matches[0];
+    }
 
     return $matches[1] . "<a href=\"$url\" rel=\"nofollow\">$url</a>" . $suffix;
 }
@@ -377,8 +378,9 @@ 
 function esc_url( $url, $protocols = null, $_context = 'display' ) {
     //$original_url = $url;
 
-    if ( '' == $url )
-        return $url;
+    if ( '' == $url ) {
+            return $url;
+    }
     $url = preg_replace('|[^a-z0-9-~+_.?#=!&;,/:%@$\|*\'()\\x80-\\xff]|i', '', $url);
     $strip = array('%0d', '%0a', '%0D', '%0A');
     $url = _deep_replace($strip, $url);
@@ -388,8 +390,9 @@ 
      * link starting with /, # or ? or a php file).
      */
     if ( strpos($url, ':') === false && ! in_array( $url[0], array( '/', '#', '?' ) ) &&
-        ! preg_match('/^[a-z0-9-]+?\.php/i', $url) )
-        $url = 'http://' . $url;
+        ! preg_match('/^[a-z0-9-]+?\.php/i', $url) ) {
+            $url = 'http://' . $url;
+    }
 
     return Security::remove_XSS($url);
 
@@ -466,8 +469,9 @@ 
     $dest = $matches[2];
     $dest = 'http://' . $dest;
     $dest = esc_url($dest);
-    if ( empty($dest) )
-        return $matches[0];
+    if ( empty($dest) ) {
+            return $matches[0];
+    }
 
     // removed trailing [.,;:)] from URL
     if ( in_array( substr($dest, -1), array('.', ',', ';', ':', ')') ) === true ) {
@@ -511,10 +515,11 @@ 
     $nested_code_pre = 0; // Keep track of how many levels link is nested inside <pre> or <code>
     foreach ( $textarr as $piece ) {
 
-        if ( preg_match( '|^<code[\s>]|i', $piece ) || preg_match( '|^<pre[\s>]|i', $piece ) )
-            $nested_code_pre++;
-        elseif ( ( '</code>' === strtolower( $piece ) || '</pre>' === strtolower( $piece ) ) && $nested_code_pre )
-            $nested_code_pre--;
+        if ( preg_match( '|^<code[\s>]|i', $piece ) || preg_match( '|^<pre[\s>]|i', $piece ) ) {
+                    $nested_code_pre++;
+        } elseif ( ( '</code>' === strtolower( $piece ) || '</pre>' === strtolower( $piece ) ) && $nested_code_pre ) {
+                    $nested_code_pre--;
+        }
 
         if ( $nested_code_pre || empty( $piece ) || ( $piece[0] === '<' && ! preg_match( '|^<\s*[\w]{1,20}+://|', $piece ) ) ) {
             $r .= $piece;
@@ -723,15 +728,13 @@ 
 			if($length>$maxchar)
 			{
 				break;
-			}
-			else
+			} else
 			{
 				$output = $output." ".$words[$i];
 				$i++;
 			};
 		};
-	}
-	else
+	} else
 	{
 		$output = $text;
 		return $output;
@@ -782,13 +785,21 @@ 
     if (isset($array['Y']) && (isset($array['F']) || isset($array['M']))  && isset($array['d']) && isset($array['H']) && isset($array['i'])) {
         $year = $array['Y'];
         $month = isset($array['F'])?$array['F']:$array['M'];
-        if (intval($month) < 10 ) $month = '0'.$month;
+        if (intval($month) < 10 ) {
+            $month = '0'.$month;
+        }
         $day = $array['d'];
-        if (intval($day) < 10 ) $day = '0'.$day;
+        if (intval($day) < 10 ) {
+            $day = '0'.$day;
+        }
         $hours = $array['H'];
-        if (intval($hours) < 10 ) $hours = '0'.$hours;
+        if (intval($hours) < 10 ) {
+            $hours = '0'.$hours;
+        }
         $minutes = $array['i'];
-        if (intval($minutes) < 10 ) $minutes = '0'.$minutes;
+        if (intval($minutes) < 10 ) {
+            $minutes = '0'.$minutes;
+        }
     }
     if (checkdate($month,$day,$year)) {
         $datetime = $year.'-'.$month.'-'.$day.' '.$hours.':'.$minutes.':'.$seconds;

main/tracking/lp_results_by_user.php

Braces +2 added lines, -1 removed lines

@@ -73,8 +73,9 @@
 //$form->addElement('submit','submit',get_lang('Filter'));
 $form->addButtonFilter(get_lang('Filter'));
 
-if (!empty($_REQUEST['course_code']))
+if (!empty($_REQUEST['course_code'])) {
     $selected_course = $_REQUEST['course_code'];
+}
 if (!empty($selected_course)) {
     $selected_course = api_get_course_info($selected_course);
     $course_list = array($selected_course);

main/tracking/toolaccess_details.php

Braces +11 added lines, -8 removed lines

@@ -90,7 +90,9 @@ 
             echo "<tr>
                     <td>
                     ";
-            if(isset($_cid)) echo "<b>$_cid : </b>";
+            if(isset($_cid)) {
+                echo "<b>$_cid : </b>";
+            }
             echo "		<b>".get_lang('ToolList')."</b>
 					</td>
                 </tr>
@@ -116,27 +118,28 @@ 
                         echo"</tr>";
                 }
 
-            }
-            else
+            } else
             {
                 echo "<tr>";
                 echo "<td colspan='2'><center>".get_lang('NoResult')."</center></td>";
                 echo"</tr>";
             }
             echo "</table></td></tr>";
-        }
-        else
+        } else
         {
             // this can prevent bug if there is special chars in $tool
             $encodedTool = urlencode($tool);
             $tool = urldecode($tool);
 
-            if( !isset($reqdate) )
-                $reqdate = time();
+            if( !isset($reqdate) ) {
+                            $reqdate = time();
+            }
             echo "<tr>
                     <td>
 					";
-            if(isset($_cid)) echo "<b>$_cid : </b>";
+            if(isset($_cid)) {
+                echo "<b>$_cid : </b>";
+            }
             echo "        <b>".get_lang($tool)."</b>
 					</td>
                 </tr>

main/tracking/logins_details.php

Braces +5 added lines, -4 removed lines

@@ -182,10 +182,11 @@
                 echo "<td style='padding-left : 40px;' valign='top'>" . $beautifulDateTime . "</td>";
                 echo"</tr>";
                 // $limit is used to select only results between $results[$j] (current login) and next one
-                if ($j == ( sizeof($results) - 1 ))
-                    $limit = date("Y-m-d H:i:s", $nextReqDate);
-                else
-                    $limit = $results[$j + 1];
+                if ($j == ( sizeof($results) - 1 )) {
+                                    $limit = date("Y-m-d H:i:s", $nextReqDate);
+                } else {
+                                    $limit = $results[$j + 1];
+                }
                 // select all access to tool between displayed date and next displayed date or now() if
                 // displayed date is the last login date
                 $sql = "SELECT access_tool, count(access_tool)

main/tracking/courseLogCSV.php

Braces +17 added lines, -9 removed lines

@@ -38,20 +38,27 @@ 
 
 $view = $_REQUEST['view'];
 
-if ($view == "0000001")
+if ($view == "0000001") {
     $nameTools = get_lang('SynthesisView');
-if ($view == "1000000")
+}
+if ($view == "1000000") {
     $nameTools = get_lang('CourseStats');
-if ($view == "0100000")
+}
+if ($view == "0100000") {
     $nameTools = get_lang('CourseAccess');
-if ($view == "0010000")
+}
+if ($view == "0010000") {
     $nameTools = get_lang('ToolsAccess');
-if ($view == "0001000")
+}
+if ($view == "0001000") {
     $nameTools = get_lang('LinksAccess');
-if ($view == "0000100")
+}
+if ($view == "0000100") {
     $nameTools = get_lang('DocumentsAccess');
-if ($view == "00000010")
+}
+if ($view == "00000010") {
     $nameTools = get_lang('ScormAccess');
+}
 
 $interbreadcrumb[] = array("url" => api_get_self() . "?view=0000000", "name" => get_lang('ToolName'));
 
@@ -79,8 +86,9 @@ 
       </div><br>
       "; */
 
-    if (!isset($view))
-        $view = "0000000";
+    if (!isset($view)) {
+            $view = "0000000";
+    }
 
 
     /* 	Reporting */

main/tracking/course_session_report.php

Braces +10 added lines, -2 removed lines

@@ -47,8 +47,16 @@
 $form->addElement('select', 'session_id', get_lang('Sessions'), $my_session_list);
 $form->addButtonFilter(get_lang('Filter'));
 
-if (!empty($_REQUEST['score']))	$filter_score = intval($_REQUEST['score']); else $filter_score = 70;
-if (!empty($_REQUEST['session_id'])) $session_id = intval($_REQUEST['session_id']); else $session_id = 0;
+if (!empty($_REQUEST['score'])) {
+    $filter_score = intval($_REQUEST['score']);
+} else {
+    $filter_score = 70;
+}
+if (!empty($_REQUEST['session_id'])) {
+    $session_id = intval($_REQUEST['session_id']);
+} else {
+    $session_id = 0;
+}
 
 if (empty($session_id)) {
 	$session_id = key($my_session_list);

main/tracking/course_access_details.php

Braces +3 added lines, -2 removed lines

@@ -57,8 +57,9 @@
 <table width="100%" cellpadding="2" cellspacing="3" border="0">
 <?php
     if ($is_allowedToTrack) {
-        if (!isset($reqdate) || $reqdate < 0 || $reqdate > 2149372861)
-            $reqdate = time();
+        if (!isset($reqdate) || $reqdate < 0 || $reqdate > 2149372861) {
+                    $reqdate = time();
+        }
             //** dislayed period
             echo "<tr><td><b>";
             switch($period) {

main/install/index.php

Braces +4 added lines, -1 removed lines

@@ -445,7 +445,10 @@
     }
 }
     ?>
-    <input type="hidden" name="updatePath"         value="<?php if (!$badUpdatePath) echo api_htmlentities($proposedUpdatePath, ENT_QUOTES); ?>" />
+    <input type="hidden" name="updatePath"         value="<?php if (!$badUpdatePath) {
+    echo api_htmlentities($proposedUpdatePath, ENT_QUOTES);
+}
+?>" />
     <input type="hidden" name="urlAppendPath"      value="<?php echo api_htmlentities($urlAppendPath, ENT_QUOTES); ?>" />
     <input type="hidden" name="pathForm"           value="<?php echo api_htmlentities($pathForm, ENT_QUOTES); ?>" />
     <input type="hidden" name="urlForm"            value="<?php echo api_htmlentities($urlForm, ENT_QUOTES); ?>" />

main/user/resume_session.php

Braces +8 added lines, -3 removed lines

@@ -149,8 +149,7 @@ 
     	<?php
     		if ($session['access_start_date'] == '00-00-0000' && $session['access_end_date']== '00-00-0000' ) {
     		    echo get_lang('NoTimeLimits');
-            }
-    		else {
+            } else {
                 if ($session['access_start_date'] != '00-00-0000') {
                 	//$session['date_start'] = Display::tag('i', get_lang('NoTimeLimits'));
                     $session['access_start_date'] =  get_lang('From').' '.$session['access_start_date'];
@@ -189,12 +188,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';