MDL-81267 mod_data: Clean field type before inclusion

* Filter out wrong field types when restoring or creating a field.
This commit is contained in:
Laurent David
2024-04-17 10:33:35 +02:00
committed by Sara Arjona
parent 4b09e99595
commit 63ba5e591d
3 changed files with 5 additions and 0 deletions
@@ -87,6 +87,7 @@ class restore_data_activity_structure_step extends restore_activity_structure_st
$oldid = $data->id;
$data->dataid = $this->get_new_parentid('data');
$data->type = clean_param($data->type, PARAM_ALPHA);
// insert the data_fields record
$newitemid = $DB->insert_record('data_fields', $data);
+2
View File
@@ -959,6 +959,7 @@ function data_get_field_from_id($fieldid, $data){
function data_get_field_new($type, $data) {
global $CFG;
$type = clean_param($type, PARAM_ALPHA);
$filepath = $CFG->dirroot.'/mod/data/field/'.$type.'/field.class.php';
// It should never access this method if the subfield class doesn't exist.
if (!file_exists($filepath)) {
@@ -986,6 +987,7 @@ function data_get_field($field, $data, $cm=null) {
if (!isset($field->type)) {
return new data_field_base($field);
}
$field->type = clean_param($field->type, PARAM_ALPHA);
$filepath = $CFG->dirroot.'/mod/data/field/'.$field->type.'/field.class.php';
if (!file_exists($filepath)) {
return new data_field_base($field);
+2
View File
@@ -976,6 +976,7 @@ function data_get_tag_title_field($dataid) {
if ($field->addtemplateposition === false) {
continue;
}
$field->type = clean_param($field->type, PARAM_ALPHA);
$filepath = $CFG->dirroot . '/mod/data/field/' . $field->type . '/field.class.php';
if (!file_exists($filepath)) {
continue;
@@ -1019,6 +1020,7 @@ function data_get_tag_title_for_entry($field, $entry) {
if (!isset($field->type)) {
return null;
}
$field->type = clean_param($field->type, PARAM_ALPHA);
$filepath = $CFG->dirroot . '/mod/data/field/' . $field->type . '/field.class.php';
if (!file_exists($filepath)) {
return null;