Project

General

Profile

Bug #1128

gradebook.php - After upgrade 1.8.7b1 not working with CSS / Template ?

Added by Carlos Brolo about 9 years ago. Updated about 9 years ago.

Status:
Feature implemented
Priority:
Normal
Assignee:
Category:
-
Target version:
Start date:
30/04/2010
Due date:
% Done:

100%

Estimated time:
Complexity:
Normal
SCRUM pts - complexity:
?

Description

/main/gradebook/gradebook.php
After upgrade not working with CSS / Template ?

Quick fix:
Add after line 437
else { Display :: display_header(''); }

Should look like this:

// --------------------------------------------------------------------------------
// - DISPLAY HEADERS AND MESSAGES
// -------------------------------------------------------------------------------

if (!isset($_GET['exportpdf']) and !isset($_GET['export_certificate'])) {
if (isset ($_GET['studentoverview'])) {
$interbreadcrumb[]= array (
'url' => $_SESSION['gradebook_dest'].'?selectcat=' . Security::remove_XSS($_GET['selectcat']),
'name' => get_lang('ToolGradebook')
);
Display :: display_header(get_lang('FlatView'));
}
elseif (isset ($_GET['search'])) {

if ($_SESSION['gradebook_dest'] == 'index.php') {
$gradebook_dest = Security::remove_XSS($_SESSION['gradebook_dest']).'?cidReq='.Security::remove_XSS($_GET['course']).'&';
} else {
$gradebook_dest = Security::remove_XSS($_SESSION['gradebook_dest']);
}
$interbreadcrumb[]= array (
'url' => $gradebook_dest,
'name' => get_lang('Gradebook')
);
if ((isset($_GET['selectcat']) && $_GET['selectcat']>0)) {
if (!empty($_GET['course'])) {
$interbreadcrumb[]= array (
'url' => $gradebook_dest.'selectcat='.Security::remove_XSS($_GET['selectcat']),
'name' => get_lang('Details')
);
} else {
$interbreadcrumb[]= array (
'url' => $_SESSION['gradebook_dest'].'?selectcat=0',
'name' => get_lang('Details')
);
}
}
Display :: display_header('');
}
else { Display :: display_header(''); }

}

Associated revisions

Revision f73d4eeb (diff)
Added by Julio Montoya about 9 years ago

Fixes #1128 adding else condition

History

#1

Updated by Yannick Warnier about 9 years ago

  • Target version set to 1.8.7 RC1
#2

Updated by Julio Montoya about 9 years ago

  • Target version changed from 1.8.7 RC1 to 1.8.7
#3

Updated by Yannick Warnier about 9 years ago

  • Status changed from New to Assigned
  • Assignee set to Julio Montoya
#4

Updated by Julio Montoya about 9 years ago

  • Status changed from Assigned to Needs more info
  • Assignee changed from Julio Montoya to Carlos Brolo
#5

Updated by Julio Montoya about 9 years ago

  • Status changed from Needs more info to Feature implemented
  • % Done changed from 90 to 100

Feel free to reopen...

Also available in: Atom PDF