Flint fixes

main/exercise/question.class.php

@@ -2489,6 +2489,21 @@ abstract class Question
         return $result;
     }
 
+    /**
+     * @return int
+     */
+    public function countAnswers()
+    {
+        $result = Database::select(
+            'COUNT(1) AS c',
+            Database::get_course_table(TABLE_QUIZ_ANSWER),
+            ['where' => ['question_id = ?' => [$this->id]]],
+            'first'
+        );
+
+        return (int) $result['c'];
+    }
+
     /**
      * Resizes a picture || Warning!: can only be called after uploadPicture,
      * or if picture is already available in object.
@@ -2553,19 +2568,4 @@ abstract class Question
 
         return false;
     }
-
-    /**
-     * @return int
-     */
-    public function countAnswers()
-    {
-        $result = Database::select(
-            'COUNT(1) AS c',
-            Database::get_course_table(TABLE_QUIZ_ANSWER),
-            ['where' => ['question_id = ?' => [$this->id]]],
-            'first'
-        );
-
-        return (int) $result['c'];
-    }
 }

main/inc/lib/course.lib.php

@@ -2843,6 +2843,7 @@ class CourseManager
     /**
      * Get the list of course IDs with the special_course field
      * set to 1. This function is access_url aware.
+     *
      * @return array
      */
     public static function get_special_course_list()

main/inc/lib/hook/interfaces/CheckLoginCredentialsHookObserverInterface.php

@@ -7,8 +7,6 @@
 interface CheckLoginCredentialsHookObserverInterface extends HookObserverInterface
 {
     /**
-     * @param CheckLoginCredentialsHookEventInterface $event
-     *
      * @return bool
      */
     public function checkLoginCredentials(CheckLoginCredentialsHookEventInterface $event);

main/inc/local.inc.php

@@ -441,7 +441,7 @@ if (!empty($_SESSION['_user']['user_id']) && !($login || $logout)) {
                             'credentials' => [
                                 'username' => $login,
                                 'password' => $password,
-                            ]
+                            ],
                         ]);
                         $validPassword = $checkLoginCredentialHook->notifyLoginCredentials();
                     }

main/lp/learnpath.class.php

@@ -13719,6 +13719,68 @@ EOD;
         }
     }
 
+    /**
+     * Save the new order for learning path items.
+     *
+     * We have to update parent_item_id, previous_item_id, next_item_id, display_order in the database.
+     *
+     * @param array $orderList A associative array with item ID as key and parent ID as value.
+     * @param int   $courseId
+     */
+    public static function sortItemByOrderList(array $orderList, $courseId = 0)
+    {
+        $courseId = $courseId ?: api_get_course_int_id();
+        $itemList = new LpItemOrderList();
+
+        foreach ($orderList as $id => $parentId) {
+            $item = new LpOrderItem($id, $parentId);
+            $itemList->add($item);
+        }
+
+        $parents = $itemList->getListOfParents();
+
+        foreach ($parents as $parentId) {
+            $sameParentLpItemList = $itemList->getItemWithSameParent($parentId);
+            $previous_item_id = 0;
+            for ($i = 0; $i < count($sameParentLpItemList->list); $i++) {
+                $item_id = $sameParentLpItemList->list[$i]->id;
+                // display_order
+                $display_order = $i + 1;
+                $itemList->setParametersForId($item_id, $display_order, 'display_order');
+                // previous_item_id
+                $itemList->setParametersForId($item_id, $previous_item_id, 'previous_item_id');
+                $previous_item_id = $item_id;
+                // next_item_id
+                $next_item_id = 0;
+                if ($i < count($sameParentLpItemList->list) - 1) {
+                    $next_item_id = $sameParentLpItemList->list[$i + 1]->id;
+                }
+                $itemList->setParametersForId($item_id, $next_item_id, 'next_item_id');
+            }
+        }
+
+        $table = Database::get_course_table(TABLE_LP_ITEM);
+
+        foreach ($itemList->list as $item) {
+            $params = [];
+            $params['display_order'] = $item->display_order;
+            $params['previous_item_id'] = $item->previous_item_id;
+            $params['next_item_id'] = $item->next_item_id;
+            $params['parent_item_id'] = $item->parent_item_id;
+
+            Database::update(
+                $table,
+                $params,
+                [
+                    'iid = ? AND c_id = ? ' => [
+                        (int) $item->id,
+                        (int) $courseId,
+                    ],
+                ]
+            );
+        }
+    }
+
     /**
      * Get the depth level of LP item.
      *
@@ -13823,66 +13885,4 @@ EOD;
 
         return '';
     }
-
-    /**
-     * Save the new order for learning path items
-     *
-     * We have to update parent_item_id, previous_item_id, next_item_id, display_order in the database.
-     *
-     * @param array $orderList A associative array with item ID as key and parent ID as value.
-     * @param int   $courseId
-     */
-    public static function sortItemByOrderList(array $orderList, $courseId = 0)
-    {
-        $courseId = $courseId ?: api_get_course_int_id();
-        $itemList = new LpItemOrderList();
-
-        foreach ($orderList as $id => $parentId) {
-            $item = new LpOrderItem($id, $parentId);
-            $itemList->add($item);
-        }
-
-        $parents = $itemList->getListOfParents();
-
-        foreach ($parents as $parentId) {
-            $sameParentLpItemList = $itemList->getItemWithSameParent($parentId);
-            $previous_item_id = 0;
-            for ($i = 0; $i < count($sameParentLpItemList->list); $i++) {
-                $item_id = $sameParentLpItemList->list[$i]->id;
-                // display_order
-                $display_order = $i + 1;
-                $itemList->setParametersForId($item_id, $display_order, 'display_order');
-                // previous_item_id
-                $itemList->setParametersForId($item_id, $previous_item_id, 'previous_item_id');
-                $previous_item_id = $item_id;
-                // next_item_id
-                $next_item_id = 0;
-                if ($i < count($sameParentLpItemList->list) - 1) {
-                    $next_item_id = $sameParentLpItemList->list[$i + 1]->id;
-                }
-                $itemList->setParametersForId($item_id, $next_item_id, 'next_item_id');
-            }
-        }
-
-        $table = Database::get_course_table(TABLE_LP_ITEM);
-
-        foreach ($itemList->list as $item) {
-            $params = [];
-            $params['display_order'] = $item->display_order;
-            $params['previous_item_id'] = $item->previous_item_id;
-            $params['next_item_id'] = $item->next_item_id;
-            $params['parent_item_id'] = $item->parent_item_id;
-
-            Database::update(
-                $table,
-                $params,
-                [
-                    'iid = ? AND c_id = ? ' => [
-                        (int) $item->id,
-                        (int) $courseId,
-                    ],
-                ]
-            );
-        }
-    }
 }

