Completed
Pull Request — 1.11.x (#1296)
by José
132:18 queued 93:54
created
main/inc/lib/MoodleImport.class.php 2 patches
Doc Comments   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -317,7 +317,7 @@  discard block
 block discarded – undo
317 317
      * Read and validate the quiz module XML
318 318
      *
319 319
      * @param resource $moduleXml XML file
320
-     * @return mixed | array if is a valid xml file, false otherwise
320
+     * @return string | array if is a valid xml file, false otherwise
321 321
      */
322 322
     public function readQuizModule($moduleXml)
323 323
     {
@@ -537,9 +537,9 @@  discard block
 block discarded – undo
537 537
      *
538 538
      * @param array $questionList
539 539
      * @param string $questionType
540
-     * @param object $questionInstance Question/Answer instance
540
+     * @param Question $questionInstance Question/Answer instance
541 541
      * @param array $currentQuestion
542
-     * @return integer db response
542
+     * @return boolean|null db response
543 543
      */
544 544
     public function processAnswers($questionList, $questionType, $questionInstance, $currentQuestion)
545 545
     {
@@ -665,7 +665,7 @@  discard block
 block discarded – undo
665 665
     /**
666 666
      * Process Chamilo Unique Answer
667 667
      *
668
-     * @param object $objAnswer
668
+     * @param Answer $objAnswer
669 669
      * @param array $answerValues
670 670
      * @param integer $position
671 671
      * @param integer $questionWeighting
@@ -698,7 +698,7 @@  discard block
 block discarded – undo
698 698
     /**
699 699
      * Process Chamilo True False
700 700
      *
701
-     * @param object $objAnswer
701
+     * @param Answer $objAnswer
702 702
      * @param array $answerValues
703 703
      * @param integer $position
704 704
      * @param integer $questionWeighting
@@ -731,7 +731,7 @@  discard block
 block discarded – undo
731 731
     /**
732 732
      * Process Chamilo FillBlanks
733 733
      *
734
-     * @param object $objAnswer
734
+     * @param Answer $objAnswer
735 735
      * @param array $questionType
736 736
      * @param array $answerValues
737 737
      * @param string $placeholder
@@ -826,7 +826,7 @@  discard block
 block discarded – undo
826 826
      * Litle utility to delete the unuseful tags
827 827
      *
828 828
      * @param $array
829
-     * @param $keys
829
+     * @param string[] $keys
830 830
      */
831 831
     public function traverseArray(&$array, $keys)
832 832
     {
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -681,7 +681,7 @@  discard block
 block discarded – undo
681 681
         if ($weighting > 0) {
682 682
             $questionWeighting += $weighting;
683 683
         }
684
-        $goodAnswer =  $correct ? true : false;
684
+        $goodAnswer = $correct ? true : false;
685 685
 
686 686
         $objAnswer->createAnswer(
687 687
             $answer,
@@ -714,7 +714,7 @@  discard block
 block discarded – undo
714 714
         if ($weighting > 0) {
715 715
             $questionWeighting += $weighting;
716 716
         }
717
-        $goodAnswer =  $correct ? true : false;
717
+        $goodAnswer = $correct ? true : false;
718 718
 
719 719
         $objAnswer->createAnswer(
720 720
             $answer,
@@ -809,7 +809,7 @@  discard block
 block discarded – undo
809 809
                     $currentAnswers = $correctAnswer.$othersAnswers;
810 810
                     $currentAnswers = '['.substr($currentAnswers, 0, -1).'] ';
811 811
 
812
-                    $placeholder .= '<p> ' . strip_tags($answer['questiontext']).' '.$currentAnswers . ' </p>';
812
+                    $placeholder .= '<p> '.strip_tags($answer['questiontext']).' '.$currentAnswers.' </p>';
813 813
                 }
814 814
 
815 815
                 return $optionsValues;
@@ -834,7 +834,7 @@  discard block
 block discarded – undo
834 834
             if (is_array($value)) {
835 835
                 $this->traverseArray($value, $keys);
836 836
             } else {
837
-                if (in_array($key, $keys)){
837
+                if (in_array($key, $keys)) {
838 838
                     unset($array[$key]);
839 839
                 }
840 840
             }
Please login to merge, or discard this patch.
main/coursecopy/import_moodle.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@
 block discarded – undo
47 47
     } else {
48 48
         Display::addFlash(Display::return_message(get_lang('ErrorImportingFile'), 'error'));
49 49
     }
50
-    header('Location: ' . api_get_self() . '?' . api_get_cidreq());
50
+    header('Location: '.api_get_self().'?'.api_get_cidreq());
51 51
 }
52 52
 
53 53
 $templateName = get_lang('ImportFromMoodle');
Please login to merge, or discard this patch.