moodle/moodle
3
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-08-06 01:16:44 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

MDL-37896 qtype_essay: correct PARAM_CLEANHTML -> PARAM_RAW.

Browse source 
It turns out that you should almost never use PARAM_CLEANHTML.
This commit is contained in:
Tim Hunt 2013-02-06 17:45:58 +00:00
parent 6319737865
commit 2d2018abda
3 changed files with 9 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

6
question/engine/questionattempt.php
View file

@ -64,7 +64,7 @@ class question_attempt {
* @var string special value to indicate a response variable that is uploaded
* files.
*/
const PARAM_CLEANHTML_FILES = 'paramcleanhtmlfiles';
const PARAM_RAW_FILES = 'paramrawfiles';
/** @var integer if this attempts is stored in the question_attempts table, the id of that row. */
protected $id = null;
@ -888,8 +888,8 @@ class question_attempt {
case self::PARAM_FILES:
return $this->process_response_files($name, $name, $postdata);
case self::PARAM_CLEANHTML_FILES:
$var = $this->get_submitted_var($name, PARAM_CLEANHTML, $postdata);
case self::PARAM_RAW_FILES:
$var = $this->get_submitted_var($name, PARAM_RAW, $postdata);
return $this->process_response_files($name, $name . ':itemid', $postdata, $var);
default:

4
question/type/essay/question.php
View file

@ -55,9 +55,7 @@ class qtype_essay_question extends question_with_responses {
public function get_expected_data() {
if ($this->responseformat == 'editorfilepicker') {
$expecteddata = array('answer' => question_attempt::PARAM_CLEANHTML_FILES);
} else if ($this->responseformat == 'editor') {
$expecteddata = array('answer' => PARAM_CLEANHTML);
$expecteddata = array('answer' => question_attempt::PARAM_RAW_FILES);
} else {
$expecteddata = array('answer' => PARAM_RAW);
}