chamilo / chamilo-lms

main/cron/lang/langstats.php

Indentation +31 added lines, -31 removed lines

@@ -22,59 +22,59 @@
 $output = false;
 $ls = new langstats();
 if ($ls === false) {
-  exit($ls->error);
+    exit($ls->error);
 }
 $list = $ls->get_popular_terms($x_most_popular);
 if ($_GET['output'] == 1) {
-  $output = true;
-  $variables_origin = $ls->get_variables_origin();
+    $output = true;
+    $variables_origin = $ls->get_variables_origin();
 }
 /**
  * Display
  */
 if (count($list) == 0) { echo 'No terms loaded so far'; }
 if (count($list) > 0) {
-  $i = 1;
-  $j = 1;
-  $k = 0;
-  $files = array();
-  $trans = array();
-  echo 'Number of records: '.count($list).'<br />';
-  echo '<table><tr><th>Index</th><th>Registration order</th><th>Term</th>'.($output == 1 ? '<th>Origin</th>' : '').'<th>Count</th></tr>';
-  foreach ($list as $elem) {
+    $i = 1;
+    $j = 1;
+    $k = 0;
+    $files = array();
+    $trans = array();
+    echo 'Number of records: '.count($list).'<br />';
+    echo '<table><tr><th>Index</th><th>Registration order</th><th>Term</th>'.($output == 1 ? '<th>Origin</th>' : '').'<th>Count</th></tr>';
+    foreach ($list as $elem) {
     if ($k > $terms_limit) { break; }
     $fixed_elem = $elem;
     if ($output) {
-      if (empty($variables_origin[$elem['term_name']]) && !empty($variables_origin['lang'.$elem['term_name']])) {
+        if (empty($variables_origin[$elem['term_name']]) && !empty($variables_origin['lang'.$elem['term_name']])) {
         $fixed_elem = array('id' => $elem['id'], 'term_name' => 'lang'.$elem['term_name'], 'term_count' => $elem['term_count']);
-      }
-      if (empty($variables_origin[$fixed_elem['term_name']])) {
+        }
+        if (empty($variables_origin[$fixed_elem['term_name']])) {
         continue; 
-      }
-      $files[$variables_origin[$fixed_elem['term_name']]][] = $fixed_elem['term_name'];
-      $translation = get_lang($fixed_elem['term_name']);
-      $k += str_word_count($translation);
-      $trans[$fixed_elem['term_name']] = $translation;
-      $j++;
+        }
+        $files[$variables_origin[$fixed_elem['term_name']]][] = $fixed_elem['term_name'];
+        $translation = get_lang($fixed_elem['term_name']);
+        $k += str_word_count($translation);
+        $trans[$fixed_elem['term_name']] = $translation;
+        $j++;
     }
     echo '<tr><td>', $i,
-      '</td><td>', $fixed_elem['id'],
-      '</td><td>', $fixed_elem['term_name'];
+        '</td><td>', $fixed_elem['id'],
+        '</td><td>', $fixed_elem['term_name'];
     if ($output) {
-      echo '</td><td>'.$variables_origin[$fixed_elem['term_name']];
+        echo '</td><td>'.$variables_origin[$fixed_elem['term_name']];
     }
     echo '</td><td>', $fixed_elem['term_count'], '</td></tr>';
     $i++;
-  }
-  echo '</table>';
-  if ($output) {
+    }
+    echo '</table>';
+    if ($output) {
     @mkdir('/tmp/lang');
     foreach ($files as $file => $terms) {
-      @touch('/tmp/lang/'.$file);
-      file_put_contents('/tmp/lang/'.$file, "<?php".PHP_EOL);
-      foreach ($terms as $term) {
+        @touch('/tmp/lang/'.$file);
+        file_put_contents('/tmp/lang/'.$file, "<?php".PHP_EOL);
+        foreach ($terms as $term) {
         file_put_contents('/tmp/lang/'.$file, '$'.$term.' = "'.str_replace('"', '\"', $trans[$term]).'";'.PHP_EOL, FILE_APPEND);
-      }
+        }
     }  
-  }
+    }
 }

main/auth/external_login/functions.inc.php

Braces +2 added lines, -1 removed lines

