diff --git a/main/exercice/exercise.lib.php b/main/exercice/exercise.lib.php index a8408eb405..ee9e50ea29 100755 --- a/main/exercice/exercise.lib.php +++ b/main/exercice/exercise.lib.php @@ -1297,7 +1297,9 @@ function get_exercise_result_ranking($my_score, $my_exe_id, $exercise_id, $cours //var_dump($my_score, $my_ranking); if ($return_string) { if (!empty($position) && !empty($my_ranking)) { - return $position.'/'.count($my_ranking); + $return_value = $position.'/'.count($my_ranking); + } else { + $return_value = '-'; } } return $return_value; diff --git a/main/inc/lib/tracking.lib.php b/main/inc/lib/tracking.lib.php index 4a0e16ca8f..156453b907 100755 --- a/main/inc/lib/tracking.lib.php +++ b/main/inc/lib/tracking.lib.php @@ -1931,7 +1931,6 @@ class Tracking { '.Display::tag('th', get_lang('Details'), array('class'=>'head')).' '; - $i = 0; foreach ($courses as $enreg) { $weighting = 0; @@ -1971,7 +1970,6 @@ class Tracking { } $html .= ''; $html .= ''; - $i = $i ? 0 : 1; } $html .= ''; } @@ -2062,14 +2060,14 @@ class Tracking { $html .= ' '.get_lang('Course').' - '.Display::tag('th', get_lang('PublishedExercises'),array('class'=>'head')).' - '.Display::tag('th', get_lang('DoneExercises'), array('class'=>'head')).' - '.Display::tag('th', get_lang('AverageExerciseResult'), array('class'=>'head')).' - '.Display::tag('th', get_lang('Time') , array('class'=>'head')).' - '.Display::tag('th', get_lang('LPProgress') , array('class'=>'head')).' + '.Display::tag('th', get_lang('PublishedExercises'), array('class'=>'head')).' + '.Display::tag('th', get_lang('DoneExercises'), array('class'=>'head')).' + '.Display::tag('th', get_lang('AverageExerciseResult'), array('class'=>'head')).' + '.Display::tag('th', get_lang('Time') , array('class'=>'head')).' + '.Display::tag('th', get_lang('LPProgress') , array('class'=>'head')).' '.Display::tag('th', get_lang('Score').Display :: return_icon('info3.gif', get_lang('ScormAndLPTestTotalAverage'), array ('align' => 'absmiddle', 'hspace' => '3px')), array('class'=>'head')).' - '.Display::tag('th', get_lang('LastConnexion'), array('class'=>'head')).' - '.Display::tag('th', get_lang('Details'), array('class'=>'head')).' + '.Display::tag('th', get_lang('LastConnexion'), array('class'=>'head')).' + '.Display::tag('th', get_lang('Details'), array('class'=>'head')).' '; foreach ($course_list as $enreg) { $weighting = 0; @@ -2085,7 +2083,7 @@ class Tracking { $html .= ''; } - $url = api_get_course_url($enreg['code'], $key); + $url = api_get_course_url($enreg['code'], $key); $course_url = Display::url($enreg['title'], $url, array('target'=>'_blank')); $html .= Display::tag('td', $course_url);/* @@ -2124,7 +2122,6 @@ class Tracking { } $details .= ''; $html .= Display::tag('td', $details, array('align'=>'center')); - $i = $i ? 0 : 1; $html .= ''; } $html .= '';