chamilo / chamilo-lms

plugin/vchamilo/cli/bulkcreatenodes.php

Braces +17 added lines, -5 removed lines

@@ -74,8 +74,14 @@ 
 
     $content = file($options['config']);
     foreach($content as $l) {
-        if (preg_match('/^\s+$/', $l)) continue; // Empty lines.
-        if (preg_match('/^[#\/!;]/', $l)) continue; // Comments (any form).
+        if (preg_match('/^\s+$/', $l)) {
+            continue;
+        }
+        // Empty lines.
+        if (preg_match('/^[#\/!;]/', $l)) {
+            continue;
+        }
+        // Comments (any form).
         if (preg_match('/^(.*?)=(.*)$/', $l, $matches)) {
             if (in_array($matches[1], $expectedoptions)){
                 $options[trim($matches[1])] = trim($matches[2]);
@@ -90,16 +96,22 @@ 
 require_once($_configuration['root_sys'] . '/plugin/vchamilo/lib/vchamilo_plugin.class.php');
 
 global $DB;
-if ($options['verbose']) echo "building database manager\n";
+if ($options['verbose']) {
+    echo "building database manager\n";
+}
 $DB = new DatabaseManager();
-if ($options['verbose']) echo "building plugin vchamilo\n";
+if ($options['verbose']) {
+    echo "building plugin vchamilo\n";
+}
 $plugin = VChamiloPlugin::create();
 
 if (empty($options['nodes'])) {
     cli_error('Missing node definition. Halt.');
 }
 
-if ($options['verbose']) echo "parsing nodelist\n";
+if ($options['verbose']) {
+    echo "parsing nodelist\n";
+}
 $nodes = vchamilo_parse_csv_nodelist($options['nodes'], $plugin);
 
 if ($options['lint']) {

plugin/vchamilo/cli/bulkdestroynodes.php

Braces +21 added lines, -7 removed lines

@@ -92,25 +92,39 @@
 }
 
 require_once($_configuration['root_sys'].'local/classes/database.class.php');       // cli only functions
-if ($options['verbose']) echo "loaded dbclass\n";
+if ($options['verbose']) {
+    echo "loaded dbclass\n";
+}
 require_once($_configuration['root_sys'].'local/classes/textlib.class.php');       // cli only functions
-if ($options['verbose']) echo "loaded textlib\n";
+if ($options['verbose']) {
+    echo "loaded textlib\n";
+}
 require_once($_configuration['root_sys'].'local/classes/mootochamlib.php');       // moodle like API
-if ($options['verbose']) echo "loaded moodle wrapping\n";
+if ($options['verbose']) {
+    echo "loaded moodle wrapping\n";
+}
 require_once($_configuration['root_sys'] . '/plugin/vchamilo/lib/vchamilo_plugin.class.php');
-if ($options['verbose']) echo "loaded vchamilo plugin\n";
+if ($options['verbose']) {
+    echo "loaded vchamilo plugin\n";
+}
 
 global $DB;
-if ($options['verbose']) echo "building database manager\n";
+if ($options['verbose']) {
+    echo "building database manager\n";
+}
 $DB = new DatabaseManager();
-if ($options['verbose']) echo "building plugin vchamilo\n";
+if ($options['verbose']) {
+    echo "building plugin vchamilo\n";
+}
 $plugin = VChamiloPlugin::create();
 
 if (empty($options['nodes'])) {
     cli_error(get_string('climissingnodes', 'block_vmoodle'));
 }
 
-if ($options['verbose']) echo "parsing nodelist\n";
+if ($options['verbose']) {
+    echo "parsing nodelist\n";
+}
 $nodes = vchamilo_parse_csv_nodelist($options['nodes'], $plugin);
 
 if ($options['lint']) {

plugin/bbb/lib/bbb_api.php

Braces +28 added lines, -40 removed lines

@@ -73,10 +73,11 @@ 
 			$data = curl_exec( $ch );
 			curl_close( $ch );
 
-			if($data)
-				return (new SimpleXMLElement($data));
-			else
-				return false;
+			if($data) {
+							return (new SimpleXMLElement($data));
+			} else {
+							return false;
+			}
 		}
 		return (simplexml_load_file($url));
 	}
@@ -85,11 +86,9 @@ 
 		/* Process required params and throw errors if we don't get values */
 		if ((isset($param)) && ($param != '')) {
 			return $param;
-		}
-		elseif (!isset($param)) {
+		} elseif (!isset($param)) {
 			throw new Exception('Missing parameter.');
-		}
-		else {
+		} else {
 			throw new Exception(''.$param.' is required.');
 		}
 	}
@@ -99,8 +98,7 @@ 
 		/* Don't know if we'll use this one, but let's build it in case. */
 		if ((isset($param)) && ($param != '')) {
 			return $param;
-		}
-		else {
+		} else {
 			$param = '';
 			return $param;
 		}
@@ -137,8 +135,9 @@ 
 		'&duration='.urlencode($creationParams['duration']);
 		//'&meta_category='.urlencode($creationParams['meta_category']);
 		$welcomeMessage = $creationParams['welcomeMsg'];
-		if(trim($welcomeMessage))
-			$params .= '&welcome='.urlencode($welcomeMessage);
+		if(trim($welcomeMessage)) {
+					$params .= '&welcome='.urlencode($welcomeMessage);
+		}
 		// Return the complete URL:
 		return ( $creationUrl.$params.'&checksum='.sha1("create".$params.$this->_securitySalt) );
 	}
