chamilo / chamilo-lms

main/auth/shibboleth/test/shibboleth_test.class.php

Braces +1 added lines, -2 removed lines

@@ -206,8 +206,7 @@
             echo $message;
             var_dump(debug_backtrace());
             die;
-        }
-        else
+        } else
         {
             $message = "Assert successful $message <br/>";
             echo $message;

main/auth/shibboleth/app/model/shibboleth_store.class.php

Braces +2 added lines, -4 removed lines

@@ -168,12 +168,10 @@
                 if (empty($result))
                 {
                     $result = $default;
-                }
-                else if (count($result) == 1)
+                } else if (count($result) == 1)
                 {
                     $result = reset($result);
-                }
-                else
+                } else
                 {
                     $result = $result;
                 }

main/auth/shibboleth/app/controller/shibboleth_controller.class.php

Braces +3 added lines, -6 removed lines

@@ -56,8 +56,7 @@ 
             if ($email = $form->get_email())
             {
                 $user->email = $email;
-            }
-            else
+            } else
             {
                 $content = $form->display();
                 Shibboleth::display()->page($content);
@@ -71,8 +70,7 @@ 
         if ($is_new_user && $user->status_request)
         {
             Shibboleth::redirect('/main/auth/shibboleth/app/view/request.php');
-        }
-        else
+        } else
         {
             Shibboleth::redirect();
         }
@@ -141,8 +139,7 @@ 
             {
                 $request_submitted = get_lang('RequestSubmitted');
                 Shibboleth::display()->message_page($request_submitted);
-            }
-            else
+            } else
             {
                 $request_failed = get_lang('RequestFailed');
                 Shibboleth::display()->error_page($request_failed);

main/auth/shibboleth/lib/store.class.php

Braces +8 added lines, -16 removed lines

@@ -107,8 +107,7 @@ 
         if ($this->is_new($object))
         {
             $result = $this->insert($object);
-        }
-        else
+        } else
         {
             $result = $this->update($object);
         }
@@ -257,8 +256,7 @@ 
             $object->{$this->id_name} = $id;
             $object->{db_name} = $db_name;
             return $id;
-        }
-        else
+        } else
         {
             return false;
         }
@@ -278,16 +276,13 @@ 
                     return '';
                 }
                 $args = array($this->pk_name, $arg);
-            }
-            else if (is_string($arg))
+            } else if (is_string($arg))
             {
                 return $arg;
-            }
-            else if (is_array($arg))
+            } else if (is_array($arg))
             {
                 $args = $arg;
-            }
-            else
+            } else
             {
                 return $arg;
             }
@@ -309,17 +304,14 @@ 
         if (is_bool($var))
         {
             return $value ? '1' : '0';
-        }
-        else if (is_numeric($value))
+        } else if (is_numeric($value))
         {
             return empty($value) ? '0' : $value;
-        }
-        else if (is_string($value))
+        } else if (is_string($value))
         {
             $value = mysql_escape_string($value);
             return "'$value'";
-        }
-        else
+        } else
         {
             return $value;
         }

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/reports/index.php

Braces +2 added lines, -1 removed lines

@@ -25,8 +25,9 @@
 $tool_name=get_lang('Reports');
 
 // Displaying the header
-foreach (array('jquery.ba-bbq.min.js', 'jquery.validate.js', 'jquery.form.js', 'jquery.form.wizard.js', 'jquery.dataTables.min.js') as $js)
+foreach (array('jquery.ba-bbq.min.js', 'jquery.validate.js', 'jquery.form.js', 'jquery.form.wizard.js', 'jquery.dataTables.min.js') as $js) {
     $htmlHeadXtra[] = '<script src="'.api_get_path(WEB_LIBRARY_PATH).'javascript/'.$js.'" type="text/javascript" language="javascript"></script>'."\n";
+}
 
 // FIXME
 $htmlHeadXtra[] = '    <style type="text/css">

main/reports/multiquery.lib.php

Braces +10 added lines, -5 removed lines

@@ -8,8 +8,9 @@ 
 		// mysql handler
 		$result[$i]['mysql'] = mysql_query($array[$i]);
 
-		if (! $result[$i]['mysql'])
-			die("error in query $i : ".$array[$i]);
+		if (! $result[$i]['mysql']) {
+					die("error in query $i : ".$array[$i]);
+		}
 
 		// fields
 		$result[$i]['num_fields'] = mysql_num_fields($result[$i]['mysql']);
@@ -25,10 +26,11 @@ 
 	$result['num_queries'] = sizeof($array);
 	// rows
 	$numberOfResult = mysql_num_rows($result[0]['mysql']);
-	for ($i = 1; $i <$result['num_queries']; $i++) 
-		if ($numberOfResult != mysql_num_rows($result[$i]['mysql']))
+	for ($i = 1; $i <$result['num_queries']; $i++) {
+			if ($numberOfResult != mysql_num_rows($result[$i]['mysql']))
 			die("wrong number of row: $numberOfResult vs ".
 				mysql_num_rows($result[$i]['mysql'])." on query $i");
+	}
 
 	$result['num_rows'] = $numberOfResult;
 	$result['num_fields'] = $field;
@@ -53,7 +55,10 @@ 
 	$pos = 0;
 	for ($i = 0; $i < $mq_h['num_queries']; $i++) {
 		$row = mysql_fetch_row($mq_h[$i]['mysql']);
-		if (!$row) return false; // last line
+		if (!$row) {
+		    return false;
+		}
+		// last line
 		for ($j = 0; $j < sizeof($row); $j++) {
 			$result[$pos] = $row[$j];
 			$pos++;

main/reports/templates/exercicesMultiCourses.reports.php

Braces +18 added lines, -11 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'].'` '; 
+				$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 ', ';
 
 

main/reports/templates/courseArticulate.reports.php

Braces +6 added lines, -4 removed lines

@@ -39,8 +39,9 @@ 
 	// Stored Value
 	$sv = array();
 	foreach ($sv as $k => $v) {
-        if (!isset($v['sql']))
-                $v['sql'] = 'FIELD';
+        if (!isset($v['sql'])) {
+                        $v['sql'] = 'FIELD';
+        }
         $sqlField = str_replace('FIELD', 'sv.sv_value', $v['sql']);
         $query = 'select '.$sqlField.' as "'.$v['title'].'" ';
 //		$query = 'select sec_to_time(sv.sv_value) as "'.$v.'" ';
@@ -110,8 +111,9 @@ 
 		$query .= ' on u.user_id = lv.user_id and lv.lp_id = '.$v['lid'];
 		$query .= ' left outer join '.Database::get_course_table(TABLE_LP_ITEM_VIEW).' liv ';
 		$query .= ' on lv.id = liv.lp_view_id ';
-		if ($v['target_view_count'])
-			$query .= ' and liv.view_count = '.$v['target_view_count'];
+		if ($v['target_view_count']) {
+					$query .= ' and liv.view_count = '.$v['target_view_count'];
+		}
 		$query .= ' and liv.lp_item_id = '.$v['liid'].' ';
 		$query .= ' where u.user_id in ('.reports_getVisibilitySQL().') ';
 		$query .= ' group by u.user_id ';