diff --git a/main/exercice/exercise.lib.php b/main/exercice/exercise.lib.php index 4d602d997f..0d1e278aee 100644 --- a/main/exercice/exercise.lib.php +++ b/main/exercice/exercise.lib.php @@ -1094,18 +1094,18 @@ function get_exam_results_data($from, $number_of_items, $column, $direction, $ex $actions .="".Display :: return_icon('quiz.gif', get_lang('Qualify')); $actions .=' '; } - $actions .=""; - if ($is_allowedToEdit) { - if ($filter==2){ - $actions .=' ' .Display :: return_icon('history.gif', get_lang('ViewHistoryChange')).''; - } - } - if (api_is_platform_admin()) { - if ($locked == false) { - $actions .=' '.Display :: return_icon('delete.png', get_lang('Delete')).''; - $actions .=' '; - } + $actions .=""; + + if ($filter == 2) { + $actions .=' ' .Display :: return_icon('history.gif', get_lang('ViewHistoryChange')).''; + } + + //Admin can always delete the attempt + if ($locked == false || api_is_platform_admin()) { + $actions .=' '.Display :: return_icon('delete.png', get_lang('Delete')).''; + $actions .=' '; } + } else { $attempt_url = api_get_path(WEB_CODE_PATH).'exercice/result.php?'.api_get_cidreq().'&id='.$results[$i]['exe_id'].'&id_session='.api_get_session_id().'&height=500&width=750'; $attempt_link = Display::url(get_lang('Show'), $attempt_url, array('class'=>'ajax btn')); diff --git a/main/exercice/exercise_report.php b/main/exercice/exercise_report.php index e710e749ab..9ae851d351 100644 --- a/main/exercice/exercise_report.php +++ b/main/exercice/exercise_report.php @@ -238,7 +238,6 @@ if ( ($is_allowedToEdit || $is_tutor || api_is_coach()) && $_GET['delete'] == 'd Database::query($sql); header('Location: exercise_report.php?cidReq=' . Security::remove_XSS($_GET['cidReq']) . '&exerciseId='.$exercise_id); exit; - } } diff --git a/main/inc/ajax/model.ajax.php b/main/inc/ajax/model.ajax.php index a7e449bc89..f8429a66ff 100644 --- a/main/inc/ajax/model.ajax.php +++ b/main/inc/ajax/model.ajax.php @@ -223,7 +223,7 @@ switch ($action) { //used inside get_exam_results_data() $documentPath = api_get_path(SYS_COURSE_PATH) . $course['path'] . "/document"; if ($is_allowedToEdit) { - $columns = array('firstname', 'lastname', 'username', 'group_name', 'exe_duration', 'start_date', 'exe_date', 'score','status','actions'); + $columns = array('firstname', 'lastname', 'username', 'group_name', 'exe_duration', 'start_date', 'exe_date', 'score', 'status', 'actions'); } else { //$columns = array('exe_duration', 'start_date', 'exe_date', 'score', 'status', 'actions'); }