chamilo / chamilo-lms

main/reports/reports.lib.php

Braces +10 added lines, -8 removed lines

@@ -15,9 +15,10 @@ 
 // load templates
 function reports_loadTemplates() {
 	global $reports_enabled_templates, $reports_template;
-	foreach ($reports_enabled_templates as $t)
-		require_once 'templates/'.$t.'.reports.php';
-}
+	foreach ($reports_enabled_templates as $t) {
+			require_once 'templates/'.$t.'.reports.php';
+	}
+	}
 
 
 // clear all reporting data
@@ -150,11 +151,12 @@ 
 // return tools ID (parametre is a constant from main_api
 function reports_getToolId($tool) {
 	$tools = array_flip(api_get_tools_lists());
-	if (array_key_exists($tool, $tools))
-		return $tools[$tool];
-	else
-		return null;
-}
+	if (array_key_exists($tool, $tools)) {
+			return $tools[$tool];
+	} else {
+			return null;
+	}
+	}
 
 // return a sql clause returning triplet of (course, $session, $uid) the
 // current user is authorized to reed

main/inc/lib/geometry.lib.php

Braces +49 added lines, -28 removed lines

@@ -65,8 +65,10 @@ 
     $bords = array_fill(0, $bord_lenght, array()); // building this array
 
     /* adding the first point of the polygone */
-    if (is_array($bords[$poly[0]['y']])) //avoid warning
+    if (is_array($bords[$poly[0]['y']])) {
+        //avoid warning
     	array_push($bords[$poly[0]['y']], $poly[0]['x']);
+    }
 
     $i = 1; // we re-use $i and $old_pente bellow the loop
     $old_pente=0;
