Minor - fixing PHP warnings.

1.9.x
Julio Montoya 12 years ago
parent 3eedd8c42a
commit fc6b904efb
  1. 52
      main/admin/session_course_edit.php

@ -12,8 +12,7 @@ require_once '../inc/global.inc.php';
$id_session = intval($_GET['id_session']);
SessionManager::protect_session_edit($id_session);
$course_code=trim(stripslashes($_GET['course_code']));
$course_code = Database::escape_string($_GET['course_code']);
$formSent=0;
$errorMsg='';
@ -23,7 +22,7 @@ $tbl_user = Database::get_main_table(TABLE_MAIN_USER);
$tbl_course = Database::get_main_table(TABLE_MAIN_COURSE);
$tbl_session = Database::get_main_table(TABLE_MAIN_SESSION);
$tbl_session_course = Database::get_main_table(TABLE_MAIN_SESSION_COURSE);
$tbl_session_rel_course_rel_user = Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER);
$tbl_session_rel_course_rel_user = Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER);
$course_info=api_get_course_info($_REQUEST['course_code']);
$tool_name=$course_info['name'];
@ -45,7 +44,8 @@ if ($_POST['formSent']) {
$formSent=1;
// get all tutor by course_code in the session
$sql = "SELECT id_user FROM $tbl_session_rel_course_rel_user WHERE id_session = '$id_session' AND course_code = '$course_code' AND status = 2";
$sql = "SELECT id_user FROM $tbl_session_rel_course_rel_user
WHERE id_session = '$id_session' AND course_code = '$course_code' AND status = 2";
$rs_coachs = Database::query($sql);
$coachs_course_session = array();
@ -76,7 +76,8 @@ if ($_POST['formSent']) {
}
} else {
$sql = "SELECT id_user FROM $tbl_session_rel_course_rel_user WHERE id_session = '$id_session' AND course_code = '$course_code' AND status = 2 ";
$sql = "SELECT id_user FROM $tbl_session_rel_course_rel_user
WHERE id_session = '$id_session' AND course_code = '$course_code' AND status = 2 ";
$rs = Database::query($sql);
if (Database::num_rows($rs) > 0) {
@ -91,25 +92,25 @@ global $_configuration;
if ($_configuration['multiple_access_urls']) {
$tbl_access_rel_user= Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
$access_url_id = api_get_current_access_url_id();
$sql="SELECT u.user_id,lastname,firstname,username FROM $tbl_user u LEFT JOIN $tbl_access_rel_user a ON(u.user_id= a.user_id) WHERE status='1' AND access_url_id = $access_url_id ".$order_clause;
$sql="SELECT u.user_id,lastname,firstname,username
FROM $tbl_user u LEFT JOIN $tbl_access_rel_user a ON(u.user_id= a.user_id)
WHERE status='1' AND access_url_id = $access_url_id ".$order_clause;
} else {
$sql="SELECT user_id,lastname,firstname,username FROM $tbl_user WHERE status='1'".$order_clause;
$sql="SELECT user_id,lastname,firstname,username
FROM $tbl_user
WHERE status='1'".$order_clause;
}
$result=Database::query($sql);
$coaches=Database::store_result($result);
$result = Database::query($sql);
$coaches = Database::store_result($result);
Display::display_header($tool_name);
$tool_name=get_lang('ModifySessionCourse');
$tool_name = get_lang('ModifySessionCourse');
api_display_tool_title($tool_name);
?>
<form method="post" action="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&page=<?php echo Security::remove_XSS($_GET['page']) ?>" style="margin:0px;">
<input type="hidden" name="formSent" value="1">
<table border="0" cellpadding="5" cellspacing="0" width="600px">
<?php
if(!empty($errorMsg)) {
?>
@ -126,25 +127,34 @@ if(!empty($errorMsg)) {
<tr>
<td width="25%" valign = "top"><?php echo get_lang("CoachName") ?>&nbsp;&nbsp;</td>
<td width="70%"><select name="id_coach[]" style="width:450px;" multiple="multiple" size="7">
<option value="0">----- <?php echo get_lang("Choose") ?> -----</option>
<option value="0" <?php if(count($arr_infos) == 0) echo 'selected="selected"'; ?>><?php echo get_lang('None') ?></option>
<td width="70%">
<select name="id_coach[]" style="width:450px;" multiple="multiple" size="7">
<option value="0">----- <?php echo get_lang("Choose") ?> -----</option>
<option value="0" <?php if(count($arr_infos) == 0) echo 'selected="selected"'; ?>>
<?php echo get_lang('None') ?>
</option>
<?php
foreach($coaches as $enreg) {
?>
<option value="<?php echo $enreg['user_id']; ?>" <?php if((!$sent && (is_array($arr_infos) && in_array($enreg['user_id'],$arr_infos))) || ($sent && $enreg['user_id'] == $id_coach)) echo 'selected="selected"'; ?>><?php echo api_get_person_name($enreg['firstname'], $enreg['lastname']).' ('.$enreg['username'].')'; ?></option>
<option value="<?php echo $enreg['user_id']; ?>" <?php if(((is_array($arr_infos) && in_array($enreg['user_id'], $arr_infos)))) echo 'selected="selected"'; ?>>
<?php echo api_get_person_name($enreg['firstname'], $enreg['lastname']).' ('.$enreg['username'].')'; ?>
</option>
<?php
}
unset($coaches);
?>
</select></td>
</select>
</td>
</tr>
<tr>
<td>&nbsp;</td>
<td><button class="save" type="submit" name="name" value="<?php echo get_lang('AssignCoach') ?>"><?php echo get_lang('AssignCoach') ?></button>
<td>
<button class="save" type="submit" name="name" value="<?php echo get_lang('AssignCoach') ?>">
<?php echo get_lang('AssignCoach') ?>
</button>
</td>
</tr>
</table>
</form>
<?php
Display::display_footer();
Display::display_footer();

Loading…
Cancel
Save