Merge branch 'hotfix' of https://git.logipro.com/raphael/Phoenix into hotfix

src/Phoenix/LMS/LearnerPath/LearnerSequence.php

@@ -31,7 +31,7 @@ class LearnerSequence
 	 *
 	 * @var \SplObjectStorage
 	 */
-    protected $tracks;
+	protected $tracks;
 
 	/**
 	 *
@@ -41,7 +41,7 @@ class LearnerSequence
 	public function __construct(Person $learner, Sequence $sequence)
 	{
 		$this->learner = $learner;
-        $this->sequence = $sequence;
+		$this->sequence = $sequence;
 
 		$this->tracks = new \SplObjectStorage();
 	}
@@ -236,16 +236,16 @@ class LearnerSequence
 
 	/**
 	 * Retourne le score moyen d'un apprenant sur la séquence
-     *
-     * !! WARNING !!
-     * Le service est injecté dans la méthode directement pour une raison de temps
-     * Cette solution n'est que temporaire
-     * Un refactoring est à prévoir avec principalement
-     * le retrait de buildLearnerPath dans l'entity Session
-     * !! WARNING !!
-     *
-     * @param ScormEngine $scormEngine
-     * @return float|null
+	 *
+	 * !! WARNING !!
+	 * Le service est injecté dans la méthode directement pour une raison de temps
+	 * Cette solution n'est que temporaire
+	 * Un refactoring est à prévoir avec principalement
+	 * le retrait de buildLearnerPath dans l'entity Session
+	 * !! WARNING !!
+	 *
+	 * @param ScormEngine $scormEngine
+	 * @return float|null
 	 */
 	public function getAverageScore(ScormEngine $scormEngine)
 	{
@@ -254,9 +254,8 @@ class LearnerSequence
 		$scoreNum = 0;
 
 		$sequenceItems = $this->sequence->getSequenceItems();
-		
-		foreach ($sequenceItems as $item)
-		{
+
+		foreach ($sequenceItems as $item) {
 			$itemScore = $this->getItemAverageScore($item, $scormEngine);
 
 			if (!is_null($itemScore)) {
@@ -276,17 +275,17 @@ class LearnerSequence
 	/**
 	 * Retourne le score moyen d'un séquenceItem de type SCORM 12 si il est complété
 	 *
-     * !! WARNING !!
-     * Le service est injecté dans la méthode directement pour une raison de temps
-     * Cette solution n'est que temporaire
-     * Un refactoring est à prévoir avec principalement
-     * le retrait de buildLearnerPath dans l'entity Session
-     * !! WARNING !!
-     *
-     * @param SequenceItem $item
-     * @param ScormEngine $scormEngine
-     *
-     * @return float|null
+	 * !! WARNING !!
+	 * Le service est injecté dans la méthode directement pour une raison de temps
+	 * Cette solution n'est que temporaire
+	 * Un refactoring est à prévoir avec principalement
+	 * le retrait de buildLearnerPath dans l'entity Session
+	 * !! WARNING !!
+	 *
+	 * @param SequenceItem $item
+	 * @param ScormEngine $scormEngine
+	 *
+	 * @return float|null
 	 */
 	public function getItemAverageScore(SequenceItem $item, ScormEngine $scormEngine)
 	{
@@ -294,53 +293,67 @@ class LearnerSequence
 
 		$sequenceItems = $this->sequence->getSequenceItems();
 
-		if ($sequenceItems->contains($item))
-		{
-			if ($item instanceof ActivityPackage)
-			{		
-                $track = $this->retrieveTrack($item);
-                
-                if ($track) {
-                    // On ne retourne le score moyen que si l'item est complété
-                    if ($track->getCompletion() == 100) {
-                        if (AbstractPackageFile::TYPE_SCORM1_2 == $item->getPackageType()) {
-                            $scoreSum = 0;
-                            $scoreNum = 0;
-                        
-                            $scorm12Tracks = $track->getScorm12Tracks();
-
-                            foreach ($scorm12Tracks as $scorm12Track) {
-                                $scoreMax = $scorm12Track->getScoreMax();
-                                // Si pas de score max, pas de score valide
-                                if ($scoreMax > 0) {
-                                    $score = $scorm12Track->getScore();
-                                    $score = $score * 100 / $scoreMax;
-                                    $score = max(min($score, 100), 0);
-
-                                    $scoreNum ++;
-                                    $scoreSum += $score;
-                                }
-                            }
-
-                            if ($scoreNum > 0) {
-                                $averageScore = $scoreSum / $scoreNum;
-                                $averageScore = round($averageScore, 2);
-                            }
-                        }
-
-                        if (AbstractPackageFile::TYPE_SCORM2004 == $item->getPackageType()) {
-                            // Récupération de la clef de l'inscription
-                            $phoenixLearner = $track->getLearner();
-                            $learnerKey = $phoenixLearner->getPersonId();
-                            $sequenceItem = $track->getSequenceItem();
-                            $courseKey = $sequenceItem->getSequenceItemId();
-                            $registrationKey = $courseKey . '|' . $learnerKey;
-        
-                            // Récupération de la complétion
-                            $averageScore = $scormEngine->getAverageScore($registrationKey);
-                        }
-                    }
-                }
+
+		if ($sequenceItems->contains($item)) {
+			if ($item instanceof ActivityPackage) {
+				$track = $this->retrieveTrack($item);
+
+				if ($track) {
+					// On ne retourne le score moyen que si l'item est complété
+					if ($track->getCompletion() == 100) {
+						if (AbstractPackageFile::TYPE_SCORM1_2 == $item->getPackageType()) {
+							$scoreSum = 0;
+							$scoreNum = 0;
+
+							$scorm12Tracks = $track->getScorm12Tracks();
+
+							foreach ($scorm12Tracks as $scorm12Track) {
+								$itemInteractions = $scorm12Track->getInteractions();
+
+								if (!$itemInteractions->isEmpty()) {
+									// Test pour savoir si le questionnaire est composé seulement de question neutre ou non
+									$trackIsNeutral = true;
+									foreach ($itemInteractions as $interaction) {
+										if ($interaction->getResult() != "neutral") {
+											$trackIsNeutral = false;
+											break;
+										}
+									}
+								}
+
+								if ($trackIsNeutral == false) {
+									$scoreMax = $scorm12Track->getScoreMax();
+									// Si pas de score max, pas de score valide
+									if ($scoreMax > 0) {
+										$score = $scorm12Track->getScore();
+										$score = $score * 100 / $scoreMax;
+										$score = max(min($score, 100), 0);
+
+										$scoreNum++;
+										$scoreSum += $score;
+									}
+								}
+							}
+
+							if ($scoreNum > 0) {
+								$averageScore = $scoreSum / $scoreNum;
+								$averageScore = round($averageScore, 2);
+							}
+						}
+
+						if (AbstractPackageFile::TYPE_SCORM2004 == $item->getPackageType()) {
+							// Récupération de la clef de l'inscription
+							$phoenixLearner = $track->getLearner();
+							$learnerKey = $phoenixLearner->getPersonId();
+							$sequenceItem = $track->getSequenceItem();
+							$courseKey = $sequenceItem->getSequenceItemId();
+							$registrationKey = $courseKey . '|' . $learnerKey;
+
+							// Récupération de la complétion
+							$averageScore = $scormEngine->getAverageScore($registrationKey);
+						}
+					}
+				}
 			}
 		}
 
@@ -349,18 +362,18 @@ class LearnerSequence
 
 	/**
 	 * Retourne un tableau contenant les quiz et leurs interactions pour une activité
-     *
-     * !! WARNING !!
-     * Le service est injecté dans la méthode directement pour une raison de temps
-     * Cette solution n'est que temporaire
-     * Un refactoring est à prévoir avec principalement
-     * le retrait de buildLearnerPath dans l'entity Session
-     * !! WARNING !!
-     * 
-     * @param SequenceItem $item
-     * @param ScormEngine $scormEngine
-     * 
-     * @return array|null
+	 *
+	 * !! WARNING !!
+	 * Le service est injecté dans la méthode directement pour une raison de temps
+	 * Cette solution n'est que temporaire
+	 * Un refactoring est à prévoir avec principalement
+	 * le retrait de buildLearnerPath dans l'entity Session
+	 * !! WARNING !!
+	 * 
+	 * @param SequenceItem $item
+	 * @param ScormEngine $scormEngine
+	 * 
+	 * @return array|null
 	 */
 	public function getItemInteractions(SequenceItem $item, ScormEngine $scormEngine)
 	{
@@ -368,85 +381,77 @@ class LearnerSequence
 
 		$sequenceItems = $this->sequence->getSequenceItems();
 
-		if ($sequenceItems->contains($item))
-		{
-            if ($item instanceof ActivityPackage) {
-                $track = $this->retrieveTrack($item);
-                    
-                if ($track) {
-                    // CAS SCORM 1.2
-                    if (AbstractPackageFile::TYPE_SCORM1_2 == $item->getPackageType()) {
-                        $scorm12Tracks = $track->getScorm12Tracks();
-
-                        foreach ($scorm12Tracks as $scorm12Track) {
-                            $itemInteractions = $scorm12Track->getInteractions();
-
-                            if (!$itemInteractions->isEmpty())
-                            {
-                                $result = array();
+		if ($sequenceItems->contains($item)) {
+			if ($item instanceof ActivityPackage) {
+				$track = $this->retrieveTrack($item);
+
+				if ($track) {
+					// CAS SCORM 1.2
+					if (AbstractPackageFile::TYPE_SCORM1_2 == $item->getPackageType()) {
+						$scorm12Tracks = $track->getScorm12Tracks();
+
+						foreach ($scorm12Tracks as $scorm12Track) {
+							$itemInteractions = $scorm12Track->getInteractions();
+
+							if (!$itemInteractions->isEmpty()) {
+								$result = array();
 								$scormItem = $scorm12Track->getItem();
 								$result['title'] = $scormItem->getTitle();
 
 								// Test pour savoir si le questionnaire est composé seulement de question neutre ou non
 								$trackIsNeutral = true;
-								foreach($itemInteractions as $interaction)
-								{
-									if ($interaction->getResult() != "neutral")
-									{
+								foreach ($itemInteractions as $interaction) {
+									if ($interaction->getResult() != "neutral") {
 										$trackIsNeutral = false;
 										break;
 									}
 								}
 
-								if ($trackIsNeutral)
-								{
+								if ($trackIsNeutral) {
 									$result['score'] = null;
 									$result['lessonStatus'] = null;
-								}
-								else
-								{
+								} else {
 									$score = null;
 									$scoreMax = $scorm12Track->getScoreMax();
 									// Si pas de score max, pas de score valide
-									if ($scoreMax > 0)
-									{
+									if ($scoreMax > 0) {
 										$score = $scorm12Track->getScore();
 										$score = $score * 100 / $scoreMax;
 										$score = max(min($score, 100), 0);
 									}
 									$result['score'] = $score;
-									
+
 									$status = $scorm12Track->getLessonStatus();
 									$result['lessonStatus'] = $status;
 								}
-								
+
 								$result['interactions'] = $itemInteractions;
 
 								$interactions[] = $result;
-                            }
-                        }
-
-                        if (count($interactions) > 0) {
-                            return $interactions;
-                        }
-                    }
-
-                    // CAS SCORM2004 (engine)
-                    if ($item->getPackageType() == AbstractPackageFile::TYPE_SCORM2004) {
-                        // Récupération de la clef de l'inscription
-                        $phoenixLearner = $track->getLearner();
-                        $learnerKey = $phoenixLearner->getPersonId();
-                        $sequenceItem = $track->getSequenceItem();
-                        $courseKey = $sequenceItem->getSequenceItemId();
-                        $registrationKey = $courseKey . '|' . $learnerKey;
-    
-                        // Récupération de la complétion
-                        $interactions = $scormEngine->getInteractions($registrationKey);
-
-                        return $interactions;
-                    }
-                }
-            }
+							}
+						}
+
+						if (count($interactions) > 0) {
+							return $interactions;
+						}
+					}
+
+					// CAS SCORM2004 (engine)
+					if ($item->getPackageType() == AbstractPackageFile::TYPE_SCORM2004) {
+						// Récupération de la clef de l'inscription
+						$phoenixLearner = $track->getLearner();
+						$learnerKey = $phoenixLearner->getPersonId();
+						$sequenceItem = $track->getSequenceItem();
+						$courseKey = $sequenceItem->getSequenceItemId();
+						$registrationKey = $courseKey . '|' . $learnerKey;
+
+						// Récupération de la complétion
+						$interactions = $scormEngine->getInteractions($registrationKey);
+
+						return $interactions;
+					}
+				}
+			}
 		}
 
 		return null;