fix for MDL-11692. There were notices about subquestions not being defined. This...

diff --git a/question/type/match/questiontype.php b/question/type/match/questiontype.php
index f300c45658493058824246bfe610b1594622dd30..f4d9996deef4c29159c11f29856969e1d00fe4c1 100644 (file)
--- a/question/type/match/questiontype.php
+++ b/question/type/match/questiontype.php
@@ -650,15 +650,18 @@ class question_match_qtype extends default_questiontype {
     function find_file_links($question, $courseid){
         // find links in the question_match_sub table.
         $urls = array();
-        foreach ($question->options->subquestions as $subquestion) {
-            $urls += question_find_file_links_from_html($subquestion->questiontext, $courseid);
-        }
+        if (isset($question->options->subquestions)){
+            foreach ($question->options->subquestions as $subquestion) {
+                $urls += question_find_file_links_from_html($subquestion->questiontext, $courseid);
+            }
 
-        //set all the values of the array to the question object
-        if ($urls){
-            $urls = array_combine(array_keys($urls), array_fill(0, count($urls), array($question->id)));
+            //set all the values of the array to the question object
+            if ($urls){
+                $urls = array_combine(array_keys($urls), array_fill(0, count($urls), array($question->id)));
+            }
         }
         $urls = array_merge_recursive($urls, parent::find_file_links($question, $courseid));
+
         return $urls;
     }