plugin/migrationmoodle/admin.php

@@ -103,7 +103,7 @@ $menu = [
         'users_learn_paths_lesson_timer',
         'users_learn_paths_lesson_branch',
         'users_learn_paths_lesson_attempts',
-        'users_learn_paths_quizzes'
+        'users_learn_paths_quizzes',
     ],
     'users_scorms_view' => [
         'users_scorms_progress',
@@ -153,7 +153,8 @@ Display::display_footer();
  *
  * @return string
  */
-function displayMenu($parent = '_') {
+function displayMenu($parent = '_')
+{
     $plugin = MigrationMoodlePlugin::create();
     $menu = $GLOBALS['menu'];
 
@@ -193,11 +194,11 @@ function displayMenu($parent = '_') {
 
 /**
  * @param string $action
- * @param array  $menu
  *
  * @return bool
  */
-function isAllowedAction($action, array $menu) {
+function isAllowedAction($action, array $menu)
+{
     foreach ($menu as $items) {
         if (in_array($action, $items)) {
             return true;

plugin/migrationmoodle/src/Extractor/BaseExtractor.php

@@ -19,8 +19,6 @@ class BaseExtractor implements ExtractorInterface
 
     /**
      * Extractor constructor.
-     *
-     * @param array $configuration
      */
     public function __construct(array $configuration)
     {
@@ -28,8 +26,6 @@ class BaseExtractor implements ExtractorInterface
     }
 
     /**
-     * @param array $sourceData
-     *
      * @return bool
      */
     public function filter(array $sourceData)

plugin/migrationmoodle/src/Extractor/LoadedCoursesFilterExtractor.php

@@ -16,8 +16,6 @@ class LoadedCoursesFilterExtractor extends FilterExtractor
 {
     /**
      * LoadedCoursesFilterExtractor constructor.
-     *
-     * @param array $configuration
      */
     public function __construct(array $configuration)
     {
@@ -27,8 +25,6 @@ class LoadedCoursesFilterExtractor extends FilterExtractor
     }
 
     /**
-     * @param array $sourceData
-     *
      * @return bool
      */
     public function filter(array $sourceData)

plugin/migrationmoodle/src/Extractor/LoadedScormsFilterExtractor.php

@@ -16,8 +16,6 @@ class LoadedScormsFilterExtractor extends FilterExtractor
 {
     /**
      * LoadedScormsFilterExtractor constructor.
-     *
-     * @param array $configuration
      */
     public function __construct(array $configuration)
     {
@@ -29,8 +27,6 @@ class LoadedScormsFilterExtractor extends FilterExtractor
     /**
      * Filter to avoid scorms not yet migrated.
      *
-     * @param array $sourceData
-     *
      * @return bool
      */
     public function filter(array $sourceData)

plugin/migrationmoodle/src/Extractor/LoadedUsersFilterExtractor.php

@@ -14,8 +14,6 @@ class LoadedUsersFilterExtractor extends FilterExtractor
 {
     /**
      * LoadedUsersFilterExtractor constructor.
-     *
-     * @param array $configuration
      */
     public function __construct(array $configuration)
     {
@@ -25,8 +23,6 @@ class LoadedUsersFilterExtractor extends FilterExtractor
     }
 
     /**
-     * @param array $sourceData
-     *
      * @return bool
      */
     public function filter(array $sourceData)

plugin/migrationmoodle/src/Interfaces/ExtractorInterface.php

@@ -9,8 +9,6 @@ namespace Chamilo\PluginBundle\MigrationMoodle\Interfaces;
 interface ExtractorInterface
 {
     /**
-     * @param array $sourceData
-     *
      * @return bool
      */
     public function filter(array $sourceData);

plugin/migrationmoodle/src/Interfaces/LoaderInterface.php

@@ -11,8 +11,6 @@ interface LoaderInterface
     /**
      * Load the data and return the ID inserted.
      *
-     * @param array $incomingData
-     *
      * @return int
      */
     public function load(array $incomingData);

plugin/migrationmoodle/src/Interfaces/TransformPropertyInterface.php

@@ -9,8 +9,6 @@ namespace Chamilo\PluginBundle\MigrationMoodle\Interfaces;
 interface TransformPropertyInterface
 {
     /**
-     * @param array $data
-     *
      * @return mixed
      */
     public function transform(array $data);

plugin/migrationmoodle/src/Interfaces/TransformerInterface.php

@@ -9,8 +9,6 @@ namespace Chamilo\PluginBundle\MigrationMoodle\Interfaces;
 interface TransformerInterface
 {
     /**
-     * @param array $sourceData
-     *
      * @throws \Exception
      *
      * @return array

plugin/migrationmoodle/src/Loader/CQuizLoader.php

@@ -15,8 +15,6 @@ class CQuizLoader implements LoaderInterface
     /**
      * Load the data and return the ID inserted.
      *
-     * @param array $incomingData
-     *
      * @throws \Doctrine\DBAL\DBALException
      *
      * @return int

plugin/migrationmoodle/src/Loader/CourseCategoriesLoader.php

@@ -15,8 +15,6 @@ class CourseCategoriesLoader implements LoaderInterface
     /**
      * Load the data and return the ID inserted.
      *
-     * @param array $incomingData
-     *
      * @return int
      */
     public function load(array $incomingData)

plugin/migrationmoodle/src/Loader/CourseFilesLoader.php

@@ -20,8 +20,6 @@ class CourseFilesLoader implements LoaderInterface
     /**
      * Load the data and return the ID inserted.
      *
-     * @param array $incomingData
-     *
      * @return int
      */
     public function load(array $incomingData)

plugin/migrationmoodle/src/Loader/CourseIntroductionLoader.php

@@ -14,7 +14,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Interfaces\LoaderInterface;
 class CourseIntroductionLoader implements LoaderInterface
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function load(array $incomingData)
     {

plugin/migrationmoodle/src/Loader/CourseModulesLessonLoader.php

@@ -17,8 +17,6 @@ class CourseModulesLessonLoader implements LoaderInterface
     /**
      * Load the data and return the ID inserted.
      *
-     * @param array $incomingData
-     *
      * @return int
      */
     public function load(array $incomingData)

plugin/migrationmoodle/src/Loader/CourseModulesQuizLoader.php

@@ -14,12 +14,9 @@ use Chamilo\PluginBundle\MigrationMoodle\Interfaces\LoaderInterface;
  */
 class CourseModulesQuizLoader implements LoaderInterface
 {
-
     /**
      * Load the data and return the ID inserted.
      *
-     * @param array $incomingData
-     *
      * @return int
      */
     public function load(array $incomingData)

plugin/migrationmoodle/src/Loader/CourseModulesScormLoader.php

@@ -14,7 +14,7 @@ use Symfony\Component\Filesystem\Filesystem;
 class CourseModulesScormLoader implements LoaderInterface
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function load(array $incomingData)
     {

plugin/migrationmoodle/src/Loader/CourseModulesUrlLoader.php

@@ -13,7 +13,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Interfaces\LoaderInterface;
 class CourseModulesUrlLoader implements LoaderInterface
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function load(array $incomingData)
     {

plugin/migrationmoodle/src/Loader/CourseSectionsLoader.php

@@ -17,8 +17,6 @@ class CourseSectionsLoader implements LoaderInterface
     /**
      * Load the data and return the ID inserted.
      *
-     * @param array $incomingData
-     *
      * @return int
      */
     public function load(array $incomingData)

plugin/migrationmoodle/src/Loader/CoursesLoader.php

@@ -13,8 +13,6 @@ use Chamilo\PluginBundle\MigrationMoodle\Interfaces\LoaderInterface;
 class CoursesLoader implements LoaderInterface
 {
     /**
-     * @param array $incomingData
-     *
      * @return int
      */
     public function load(array $incomingData)

plugin/migrationmoodle/src/Loader/FilesForScormScoLoader.php

@@ -16,7 +16,7 @@ class FilesForScormScoLoader extends CourseFilesLoader
     use FileFinderTrait;
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function load(array $incomingData)
     {

plugin/migrationmoodle/src/Loader/LessonAnswersEssayLoader.php

@@ -17,8 +17,6 @@ class LessonAnswersEssayLoader implements LoaderInterface
     /**
      * Load the data and return the ID inserted.
      *
-     * @param array $incomingData
-     *
      * @return int
      */
     public function load(array $incomingData)

plugin/migrationmoodle/src/Loader/LessonAnswersMatchingLoader.php

@@ -17,8 +17,6 @@ class LessonAnswersMatchingLoader implements LoaderInterface
     /**
      * Load the data and return the ID inserted.
      *
-     * @param array $incomingData
-     *
      * @return int
      */
     public function load(array $incomingData)

plugin/migrationmoodle/src/Loader/LessonAnswersMultipleAnswerLoader.php

@@ -15,8 +15,6 @@ use Chamilo\PluginBundle\MigrationMoodle\Interfaces\LoaderInterface;
 class LessonAnswersMultipleAnswerLoader implements LoaderInterface
 {
     /**
-     * @param array $incomingData
-     *
      * @return int
      */
     public function load(array $incomingData)

plugin/migrationmoodle/src/Loader/LessonAnswersMultipleChoiceLoader.php

@@ -13,8 +13,6 @@ namespace Chamilo\PluginBundle\MigrationMoodle\Loader;
 class LessonAnswersMultipleChoiceLoader extends LessonAnswersTrueFalseLoader
 {
     /**
-     * @param array $incomingData
-     *
      * @return int
      */
     public function load(array $incomingData)

plugin/migrationmoodle/src/Loader/LessonAnswersShortAnswerLoader.php

@@ -19,8 +19,6 @@ class LessonAnswersShortAnswerLoader implements LoaderInterface
     /**
      * Load the data and return the ID inserted.
      *
-     * @param array $incomingData
-     *
      * @return int
      */
     public function load(array $incomingData)

plugin/migrationmoodle/src/Loader/LessonAnswersTrueFalseLoader.php

@@ -17,8 +17,6 @@ class LessonAnswersTrueFalseLoader implements LoaderInterface
     /**
      * Load the data and return the ID inserted.
      *
-     * @param array $incomingData
-     *
      * @return int
      */
     public function load(array $incomingData)

plugin/migrationmoodle/src/Loader/LessonPagesDocumentLoader.php

@@ -17,8 +17,6 @@ class LessonPagesDocumentLoader implements LoaderInterface
     /**
      * Load the data and return the ID inserted.
      *
-     * @param array $incomingData
-     *
      * @return int
      */
     public function load(array $incomingData)

plugin/migrationmoodle/src/Loader/LessonPagesLoader.php

@@ -17,8 +17,6 @@ class LessonPagesLoader implements LoaderInterface
     /**
      * Load the data and return the ID inserted.
      *
-     * @param array $incomingData
-     *
      * @return int
      */
     public function load(array $incomingData)

plugin/migrationmoodle/src/Loader/LessonQuestionPagesQuestionLoader.php

@@ -18,8 +18,6 @@ class LessonQuestionPagesQuestionLoader implements LoaderInterface
     /**
      * Load the data and return the ID inserted.
      *
-     * @param array $incomingData
-     *
      * @return int
      */
     public function load(array $incomingData)

plugin/migrationmoodle/src/Loader/LessonQuestionPagesQuizLoader.php

@@ -17,8 +17,6 @@ class LessonQuestionPagesQuizLoader implements LoaderInterface
     /**
      * Load the data and return the ID inserted.
      *
-     * @param array $incomingData
-     *
      * @throws \Doctrine\DBAL\DBALException
      * @throws \Exception
      *

plugin/migrationmoodle/src/Loader/QuestionCategoriesLoader.php

@@ -17,8 +17,6 @@ class QuestionCategoriesLoader implements LoaderInterface
     /**
      * Load the data and return the ID inserted.
      *
-     * @param array $incomingData
-     *
      * @throws \Exception
      *
      * @return int

plugin/migrationmoodle/src/Loader/QuestionGapselectLoader.php

@@ -13,7 +13,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Interfaces\LoaderInterface;
 class QuestionGapselectLoader implements LoaderInterface
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function load(array $incomingData)
     {

plugin/migrationmoodle/src/Loader/QuizzesLoader.php

@@ -17,8 +17,6 @@ class QuizzesLoader implements LoaderInterface
     /**
      * Load the data and return the ID inserted.
      *
-     * @param array $incomingData
-     *
      * @return int
      */
     public function load(array $incomingData)

plugin/migrationmoodle/src/Loader/QuizzesScoresLoader.php

@@ -13,7 +13,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Interfaces\LoaderInterface;
 class QuizzesScoresLoader implements LoaderInterface
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function load(array $incomingData)
     {

plugin/migrationmoodle/src/Loader/RoleAssignmentsLoader.php

@@ -16,8 +16,6 @@ class RoleAssignmentsLoader implements LoaderInterface
     /**
      * Load the data and return the ID inserted.
      *
-     * @param array $incomingData
-     *
      * @return int
      */
     public function load(array $incomingData)

plugin/migrationmoodle/src/Loader/ScormScoLoader.php

@@ -13,7 +13,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Interfaces\LoaderInterface;
 class ScormScoLoader implements LoaderInterface
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function load(array $incomingData)
     {

plugin/migrationmoodle/src/Loader/SortSectionModuleLoader.php

@@ -13,7 +13,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Interfaces\LoaderInterface;
 class SortSectionModuleLoader implements LoaderInterface
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function load(array $incomingData)
     {

plugin/migrationmoodle/src/Loader/UrlLoader.php

@@ -15,7 +15,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Interfaces\LoaderInterface;
 class UrlLoader implements LoaderInterface
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function load(array $incomingData)
     {

plugin/migrationmoodle/src/Loader/UserLearnPathLessonAttemptLoader.php

@@ -11,8 +11,6 @@ namespace Chamilo\PluginBundle\MigrationMoodle\Loader;
 class UserLearnPathLessonAttemptLoader extends UserLearnPathLessonBranchLoader
 {
     /**
-     * @param array $incomingData
-     *
      * @throws \Doctrine\ORM\ORMException
      * @throws \Doctrine\ORM\OptimisticLockException
      * @throws \Doctrine\ORM\TransactionRequiredException

plugin/migrationmoodle/src/Loader/UserLearnPathLessonBranchLoader.php

@@ -14,7 +14,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Interfaces\LoaderInterface;
 class UserLearnPathLessonBranchLoader implements LoaderInterface
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function load(array $incomingData)
     {
@@ -49,8 +49,6 @@ class UserLearnPathLessonBranchLoader implements LoaderInterface
     }
 
     /**
-     * @param array $incomingData
-     *
      * @throws \Doctrine\ORM\NonUniqueResultException
      *
      * @return CLpItemView
@@ -68,16 +66,13 @@ class UserLearnPathLessonBranchLoader implements LoaderInterface
             ->getOneOrNullResult();
 
         if (!$itemView) {
-            throw new \Exception("Item view not found for "
+            throw new \Exception("Item view not found for "."item ({$incomingData['item_id']}) and user ({$incomingData['user_id']}).");
-                ."item ({$incomingData['item_id']}) and user ({$incomingData['user_id']}).");
         }
 
         return $itemView;
     }
 
     /**
-     * @param array $incomingData
-     *
      * @throws \Doctrine\ORM\NonUniqueResultException
      *
      * @return CLpItemView
@@ -96,8 +91,7 @@ class UserLearnPathLessonBranchLoader implements LoaderInterface
             ->getOneOrNullResult();
 
         if (!$previuousItemView) {
-            throw new \Exception("Item view not found for "
+            throw new \Exception("Item view not found for "."previous item ({$incomingData['item_id']}) and user ({$incomingData['user_id']}).");
-                ."previous item ({$incomingData['item_id']}) and user ({$incomingData['user_id']}).");
         }
 
         return $previuousItemView;

plugin/migrationmoodle/src/Loader/UserLearnPathLessonTimerLoader.php

@@ -14,7 +14,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Interfaces\LoaderInterface;
 class UserLearnPathLessonTimerLoader implements LoaderInterface
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function load(array $incomingData)
     {
@@ -32,8 +32,6 @@ class UserLearnPathLessonTimerLoader implements LoaderInterface
     }
 
     /**
-     * @param array $incomingData
-     *
      * @throws \Doctrine\ORM\NonUniqueResultException
      *
      * @return CLpItemView
@@ -58,8 +56,6 @@ class UserLearnPathLessonTimerLoader implements LoaderInterface
     }
 
     /**
-     * @param array $incomingData
-     *
      * @throws \Doctrine\ORM\NonUniqueResultException
      *
      * @return CLpItemView
@@ -93,8 +89,7 @@ class UserLearnPathLessonTimerLoader implements LoaderInterface
             ->getOneOrNullResult();
 
         if (!$itemView) {
-            throw new \Exception("Item view not found for item with"
+            throw new \Exception("Item view not found for item with"." parent item ({$incomingData['parent_item_id']}) and user ({$incomingData['user_id']})");
-                ." parent item ({$incomingData['parent_item_id']}) and user ({$incomingData['user_id']})");
         }
 
         return $itemView;

plugin/migrationmoodle/src/Loader/UserLearnPathQuizLoader.php

@@ -14,7 +14,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Interfaces\LoaderInterface;
 class UserLearnPathQuizLoader implements LoaderInterface
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function load(array $incomingData)
     {

plugin/migrationmoodle/src/Loader/UserLearnPathsLoader.php

@@ -14,7 +14,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Interfaces\LoaderInterface;
 class UserLearnPathsLoader implements LoaderInterface
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function load(array $incomingData)
     {

plugin/migrationmoodle/src/Loader/UserQuestionAttemptLoader.php

@@ -13,7 +13,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Interfaces\LoaderInterface;
 class UserQuestionAttemptLoader implements LoaderInterface
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function load(array $incomingData)
     {

plugin/migrationmoodle/src/Loader/UserQuizAttemptLoader.php

@@ -3,8 +3,6 @@
 
 namespace Chamilo\PluginBundle\MigrationMoodle\Loader;
 
-use Chamilo\CoreBundle\Entity\TrackEExercises;
-use Chamilo\CourseBundle\Entity\CLpItemView;
 use Chamilo\PluginBundle\MigrationMoodle\Interfaces\LoaderInterface;
 
 /**
@@ -15,7 +13,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Interfaces\LoaderInterface;
 class UserQuizAttemptLoader implements LoaderInterface
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function load(array $incomingData)
     {

plugin/migrationmoodle/src/Loader/UserScormProgressLoader.php

@@ -13,7 +13,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Interfaces\LoaderInterface;
 class UserScormProgressLoader implements LoaderInterface
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function load(array $incomingData)
     {

plugin/migrationmoodle/src/Loader/UserSessionLoader.php

@@ -14,7 +14,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Task\UserSessionsTask;
 class UserSessionLoader implements LoaderInterface
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function load(array $incomingData)
     {
@@ -28,9 +28,7 @@ class UserSessionLoader implements LoaderInterface
             $courseId = api_get_course_int_id($courseCode);
 
             if (empty($courseId)) {
-                throw new \Exception(
+                throw new \Exception("Course ($courseCode) not found when creating course session for user ({$incomingData['user_id']})");
-                    "Course ($courseCode) not found when creating course session for user ({$incomingData['user_id']})"
-                );
             }
 
             $courseIds[] = $courseId;

plugin/migrationmoodle/src/Loader/UsersLoader.php

@@ -16,8 +16,6 @@ use Chamilo\UserBundle\Entity\User;
 class UsersLoader implements LoaderInterface
 {
     /**
-     * @param array $incomingData
-     *
      * @throws \Doctrine\ORM\OptimisticLockException
      *
      * @return int

plugin/migrationmoodle/src/Loader/UsersScormsViewLoader.php

@@ -4,7 +4,6 @@
 namespace Chamilo\PluginBundle\MigrationMoodle\Loader;
 
 use Chamilo\CoreBundle\Entity\Session;
-use Chamilo\CourseBundle\Entity\CLpView;
 use Chamilo\PluginBundle\MigrationMoodle\Interfaces\LoaderInterface;
 
 /**
@@ -15,7 +14,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Interfaces\LoaderInterface;
 class UsersScormsViewLoader implements LoaderInterface
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function load(array $incomingData)
     {
@@ -77,9 +76,7 @@ class UsersScormsViewLoader implements LoaderInterface
             ->findOneBy(['user' => $userId, 'course' => $courseId]);
 
         if (empty($subscription)) {
-            throw new \Exception(
+            throw new \Exception("Session not found for user ($userId) with course ($courseId)");
-                "Session not found for user ($userId) with course ($courseId)"
-            );
         }
 
         return $subscription->getSession();
@@ -102,7 +99,7 @@ class UsersScormsViewLoader implements LoaderInterface
                     'userId' => $userId,
                     'lpId' => $lpId,
                     'cId' => $cId,
-                    'sessionId' => $sessionId
+                    'sessionId' => $sessionId,
                 ],
                 ['viewCount' => 'DESC']
             );

plugin/migrationmoodle/src/Messages/ExtractMessage.php

@@ -14,8 +14,6 @@ class ExtractMessage extends Message
 {
     /**
      * ExtractMessage constructor.
-     *
-     * @param Throwable|null $previous
      */
     public function __construct(Throwable $previous = null)
     {

plugin/migrationmoodle/src/Messages/LoadMessage.php

@@ -20,8 +20,7 @@ class LoadMessage extends Message
     /**
      * LoadMessage constructor.
      *
-     * @param                $incomingData
+     * @param $incomingData
-     * @param Throwable|null $previous
      */
     public function __construct($incomingData, Throwable $previous = null)
     {

plugin/migrationmoodle/src/Messages/Message.php

@@ -24,8 +24,7 @@ abstract class Message
     /**
      * Message constructor.
      *
-     * @param string         $message
+     * @param string $message
-     * @param Throwable|null $previous
      */
     public function __construct($message = "", Throwable $previous = null)
     {

plugin/migrationmoodle/src/Messages/TransformMessage.php

@@ -19,9 +19,6 @@ class TransformMessage extends Message
 
     /**
      * TransformMessage constructor.
-     *
-     * @param array          $extractedData
-     * @param Throwable|null $previous
      */
     public function __construct(array $extractedData, Throwable $previous = null)
     {

plugin/migrationmoodle/src/MigrationMoodleCheckLoginCredentialsHook.php

@@ -1,15 +1,13 @@
 <?php
 /* For licensing terms, see /license.txt */
 
-use Chamilo\CoreBundle\Entity\ExtraFieldValues;
 use Chamilo\CoreBundle\Entity\ExtraField;
+use Chamilo\CoreBundle\Entity\ExtraFieldValues;
 
 /**
  * Class MigrationMoodleCheckLoginCredentialsHook.
  */
-class MigrationMoodleCheckLoginCredentialsHook
+class MigrationMoodleCheckLoginCredentialsHook extends HookObserver implements CheckLoginCredentialsHookObserverInterface
-    extends HookObserver
-    implements CheckLoginCredentialsHookObserverInterface
 {
     /**
      * MigrationMoodleCheckLoginCredentialsHook constructor.
@@ -23,8 +21,6 @@ class MigrationMoodleCheckLoginCredentialsHook
     }
 
     /**
-     * @param CheckLoginCredentialsHookEventInterface $event
-     *
      * @return bool
      */
     public function checkLoginCredentials(CheckLoginCredentialsHookEventInterface $event)
@@ -75,9 +71,6 @@ class MigrationMoodleCheckLoginCredentialsHook
     }
 
     /**
-     * @param ExtraField $extraField
-     * @param array      $userData
-     *
      * @return ExtraFieldValues|null
      */
     private function getExtraFieldValue(ExtraField $extraField, array $userData)

plugin/migrationmoodle/src/Task/BaseTask.php

@@ -132,7 +132,7 @@ abstract class BaseTask
 
             yield md5("{$extractedData['id']}@@$loadedId") => [
                 'extracted' => $extractedData['id'],
-                'loaded' => $loadedId
+                'loaded' => $loadedId,
             ];
         }
     }

plugin/migrationmoodle/src/Task/CQuizTask.php

@@ -58,7 +58,7 @@ class CQuizTask extends BaseTask
                 'enabletimercontroltotalminutes' => 'timelimit',
                 'pass_percentage' => [
                     'class' => Percentage::class,
-                    'properties' => ['sumgrades', 'grade']
+                    'properties' => ['sumgrades', 'grade'],
                 ],
                 //'text_when_finished',
             ],

plugin/migrationmoodle/src/Task/CourseCategoriesTask.php

@@ -6,8 +6,8 @@ namespace Chamilo\PluginBundle\MigrationMoodle\Task;
 use Chamilo\PluginBundle\MigrationMoodle\Extractor\BaseExtractor;
 use Chamilo\PluginBundle\MigrationMoodle\Loader\CourseCategoriesLoader;
 use Chamilo\PluginBundle\MigrationMoodle\Transformer\BaseTransformer;
-use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\CourseCode;
 use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\CourseCategoryLookup;
+use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\CourseCode;
 
 /**
  * Class CourseCategoriesTask.

plugin/migrationmoodle/src/Task/CourseIntroductionsTask.php

@@ -19,7 +19,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\ReplaceFilePaths;
 class CourseIntroductionsTask extends BaseTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
@@ -32,7 +32,7 @@ class CourseIntroductionsTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getTransformConfiguration()
     {
@@ -53,7 +53,7 @@ class CourseIntroductionsTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getLoadConfiguration()
     {

plugin/migrationmoodle/src/Task/CourseModulesQuizTask.php

@@ -49,7 +49,7 @@ class CourseModulesQuizTask extends BaseTask
                 ],
                 'lp_id' => [
                     'class' => LoadedCourseSectionLookup::class,
-                    'properties' => ['section']
+                    'properties' => ['section'],
                 ],
                 'title' => 'name',
             ],

plugin/migrationmoodle/src/Task/CourseModulesScormTask.php

@@ -19,7 +19,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\LoadedCourseLookup
 class CourseModulesScormTask extends BaseTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
@@ -48,7 +48,7 @@ class CourseModulesScormTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getTransformConfiguration()
     {
@@ -66,22 +66,22 @@ class CourseModulesScormTask extends BaseTask
                 'hide_toc_frame' => 'hidetoc',
                 'created_on' => [
                     'class' => DateTimeObject::class,
-                    'properties' => ['added']
+                    'properties' => ['added'],
                 ],
                 'modified_on' => [
                     'class' => DateTimeObject::class,
-                    'properties' => ['timemodified']
+                    'properties' => ['timemodified'],
                 ],
                 'publicated_on' => [
                     'class' => DateTimeObject::class,
-                    'properties' => ['added']
+                    'properties' => ['added'],
                 ],
             ],
         ];
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getLoadConfiguration()
     {

plugin/migrationmoodle/src/Task/CourseModulesUrlTask.php

@@ -19,7 +19,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\LoadedCourseSectio
 class CourseModulesUrlTask extends BaseTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
@@ -43,7 +43,7 @@ class CourseModulesUrlTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getTransformConfiguration()
     {
@@ -64,7 +64,7 @@ class CourseModulesUrlTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getLoadConfiguration()
     {

plugin/migrationmoodle/src/Task/CourseSectionsTask.php

@@ -7,7 +7,6 @@ use Chamilo\PluginBundle\MigrationMoodle\Extractor\LoadedCoursesFilterExtractor;
 use Chamilo\PluginBundle\MigrationMoodle\Loader\CourseSectionsLoader;
 use Chamilo\PluginBundle\MigrationMoodle\Transformer\BaseTransformer;
 use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\LoadedCourseCodeLookup;
-use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\ReplaceFilePaths;
 use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\WrapHtmlReplacingFilePaths;
 
 /**

plugin/migrationmoodle/src/Task/FilesForCourseIntroductionsTask.php

@@ -13,7 +13,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Extractor\LoadedCoursesFilterExtractor;
 class FilesForCourseIntroductionsTask extends CourseFilesTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
@@ -35,7 +35,7 @@ class FilesForCourseIntroductionsTask extends CourseFilesTask
                     AND f.filearea = 'section'
                     AND ctx.contextlevel = 50
                     AND f.filename NOT IN ('.', '..')
-                    AND cs.section = 0 AND (cs.summary != '' AND cs.summary IS NOT NULL)"
+                    AND cs.section = 0 AND (cs.summary != '' AND cs.summary IS NOT NULL)",
         ];
     }
 }

plugin/migrationmoodle/src/Task/FilesForCourseSectionsTask.php

@@ -13,7 +13,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Extractor\LoadedCoursesFilterExtractor;
 class FilesForCourseSectionsTask extends CourseFilesTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {

plugin/migrationmoodle/src/Task/FilesForScormScoesTask.php

@@ -16,7 +16,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\LoadedCourseLookup
 class FilesForScormScoesTask extends BaseTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
@@ -47,8 +47,9 @@ class FilesForScormScoesTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
+
     /**
      * @return array
      */
@@ -71,7 +72,7 @@ class FilesForScormScoesTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getLoadConfiguration()
     {

plugin/migrationmoodle/src/Task/LessonAnswersMultipleAnswerTask.php

@@ -7,7 +7,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Extractor\LoadedCoursesFilterExtractor;
 use Chamilo\PluginBundle\MigrationMoodle\Loader\LessonAnswersMultipleAnswerLoader;
 
 /**
- * Class LessonAnswersMultipleAnswerTask
+ * Class LessonAnswersMultipleAnswerTask.
  *
  * Task to convert Multiple Choice answers from a Moodle lesson page in answers for Unique Answer question for Chamilo.
  *

plugin/migrationmoodle/src/Task/LessonAnswersMultipleChoiceTask.php

@@ -7,7 +7,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Extractor\LoadedCoursesFilterExtractor;
 use Chamilo\PluginBundle\MigrationMoodle\Loader\LessonAnswersMultipleChoiceLoader;
 
 /**
- * Class LessonAnswersMultipleChoiceTask
+ * Class LessonAnswersMultipleChoiceTask.
  *
  * Task to convert Multiple Choice answers from a Moodle lesson page in answers for Unique Answer question for Chamilo.
  *

plugin/migrationmoodle/src/Task/LessonPagesTask.php

@@ -6,11 +6,11 @@ namespace Chamilo\PluginBundle\MigrationMoodle\Task;
 use Chamilo\PluginBundle\MigrationMoodle\Extractor\LoadedCoursesFilterExtractor;
 use Chamilo\PluginBundle\MigrationMoodle\Loader\LessonPagesLoader;
 use Chamilo\PluginBundle\MigrationMoodle\Transformer\BaseTransformer;
+use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\LessonPageType;
 use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\LoadedCourseCodeLookup;
 use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\LoadedCourseModuleLessonLookup;
 use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\LoadedCourseSectionFromLessonLookup;
 use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\LoadedLessonPageLookup;
-use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\LessonPageType;
 
 /**
  * Class LessonPagesTask.
@@ -72,11 +72,11 @@ class LessonPagesTask extends BaseTask
                 ],
                 'lp_id' => [
                     'class' => LoadedCourseSectionFromLessonLookup::class,
-                    'properties' => ['lesson']
+                    'properties' => ['lesson'],
                 ],
                 'parent' => [
                     'class' => LoadedCourseModuleLessonLookup::class,
-                    'properties' => ['lesson']
+                    'properties' => ['lesson'],
                 ],
                 'previous' => [
                     'class' => LoadedLessonPageLookup::class,

plugin/migrationmoodle/src/Task/QuestionCategoriesTask.php

@@ -42,7 +42,7 @@ class QuestionCategoriesTask extends BaseTask
             'map' => [
                 'c_id' => [
                     'class' => LoadedCourseFromQuestionCategoryLookup::class,
-                    'properties' => ['contextlevel', 'instanceid']
+                    'properties' => ['contextlevel', 'instanceid'],
                 ],
                 'name' => 'name',
                 'description' => 'info',

plugin/migrationmoodle/src/Task/QuestionGapselectTask.php

@@ -19,7 +19,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\QuestionGapselectA
 class QuestionGapselectTask extends BaseTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
@@ -43,12 +43,12 @@ class QuestionGapselectTask extends BaseTask
                 INNER JOIN mdl_quiz q ON qs.quizid = q.id
                 WHERE qq.qtype = 'gapselect'
                 GROUP BY q.id, qq.id
-                ORDER BY q.id, qq.id"
+                ORDER BY q.id, qq.id",
         ];
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getTransformConfiguration()
     {
@@ -78,7 +78,7 @@ class QuestionGapselectTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getLoadConfiguration()
     {

plugin/migrationmoodle/src/Task/QuestionMultiChoiceMultipleTask.php

@@ -14,7 +14,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Loader\LessonAnswersMultipleAnswerLoade
 class QuestionMultiChoiceMultipleTask extends QuestionMultiChoiceSingleTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
@@ -40,7 +40,7 @@ class QuestionMultiChoiceMultipleTask extends QuestionMultiChoiceSingleTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getLoadConfiguration()
     {

plugin/migrationmoodle/src/Task/QuestionMultiChoiceSingleTask.php

@@ -21,7 +21,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\ReplaceFilePaths;
 class QuestionMultiChoiceSingleTask extends BaseTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
@@ -47,7 +47,7 @@ class QuestionMultiChoiceSingleTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getTransformConfiguration()
     {
@@ -81,7 +81,7 @@ class QuestionMultiChoiceSingleTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getLoadConfiguration()
     {

plugin/migrationmoodle/src/Task/QuestionShortAnswerTask.php

@@ -17,7 +17,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\LoadedQuestionLook
 class QuestionShortAnswerTask extends LessonAnswersShortAnswerTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
@@ -44,7 +44,7 @@ class QuestionShortAnswerTask extends LessonAnswersShortAnswerTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getTransformConfiguration()
     {

plugin/migrationmoodle/src/Task/QuestionsTrueFalseTask.php

@@ -15,7 +15,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Extractor\LoadedCoursesFilterExtractor;
 class QuestionsTrueFalseTask extends QuestionMultiChoiceSingleTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {

plugin/migrationmoodle/src/Task/QuizzesScoresTask.php

@@ -18,7 +18,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\LoadedQuizLookup;
 class QuizzesScoresTask extends BaseTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
@@ -39,7 +39,7 @@ class QuizzesScoresTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getTransformConfiguration()
     {
@@ -63,7 +63,7 @@ class QuizzesScoresTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getLoadConfiguration()
     {

plugin/migrationmoodle/src/Task/QuizzesTask.php

@@ -79,7 +79,7 @@ class QuizzesTask extends BaseTask
                 'enabletimercontroltotalminutes' => 'timelimit',
                 'pass_percentage' => [
                     'class' => Percentage::class,
-                    'properties' => ['gradepass', 'grade']
+                    'properties' => ['gradepass', 'grade'],
                 ],
             ],
         ];

plugin/migrationmoodle/src/Task/ScormScoesTask.php

@@ -18,7 +18,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\ScormScoParentLook
 class ScormScoesTask extends BaseTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
@@ -32,7 +32,7 @@ class ScormScoesTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getTransformConfiguration()
     {
@@ -60,7 +60,7 @@ class ScormScoesTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getLoadConfiguration()
     {

plugin/migrationmoodle/src/Task/SortSectionModulesTask.php

@@ -20,7 +20,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\SectionSequenceLoo
 class SortSectionModulesTask extends BaseTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
@@ -43,7 +43,7 @@ class SortSectionModulesTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getTransformConfiguration()
     {
@@ -67,7 +67,7 @@ class SortSectionModulesTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getLoadConfiguration()
     {

plugin/migrationmoodle/src/Task/UrlsTask.php

@@ -18,7 +18,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\LoadedCourseSectio
 class UrlsTask extends BaseTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
@@ -42,7 +42,7 @@ class UrlsTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getTransformConfiguration()
     {
@@ -68,7 +68,7 @@ class UrlsTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getLoadConfiguration()
     {

plugin/migrationmoodle/src/Task/UserQuestionAttemptsGapselectTask.php

@@ -11,7 +11,7 @@ namespace Chamilo\PluginBundle\MigrationMoodle\Task;
 class UserQuestionAttemptsGapselectTask extends UserQuestionAttemptsTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {

plugin/migrationmoodle/src/Task/UserQuestionAttemptsShortanswerTask.php

@@ -11,7 +11,7 @@ namespace Chamilo\PluginBundle\MigrationMoodle\Task;
 class UserQuestionAttemptsShortanswerTask extends UserQuestionAttemptsTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {

plugin/migrationmoodle/src/Task/UserQuestionAttemptsTask.php

@@ -24,7 +24,7 @@ abstract class UserQuestionAttemptsTask extends BaseTask
     protected $questionType;
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
@@ -64,7 +64,7 @@ abstract class UserQuestionAttemptsTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getTransformConfiguration()
     {
@@ -113,7 +113,7 @@ abstract class UserQuestionAttemptsTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getLoadConfiguration()
     {

plugin/migrationmoodle/src/Task/UserQuestionAttemptsTruefalseTask.php

@@ -11,7 +11,7 @@ namespace Chamilo\PluginBundle\MigrationMoodle\Task;
 class UserQuestionAttemptsTruefalseTask extends UserQuestionAttemptsTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {

plugin/migrationmoodle/src/Task/UserSessionsTask.php

@@ -19,7 +19,7 @@ class UserSessionsTask extends BaseTask
     const SEPARATOR_NAME = ' - ';
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
@@ -49,7 +49,7 @@ class UserSessionsTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getTransformConfiguration()
     {
@@ -58,7 +58,7 @@ class UserSessionsTask extends BaseTask
             'map' => [
                 'name' => [
                     'class' => SessionName::class,
-                    'properties' => ['username', 'session_name']
+                    'properties' => ['username', 'session_name'],
                 ],
                 'user_id' => [
                     'class' => LoadedUserLookup::class,
@@ -70,7 +70,7 @@ class UserSessionsTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getLoadConfiguration()
     {

plugin/migrationmoodle/src/Task/UsersLearnPathsLessonAttemptsTask.php

@@ -14,7 +14,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Loader\UserLearnPathLessonAttemptLoader
 class UsersLearnPathsLessonAttemptsTask extends UsersLearnPathsLessonBranchTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
@@ -25,7 +25,7 @@ class UsersLearnPathsLessonAttemptsTask extends UsersLearnPathsLessonBranchTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getTransformConfiguration()
     {
@@ -37,7 +37,7 @@ class UsersLearnPathsLessonAttemptsTask extends UsersLearnPathsLessonBranchTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getLoadConfiguration()
     {

plugin/migrationmoodle/src/Task/UsersLearnPathsLessonBranchTask.php

@@ -19,7 +19,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\LoadedUserSessionL
 class UsersLearnPathsLessonBranchTask extends BaseTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
@@ -30,7 +30,7 @@ class UsersLearnPathsLessonBranchTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getTransformConfiguration()
     {
@@ -60,7 +60,7 @@ class UsersLearnPathsLessonBranchTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getLoadConfiguration()
     {

plugin/migrationmoodle/src/Task/UsersLearnPathsLessonTimerTask.php

@@ -20,7 +20,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\LoadedUserSessionL
 class UsersLearnPathsLessonTimerTask extends BaseTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
@@ -31,7 +31,7 @@ class UsersLearnPathsLessonTimerTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getTransformConfiguration()
     {
@@ -56,7 +56,7 @@ class UsersLearnPathsLessonTimerTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getLoadConfiguration()
     {

plugin/migrationmoodle/src/Task/UsersLearnPathsQuizzesTask.php

@@ -20,7 +20,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\Subtract;
 class UsersLearnPathsQuizzesTask extends BaseTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
@@ -33,7 +33,7 @@ class UsersLearnPathsQuizzesTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getTransformConfiguration()
     {
@@ -67,7 +67,7 @@ class UsersLearnPathsQuizzesTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getLoadConfiguration()
     {

plugin/migrationmoodle/src/Task/UsersLearnPathsTask.php

@@ -18,7 +18,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\LoadedUserLookup;
 class UsersLearnPathsTask extends BaseTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
@@ -37,7 +37,7 @@ class UsersLearnPathsTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getTransformConfiguration()
     {
@@ -53,7 +53,7 @@ class UsersLearnPathsTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getLoadConfiguration()
     {

plugin/migrationmoodle/src/Task/UsersQuizzesAttemptsTask.php

@@ -21,7 +21,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\Subtract;
 class UsersQuizzesAttemptsTask extends BaseTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
@@ -44,7 +44,7 @@ class UsersQuizzesAttemptsTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getTransformConfiguration()
     {
@@ -61,7 +61,7 @@ class UsersQuizzesAttemptsTask extends BaseTask
                 ],
                 'exo_id' => [
                     'class' => LoadedQuizLookup::class,
-                    'properties' => ['quiz']
+                    'properties' => ['quiz'],
                 ],
                 'result' => 'real_result',
                 'weighting' => 'weighting',
@@ -83,7 +83,7 @@ class UsersQuizzesAttemptsTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getLoadConfiguration()
     {

plugin/migrationmoodle/src/Task/UsersScormsProgressTask.php

@@ -16,18 +16,18 @@ use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\LoadedUserLookup;
 class UsersScormsProgressTask extends BaseTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
         return [
             'class' => LoadedUsersFilterExtractor::class,
-            'query' => "SELECT DISTINCT userid id FROM mdl_scorm_scoes_track"
+            'query' => "SELECT DISTINCT userid id FROM mdl_scorm_scoes_track",
         ];
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getTransformConfiguration()
     {
@@ -43,7 +43,7 @@ class UsersScormsProgressTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getLoadConfiguration()
     {

plugin/migrationmoodle/src/Task/UsersScormsViewTask.php

@@ -22,7 +22,7 @@ use Chamilo\PluginBundle\MigrationMoodle\Transformer\Property\ScormScoTrackData;
 class UsersScormsViewTask extends BaseTask
 {
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getExtractConfiguration()
     {
@@ -45,7 +45,7 @@ class UsersScormsViewTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getTransformConfiguration()
     {
@@ -78,7 +78,7 @@ class UsersScormsViewTask extends BaseTask
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getLoadConfiguration()
     {

plugin/migrationmoodle/src/Transformer/BaseTransformer.php

@@ -15,8 +15,6 @@ class BaseTransformer implements TransformerInterface
 
     /**
      * BaseTransformer constructor.
-     *
-     * @param array $configuration
      */
     public function __construct(array $configuration)
     {
@@ -24,8 +22,6 @@ class BaseTransformer implements TransformerInterface
     }
 
     /**
-     * @param array $sourceData
-     *
      * @throws \Exception
      *
      * @return array

plugin/migrationmoodle/src/Transformer/Property/AuthLookup.php

@@ -11,8 +11,6 @@ use Chamilo\PluginBundle\MigrationMoodle\Interfaces\TransformPropertyInterface;
 class AuthLookup implements TransformPropertyInterface
 {
     /**
-     * @param array $data
-     *
      * @return string
      */
     public function transform(array $data)

plugin/migrationmoodle/src/Transformer/Property/Copy.php

@@ -11,8 +11,6 @@ use Chamilo\PluginBundle\MigrationMoodle\Interfaces\TransformPropertyInterface;
 class Copy implements TransformPropertyInterface
 {
     /**
-     * @param array $data
-     *
      * @return array
      */
     public function transform(array $data)