Преглед на файлове

Removing notices/warnings

Julio Montoya преди 13 години
родител
ревизия
6e27190325
променени са 6 файла, в които са добавени 12 реда и са изтрити 21 реда
  1. 2 1
      main/inc/lib/social.lib.php
  2. 1 1
      main/inc/lib/tracking.lib.php
  3. 3 6
      main/mySpace/index.php
  4. 2 4
      main/social/groups.php
  5. 2 4
      main/social/profile.php
  6. 2 5
      main/tracking/courseLog.php

+ 2 - 1
main/inc/lib/social.lib.php

@@ -517,7 +517,7 @@ class SocialManager extends UserManager {
 
 				$session = array();
 				$session['title'] = $my_course['session_name'];
-				if ( $my_course['date_start']=='0000-00-00' ) {
+				if ($my_course['date_start']=='0000-00-00') {
 					$session['dates'] = get_lang('WithoutTimeLimits');
 					if ( api_get_setting('show_session_coach') === 'true' ) {
 						$session['coach'] = get_lang('GeneralCoach').': '.api_get_person_name($sessioncoach['firstname'], $sessioncoach['lastname']);
@@ -531,6 +531,7 @@ class SocialManager extends UserManager {
 					$active = ($date_start <= $now && $date_end >= $now)?true:false;
 				}
 			}			
+			$my_course['id_session'] = isset($my_course['id_session']) ? $my_course['id_session'] : 0;
 			$output = array ($my_course['user_course_cat'], $result, $my_course['id_session'], $session, 'active'=>$active);
 		} else {
 			$output = array ($my_course['user_course_cat'], $result);

+ 1 - 1
main/inc/lib/tracking.lib.php

@@ -3065,7 +3065,7 @@ class TrackingCourseLog {
         foreach ($extra_fields as $key=>$field) {
             // show only extra fields that are visible + and can be filtered, added by J.Montoya
             if ($field[6]==1 && $field[8] == 1) {
-                if ($field[0] == $_GET['additional_profile_field'] ) {
+                if (isset($_GET['additional_profile_field']) && $field[0] == $_GET['additional_profile_field'] ) {
                     $selected = 'selected="selected"';
                 } else {
                     $selected = '';

+ 3 - 6
main/mySpace/index.php

@@ -1,9 +1,6 @@
 <?php
 /* For licensing terms, see /license.txt */
-/**
- * @todo use constant for $this_section
- */
-// name of the language file that needs to be included
+
 $language_file = array('registration', 'index', 'tracking');
 
 // resetting the course id
@@ -137,7 +134,7 @@ global $_configuration;
 // && isset($_GET['display']) && $_GET['display'] != 'yourstudents'
 
 if (api_is_allowed_to_create_course()) {
-	$session_id = intval($_GET['session_id']);
+	$session_id = isset($_GET['session_id']) ? intval($_GET['session_id']): 0;
 	if (!empty($session_id)) {
 		$courses = Tracking::get_courses_followed_by_coach($user_id, $session_id);
 	} else {
@@ -578,7 +575,7 @@ if (api_is_allowed_to_create_course() && $view == 'teacher') {
 			$tracking_column = 0;
 		}
 
-		if ($_GET['tracking_direction'] == 'DESC') {
+		if (isset($_GET['tracking_direction']) &&  $_GET['tracking_direction'] == 'DESC') {
 			usort($all_data, 'rsort_sessions');
 		} else {
 			usort($all_data, 'sort_sessions');

+ 2 - 4
main/social/groups.php

@@ -4,9 +4,7 @@
  * @package chamilo.social
  * @author Julio Montoya <gugli100@gmail.com>
  */
-/**
- * Initialization
- */
+
 $cidReset = true;
 $language_file = array('userInfo');
 require_once '../inc/global.inc.php';
@@ -369,7 +367,7 @@ if ($group_id != 0 ) {
 				
                 /*$join_url = '<a href="groups.php?id='.$group_id.'&action=join&u='.api_get_user_id().'">'.Display::return_icon('group_join.png', get_lang('JoinGroup'), array('hspace'=>'6')).''.get_lang('JoinGroup').'</a> ';                
 				$item_4 = '<div class="box_description_group_actions" >'.$join_url. $url_open.get_lang('SeeMore').$url_close.'</div>';*/				
-				$grid_item_2 = $item_0.$item_1.$item_2.$item_3.$item_4;
+				$grid_item_2 = $item_0.$item_1.$item_2.$item_3;
 				$grid_my_groups[]= array($grid_item_2);
 			}
 		}

+ 2 - 4
main/social/profile.php

@@ -6,9 +6,7 @@
 * @author Isaac Flores Paz <florespaz_isaac@hotmail.com>
 * @package chamilo.social
 */
-/**
- * Initialization
- */
+
 $language_file = array('userInfo');
 $cidReset = true;
 require_once '../inc/global.inc.php';
@@ -469,7 +467,7 @@ if ($show_full_profile) {
 			if (api_get_user_id() == $user_id) {
 				//$item_actions = '<div class="box_shared_profile_group_actions"><a href="groups.php?id='.$id.'">'.get_lang('SeeMore').$url_close.'</div>';
 			}
-			$grid_my_groups[]= array($item_name,$url_open.$result['picture_uri'].$url_close, $item_description.$item_actions);
+			$grid_my_groups[]= array($item_name,$url_open.$result['picture_uri'].$url_close, $item_actions);
 			$i++;
 		}
 	}

+ 2 - 5
main/tracking/courseLog.php

@@ -4,9 +4,6 @@
 /**
  *	@package chamilo.tracking
  */
-/**
- * Code
- */
 
 /* INIT SECTION */
 
@@ -161,13 +158,13 @@ echo '<a href="javascript: void(0);" onclick="javascript: window.print();">'.Dis
 
 if ($_GET['studentlist'] == 'false') {
     echo '<a href="'.api_get_self().'?'.api_get_cidreq().'&id_session='.api_get_session_id().'&export=csv&studentlist=false">
-	'.Display::return_icon('export_csv.png', get_lang('ExportAsCSV'),'','32').'</a>';
-	
+	'.Display::return_icon('export_csv.png', get_lang('ExportAsCSV'),'','32').'</a>';	
 } elseif ($_GET['studentlist'] == '' || $_GET['studentlist'] == 'true') {
     $addional_param = '';
     if (isset($_GET['additional_profile_field'])) {
         $addional_param ='additional_profile_field='.intval($_GET['additional_profile_field']);
     }
+    $users_tracking_per_page = '';
     if (isset($_GET['users_tracking_per_page'])) {
         $users_tracking_per_page= '&users_tracking_per_page='.intval($_GET['users_tracking_per_page']);
     }