diff --git a/mod/lesson/import_form.php b/mod/lesson/import_form.php index b152c0abe24..0ca0b1e5d68 100644 --- a/mod/lesson/import_form.php +++ b/mod/lesson/import_form.php @@ -59,6 +59,7 @@ class lesson_import_form extends moodleform { * @param array $data the submitted data. * @param array $errors the errors so far. * @return array the updated errors. + * @throws moodle_exception */ protected function validate_uploaded_file($data, $errors) { global $CFG; @@ -69,7 +70,7 @@ class lesson_import_form extends moodleform { } $files = $this->get_draft_files('questionfile'); - if (count($files) < 1) { + if (!is_array($files) || count($files) < 1) { $errors['questionfile'] = get_string('required'); return $errors; } diff --git a/question/import_form.php b/question/import_form.php index ad2cc18feb6..f4693e40184 100644 --- a/question/import_form.php +++ b/question/import_form.php @@ -112,6 +112,7 @@ class question_import_form extends moodleform { * @param array $data the submitted data. * @param array $errors the errors so far. * @return array the updated errors. + * @throws moodle_exception */ protected function validate_uploaded_file($data, $errors) { if (empty($data['newfile'])) { @@ -120,7 +121,7 @@ class question_import_form extends moodleform { } $files = $this->get_draft_files('newfile'); - if (count($files) < 1) { + if (!is_array($files) || count($files) < 1) { $errors['newfile'] = get_string('required'); return $errors; }