Skip to content

Commit

Permalink
Set 'is_same_response' to false always - reduces author confusion when
Browse files Browse the repository at this point in the history
developing questions.
  • Loading branch information
trampgeek committed Jul 6, 2017
1 parent 3623be5 commit bb472f1
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion question.php
Expand Up @@ -104,15 +104,22 @@ public function get_validation_error(array $response) {


/** This function is used by the question engine to prevent regrading of
* unchanged submissions.
* unchanged submissions. This has been disabled (it always returns false)
* to avoid confusion by authors and students when changing templates
* or other question data. It seems that this is more of a problem for
* CodeRunner than normal question types. The slight downside is that
* students pay a penalty for submitting the same code twice.
*
* @param array $prevresponse
* @param array $newresponse
* @return boolean
*/
public function is_same_response(array $prevresponse, array $newresponse) {
/*
return question_utils::arrays_same_at_key_missing_is_blank(
$prevresponse, $newresponse, 'answer');
*/
return false;
}

public function get_correct_response() {
Expand Down

0 comments on commit bb472f1

Please sign in to comment.