@@ -75,11 +77,14 @@ 
 
         /* special cases */
         if ($poly[$i-1]['y'] == $poly[$i]['y']) {
-            if ($poly[$i-1]['x'] == $poly[$i]['x'])
-                continue; // twice the same point
+            if ($poly[$i-1]['x'] == $poly[$i]['x']) {
+                            continue;
+            }
+            // twice the same point
             else {    //  infinite elevation of the edge
-            	if (is_array($bords[$poly[$i]['y']]))
-                	array_push($bords[$poly[$i]['y']],$poly[$i]['x']);
+            	if (is_array($bords[$poly[$i]['y']])) {
+            	                	array_push($bords[$poly[$i]['y']],$poly[$i]['x']);
+            	}
                 $old_pente=0;
                 continue;
             }
@@ -87,9 +92,13 @@ 
 
 		//echo 'point:'.$poly[$i]['y']; bug here
         // adding the point as a part of an edge
-        if (is_array($bords[$poly[$i]['y']])) //avoid warning
+        if (is_array($bords[$poly[$i]['y']])) {
+            //avoid warning
         array_push($bords[$poly[$i]['y']], $poly[$i]['x']);
-        if (DEBUG) echo '('.$poly[$i]['x'].';'.$poly[$i]['y'].')   ';
+        }
+        if (DEBUG) {
+            echo '('.$poly[$i]['x'].';'.$poly[$i]['y'].')   ';
+        }
 
         /* computing the elevation of the edge going */
         //        from $poly[$i-1] to $poly[$i]
@@ -99,15 +108,17 @@ 
         // if the sign of the elevation change from the one of the
         // previous edge, the point must be added a second time inside
         // $bords
-        if ($i>1)
-            if (($old_pente<0 && $pente>0)
+        if ($i>1) {
+                    if (($old_pente<0 && $pente>0)
                     || ($old_pente>0 && $pente<0)) {
 				if (is_array($bords[$poly[$i]['y']])) //avoid warning
                 	array_push($bords[$poly[$i]['y']],$poly[$i]['x']);
+        }
 
-                if (DEBUG)
-                    echo '*('.$poly[$i]['x'].
+                if (DEBUG) {
+                                    echo '*('.$poly[$i]['x'].
                         ';'.$poly[$i]['y'].')   ';
+                }
         	}
 
         /* detect the direction of the elevation in Y */
@@ -136,10 +147,11 @@ 
 
         // elevation between $poly[0]['x'] and $poly[1]['x'])
         $rest = $poly[0]['y']-$poly[1]['y'];
-        if ($rest!=0)
-        	$pente1 = ($poly[0]['x']-$poly[1]['x'])/($rest);
-        else
-			$pente1 = 0;
+        if ($rest!=0) {
+                	$pente1 = ($poly[0]['x']-$poly[1]['x'])/($rest);
+        } else {
+        			$pente1 = 0;
+        }
 
         // elevation between $poly[$i-1]['x'] and $poly[0]['x'])
         $pente = ($poly[$i-1]['x']-$poly[0]['x'])/
@@ -151,14 +163,17 @@ 
 
         // doubling the first point if needed (see above)
         if (($pente1<0 && $pente>0) || ($pente1>0 && $pente<0)) {
-        	if (is_array($bords[$poly[$i - 1]['y']]))
-            	array_push($bords[$poly[$i - 1]['y']],  round($poly[$i - 1]['x']));
+        	if (is_array($bords[$poly[$i - 1]['y']])) {
+        	            	array_push($bords[$poly[$i - 1]['y']],  round($poly[$i - 1]['x']));
+        	}
             //if (DEBUG) echo '('.$poly[$i-1]['x'].';'.$poly[$i-1]['y'].')   ';
         }
         //  doubling the last point if neededd
         if (($old_pente<0 && $pente>0) || ($old_pente>0 && $pente<0)) {
-        	if (is_array($bords[$poly[$i-1]['y']])) //avoid warning
+        	if (is_array($bords[$poly[$i-1]['y']])) {
+        	    //avoid warning
             	array_push($bords[$poly[$i-1]['y']], round($poly[$i-1]['x']));
+        	}
             //if (DEBUG) echo '*('.$poly[$i-1]['x'].';'.$poly[$i-1]['y'].')   ';
         }
 
@@ -227,11 +242,13 @@ 
         $s = "<div style='font-size: 8px; line-height:3px'><pre>\n";
     }
     for ($i=0; $i<$max['y']; $i++) {
-        for($j=0; $j<$max['x']; $j++)
-            if($poly[$j][$i] == TRUE)
+        for($j=0; $j<$max['x']; $j++) {
+                    if($poly[$j][$i] == TRUE)
                 $s .= ($format=='html'?"<b>1</b>":'1');
-            else
-                $s .= "0";
+        }
+            else {
+                            $s .= "0";
+            }
         $s .= ($format=='html'?"<br />\n":"\n");
     }
     $s .= ($format=='html'?"</pre></div>\n":"\n");
@@ -252,15 +269,19 @@ 
     $surfaceOf1 = 0;
     $surfaceOf2 = 0;
 
-    for ($i=0; $i<$max['x']; $i++)
-        for($j=0; $j<$max['y']; $j++) {
+    for ($i=0; $i<$max['x']; $i++) {
+            for($j=0;
+    }
+    $j<$max['y']; $j++) {
             if (isset($poly1[$i][$j]) && ($poly1[$i][$j] == TRUE)) {
                 $surfaceOf1++;
-                if (isset($poly2[$i][$j]) && ($poly2[$i][$j] == FALSE))
-                    $onlyIn1++;
+                if (isset($poly2[$i][$j]) && ($poly2[$i][$j] == FALSE)) {
+                                    $onlyIn1++;
+                }
+            }
+            if (isset($poly2[$i][$j]) && ($poly2[$i][$j] == TRUE)) {
+                            $surfaceOf2++;
             }
-            if (isset($poly2[$i][$j]) && ($poly2[$i][$j] == TRUE))
-                $surfaceOf2++;
         }
 
     return array (

main/reports/templates/exercicesMultiCourses.reports.php

Braces +17 added lines, -10 removed lines

@@ -41,28 +41,35 @@
 		'where r.course_id=c.id and r.tool_id='.
 		reports_getToolId(TOOL_QUIZ).
 		' order by r.course_id, r.child_name');
-	if (Database::num_rows($columns) == 0)
-		die('<b>'.get_lang('no data found').'</b>');
+	if (Database::num_rows($columns) == 0) {
+			die('<b>'.get_lang('no data found').'</b>');
+	}
 	$query = 'select u.lastname Name, u.firstname Firstname';
 	$columns = Database::store_result($columns);
-	if ($_REQUEST['tattempt'] == 'min' || $_REQUEST['tattempt'] == 'max')
-		$function = $_REQUEST['tattempt'];
-	else
-		$function = 'avg';
-	foreach ($columns as $key => $column)
-		$query .= ', '.$function.'(k'.$key.'.score) as `'.
+	if ($_REQUEST['tattempt'] == 'min' || $_REQUEST['tattempt'] == 'max') {
+			$function = $_REQUEST['tattempt'];
+	} else {
+			$function = 'avg';
+	}
+	foreach ($columns as $key => $column) {
+			$query .= ', '.$function.'(k'.$key.'.score) as `'.
 				$column['course'].' - '.
 				$column['test'].'` ';
+	}
 	$query .= ' from '.Database::get_main_table(TABLE_MAIN_USER).' u ';
-	foreach ($columns as $key => $column) // fixme sessions
+	foreach ($columns as $key => $column) {
+	    // fixme sessions
 		$query .= 'left outer join '.
 			Database::get_main_table(TABLE_MAIN_REPORTS_VALUES).
 			' k'.$key.
 			' on k'.$key.'.key_id = '.$column['kid'].
 				' and k'.$key.'.user_id = u.user_id ';
+	}
 	$query .= ' group by ';
-	foreach ($columns as $key => $column) // grouping attempt
+	foreach ($columns as $key => $column) {
+	    // grouping attempt
 		$query .= 'k'.$key.'.attempt, ';
+	}
 	$query = substr($query, 0, -2); // removing last ', ';
 
 

app/Resources/public/assets/ckeditor/samples/old/assets/posteddata.php

Braces +9 added lines, -7 removed lines

@@ -29,14 +29,16 @@
 {

 	foreach ( $_POST as $key => $value )

 	{

-		if ( ( !is_string($value) && !is_numeric($value) ) || !is_string($key) )

-			continue;

+		if ( ( !is_string($value) && !is_numeric($value) ) || !is_string($key) ) {
+					continue;
+		}
 

-		if ( get_magic_quotes_gpc() )

-			$value = htmlspecialchars( stripslashes((string)$value) );

-		else

-			$value = htmlspecialchars( (string)$value );

-?>

+		if ( get_magic_quotes_gpc() ) {
+					$value = htmlspecialchars( stripslashes((string)$value) );
+		} else {
+					$value = htmlspecialchars( (string)$value );
+		}
+		?>

 		<tr>

 			<th style="vertical-align: top"><?php echo htmlspecialchars( (string)$key ); ?></th>

 			<td><pre class="samples"><?php echo $value; ?></pre></td>

plugin/vchamilo/cli/clilib.php

Braces +10 added lines, -3 removed lines

@@ -133,7 +133,9 @@ 
             $key = $headers[$f];
             if (preg_match('/\|/', $key)) {
                 list($plugin, $variable) = explode('|', str_replace('plugin_', '', $key));
-                if (empty($variable)) die("Key error in CSV : $key ");
+                if (empty($variable)) {
+                    die("Key error in CSV : $key ");
+                }
                 if (!isset($vnode->$plugin)) {
                     $vnode->$plugin = new StdClass();
                 }
@@ -170,8 +172,13 @@ 
     static $first = true;
 
     // We may have a risk the BOM is present on first line
-    if ($resetfirst) $first = true;
-    if (!isset($textlib)) $textlib = new textlib(); // Singleton
+    if ($resetfirst) {
+        $first = true;
+    }
+    if (!isset($textlib)) {
+        $textlib = new textlib();
+    }
+    // Singleton
     $text = $textlib->trim_utf8_bom($text);
     $first = false;
 

main/inc/lib/javascript/svgedit/extensions/imagelib/index.php

Braces +1 added lines, -2 removed lines

@@ -65,8 +65,7 @@
 		if (strpos($filename, "svg")){
 			$new_sizes['width'] = 60;
 			$new_sizes['height'] = 60;
-		}
-		else {
+		} else {
 			$new_sizes = api_resize_image($image, 60, 60);
 		}
 

main/inc/lib/api.lib.php

Braces +19 added lines, -16 removed lines

@@ -1701,7 +1701,9 @@ 
     }
 
     global $_course;
-    if ($_course == '-1') $_course = array();
+    if ($_course == '-1') {
+        $_course = array();
+    }
     return $_course;
 }
 
@@ -4154,18 +4156,21 @@ 
     switch ($lang_type) {
         case 'platform_lang':
             $temp_lang = api_get_setting('platformLanguage');
-            if (!empty($temp_lang))
-                $return = $temp_lang;
+            if (!empty($temp_lang)) {
+                            $return = $temp_lang;
+            }
             break;
         case 'user_profil_lang':
             $_user = api_get_user_info();
 
-            if (isset($_user['language']) && !empty($_user['language']))
-                $return = $_user['language'];
+            if (isset($_user['language']) && !empty($_user['language'])) {
+                            $return = $_user['language'];
+            }
             break;
         case 'user_selected_lang':
-            if (isset($_SESSION['user_language_choice']) && !empty($_SESSION['user_language_choice']))
-                $return = $_SESSION['user_language_choice'];
+            if (isset($_SESSION['user_language_choice']) && !empty($_SESSION['user_language_choice'])) {
+                            $return = $_SESSION['user_language_choice'];
+            }
             break;
         case 'course_lang':
             global $_course;
@@ -5434,8 +5439,7 @@ 
             $is_courseAdmin = false;
             $is_courseCoach = true;
             $is_sessionAdmin = false;
-        }
-        elseif ($row[0]['session_admin_id'] == $userid) {
+        } elseif ($row[0]['session_admin_id'] == $userid) {
             $is_courseMember = false;
             $is_courseTutor = false;
             $is_courseAdmin = false;
@@ -5778,8 +5782,7 @@ 
     elseif (isset($_ENV['OS'])) {
         // Sometimes $_ENV['OS'] may not be present (bugs?)
         $os = $_ENV['OS'];
-    }
-    elseif (defined('PHP_OS')) {
+    } elseif (defined('PHP_OS')) {
         // PHP_OS means on which OS PHP was compiled, this is why
         // using PHP_OS is the last choice for detection.
         $os = PHP_OS;
@@ -5843,12 +5846,10 @@ 
     if ($delta_width > $delta_height) {
         $result['width'] = ceil($image_width * $resize_factor_height);
         $result['height'] = ceil($image_height * $resize_factor_height);
-    }
-    elseif ($delta_width < $delta_height) {
+    } elseif ($delta_width < $delta_height) {
         $result['width'] = ceil($image_width * $resize_factor_width);
         $result['height'] = ceil($image_height * $resize_factor_width);
-    }
-    else {
+    } else {
         $result['width'] = ceil($target_width);
         $result['height'] = ceil($target_height);
     }
@@ -6629,7 +6630,9 @@ 
         }
         $ip = trim($ip1);
     }
-    if (!empty($debug)) error_log('Real IP: '.$ip);
+    if (!empty($debug)) {
+        error_log('Real IP: '.$ip);
+    }
     return $ip;
 }
 

main/ticket/report.php

Braces +3 added lines, -2 removed lines

@@ -289,8 +289,9 @@
 
     if ($start_date != '' || $end_date != '') {
         $sql .= " HAVING ";
-        if ($start_date != '')
-            $sql .= "  access_date >= '$start_date'   ";
+        if ($start_date != '') {
+                    $sql .= "  access_date >= '$start_date'   ";
+        }
         if ($end_date != '') {
             $sql = ($start_date == '') ? $sql : ($sql . " AND ");
             $sql .= "  access_date <= '$end_date'   ";

main/exercise/overview.php

Braces +1 added lines, -2 removed lines

@@ -250,8 +250,7 @@
         case RESULT_DISABLE_SHOW_SCORE_ONLY:
             if ($objExercise->feedback_type != EXERCISE_FEEDBACK_TYPE_END) {
                $header_names = array(get_lang('Attempt'), get_lang('StartDate'), get_lang('IP'), get_lang('Score'));
-            }
-            else {
+            } else {
                 $header_names = array(get_lang('Attempt'), get_lang('StartDate'), get_lang('IP'), get_lang('Score'), get_lang('Details'));
             }
             break;