From 7e252acff1e0a118d0f0a56b4fef39a68731c3fa Mon Sep 17 00:00:00 2001
From: Tim Hunt <T.J.Hunt@open.ac.uk>
Date: Fri, 30 Nov 2012 11:28:38 +0000
Subject: [PATCH] MDL-36942 xml question import: don't add spaces.

They are not necessary anyway, there is a \n character at the end of
each line.

Thanks to Richard Lobb for fining the problem and how to fix it.
---
 question/format/xml/format.php | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/question/format/xml/format.php b/question/format/xml/format.php
index 7186582927123..21997d5ddff57 100644
--- a/question/format/xml/format.php
+++ b/question/format/xml/format.php
@@ -905,17 +905,18 @@ protected function import_category($question) {
      */
     protected function readquestions($lines) {
         // We just need it as one big string
-        $text = implode($lines, ' ');
-        unset($lines);
+        $lines = implode('', $lines);
 
         // This converts xml to big nasty data structure
         // the 0 means keep white space as it is (important for markdown format)
         try {
-            $xml = xmlize($text, 0, 'UTF-8', true);
+            $xml = xmlize($lines, 0, 'UTF-8', true);
         } catch (xml_format_exception $e) {
             $this->error($e->getMessage(), '');
             return false;
         }
+        unset($lines); // No need to keep this in memory.
+
         // Set up array to hold all our questions
         $questions = array();