chamilo / chamilo-lms

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

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

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]);
     }
 

main/inc/lib/search/ChamiloQuery.php

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);
 }

main/session/session_category_add.php

Braces +172 added lines, -43 removed lines

@@ -72,7 +72,10 @@ 
             <div class="form-group">
                 <label class="col-sm-3 control-label"><?php echo get_lang('SessionCategoryName') ?></label>
                 <div class="col-sm-6">
-                    <input type="text" class="form-control" name="name" placeholder="<?php echo get_lang('Category') ?>" size="50" maxlength="50" value="<?php if ($formSent) echo api_htmlentities($name, ENT_QUOTES, $charset); ?>">
+                    <input type="text" class="form-control" name="name" placeholder="<?php echo get_lang('Category') ?>" size="50" maxlength="50" value="<?php if ($formSent) {
+    echo api_htmlentities($name, ENT_QUOTES, $charset);
+}
+?>">
                 </div>
                 <div class="col-md-3"></div>
             </div>
@@ -90,58 +93,184 @@ 
                 <div class="col-sm-6">
                     <select name="day_start">
                         <option value="1">01</option>
-                        <option value="2" <?php if ((!$formSent && $thisDay == 2) || ($formSent && $day_start == 2)) echo 'selected="selected"'; ?> >02</option>
-                        <option value="3" <?php if ((!$formSent && $thisDay == 3) || ($formSent && $day_start == 3)) echo 'selected="selected"'; ?> >03</option>
-                        <option value="4" <?php if ((!$formSent && $thisDay == 4) || ($formSent && $day_start == 4)) echo 'selected="selected"'; ?> >04</option>
-                        <option value="5" <?php if ((!$formSent && $thisDay == 5) || ($formSent && $day_start == 5)) echo 'selected="selected"'; ?> >05</option>
-                        <option value="6" <?php if ((!$formSent && $thisDay == 6) || ($formSent && $day_start == 6)) echo 'selected="selected"'; ?> >06</option>
-                        <option value="7" <?php if ((!$formSent && $thisDay == 7) || ($formSent && $day_start == 7)) echo 'selected="selected"'; ?> >07</option>
-                        <option value="8" <?php if ((!$formSent && $thisDay == 8) || ($formSent && $day_start == 8)) echo 'selected="selected"'; ?> >08</option>
-                        <option value="9" <?php if ((!$formSent && $thisDay == 9) || ($formSent && $day_start == 9)) echo 'selected="selected"'; ?> >09</option>
-                        <option value="10" <?php if ((!$formSent && $thisDay == 10) || ($formSent && $day_start == 10)) echo 'selected="selected"'; ?> >10</option>
-                        <option value="11" <?php if ((!$formSent && $thisDay == 11) || ($formSent && $day_start == 11)) echo 'selected="selected"'; ?> >11</option>
-                        <option value="12" <?php if ((!$formSent && $thisDay == 12) || ($formSent && $day_start == 12)) echo 'selected="selected"'; ?> >12</option>
-                        <option value="13" <?php if ((!$formSent && $thisDay == 13) || ($formSent && $day_start == 13)) echo 'selected="selected"'; ?> >13</option>
-                        <option value="14" <?php if ((!$formSent && $thisDay == 14) || ($formSent && $day_start == 14)) echo 'selected="selected"'; ?> >14</option>
-                        <option value="15" <?php if ((!$formSent && $thisDay == 15) || ($formSent && $day_start == 15)) echo 'selected="selected"'; ?> >15</option>
-                        <option value="16" <?php if ((!$formSent && $thisDay == 16) || ($formSent && $day_start == 16)) echo 'selected="selected"'; ?> >16</option>
-                        <option value="17" <?php if ((!$formSent && $thisDay == 17) || ($formSent && $day_start == 17)) echo 'selected="selected"'; ?> >17</option>
-                        <option value="18" <?php if ((!$formSent && $thisDay == 18) || ($formSent && $day_start == 18)) echo 'selected="selected"'; ?> >18</option>
-                        <option value="19" <?php if ((!$formSent && $thisDay == 19) || ($formSent && $day_start == 19)) echo 'selected="selected"'; ?> >19</option>
-                        <option value="20" <?php if ((!$formSent && $thisDay == 20) || ($formSent && $day_start == 20)) echo 'selected="selected"'; ?> >20</option>
-                        <option value="21" <?php if ((!$formSent && $thisDay == 21) || ($formSent && $day_start == 21)) echo 'selected="selected"'; ?> >21</option>
-                        <option value="22" <?php if ((!$formSent && $thisDay == 22) || ($formSent && $day_start == 22)) echo 'selected="selected"'; ?> >22</option>
-                        <option value="23" <?php if ((!$formSent && $thisDay == 23) || ($formSent && $day_start == 23)) echo 'selected="selected"'; ?> >23</option>
-                        <option value="24" <?php if ((!$formSent && $thisDay == 24) || ($formSent && $day_start == 24)) echo 'selected="selected"'; ?> >24</option>
-                        <option value="25" <?php if ((!$formSent && $thisDay == 25) || ($formSent && $day_start == 25)) echo 'selected="selected"'; ?> >25</option>
-                        <option value="26" <?php if ((!$formSent && $thisDay == 26) || ($formSent && $day_start == 26)) echo 'selected="selected"'; ?> >26</option>
-                        <option value="27" <?php if ((!$formSent && $thisDay == 27) || ($formSent && $day_start == 27)) echo 'selected="selected"'; ?> >27</option>
-                        <option value="28" <?php if ((!$formSent && $thisDay == 28) || ($formSent && $day_start == 28)) echo 'selected="selected"'; ?> >28</option>
-                        <option value="29" <?php if ((!$formSent && $thisDay == 29) || ($formSent && $day_start == 29)) echo 'selected="selected"'; ?> >29</option>
-                        <option value="30" <?php if ((!$formSent && $thisDay == 30) || ($formSent && $day_start == 30)) echo 'selected="selected"'; ?> >30</option>
-                        <option value="31" <?php if ((!$formSent && $thisDay == 31) || ($formSent && $day_start == 31)) echo 'selected="selected"'; ?> >31</option>
+                        <option value="2" <?php if ((!$formSent && $thisDay == 2) || ($formSent && $day_start == 2)) {
+    echo 'selected="selected"';
+}
+?> >02</option>
+                        <option value="3" <?php if ((!$formSent && $thisDay == 3) || ($formSent && $day_start == 3)) {
+    echo 'selected="selected"';
+}
+?> >03</option>
+                        <option value="4" <?php if ((!$formSent && $thisDay == 4) || ($formSent && $day_start == 4)) {
+    echo 'selected="selected"';
+}
+?> >04</option>
+                        <option value="5" <?php if ((!$formSent && $thisDay == 5) || ($formSent && $day_start == 5)) {
+    echo 'selected="selected"';
+}
+?> >05</option>
+                        <option value="6" <?php if ((!$formSent && $thisDay == 6) || ($formSent && $day_start == 6)) {
+    echo 'selected="selected"';
+}
+?> >06</option>
+                        <option value="7" <?php if ((!$formSent && $thisDay == 7) || ($formSent && $day_start == 7)) {
+    echo 'selected="selected"';
+}
+?> >07</option>
+                        <option value="8" <?php if ((!$formSent && $thisDay == 8) || ($formSent && $day_start == 8)) {
+    echo 'selected="selected"';
+}
+?> >08</option>
+                        <option value="9" <?php if ((!$formSent && $thisDay == 9) || ($formSent && $day_start == 9)) {
+    echo 'selected="selected"';
+}
+?> >09</option>
+                        <option value="10" <?php if ((!$formSent && $thisDay == 10) || ($formSent && $day_start == 10)) {
+    echo 'selected="selected"';
+}
+?> >10</option>
+                        <option value="11" <?php if ((!$formSent && $thisDay == 11) || ($formSent && $day_start == 11)) {
+    echo 'selected="selected"';
+}
+?> >11</option>
+                        <option value="12" <?php if ((!$formSent && $thisDay == 12) || ($formSent && $day_start == 12)) {
+    echo 'selected="selected"';
+}
+?> >12</option>
+                        <option value="13" <?php if ((!$formSent && $thisDay == 13) || ($formSent && $day_start == 13)) {
+    echo 'selected="selected"';
+}
+?> >13</option>
+                        <option value="14" <?php if ((!$formSent && $thisDay == 14) || ($formSent && $day_start == 14)) {
+    echo 'selected="selected"';
+}
+?> >14</option>
+                        <option value="15" <?php if ((!$formSent && $thisDay == 15) || ($formSent && $day_start == 15)) {
+    echo 'selected="selected"';
+}
+?> >15</option>
+                        <option value="16" <?php if ((!$formSent && $thisDay == 16) || ($formSent && $day_start == 16)) {
+    echo 'selected="selected"';
+}
+?> >16</option>
+                        <option value="17" <?php if ((!$formSent && $thisDay == 17) || ($formSent && $day_start == 17)) {
+    echo 'selected="selected"';
+}
+?> >17</option>
+                        <option value="18" <?php if ((!$formSent && $thisDay == 18) || ($formSent && $day_start == 18)) {
+    echo 'selected="selected"';
+}
+?> >18</option>
+                        <option value="19" <?php if ((!$formSent && $thisDay == 19) || ($formSent && $day_start == 19)) {
+    echo 'selected="selected"';
+}
+?> >19</option>
+                        <option value="20" <?php if ((!$formSent && $thisDay == 20) || ($formSent && $day_start == 20)) {
+    echo 'selected="selected"';
+}
+?> >20</option>
+                        <option value="21" <?php if ((!$formSent && $thisDay == 21) || ($formSent && $day_start == 21)) {
+    echo 'selected="selected"';
+}
+?> >21</option>
+                        <option value="22" <?php if ((!$formSent && $thisDay == 22) || ($formSent && $day_start == 22)) {
+    echo 'selected="selected"';
+}
+?> >22</option>
+                        <option value="23" <?php if ((!$formSent && $thisDay == 23) || ($formSent && $day_start == 23)) {
+    echo 'selected="selected"';
+}
+?> >23</option>
+                        <option value="24" <?php if ((!$formSent && $thisDay == 24) || ($formSent && $day_start == 24)) {
+    echo 'selected="selected"';
+}
+?> >24</option>
+                        <option value="25" <?php if ((!$formSent && $thisDay == 25) || ($formSent && $day_start == 25)) {
+    echo 'selected="selected"';
+}
+?> >25</option>
+                        <option value="26" <?php if ((!$formSent && $thisDay == 26) || ($formSent && $day_start == 26)) {
+    echo 'selected="selected"';
+}
+?> >26</option>
+                        <option value="27" <?php if ((!$formSent && $thisDay == 27) || ($formSent && $day_start == 27)) {
+    echo 'selected="selected"';
+}
+?> >27</option>
+                        <option value="28" <?php if ((!$formSent && $thisDay == 28) || ($formSent && $day_start == 28)) {
+    echo 'selected="selected"';
+}
+?> >28</option>
+                        <option value="29" <?php if ((!$formSent && $thisDay == 29) || ($formSent && $day_start == 29)) {
+    echo 'selected="selected"';
+}
+?> >29</option>
+                        <option value="30" <?php if ((!$formSent && $thisDay == 30) || ($formSent && $day_start == 30)) {
+    echo 'selected="selected"';
+}
+?> >30</option>
+                        <option value="31" <?php if ((!$formSent && $thisDay == 31) || ($formSent && $day_start == 31)) {
+    echo 'selected="selected"';
+}
+?> >31</option>
                     </select>
                     /
                     <select name="month_start">
                           <option value="1">01</option>
