From 181190d81b8897a3a6f49ea7b8cf5b6729e2357c Mon Sep 17 00:00:00 2001 From: tjhunt Date: Thu, 19 Apr 2007 17:37:25 +0000 Subject: [PATCH] MDL-9432 - Forgot to return success/failure in two places. --- question/type/match/questiontype.php | 4 ++++ question/type/multichoice/questiontype.php | 4 ++++ 2 files changed, 8 insertions(+) diff --git a/question/type/match/questiontype.php b/question/type/match/questiontype.php index d0e8600e57f6a..25e57b505bf38 100644 --- a/question/type/match/questiontype.php +++ b/question/type/match/questiontype.php @@ -610,6 +610,8 @@ function restore_recode_answer($state, $restore) { * @return bool success or failure. */ function decode_content_links_caller($questionids, $restore, &$i) { + $status = true; + // Decode links in the question_match_sub table. if ($subquestions = get_records_list('question_match_sub', 'question', implode(',', $questionids), '', 'id, questiontext')) { @@ -633,6 +635,8 @@ function decode_content_links_caller($questionids, $restore, &$i) { } } } + + return $status; } } //// END OF CLASS //// diff --git a/question/type/multichoice/questiontype.php b/question/type/multichoice/questiontype.php index 014c452057852..522426f232fdc 100644 --- a/question/type/multichoice/questiontype.php +++ b/question/type/multichoice/questiontype.php @@ -560,6 +560,8 @@ function restore_recode_answer($state, $restore) { * @return bool success or failure. */ function decode_content_links_caller($questionids, $restore, &$i) { + $status = true; + // Decode links in the question_multichoice table. if ($multichoices = get_records_list('question_multichoice', 'question', implode(',', $questionids), '', 'id, correctfeedback, partiallycorrectfeedback, incorrectfeedback')) { @@ -589,6 +591,8 @@ function decode_content_links_caller($questionids, $restore, &$i) { } } } + + return $status; } } //// END OF CLASS ////