Bug #879 - Fixing a bug that prevented proper importing tests from course zip-archives created on systems prior Chamilo 1.8.6.2.

documentation/changelog.html

@@ -65,13 +65,14 @@
   <li>Quiz tool: Answer list fixed when using "matching" question type (BT#801). Reported by Noa Orizales</li>
   <li>Quiz tool + LP : Fixed bug when adding a exercise twice in the same LP with multiple attempts or not.</li>
   <li>Quiz tool + LP + Multiple attempts : Pre requisites will work the first time a student pass the LP.
- 	  If an student finished all steps correctly he will view all steps, prerequisites will not work here.</li>
+ 	  If a student finished all steps correctly he will view all steps, prerequisites will not work here.</li>
   <li>Learning Path (Courses): PHP sessions of a LP are destroyed when visiting the main course page.</li>
   <li>Assignment tool: Work.php deletes all homeworks bugs fixed. Reported and fixed by carlosbrolotobar <a href="http://www.chamilo.org/node/143" target="_blank">http://www.chamilo.org/node/143</a></li>
   <li>Core API: Support has been implemented for json_encode() function for PHP version < 5.2. The correspondent bug-report "Hide and show not working" and testing have been done by Hubert Borderiou <a href="http://support.chamilo.org/issues/761" target="_blank">http://support.chamilo.org/issues/761</a></li>
   <li>Fixed target problems in links tool link edition</li>
   <li>Fixed minor bug in online mail editor in order to show the sender's name and e-mail</li>
-  <li>Fixing a bug that prevented importing SCORM resource from course archives (Bug #872)</li>  
+  <li>A bug that prevented importing SCORM resource from course archives has been fixed (Bug #872)</li>
+  <li>A bug that prevented proper importing tests from course zip-archives created on systems prior Chamilo 1.8.6.2 has been fixed (Bug #879)</li>
 </ul>
 <br />
 <h3>CSS changes</h3>

main/coursecopy/classes/CourseRestorer.class.php

@@ -871,7 +871,8 @@ class CourseRestorer
 				$this->course->resources[RESOURCE_QUIZ][$id]->destination_id = $new_id;
 				foreach ($quiz->question_ids as $index => $question_id) {
 					$qid = $this->restore_quiz_question($question_id);
-					$sql = "INSERT IGNORE INTO ".$table_rel." SET question_id = ".$qid.", exercice_id = ".$new_id.", question_order = ".$quiz->question_orders[$index]."";
+					$question_order = $quiz->question_orders[$index] ? $quiz->question_orders[$index] : 1;
+					$sql = "INSERT IGNORE INTO ".$table_rel." SET question_id = ".$qid.", exercice_id = ".$new_id.", question_order = ".$question_order;
 					Database::query($sql);
 				}
 			}