TLD-577 Correction suite au test

LearningModels/DOMSCORM2004.php

@@ -30,18 +30,22 @@ class DOMSCORM2004 extends DOMSCORM
      */
     public function getTypicalLearningTime()
     {
+        $result = array();
+
         $tlt = $this->manifest->getElementsByTagName("typicalLearningTime");
         $tltNode = $tlt->item(0);
-        $children = $tltNode->childNodes;
-        $result = array();
-        foreach ($children as $node) {
-            switch ($node->nodeName) {
-                case 'duration':
-                case 'description': // IMPORTANT TODO: ce champ peut etre multilangue dans le manifest
-                    $result[$node->nodeName] = $node->nodeValue;
-                    break;
+        if ($tltNode) {
+            $children = $tltNode->childNodes;
+            foreach ($children as $node) {
+                switch ($node->nodeName) {
+                    case 'duration':
+                    case 'description': // IMPORTANT TODO: ce champ peut etre multilangue dans le manifest
+                        $result[$node->nodeName] = $node->nodeValue;
+                        break;
+                }
             }
         }
+
         return $result;
     }
 

Package/PackageValidator.php

@@ -25,7 +25,7 @@ class PackageValidator
             'validation' => null
         );
 
-    // Identification du standard
+        // Identification du standard
         // Accès au contenu du zip
         $zip = new \ZipArchive();
         $result = $zip->open($zipPath);
@@ -74,7 +74,7 @@ class PackageValidator
                 }
             }
 
-// @TODO Validation XML reportée
+            // @TODO Validation XML reportée
             // $response['validation'] = $dom->validate();
             $response['validation'] = array(
                 'status' => true

Services/ScormEngine.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace Logipro\Bundle\SCORMBundle\Services;
 
 use Doctrine\ORM\EntityManagerInterface;
@@ -115,7 +116,7 @@ class ScormEngine
                 'duration' => ''
             )
         );
-        
+
         // Controle sur la clef fournie (elle doit etre non null et non existante en BD)
         if (!empty($courseKey)) {
             $course = $this->requireCourse($courseKey);
@@ -126,7 +127,7 @@ class ScormEngine
                 return $response;
             }
 
-        // ZIP
+            // ZIP
             // Test de l'existence du fichier à uploader
             if (!file_exists($zipPath)) {
                 $response['code'] = 400;
@@ -192,7 +193,7 @@ class ScormEngine
                 $zipFile->extractZipFile($this->packageFolder, $this->contentFolder);
             }
 
-        // COURS
+            // COURS
             // Création du cours
             $course = new Course();
             $course->setCourseKey($courseKey);
@@ -203,7 +204,7 @@ class ScormEngine
             $response['course']['courseKey'] = $course->getCourseKey();
             $response['course']['maxVersion'] = $course->getMaxVersion();
 
-        // Mapping Zip/Cours
+            // Mapping Zip/Cours
             $mapping = new MapCourseZipfile();
             $mapping->setCourse($course);
             $mapping->setZipFile($zipFile);
@@ -248,7 +249,7 @@ class ScormEngine
                 'duration' => ''
             )
         );
-        
+
         // Controle sur la clef fournie (elle doit etre non null et non existante en BD)
         if (!empty($courseKey)) {
             $course = $this->requireCourse($courseKey);
@@ -259,7 +260,7 @@ class ScormEngine
                 return $response;
             }
 
-        // ZIP
+            // ZIP
             // Test de l'existence du fichier à uploader
             if (!file_exists($zipPath)) {
                 $response['code'] = 400;
@@ -321,7 +322,7 @@ class ScormEngine
                 $zipFile->extractZipFile($this->packageFolder, $this->contentFolder);
             }
 
-        // COURS
+            // COURS
             // MaJ du cours
             $course->setVersionRule($versioningRule);
             $newMaxVersion = $course->getMaxVersion();
@@ -333,7 +334,7 @@ class ScormEngine
             $response['course']['courseKey'] = $course->getCourseKey();
             $response['course']['maxVersion'] = $newMaxVersion;
 
-        // Mapping Zip/Cours
+            // Mapping Zip/Cours
             $mapping = new MapCourseZipfile();
             $mapping->setCourse($course);
             $mapping->setZipFile($zipFile);
@@ -385,7 +386,7 @@ class ScormEngine
             $response['error'] = 'La clef du nouveau cours dupliqué est requise.';
             return $response;
         }
