ReplaceDocumentFileAction::__invoke() - Code Metrics - Inspection of "Language: Add new language terms" - chamilo/chamilo-lms - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( edd7fd...017cdc )

by Yannick

created 2025-03-22 07:16 UTC

ReplaceDocumentFileAction::__invoke() B

↳ Parent: ReplaceDocumentFileAction

Complexity

Conditions	7
Paths	7

Size

Total Lines	52
Code Lines	30

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	7
eloc	30
c	1
b	0
f	0
nc	7
nop	4
dl	0
loc	52
rs	8.5066

How to fix Long Method

<?php

declare(strict_types=1);

/* For licensing terms, see /license.txt */

namespace Chamilo\CoreBundle\Controller\Api;

use Chamilo\CoreBundle\Repository\ResourceNodeRepository;
use Chamilo\CourseBundle\Entity\CDocument;
use Doctrine\ORM\EntityManagerInterface;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\HttpKernel\Exception\BadRequestHttpException;
use Symfony\Component\HttpKernel\KernelInterface;
use Symfony\Component\HttpFoundation\File\Exception\FileException;

class ReplaceDocumentFileAction extends BaseResourceFileAction
{
    private string $uploadBasePath;

    public function __construct(KernelInterface $kernel)
    {
        $this->uploadBasePath = $kernel->getProjectDir() . '/var/upload/resource';
    }

    public function __invoke(
        CDocument $document,
        Request $request,
        ResourceNodeRepository $resourceNodeRepository,
        EntityManagerInterface $em
    ): Response {
        $uploadedFile = $request->files->get('file');
        if (!$uploadedFile) {
            throw new BadRequestHttpException('"file" is required.');
        }

        $resourceNode = $document->getResourceNode();
        if (!$resourceNode) {
            throw new BadRequestHttpException('ResourceNode not found.');
        }

        $resourceFile = $resourceNode->getFirstResourceFile();
        if (!$resourceFile) {
            throw new BadRequestHttpException('No file found in the resource node.');
        }

        $filePath = $this->uploadBasePath . $resourceNodeRepository->getFilename($resourceFile);
        if (!$filePath) {
            throw new BadRequestHttpException('File path could not be resolved.');
        }

        $this->prepareDirectory($filePath);

        try {
            $uploadedFile->move(dirname($filePath), basename($filePath));
        } catch (FileException $e) {
            throw new BadRequestHttpException(sprintf('Failed to move the file: %s', $e->getMessage()));
        }

        $movedFilePath = $filePath;
        if (!file_exists($movedFilePath)) {
            throw new \RuntimeException('The moved file does not exist at the expected location.');
        }
        $fileSize = filesize($movedFilePath);
        $resourceFile->setSize($fileSize);

        $newFileName = $uploadedFile->getClientOriginalName();
        $document->setTitle($newFileName);
        $resourceFile->setOriginalName($newFileName);

        $resourceNode->setUpdatedAt(new \DateTime());

        $em->persist($document);
        $em->persist($resourceFile);
        $em->flush();

        return new Response('Document replaced successfully.', Response::HTTP_OK);
    }

    /**
     * Prepares the directory to ensure it exists and is writable.
     */
    protected function prepareDirectory(string $filePath): void
    {
        $directory = dirname($filePath);

        if (!is_dir($directory)) {
            if (!mkdir($directory, 0775, true) && !is_dir($directory)) {
                throw new \RuntimeException(sprintf('Unable to create directory "%s".', $directory));
            }
        }

        if (!is_writable($directory)) {
            throw new \RuntimeException(sprintf('Directory "%s" is not writable.', $directory));
        }
    }

}


1			<?php
2
3			declare(strict_types=1);
4
5			/* For licensing terms, see /license.txt */
6
7			namespace Chamilo\CoreBundle\Controller\Api;
8
9			use Chamilo\CoreBundle\Repository\ResourceNodeRepository;
10			use Chamilo\CourseBundle\Entity\CDocument;
11			use Doctrine\ORM\EntityManagerInterface;
12			use Symfony\Component\HttpFoundation\Request;
13			use Symfony\Component\HttpFoundation\Response;
14			use Symfony\Component\HttpKernel\Exception\BadRequestHttpException;
15			use Symfony\Component\HttpKernel\KernelInterface;
16			use Symfony\Component\HttpFoundation\File\Exception\FileException;
17
18			class ReplaceDocumentFileAction extends BaseResourceFileAction
19			{
20			private string $uploadBasePath;
21
22			public function __construct(KernelInterface $kernel)
23			{
24			$this->uploadBasePath = $kernel->getProjectDir() . '/var/upload/resource';
25			}
26
27			public function __invoke(
28			CDocument $document,
29			Request $request,
30			ResourceNodeRepository $resourceNodeRepository,
31			EntityManagerInterface $em
32			): Response {
33			$uploadedFile = $request->files->get('file');
34			if (!$uploadedFile) {
35			throw new BadRequestHttpException('"file" is required.');
36			}
37
38			$resourceNode = $document->getResourceNode();
39			if (!$resourceNode) {
40			throw new BadRequestHttpException('ResourceNode not found.');
41			}
42
43			$resourceFile = $resourceNode->getFirstResourceFile();
44			if (!$resourceFile) {
45			throw new BadRequestHttpException('No file found in the resource node.');
46			}
47
48			$filePath = $this->uploadBasePath . $resourceNodeRepository->getFilename($resourceFile);
49			if (!$filePath) {
50			throw new BadRequestHttpException('File path could not be resolved.');
51			}
52
53			$this->prepareDirectory($filePath);
54
55			try {
56			$uploadedFile->move(dirname($filePath), basename($filePath));
57			} catch (FileException $e) {
58			throw new BadRequestHttpException(sprintf('Failed to move the file: %s', $e->getMessage()));
59			}
60
61			$movedFilePath = $filePath;
62			if (!file_exists($movedFilePath)) {
63			throw new \RuntimeException('The moved file does not exist at the expected location.');
64			}
65			$fileSize = filesize($movedFilePath);
66			$resourceFile->setSize($fileSize);
67
68			$newFileName = $uploadedFile->getClientOriginalName();
69			$document->setTitle($newFileName);
70			$resourceFile->setOriginalName($newFileName);
71
72			$resourceNode->setUpdatedAt(new \DateTime());
73
74			$em->persist($document);
75			$em->persist($resourceFile);
76			$em->flush();
77
78			return new Response('Document replaced successfully.', Response::HTTP_OK);
79			}
80
81			/**
82			* Prepares the directory to ensure it exists and is writable.
83			*/
84			protected function prepareDirectory(string $filePath): void
85			{
86			$directory = dirname($filePath);
87
88			if (!is_dir($directory)) {
89			if (!mkdir($directory, 0775, true) && !is_dir($directory)) {
90			throw new \RuntimeException(sprintf('Unable to create directory "%s".', $directory));
91			}
92			}
93
94			if (!is_writable($directory)) {
95			throw new \RuntimeException(sprintf('Directory "%s" is not writable.', $directory));
96			}
97			}
98
99			}
100

chamilo / chamilo-lms

Push — master ( edd7fd...017cdc )

ReplaceDocumentFileAction::__invoke() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like