@@ -24,7 +24,8 @@
     $sql = "SELECT * from $table where username='".Database::escape_string($login)."'";
     $result = Database::query($sql);
 
-    if (Database::num_rows($result) == 0) { //false password
+    if (Database::num_rows($result) == 0) {
+//false password
         return false;
     }
     $user_info = Database::fetch_assoc($result);

plugin/jcapture/action.php

Indentation +11 added lines, -11 removed lines

@@ -11,18 +11,18 @@
  
 class action_plugin_jcapture extends DokuWiki_Action_Plugin {
  
-  /**
-   * return some info
-   */
-  function getInfo() {
+    /**
+     * return some info
+     */
+    function getInfo() {
     return array(
-                 'author' => 'Pavel Vlasov',
-                 'email'  => 'Pavel.Vlasov@nasdanika.com',
-                 'name'   => 'JCapture',
-                 'desc'   => 'Plugin for making screen captures.',
-                 'url'    => 'http://www.nasdanika.com/wiki/doku.php?id=products:jcapture:start',
-                 );
-  }
+                    'author' => 'Pavel Vlasov',
+                    'email'  => 'Pavel.Vlasov@nasdanika.com',
+                    'name'   => 'JCapture',
+                    'desc'   => 'Plugin for making screen captures.',
+                    'url'    => 'http://www.nasdanika.com/wiki/doku.php?id=products:jcapture:start',
+                    );
+    }
 
     /**
      * Register the eventhandlers

Braces +14 added lines, -6 removed lines

@@ -5,16 +5,22 @@ 
  * @author Pavel Vlasov
  */
  
-if (!defined('DOKU_INC')) die();
-if (!defined('DOKU_PLUGIN')) define('DOKU_PLUGIN', DOKU_INC.'lib/plugins/');
+if (!defined('DOKU_INC')) {
+    die();
+}
+if (!defined('DOKU_PLUGIN')) {
+    define('DOKU_PLUGIN', DOKU_INC.'lib/plugins/');
+}
 require_once (DOKU_PLUGIN.'action.php');
  
-class action_plugin_jcapture extends DokuWiki_Action_Plugin {
+class action_plugin_jcapture extends DokuWiki_Action_Plugin
+{
  
   /**
    * return some info
    */
-  function getInfo() {
+  function getInfo()
+  {
     return array(
                  'author' => 'Pavel Vlasov',
                  'email'  => 'Pavel.Vlasov@nasdanika.com',
@@ -27,14 +33,16 @@ 
     /**
      * Register the eventhandlers
      */
-    function register(&$controller) {
+    function register(&$controller)
+    {
         $controller->register_hook('TOOLBAR_DEFINE', 'AFTER', $this, 'insert_button', array());
     }
  
     /**
      * Inserts the toolbar button
      */
-    function insert_button(& $event, $param) {
+    function insert_button(& $event, $param)
+    {
         $event->data[] = array(
             'type' => 'JCapture',
             'title' => 'Screen capture',

plugin/date/index.act.php

Indentation +8 added lines, -8 removed lines

@@ -7,15 +7,15 @@
  * Initialization
  */
 $convert_lang_to_code = array(
-  "english" => "en_US",
-  "french"  => "fr_BE",
-  "dutch" => "nl_NL",
-  "german" => "de_DE",
-  "japanese" => "ja_JP",
-  "danish" => "da_DK",
+    "english" => "en_US",
+    "french"  => "fr_BE",
+    "dutch" => "nl_NL",
+    "german" => "de_DE",
+    "japanese" => "ja_JP",
+    "danish" => "da_DK",
 );
 if (!empty($_SESSION['user_language_choice']) && !empty($convert_lang_to_code[$_SESSION['user_language_choice']])) {
-  $code = $convert_lang_to_code[$_SESSION['user_language_choice']];
-  $locale = setlocale(LC_TIME, $code);
+    $code = $convert_lang_to_code[$_SESSION['user_language_choice']];
+    $locale = setlocale(LC_TIME, $code);
 }
 $date = strftime('%c');

main/gradebook/lib/fe/gradebooktable.class.php

Doc Comments +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
      * @param array $links
      * @param null $addparams
      * @param bool $exportToPdf
-     * @param null $showTeacherView
+     * @param boolean $showTeacherView
      * @param int $userId
      * @param array $studentList
      */
@@ -918,7 +918,7 @@ 
 
     /**
      * @param $item
-     * @return mixed
+     * @return string|null
      */
     private function build_course_code($item)
     {

main/inc/lib/search/ChamiloQuery.php

Doc Comments +4 added lines, -1 removed lines

@@ -23,6 +23,9 @@ 
     return chamilo_preprocess_results($results);
 }
 
+/**
+ * @param string $query_string
+ */
 function chamilo_query_simple_query($query_string, $offset = 0, $length = 10, $extra = NULL) {
     return xapian_query($query_string, NULL, $offset, $length, $extra);
 }
@@ -30,7 +33,7 @@ 
 /**
  * Wrapper for getting boolean queries
  *
- * @param   string    $query_string   The term string
+ * @param string $term
  */
 function chamilo_get_boolean_query($term) {
   return xapian_get_boolean_query($term);

Indentation +16 added lines, -16 removed lines

@@ -33,7 +33,7 @@ 
  * @param   string    $query_string   The term string
  */
 function chamilo_get_boolean_query($term) {
-  return xapian_get_boolean_query($term);
+    return xapian_get_boolean_query($term);
 }
 
 /**
@@ -44,22 +44,22 @@ 
     $results_by_tool = array();
     if (count($results) > 0) {
 
-	    foreach ($results as $key => $row) {
-	        $results_by_tool[$row['toolid']][] = $row;
-	    }
+        foreach ($results as $key => $row) {
+            $results_by_tool[$row['toolid']][] = $row;
+        }
 
-	    $processed_results = array();
-	    foreach ($results_by_tool as $toolid => $rows) {
-	        $tool_processor_class = $toolid.'_processor';
-	        $tool_processor_path = api_get_path(LIBRARY_PATH).'search/tool_processors/'.$tool_processor_class.'.class.php';
-	        if (file_exists($tool_processor_path)) {
-	            require_once($tool_processor_path);
-	            $tool_processor = new $tool_processor_class($rows);
-	            $processed_results = array_merge($tool_processor->process(), $processed_results);
-	        }
-	    }
+        $processed_results = array();
+        foreach ($results_by_tool as $toolid => $rows) {
+            $tool_processor_class = $toolid.'_processor';
+            $tool_processor_path = api_get_path(LIBRARY_PATH).'search/tool_processors/'.$tool_processor_class.'.class.php';
+            if (file_exists($tool_processor_path)) {
+                require_once($tool_processor_path);
+                $tool_processor = new $tool_processor_class($rows);
+                $processed_results = array_merge($tool_processor->process(), $processed_results);
+            }
+        }
 
-	    return array(count($processed_results), $processed_results);
+        return array(count($processed_results), $processed_results);
     }
 }
 
@@ -72,5 +72,5 @@ 
  * @return XapianQuery query joined
  */
 function chamilo_join_queries($query1, $query2 = NULL, $op = 'or') {
-	return xapian_join_queries($query1, $query2, $op);
+    return xapian_join_queries($query1, $query2, $op);
 }

Braces +10 added lines, -5 removed lines

@@ -18,12 +18,14 @@ 
  * @param   array     extra           Extra queries to join with. Optional
  * @return  array
  */
-function chamilo_query_query($query_string, $offset = 0, $length = 10, $extra = NULL) {
+function chamilo_query_query($query_string, $offset = 0, $length = 10, $extra = NULL)
+{
     list($count, $results) = xapian_query($query_string, NULL, $offset, $length, $extra);
     return chamilo_preprocess_results($results);
 }
 
-function chamilo_query_simple_query($query_string, $offset = 0, $length = 10, $extra = NULL) {
+function chamilo_query_simple_query($query_string, $offset = 0, $length = 10, $extra = NULL)
+{
     return xapian_query($query_string, NULL, $offset, $length, $extra);
 }
 
@@ -32,14 +34,16 @@ 
  *
  * @param   string    $query_string   The term string
  */
-function chamilo_get_boolean_query($term) {
+function chamilo_get_boolean_query($term)
+{
   return xapian_get_boolean_query($term);
 }
 
 /**
  * Preprocess all results depending on the toolid
  */
-function chamilo_preprocess_results($results) {
+function chamilo_preprocess_results($results)
+{
     // group by toolid
     $results_by_tool = array();
     if (count($results) > 0) {
@@ -71,6 +75,7 @@ 
  * @param string $op
  * @return XapianQuery query joined
  */
-function chamilo_join_queries($query1, $query2 = NULL, $op = 'or') {
+function chamilo_join_queries($query1, $query2 = NULL, $op = 'or')
+{
 	return xapian_join_queries($query1, $query2, $op);
 }

Upper-Lower-Casing +5 added lines, -5 removed lines

@@ -18,13 +18,13 @@ 
  * @param   array     extra           Extra queries to join with. Optional
  * @return  array
  */
-function chamilo_query_query($query_string, $offset = 0, $length = 10, $extra = NULL) {
-    list($count, $results) = xapian_query($query_string, NULL, $offset, $length, $extra);
+function chamilo_query_query($query_string, $offset = 0, $length = 10, $extra = null) {
+    list($count, $results) = xapian_query($query_string, null, $offset, $length, $extra);
     return chamilo_preprocess_results($results);
 }
 
-function chamilo_query_simple_query($query_string, $offset = 0, $length = 10, $extra = NULL) {
-    return xapian_query($query_string, NULL, $offset, $length, $extra);
+function chamilo_query_simple_query($query_string, $offset = 0, $length = 10, $extra = null) {
+    return xapian_query($query_string, null, $offset, $length, $extra);
 }
 
 /**
@@ -71,6 +71,6 @@ 
  * @param string $op
  * @return XapianQuery query joined
  */
-function chamilo_join_queries($query1, $query2 = NULL, $op = 'or') {
+function chamilo_join_queries($query1, $query2 = null, $op = 'or') {
 	return xapian_join_queries($query1, $query2, $op);
 }

main/inc/lib/formvalidator/Rule/MultipleRequired.php

Indentation +4 added lines, -4 removed lines

@@ -27,10 +27,10 @@
      */
     function validate($value, $options = null)
     {
-    	if (is_array($value))
-    	{
-    		$value = implode(null, $value);
-    	}
+        if (is_array($value))
+        {
+            $value = implode(null, $value);
+        }
         if ((string) $value == '') {
             return false;
         }

Braces +1 added lines, -2 removed lines

@@ -27,8 +27,7 @@
      */
     function validate($value, $options = null)
     {
-    	if (is_array($value))
-    	{
+    	if (is_array($value)) {
     		$value = implode(null, $value);
     	}
         if ((string) $value == '') {

main/inc/lib/rights.lib.php

Indentation +33 added lines, -33 removed lines

@@ -6,42 +6,42 @@
  * @deprecated Don't use this class
  */
 class Rights {
-	private static $rights_cache = array();
-	private static $rights = array(
-		'show_tabs:reports' =>
-			array(
-				'type' => 'const',
-				'const' => 'true' )
-		);
+    private static $rights_cache = array();
+    private static $rights = array(
+        'show_tabs:reports' =>
+            array(
+                'type' => 'const',
+                'const' => 'true' )
+        );
 
-	// warning the goal of this function is to enforce rights managment in Chamilo
-	// thus default return value is always true
-	public static function hasRight($handler) {
-		if (array_key_exists($handler, self::$rights_cache))
-			return self::$rights_cache[$handler];
+    // warning the goal of this function is to enforce rights managment in Chamilo
+    // thus default return value is always true
+    public static function hasRight($handler) {
+        if (array_key_exists($handler, self::$rights_cache))
+            return self::$rights_cache[$handler];
 
-		if (!array_key_exists($handler, self::$rights))
-			return true; // handler does not exists
+        if (!array_key_exists($handler, self::$rights))
+            return true; // handler does not exists
 
-		if (self::$rights[$handler]['type'] == 'sql') {
-			$result = Database::query(self::$rights[$handler]['sql']);
-			if (Database::num_rows($result) > 0)
-				$result = true;
-			else
-				$result = false;
-		} else if (self::$rights[$handler]['type'] == 'const')
-			$result = self::$rights[$handler]['const'];
-		else if (self::$rights[$handler]['type'] == 'func')
-			$result = self::$rights[$handler]['func']();
-		else // handler type not implemented
-			return true;
-		self::$rights_cache[$handler] = $result;
-		return $result;
-	}
+        if (self::$rights[$handler]['type'] == 'sql') {
+            $result = Database::query(self::$rights[$handler]['sql']);
+            if (Database::num_rows($result) > 0)
+                $result = true;
+            else
+                $result = false;
+        } else if (self::$rights[$handler]['type'] == 'const')
+            $result = self::$rights[$handler]['const'];
+        else if (self::$rights[$handler]['type'] == 'func')
+            $result = self::$rights[$handler]['func']();
+        else // handler type not implemented
+            return true;
+        self::$rights_cache[$handler] = $result;
+        return $result;
+    }
 
-	public static function hasRightClosePageWithError($handler) {
-		if (hasRight($handler) == false)
-			die("You are not allowed here"); //FIXME
-	}
+    public static function hasRightClosePageWithError($handler) {
+        if (hasRight($handler) == false)
+            die("You are not allowed here"); //FIXME
+    }
 
 }

Braces +29 added lines, -18 removed lines

@@ -5,7 +5,8 @@ 
  * Class Rights
  * @deprecated Don't use this class
  */
-class Rights {
+class Rights
+{
 	private static $rights_cache = array();
 	private static $rights = array(
 		'show_tabs:reports' =>
@@ -16,32 +17,42 @@ 
 
 	// warning the goal of this function is to enforce rights managment in Chamilo
 	// thus default return value is always true
-	public static function hasRight($handler) {
-		if (array_key_exists($handler, self::$rights_cache))
-			return self::$rights_cache[$handler];
+	public static function hasRight($handler)
+	{
+		if (array_key_exists($handler, self::$rights_cache)) {
+					return self::$rights_cache[$handler];
+		}
 
-		if (!array_key_exists($handler, self::$rights))
-			return true; // handler does not exists
+		if (!array_key_exists($handler, self::$rights)) {
+					return true;
+		}
+		// handler does not exists
 
 		if (self::$rights[$handler]['type'] == 'sql') {
 			$result = Database::query(self::$rights[$handler]['sql']);
-			if (Database::num_rows($result) > 0)
-				$result = true;
-			else
-				$result = false;
-		} else if (self::$rights[$handler]['type'] == 'const')
-			$result = self::$rights[$handler]['const'];
-		else if (self::$rights[$handler]['type'] == 'func')
-			$result = self::$rights[$handler]['func']();
-		else // handler type not implemented
+			if (Database::num_rows($result) > 0) {
+							$result = true;
+			} else {
+							$result = false;
+			}
+		} else if (self::$rights[$handler]['type'] == 'const') {
+					$result = self::$rights[$handler]['const'];
+		} else if (self::$rights[$handler]['type'] == 'func') {
+					$result = self::$rights[$handler]['func']();
+		} else {
+		    // handler type not implemented
 			return true;
+		}
 		self::$rights_cache[$handler] = $result;
 		return $result;
 	}
 
-	public static function hasRightClosePageWithError($handler) {
-		if (hasRight($handler) == false)
-			die("You are not allowed here"); //FIXME
+	public static function hasRightClosePageWithError($handler)
+	{
+		if (hasRight($handler) == false) {
+					die("You are not allowed here");
+		}
+		//FIXME
 	}
 
 }

main/inc/lib/system/web/request.class.php

Braces +2 added lines, -1 removed lines

@@ -15,7 +15,8 @@
         return isset($_REQUEST[$key]) ? $_REQUEST[$key] : $default;
     }
 
-    public static function has($key) {
+    public static function has($key)
+    {
         return isset($_REQUEST[$key]);
     }
 

Doc Comments +7 added lines

@@ -9,6 +9,9 @@ 
 class Request
 {
 
+    /**
+     * @param string $key
+     */
     public static function get($key, $default = null)
     {
         return isset($_REQUEST[$key]) ? $_REQUEST[$key] : $default;
@@ -30,6 +33,10 @@ 
         return $method == 'GET';
     }
 
+    /**
+     * @param string $key
+     * @param string $default
+     */
     public static function post($key, $default = null)
     {
         return isset($_POST[$key]) ? $_POST[$key] : $default;