-        
+
         // Contrôle de l'existence du cours à dupliquer
         $originalCourse = $this->requireCourse($orginalCourseKey);
 
@@ -403,7 +404,7 @@ class ScormEngine
             $response['error'] = 'La clef du nouveau cours existe déjà. Utilser \'updateCourse\' pour mettre à jour un cours existant.';
             return $response;
         }
-        
+
         // COURS
         $newCourse = clone $originalCourse;
         $newCourse->setCourseKey($newCourseKey);
@@ -420,7 +421,7 @@ class ScormEngine
         $mapping->setCourse($newCourse);
         $mapping->setZipFile($zipFile);
         $mapping->setVersion($newCourse->getMaxVersion());
-             
+
         $this->em->persist($mapping);
         $this->em->flush();
 
@@ -471,7 +472,7 @@ class ScormEngine
             $this->em->flush();
             return true;
         }
-        
+
         return false;
     }
 
@@ -503,11 +504,11 @@ class ScormEngine
         return null;
     }
 
-    public function getCourses() : array
+    public function getCourses(): array
     {
         $repository = $this->em->getRepository(Course::class);
         $courses = $repository->findAll();
-        
+
         return $courses;
     }
 
@@ -560,7 +561,7 @@ class ScormEngine
         } else {
             $response['error'] = 'La clef d\'identification est necessaire';
         }
-        
+
         return $response;
     }
 
@@ -585,7 +586,7 @@ class ScormEngine
                 $this->em->flush();
             }
         }
-            return $learner->getLearnerKey();
+        return $learner->getLearnerKey();
     }
 
     /**
@@ -626,7 +627,7 @@ class ScormEngine
             $this->em->flush();
             return true;
         }
-        
+
         return false;
     }
 
@@ -682,7 +683,7 @@ class ScormEngine
         } else {
             $response['error'] = 'La clef d\'identification est necessaire';
         }
-        
+
         return $response;
     }
 
@@ -695,7 +696,7 @@ class ScormEngine
      *
      * @return array
      */
-    public function getPlayerVariables(string $registrationKey, string $data = null, Response $response = null) : array
+    public function getPlayerVariables(string $registrationKey, string $data = null, Response $response = null): array
     {
         $result = array();
         // Vérification de l'existance de l'inscription
@@ -705,7 +706,7 @@ class ScormEngine
             // Erreur : une inscription valide doit être passée
             return null;
         }
-        
+
         // Récupération du cours et des infos de version
         $course = $registration->getCourse();
         $maxVersion = $course->getMaxVersion();
@@ -732,11 +733,11 @@ class ScormEngine
             $currentVersion = $maxVersion;
 
             if (!$needCreation) {
-// TODO Indiquer dans la réponse que l'on a effectué une modification de version pour l'apprenant
+                // TODO Indiquer dans la réponse que l'on a effectué une modification de version pour l'apprenant
                 $data = null;
             }
         }
-        
+
         // Récupération du zip correspondant et ainsi de la version du standard utilisée
         $mapCourseZipRepository = $this->em->getRepository(MapCourseZipfile::class);
         $mapCourseZip = $mapCourseZipRepository->findOneBy(array('course' => $course, 'version' => $currentVersion));
@@ -792,13 +793,13 @@ class ScormEngine
         if (empty($registration)) {
             return 0;
         }
-        
+
         // Récupération du dernier essai moteur pour cette inscription
         $registrationAttemptRepository = $this->em->getRepository(RegistrationAttempt::class);
         $lastAttempt = $registrationAttemptRepository->findOneBy(array('registration' => $registration), array('registrationAttemptId' => 'desc'));
 
-// DETECTER AVANT LE TYPE DE SCORM POUR SAVOIR QUEL STANDARD
-// CODE SCORM 2004
+        // DETECTER AVANT LE TYPE DE SCORM POUR SAVOIR QUEL STANDARD
+        // CODE SCORM 2004
         // On récupère le track correspondant
         $track2004Repository = $this->em->getRepository(Scorm2004Track::class);
         $track2004 = $track2004Repository->findOneBy(array('registrationAttempt' => $lastAttempt));
