MDL-50614 mod_scorm: Improve validation on currentorg
This commit is contained in:
committed by
David Monllao
parent
630b8fdaea
commit
f3e7afedb9
@@ -44,6 +44,13 @@ if (!empty($id)) {
|
||||
print_error('missingparameter');
|
||||
}
|
||||
|
||||
// PARAM_RAW is used for $currentorg, validate it against records stored in the table.
|
||||
if (!empty($currentorg)) {
|
||||
if (!$DB->record_exists('scorm_scoes', array('scorm' => $scorm->id, 'identifier' => $currentorg))) {
|
||||
$currentorg = '';
|
||||
}
|
||||
}
|
||||
|
||||
$PAGE->set_url('/mod/scorm/loaddatamodel.php', array('scoid'=>$scoid, 'id'=>$cm->id));
|
||||
|
||||
require_login($course, false, $cm);
|
||||
|
||||
@@ -51,6 +51,14 @@ if (!empty($id)) {
|
||||
} else {
|
||||
print_error('missingparameter');
|
||||
}
|
||||
|
||||
// PARAM_RAW is used for $currentorg, validate it against records stored in the table.
|
||||
if (!empty($currentorg)) {
|
||||
if (!$DB->record_exists('scorm_scoes', array('scorm' => $scorm->id, 'identifier' => $currentorg))) {
|
||||
$currentorg = '';
|
||||
}
|
||||
}
|
||||
|
||||
// If new attempt is being triggered set normal mode and increment attempt number.
|
||||
$attempt = scorm_get_last_attempt($scorm->id, $USER->id);
|
||||
|
||||
|
||||
@@ -50,6 +50,13 @@ if (!empty($id)) {
|
||||
print_error('missingparameter');
|
||||
}
|
||||
|
||||
// PARAM_RAW is used for $currentorg, validate it against records stored in the table.
|
||||
if (!empty($currentorg)) {
|
||||
if (!$DB->record_exists('scorm_scoes', array('scorm' => $scorm->id, 'identifier' => $currentorg))) {
|
||||
$currentorg = '';
|
||||
}
|
||||
}
|
||||
|
||||
$PAGE->set_url('/mod/scorm/prereqs.php', array('scoid'=>$scoid, 'attempt'=>$attempt, 'id'=>$cm->id));
|
||||
|
||||
require_login($course, false, $cm);
|
||||
|
||||
Reference in New Issue
Block a user