Cleaning code + adding intval value

skala
Julio Montoya 15 years ago
parent a14060e23c
commit 168c093445
  1. 22
      main/inc/lib/group_portal_manager.lib.php

@ -1,11 +1,11 @@
<?php <?php
/* For licensing terms, see /chamilo_license.txt */ /* For licensing terms, see /license.txt */
/** /**
============================================================================== ==============================================================================
* This library provides functions for the group management. * This library provides functions for the group management.
* Include/require it in your code to use its functionality. * Include/require it in your code to use its functionality.
* @author Julio Montoya <gugli100@gmail.com> * @author Julio Montoya <gugli100@gmail.com>
* @package dokeos.library * @package chamilo.library
============================================================================== ==============================================================================
*/ */
// Group permissions // Group permissions
@ -163,7 +163,7 @@ class GroupPortalManager
$tags = array(); $tags = array();
if (Database::num_rows($res)>0) { if (Database::num_rows($res)>0) {
while ($row = Database::fetch_array($res,'ASSOC')) { while ($row = Database::fetch_array($res,'ASSOC')) {
$tags[] = $row; $tags[] = $row;
} }
} }
@ -247,12 +247,12 @@ class GroupPortalManager
$result=Database::query($sql); $result=Database::query($sql);
$array = array(); $array = array();
while ($row = Database::fetch_array($result, 'ASSOC')) { while ($row = Database::fetch_array($result, 'ASSOC')) {
if ($with_image == true) { if ($with_image == true) {
$picture = self::get_picture_group($row['id'], $row['picture_uri'],80); $picture = self::get_picture_group($row['id'], $row['picture_uri'],80);
$img = '<img src="'.$picture['file'].'" />'; $img = '<img src="'.$picture['file'].'" />';
$row['picture_uri'] = $img; $row['picture_uri'] = $img;
} }
$array[$row['id']] = $row; $array[$row['id']] = $row;
} }
return $array; return $array;
} }
@ -434,8 +434,7 @@ class GroupPortalManager
$role = self::get_user_group_role($user_id,$group_id); $role = self::get_user_group_role($user_id,$group_id);
if ($role == 0) { if ($role == 0) {
$sql = "INSERT INTO $table_url_rel_group $sql = "INSERT INTO $table_url_rel_group
SET user_id = ".intval($user_id).", group_id = ".intval($group_id).", relation_type = ".intval($relation_type).""; SET user_id = ".intval($user_id).", group_id = ".intval($group_id).", relation_type = ".intval($relation_type);
$result = Database::query($sql); $result = Database::query($sql);
if ($result) if ($result)
@ -461,6 +460,7 @@ class GroupPortalManager
$table_ = Database :: get_main_table(TABLE_MAIN_USER_REL_GROUP); $table_ = Database :: get_main_table(TABLE_MAIN_USER_REL_GROUP);
$condition_relation = ""; $condition_relation = "";
if (!empty($relation_type)) { if (!empty($relation_type)) {
$relation_type = intval($relation_type);
$condition_relation = " AND relation_type = '$relation_type'"; $condition_relation = " AND relation_type = '$relation_type'";
} }
$sql = "DELETE FROM $table_ WHERE group_id = ".intval($group_id).$condition_relation; $sql = "DELETE FROM $table_ WHERE group_id = ".intval($group_id).$condition_relation;

Loading…
Cancel
Save