@@ -810,7 +811,7 @@ class ScormEngine
         // en se basant sur l'état de complétion de l'ensemble des items terminaux
         $mapTrackItemRepository = $this->em->getRepository(Scorm2004MapTrackItem::class);
         $attemptItemRepository = $this->em->getRepository(Scorm2004AttemptProgressInformation::class);
-        
+
         $mapTrackItems = $mapTrackItemRepository->findBy(array('scorm2004Track' => $track2004));
 
         if (!empty($mapTrackItems)) {
@@ -819,14 +820,14 @@ class ScormEngine
             foreach ($mapTrackItems as $item) {
                 // Si les items sont terminaux
                 if ($item->getAvailableOrder() > -1) {
-                    $numFinalItems ++;
+                    $numFinalItems++;
                     $identifier = $item->getItemIdentifier();
-                    $attemptItem = $attemptItemRepository->findOneBy(array('scorm2004Track'=> $track2004, 'item_identifier' => $identifier), array('attempt_id' => 'desc'));
+                    $attemptItem = $attemptItemRepository->findOneBy(array('scorm2004Track' => $track2004, 'item_identifier' => $identifier), array('attempt_id' => 'desc'));
                     if (!empty($attemptItem)) {
                         $progressStatus = $attemptItem->getProgressStatus();
                         $completionStatus = $attemptItem->getCompletionStatus();
                         if (true == $progressStatus && true == $completionStatus) {
-                            $numCompletedItems ++;
+                            $numCompletedItems++;
                         }
                     }
                 }
@@ -850,7 +851,7 @@ class ScormEngine
         }
 
         //Récupération du dernier essai SCORM sur l'organisation
-        $lastOrganizationAttempt = $attemptItemRepository->findOneBy(array('scorm2004Track'=> $track2004, 'item_identifier' => $organization), array('attempt_id' => 'desc'));
+        $lastOrganizationAttempt = $attemptItemRepository->findOneBy(array('scorm2004Track' => $track2004, 'item_identifier' => $organization), array('attempt_id' => 'desc'));
         if (empty($lastOrganizationAttempt)) {
             return 0;
         }
@@ -884,22 +885,22 @@ class ScormEngine
         if (empty($registration)) {
             return null;
         }
-        
+
         // Récupération du dernier essai pour cette inscription
         $registrationAttemptRepository = $this->em->getRepository(RegistrationAttempt::class);
         $lastAttempt = $registrationAttemptRepository->findOneBy(array('registration' => $registration), array('registrationAttemptId' => 'desc'));
 
-// DETECTER AVANT LE TYPE DE SCORM POUR SAVOIR QUEL STANDARD
-// CODE SCORM 2004
+        // DETECTER AVANT LE TYPE DE SCORM POUR SAVOIR QUEL STANDARD
+        // CODE SCORM 2004
         // On récupère le track correspondant
         $track2004Repository = $this->em->getRepository(Scorm2004Track::class);
         $track2004 = $track2004Repository->findOneBy(array('registrationAttempt' => $lastAttempt));
         if (empty($track2004)) {
             return null;
         }
-        
+
         $mapTrackItemRepository = $this->em->getRepository(Scorm2004MapTrackItem::class);
-        
+
         $mapTrackItems = $mapTrackItemRepository->findBy(array('scorm2004Track' => $track2004));
 
         $averageScore = null;
@@ -916,7 +917,7 @@ class ScormEngine
                     $score = $score * 100 / $scoreMax;
                     $score = max(min($score, 100), 0);
 
-                    $scoreNum ++;
+                    $scoreNum++;
                     $scoreSum += $score;
                 }
             }
@@ -943,22 +944,22 @@ class ScormEngine
         if (empty($registration)) {
             return null;
         }
-        
+
         // Récupération du dernier essai pour cette inscription
         $registrationAttemptRepository = $this->em->getRepository(RegistrationAttempt::class);
         $lastAttempt = $registrationAttemptRepository->findOneBy(array('registration' => $registration), array('registrationAttemptId' => 'desc'));
 
-// DETECTER AVANT LE TYPE DE SCORM POUR SAVOIR QUEL STANDARD
-// CODE SCORM 2004
+        // DETECTER AVANT LE TYPE DE SCORM POUR SAVOIR QUEL STANDARD
+        // CODE SCORM 2004
         // On récupère le track correspondant
         $track2004Repository = $this->em->getRepository(Scorm2004Track::class);
         $track2004 = $track2004Repository->findOneBy(array('registrationAttempt' => $lastAttempt));
         if (empty($track2004)) {
             return null;
         }