-                          <option value="2" <?php if ((!$formSent && $thisMonth == 2) || ($formSent && $month_start == 2)) echo 'selected="selected"'; ?> >02</option>
-                          <option value="3" <?php if ((!$formSent && $thisMonth == 3) || ($formSent && $month_start == 3)) echo 'selected="selected"'; ?> >03</option>
-                          <option value="4" <?php if ((!$formSent && $thisMonth == 4) || ($formSent && $month_start == 4)) echo 'selected="selected"'; ?> >04</option>
-                          <option value="5" <?php if ((!$formSent && $thisMonth == 5) || ($formSent && $month_start == 5)) echo 'selected="selected"'; ?> >05</option>
-                          <option value="6" <?php if ((!$formSent && $thisMonth == 6) || ($formSent && $month_start == 6)) echo 'selected="selected"'; ?> >06</option>
-                          <option value="7" <?php if ((!$formSent && $thisMonth == 7) || ($formSent && $month_start == 7)) echo 'selected="selected"'; ?> >07</option>
-                          <option value="8" <?php if ((!$formSent && $thisMonth == 8) || ($formSent && $month_start == 8)) echo 'selected="selected"'; ?> >08</option>
-                          <option value="9" <?php if ((!$formSent && $thisMonth == 9) || ($formSent && $month_start == 9)) echo 'selected="selected"'; ?> >09</option>
-                          <option value="10" <?php if ((!$formSent && $thisMonth == 10) || ($formSent && $month_start == 10)) echo 'selected="selected"'; ?> >10</option>
-                          <option value="11" <?php if ((!$formSent && $thisMonth == 11) || ($formSent && $month_start == 11)) echo 'selected="selected"'; ?> >11</option>
-                          <option value="12" <?php if ((!$formSent && $thisMonth == 12) || ($formSent && $month_start == 12)) echo 'selected="selected"'; ?> >12</option>
+                          <option value="2" <?php if ((!$formSent && $thisMonth == 2) || ($formSent && $month_start == 2)) {
+    echo 'selected="selected"';
+}
+?> >02</option>
+                          <option value="3" <?php if ((!$formSent && $thisMonth == 3) || ($formSent && $month_start == 3)) {
+    echo 'selected="selected"';
+}
+?> >03</option>
+                          <option value="4" <?php if ((!$formSent && $thisMonth == 4) || ($formSent && $month_start == 4)) {
+    echo 'selected="selected"';
+}
+?> >04</option>
+                          <option value="5" <?php if ((!$formSent && $thisMonth == 5) || ($formSent && $month_start == 5)) {
+    echo 'selected="selected"';
+}
+?> >05</option>
+                          <option value="6" <?php if ((!$formSent && $thisMonth == 6) || ($formSent && $month_start == 6)) {
+    echo 'selected="selected"';
+}
+?> >06</option>
+                          <option value="7" <?php if ((!$formSent && $thisMonth == 7) || ($formSent && $month_start == 7)) {
+    echo 'selected="selected"';
+}
+?> >07</option>
+                          <option value="8" <?php if ((!$formSent && $thisMonth == 8) || ($formSent && $month_start == 8)) {
+    echo 'selected="selected"';
+}
+?> >08</option>
+                          <option value="9" <?php if ((!$formSent && $thisMonth == 9) || ($formSent && $month_start == 9)) {
+    echo 'selected="selected"';
+}
+?> >09</option>
+                          <option value="10" <?php if ((!$formSent && $thisMonth == 10) || ($formSent && $month_start == 10)) {
+    echo 'selected="selected"';
+}
+?> >10</option>
+                          <option value="11" <?php if ((!$formSent && $thisMonth == 11) || ($formSent && $month_start == 11)) {
+    echo 'selected="selected"';
+}
+?> >11</option>
+                          <option value="12" <?php if ((!$formSent && $thisMonth == 12) || ($formSent && $month_start == 12)) {
+    echo 'selected="selected"';
+}
+?> >12</option>
                     </select>
                     /
                       <select name="year_start">
                       <?php
                       for ($i = $thisYear - 5; $i <= ($thisYear + 5); $i++) {
                       ?>
-                              <option value="<?php echo $i; ?>" <?php if ((!$formSent && $thisYear == $i) || ($formSent && $year_start == $i)) echo 'selected="selected"'; ?> ><?php echo $i; ?></option>
+                              <option value="<?php echo $i; ?>" <?php if ((!$formSent && $thisYear == $i) || ($formSent && $year_start == $i)) {
+    echo 'selected="selected"';
+}
+?> ><?php echo $i; ?></option>
                       <?php
                       }
                       ?>