@@ -165,8 +164,8 @@ 
 		$xml = $this->_processXmlResponse($this->getCreateMeetingURL($creationParams));
 
         if ($xml) {
-			if($xml->meetingID)
-				return array(
+			if($xml->meetingID) {
+							return array(
 					'returncode' => $xml->returncode->__toString(),
 					'message' => $xml->message->__toString(),
 					'messageKey' => $xml->messageKey->__toString(),
@@ -176,14 +175,14 @@ 
 					'hasBeenForciblyEnded' => $xml->hasBeenForciblyEnded->__toString(),
 					'createTime' => $xml->createTime->__toString()
 					);
-			else
-				return array(
+			} else {
+							return array(
 					'returncode' => $xml->returncode->__toString(),
 					'message' => $xml->message->__toString(),
 					'messageKey' => $xml->messageKey->__toString()
 					);
-		}
-		else {
+			}
+		} else {
 			return null;
 		}
 	}
@@ -253,8 +252,7 @@ 
 				'message' => $xml->message->__toString(),
 				'messageKey' => $xml->messageKey->__toString()
 				);
-		}
-		else {
+		} else {
 			return null;
 		}
 
@@ -288,8 +286,7 @@ 
 				'returncode' => $xml->returncode->__toString(),
 				'running' => $xml->running->__toString() 	// -- Returns true/false.
 			);
-		}
-		else {
+		} else {
 			return null;
 		}
 
@@ -317,8 +314,7 @@ 
 					'returncode' => $xml->returncode->__toString()
 				);
 				return $result;
-			}
-			elseif ($xml->messageKey == 'noMeetings') {
+			} elseif ($xml->messageKey == 'noMeetings') {
 				/* No meetings on server, so return just this info: */
 				$result = array(
 					'returncode' => $xml->returncode->__toString(),
@@ -326,8 +322,7 @@ 
 					'message' => $xml->message->__toString()
 				);
 				return $result;
-			}
-			else {
+			} else {
 				// In this case, we have success and meetings. First return general response:
 				$result = array(
 					'returncode' => $xml->returncode->__toString(),
@@ -348,8 +343,7 @@ 
 				}
 				return $result;
 			}
-		}
-		else {
+		} else {
 			return null;
 		}
 
@@ -388,8 +382,7 @@ 
 					'message' => $xml->message->__toString()
 				);
 				return $result;
-			}
-			else {
+			} else {
 				// In this case, we have success and meeting info:
 				$result = array(
 					'returncode' => $xml->returncode->__toString(),
@@ -418,8 +411,7 @@ 
 				}
 				return $result;
 			}
-		}
-		else {
+		} else {
 			return null;
 		}
 
@@ -463,8 +455,7 @@ 
 					'message' => $xml->message->__toString()
 				);
 				return $result;
-			}
-			else {
+			} else {
 				// In this case, we have success and recording info:
 				$result = array(
 					'returncode' => $xml->returncode->__toString(),
@@ -495,8 +486,7 @@ 
 					}
 				return $result;
 			}
