Internal: Add platform file/dir permissions in structured-file course import - refs BT#22644

Author: christianbeeznest

src/CoreBundle/Command/CreateCoursesFromStructuredFileCommand.php

@@ -18,11 +18,11 @@
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
 use Symfony\Component\Console\Output\OutputInterface;
+use Symfony\Component\Console\Style\SymfonyStyle;
+use Symfony\Component\DependencyInjection\ParameterBag\ParameterBagInterface;
 use Symfony\Component\Finder\Finder;
 
-
 #[AsCommand(
     name: 'app:create-courses-from-structured-file',
     description: 'Create courses and learning paths from a folder containing files',
@@ -32,20 +32,24 @@ class CreateCoursesFromStructuredFileCommand extends Command
     public function __construct(
         private readonly EntityManagerInterface $em,
         private readonly CourseService $courseService,
-        private readonly SettingsManager $settingsManager
+        private readonly SettingsManager $settingsManager,
+        private readonly ParameterBagInterface $parameterBag,
     ) {
         parent::__construct();
     }
 
     protected function configure(): void
     {
-        $this
-            ->addArgument('folder', InputArgument::REQUIRED, 'Path to folder with course files');
+        $this->addArgument(
+            'folder',
+            InputArgument::REQUIRED,
+            'Absolute path to the folder that contains course files'
+        );
     }
 
     protected function execute(InputInterface $input, OutputInterface $output): int
     {
-        $io = new SymfonyStyle($input, $output);
+        $io        = new SymfonyStyle($input, $output);
         $adminUser = $this->getFirstAdmin();
         if (!$adminUser) {
             $io->error('No admin user found in the system.');
@@ -58,41 +62,52 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             return Command::FAILURE;
         }
 
+        // Retrieve Unix permissions from platform settings
+        $dirPermOct  = octdec(
+            $this->settingsManager->getSetting('document.permissions_for_new_directories') ?? '0777'
+        );
+        $filePermOct = octdec(
+            $this->settingsManager->getSetting('document.permissions_for_new_files') ?? '0666'
+        );
+
+        // Absolute base to /var/upload/resource
+        $uploadBase = $this->parameterBag->get('kernel.project_dir') . '/var/upload/resource';
+
         $finder = new Finder();
         $finder->files()->in($folder);
 
         foreach ($finder as $file) {
-            $basename = $file->getBasename();
+            $basename   = $file->getBasename();
             $courseCode = pathinfo($basename, PATHINFO_FILENAME);
-            $filePath = $file->getRealPath();
+            $filePath   = $file->getRealPath();
 
-            // Skip unsupported extensions
+            // 1. Skip unsupported file extensions
             $allowedExtensions = ['pdf', 'html', 'htm', 'mp4'];
-            if (!in_array(strtolower($file->getExtension()), $allowedExtensions)) {
+            if (!in_array(strtolower($file->getExtension()), $allowedExtensions, true)) {
                 $io->warning("Skipping unsupported file: $basename");
                 continue;
             }
 
             $io->section("Creating course: $courseCode");
 
-            // Step 1: Create the course
+            // 2. Create course
             $course = $this->courseService->createCourse([
-                'title' => $courseCode,
-                'wanted_code' => $courseCode,
+                'title'               => $courseCode,
+                'wanted_code'         => $courseCode,
                 'add_user_as_teacher' => true,
-                'course_language' => $this->settingsManager->getSetting('language.platform_language'),
-                'visibility' => Course::OPEN_PLATFORM,
-                'subscribe' => true,
-                'unsubscribe' => true,
-                'disk_quota' => $this->settingsManager->getSetting('document.default_document_quotum'),
-                'expiration_date' => (new \DateTime('+1 year'))->format('Y-m-d H:i:s')
+                'course_language'     => $this->settingsManager->getSetting('language.platform_language'),
+                'visibility'          => Course::OPEN_PLATFORM,
+                'subscribe'           => true,
+                'unsubscribe'         => true,
+                'disk_quota'          => $this->settingsManager->getSetting('document.default_document_quotum'),
+                'expiration_date'     => (new \DateTime('+1 year'))->format('Y-m-d H:i:s'),
             ]);
 
             if (!$course) {
-                throw new \RuntimeException('Error: Course could not be created.');
+                throw new \RuntimeException("Course '$courseCode' could not be created.");
             }
 
-            // Step 2: Create learning path (CLp)
+            // 3. Create learning path
             $lp = (new CLp())
                 ->setLpType(1)
                 ->setTitle($courseCode)
@@ -101,15 +116,14 @@ protected function execute(InputInterface $input, OutputInterface $output): int
                 ->setExpiredOn(null)
                 ->setCategory(null)
                 ->setParent($course)
-                ->addCourseLink($course);
-            $lp->setCreator($adminUser);
+                ->addCourseLink($course)
+                ->setCreator($adminUser);
 
-            $lpRepo = $this->em->getRepository(CLp::class);
-            $lpRepo->createLp($lp);
+            $this->em->getRepository(CLp::class)->createLp($lp);
 
-            // Step 3: Create CDocument from uploaded file
-            $document = new CDocument();
-            $document->setFiletype('file')
+            // 4. Create document
+            $document = (new CDocument())
+                ->setFiletype('file')
                 ->setTitle($basename)
                 ->setComment(null)
                 ->setReadonly(false)
@@ -121,12 +135,26 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             $this->em->flush();
 
             $documentRepo = $this->em->getRepository(CDocument::class);
-            $documentRepo->addFileFromPath($document, $basename, $filePath);
+            $resourceFile = $documentRepo->addFileFromPath($document, $basename, $filePath);
+
+            // 4.1  Apply permissions to the real file & its directory
+            if ($resourceFile) {
+                $resourceNodeRepo = $this->em->getRepository(\Chamilo\CoreBundle\Entity\ResourceNode::class);
+                $relativePath = $resourceNodeRepo->getFilename($resourceFile); // e.g. /2025/06/16/abc.pdf
+                $fullPath     = realpath($uploadBase . $relativePath);
+
+                if ($fullPath && is_file($fullPath)) {
+                    @chmod($fullPath, $filePermOct);
+                }
+                $fullDir = dirname($fullPath ?: '');
+                if ($fullDir && is_dir($fullDir)) {
+                    @chmod($fullDir, $dirPermOct);
+                }
+            }
 
-            // Step 4: Create LP item linked to the document
-            // Ensure root item exists
+            // 5. Ensure learning path root item exists
             $lpItemRepo = $this->em->getRepository(CLpItem::class);
-            $rootItem = $lpItemRepo->getRootItem((int) $lp->getIid());
+            $rootItem   = $lpItemRepo->getRootItem((int) $lp->getIid());
 
             if (!$rootItem) {
                 $rootItem = (new CLpItem())
@@ -138,6 +166,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
                 $this->em->flush();
             }
 
+            // 6. Create LP item linked to the document
             $lpItem = (new CLpItem())
                 ->setLp($lp)
                 ->setTitle($basename)
@@ -155,12 +184,15 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             $this->em->persist($lpItem);
             $this->em->flush();
 
-            $io->success("Course '$courseCode' created with LP and document item '$basename'");
+            $io->success("Course '$courseCode' created with LP and document '$basename'");
         }
 
         return Command::SUCCESS;
     }
 
+    /**
+     * Return the first user that has ROLE_ADMIN.
+     */
     private function getFirstAdmin(): ?User
     {
         return $this->em->getRepository(User::class)