custompages/first_login-dist.php

Braces +4 added lines, -2 removed lines

@@ -11,8 +11,9 @@ 
 /**
  * Security checks
  */
-if (!isset($_SESSION['conditional_login']['uid']))
+if (!isset($_SESSION['conditional_login']['uid'])) {
   die("Not Authorised");
+}
 
 if (isset($_POST['password'])) {
     $u = api_get_user_info($_SESSION['conditional_login']['uid']);
@@ -20,7 +21,8 @@ 
         header('Location: '.api_get_self().'?invalid=2');
         exit();
     }
-    if (empty($_POST['password'])) { //|| !api_check_password($password)) { //Pass must be at least 5 char long with 2 digits and 3 letters
+    if (empty($_POST['password'])) {
+//|| !api_check_password($password)) { //Pass must be at least 5 char long with 2 digits and 3 letters
         header('Location: '.api_get_self().'?invalid=1');
         exit();
     }

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

Braces +2 added lines, -1 removed lines

@@ -13,7 +13,8 @@
      * @param string $current_username
      * @return boolean True if username is available
      */
-    function validate($username, $current_username = null) {
+    function validate($username, $current_username = null)
+    {
         $user_table = Database::get_main_table(TABLE_MAIN_USER);
         $username = Database::escape_string($username);
         $current_username = Database::escape_string($current_username);

main/document/create_document.php

Braces +2 added lines, -1 removed lines

@@ -253,7 +253,8 @@
 // form title
 $form->addElement('header', $nameTools);
 
-if ($is_certificate_mode) {//added condition for certicate in gradebook
+if ($is_certificate_mode) {
+//added condition for certicate in gradebook
     $form->addElement(
         'hidden',
         'certificate',

main/permissions/permissions_functions.inc.php

Braces +10 added lines, -17 removed lines

@@ -206,8 +206,7 @@ 
 function display_checkbox_matrix($permission_array, $tool, $permission, $inherited_permissions = array())
 {
     $checked = "";
-    if (is_array($permission_array[$tool]) AND in_array($permission, $permission_array[$tool]))
-    {
+    if (is_array($permission_array[$tool]) AND in_array($permission, $permission_array[$tool])) {
         $checked = "checked";
     }
     echo "\t\t\t<input type=\"checkbox\" name=\"permission*$tool*$permission\" $checked>\n";
@@ -254,8 +253,7 @@ 
                     echo "</a>";
                 }
             } else {
-                if ($editable)
-                {
+                if ($editable) {
                     $url = api_get_self();
                     $urlparameters = '';
                     foreach ($_GET as $key => $value) {
@@ -273,8 +271,7 @@ 
                     echo "\t\t\t <a href=\"".$url."\">";
                 }
                 echo "<img src=\"../img/wrong.gif\" border=\"0\"/>";
-                if ($editable)
-                {
+                if ($editable) {
                     echo "</a>";
                 }
             }
@@ -306,8 +303,7 @@ 
                 if ($editable) {
                     $url = api_get_self();
                     $urlparameters = '';
-                    foreach ($_GET as $key => $value)
-                    {
+                    foreach ($_GET as $key => $value) {
                         $parameter[$key] = $value;
                     }
                     $parameter['action'] = 'manage_rights';
@@ -315,8 +311,7 @@ 
                     $parameter['permission'] = $permission;
                     $parameter['tool'] = $tool;
                     $parameter['user_id'] = $user_id;
-                    foreach ($parameter as $key=>$value)
-                    {
+                    foreach ($parameter as $key=>$value) {
                         $urlparameters .= $key.'='.$value.'&amp;';
                     }
                     $url = $url.'?'.$urlparameters;
@@ -433,16 +428,15 @@ 
 * @author Patrick Cool <patrick.cool@ugent.be>, Ghent University
 * @version 1.0
 */
-function get_all_roles($content = 'course') {
+function get_all_roles($content = 'course')
+{
     $course_id = api_get_course_int_id();
     $course_id_condition = " WHERE c_id = $course_id ";
 
-    if ($content == 'course')
-    {
+    if ($content == 'course') {
         $table_role = Database::get_course_table(TABLE_ROLE);
     }
-    if ($content == 'platform')
-    {
+    if ($content == 'platform') {
         $table_role = Database::get_main_table(TABLE_ROLE);
         $course_id_condition = '';
     }
@@ -450,8 +444,7 @@ 
     $current_roles = array();
     $sql = "SELECT * FROM $table_role $course_id_condition ";
     $result = Database::query($sql);
-    while ($row = Database::fetch_array($result))
-    {
+    while ($row = Database::fetch_array($result)) {
         $roles[] = $row;
     }