-		}
-		else {
+		} else {
 			return null;
 		}
 	}
@@ -529,8 +519,7 @@ 
 				'returncode' => $xml->returncode->__toString(),
 				'published' => $xml->published->__toString() 	// -- Returns true/false.
 			);
-		}
-		else {
+		} else {
 			return null;
 		}
 
@@ -562,8 +551,7 @@ 
 				'returncode' => $xml->returncode->__toString(),
 				'deleted' => $xml->deleted->__toString() 	// -- Returns true/false.
 			);
-		}
-		else {
+		} else {
 			return null;
 		}
 

main/lp/openoffice_text.class.php

Braces +3 added lines, -2 removed lines

@@ -123,8 +123,9 @@
 
         for ($i = 0; $i < count($matches[0]); $i++) {
 
-            if (empty($matches[1][$i]))
-                continue;
+            if (empty($matches[1][$i])) {
+                            continue;
+            }
 
             $content = strstr($content,$matches[0][$i]);
             if ($i + 1 !== count($matches[0])) {

main/gradebook/lib/be/learnpathlink.class.php

Braces +12 added lines, -8 removed lines

@@ -29,8 +29,9 @@ 
 	public function get_not_created_links()
 	{
 		return false;
-		if (empty($this->course_code))
-			die('Error in get_not_created_links() : course code not set');
+		if (empty($this->course_code)) {
+					die('Error in get_not_created_links() : course code not set');
+		}
 
 		$tbl_grade_links = Database :: get_main_table(TABLE_MAIN_GRADEBOOK_LINK);
 
@@ -57,8 +58,9 @@ 
 	 */
 	public function get_all_links()
 	{
-		if (empty($this->course_code))
-			die('Error in get_not_created_links() : course code not set');
+		if (empty($this->course_code)) {
+					die('Error in get_not_created_links() : course code not set');
+		}
 
 		$session_id = api_get_session_id();
 		if (empty($session_id)) {
@@ -111,8 +113,9 @@ 
                     lp_id = ".$this->get_ref_id()." AND
                     session_id = $session_id ";
 
-		if (isset($stud_id))
-			$sql .= ' AND user_id = '.intval($stud_id);
+		if (isset($stud_id)) {
+					$sql .= ' AND user_id = '.intval($stud_id);
+		}
 
 		// order by id, that way the student's first attempt is accessed first
 		$sql .= ' ORDER BY view_count DESC';
@@ -122,8 +125,9 @@ 
 		if (isset($stud_id)) {
 			if ($data = Database::fetch_assoc($scores)) {
 				return array ($data['progress'], 100);
-			} else
-				return null;
+			} else {
+							return null;
+			}
 		} else {
 			// all students -> get average
 			$students = array();  // user list, needed to make sure we only

main/gradebook/lib/be/evaluation.class.php

Braces +27 added lines, -12 removed lines

@@ -231,36 +231,51 @@ 
 		}
 
 		if (isset($user_id)) {
-			if ($paramcount != 0) $sql .= ' AND';
-			else $sql .= ' WHERE';
+			if ($paramcount != 0) {
+			    $sql .= ' AND';
+			} else {
+			    $sql .= ' WHERE';
+			}
 			$sql .= ' user_id = '.intval($user_id);
 			$paramcount ++;
 		}
 
 		if (isset($course_code) && $course_code <> '-1') {
-			if ($paramcount != 0) $sql .= ' AND';
-			else $sql .= ' WHERE';
+			if ($paramcount != 0) {
+			    $sql .= ' AND';
+			} else {
+			    $sql .= ' WHERE';
+			}
 			$sql .= " course_code = '".Database::escape_string($course_code)."'";
 			$paramcount ++;
 		}
 
 		if (isset($category_id)) {
-			if ($paramcount != 0) $sql .= ' AND';
-			else $sql .= ' WHERE';
+			if ($paramcount != 0) {
+			    $sql .= ' AND';
+			} else {
+			    $sql .= ' WHERE';
+			}
 			$sql .= ' category_id = '.intval($category_id);
 			$paramcount ++;
 		}
 
 		if (isset($visible)) {
-			if ($paramcount != 0) $sql .= ' AND';
-			else $sql .= ' WHERE';
+			if ($paramcount != 0) {
+			    $sql .= ' AND';
+			} else {
+			    $sql .= ' WHERE';
+			}
 			$sql .= ' visible = '.intval($visible);
 			$paramcount ++;
 		}
 
 		if (isset($locked)) {
-			if ($paramcount != 0) $sql .= ' AND';
-			else $sql .= ' WHERE';
+			if ($paramcount != 0) {
+			    $sql .= ' AND';
+			} else {
+			    $sql .= ' WHERE';
+			}
 			$sql .= ' locked = '.intval($locked);
 		}
 
@@ -398,7 +413,7 @@ 
 			.', description = ';
 		if (isset($this->description)) {
 			$sql .= "'".Database::escape_string($this->get_description())."'";
-		}else {
+		} else {
 			$sql .= 'null';
 		}
 		$sql .= ', user_id = '.intval($this->get_user_id())
@@ -469,7 +484,7 @@ 
 				$sql .= ' AND user_id = '.api_get_user_id();
 			}
 
-		}else {
+		} else {
 			$sql .= ' AND user_id = '.api_get_user_id();
 		}
 

main/admin/add_sessions_to_usergroup.php

Braces +4 added lines, -1 removed lines

@@ -173,7 +173,10 @@
 echo '<div id="advancedSearch" style="display: none">'. get_lang('SearchSessions'); ?> :
      <input name="SearchSession" onchange = "xajax_search_usergroup_sessions(this.value,'searchbox')" onkeyup="this.onchange()">
      </div>
-<form name="formulaire" method="post" action="<?php echo api_get_self(); ?>?id=<?php echo $id; if(!empty($_GET['add'])) echo '&add=true' ; ?>" style="margin:0px;" <?php if($ajax_search){echo ' onsubmit="valide();"';}?>>
+<form name="formulaire" method="post" action="<?php echo api_get_self(); ?>?id=<?php echo $id; if(!empty($_GET['add'])) {
+    echo '&add=true' ;
+}
+?>" style="margin:0px;" <?php if($ajax_search){echo ' onsubmit="valide();"';}?>>
 <?php
 echo '<legend>'.$data['name'].': '.$tool_name.'</legend>';
 echo Display::input('hidden','id',$id);

main/document/edit_draw.php

Braces +3 added lines, -2 removed lines

@@ -96,13 +96,14 @@
 
 $is_certificate_mode = DocumentManager::is_certificate_mode($dir);
 
-if (!$is_certificate_mode)
+if (!$is_certificate_mode) {
     $interbreadcrumb[] = array(
         "url" => "./document.php?curdirpath=".urlencode($my_cur_dir_path).'&'.api_get_cidreq(),
         "name" => get_lang('Documents'),
     );
-else
+} else {
     $interbreadcrumb[] = array('url' => '../gradebook/'.$_SESSION['gradebook_dest'], 'name' => get_lang('Gradebook'));
+}
 
 // Interbreadcrumb for the current directory root path
 if (empty($document_data['parents'])) {

main/dropbox/dropbox_class.inc.php

Braces +9 added lines, -3 removed lines

@@ -491,13 +491,19 @@
 		$id = intval($id);
 		$sql = "DELETE FROM ". Database::get_course_table(TABLE_DROPBOX_FILE) ."
 		        WHERE c_id = $course_id AND cat_id = '".$id."' ";
-		if (!Database::query($sql)) return false;
+		if (!Database::query($sql)) {
+		    return false;
+		}
 		$sql = "DELETE FROM ". Database::get_course_table(TABLE_DROPBOX_CATEGORY) ."
 		        WHERE c_id = $course_id AND cat_id = '".$id."' ";
-		if (!Database::query($sql)) return false;
+		if (!Database::query($sql)) {
+		    return false;
+		}
 		$sql = "DELETE FROM ".Database::get_course_table(TABLE_DROPBOX_POST)."
 		        WHERE c_id = $course_id AND cat_id = '".$id."' ";
-		if (!Database::query($sql)) return false;
+		if (!Database::query($sql)) {
+		    return false;
+		}
 		return true;
 	}