-        
+
         $mapTrackItemRepository = $this->em->getRepository(Scorm2004MapTrackItem::class);
-        
+
         // Récupération des données pour chaque items
         $mapTrackItems = $mapTrackItemRepository->findBy(array('scorm2004Track' => $track2004));
 
@@ -977,7 +978,7 @@ class ScormEngine
                 $identifier = $item->getItemIdentifier();
 
                 // On récupère le dernier AttemptProgressInfo en cours
-                $attemptItem = $attemptItemRepository->findOneBy(array('scorm2004Track'=> $track2004, 'item_identifier' => $identifier), array('attempt_id' => 'desc'));
+                $attemptItem = $attemptItemRepository->findOneBy(array('scorm2004Track' => $track2004, 'item_identifier' => $identifier), array('attempt_id' => 'desc'));
 
                 // On récupère les interactions pour cet item et cet Attempt
                 $itemInteractions = $interactionRepository->findBy(array('scorm2004Track' => $track2004, 'scorm2004AttemptProgressInformation' => $attemptItem, 'itemIdentifier' => $identifier), array('interaction_id' => 'asc'));
@@ -985,10 +986,10 @@ class ScormEngine
                 // Si il existe des interactions
                 if (!empty($itemInteractions)) {
                     $result = array();
-                    
+
                     // On récupère le nom de l'item
                     $result['title'] = $domScorm2004->getTitle($identifier);
-                    
+
                     // On Récupère le Score sur le mapTrackItem
                     $score = null;
                     $scoreMax = $item->getScoreMax();
@@ -1036,7 +1037,7 @@ class ScormEngine
             $this->em->flush();
             return true;
         }
-        
+
         return false;
     }
 
@@ -1074,7 +1075,7 @@ class ScormEngine
             $this->em->flush();
             return true;
         }
-        
+
         return false;
     }
 
@@ -1155,8 +1156,7 @@ class ScormEngine
      * @return array
      */
     public function getAttemptStats(int $attemptId)
-    {
-    }
+    { }
 
     // TOOLS
 
@@ -1167,7 +1167,7 @@ class ScormEngine
      * @param Response $response
      * @return void
      */
-    private function prepareContentAccess(string $courseKey, Response $response = null) : void
+    private function prepareContentAccess(string $courseKey, Response $response = null): void
     {
         if (!$this->session->has('scormkey')) {
             $scormkey = rand();
@@ -1182,7 +1182,7 @@ class ScormEngine
             $this->response = $response;
         }
 
-        $pathCookie = $this->contentUrlPrefix."/$scormkey/$courseKey/";
+        $pathCookie = $this->contentUrlPrefix . "/$scormkey/$courseKey/";
 
         $md5cookiekey = md5($scormkey);
         $cookie = new Cookie('SCORMBundle', $md5cookiekey, 0, $pathCookie);
@@ -1195,7 +1195,7 @@ class ScormEngine
      *
      * @return Response
      */
-    public function getResponse() : Response
+    public function getResponse(): Response
     {
         return $this->response;
     }
@@ -1210,11 +1210,11 @@ class ScormEngine
      *
      * @return string
      */
-    public function getBaseUrl($courseKey) : string
+    public function getBaseUrl($courseKey): string
     {
-        return $this->contentUrlPrefix.'/'.$this->session->get('scormkey').'/'.$courseKey;
+        return $this->contentUrlPrefix . '/' . $this->session->get('scormkey') . '/' . $courseKey;
     }
-    
+
     /**
      * Retourne l'activité
      *
@@ -1224,12 +1224,12 @@ class ScormEngine
     protected function requireCourse(string $courseKey)
     {
         $repository = $this->em->getRepository(Course::class);
-        
+
         $course = $repository->findOneBy(array('courseKey' => $courseKey));
 
         return $course;
     }
-    
+
     /**
      * Retourne l'apprenant
      *
@@ -1244,7 +1244,7 @@ class ScormEngine
 
         return $learner;
     }
-    
+
     /**
      * Retourne l'inscription
      *
@@ -1260,8 +1260,8 @@ class ScormEngine
 
         return $registration;
     }
-    
-     /**
+
+    /**
      * Retourne un essais relatif à une inscription
      *
      * @param int $attemptId