Version20231026231100::up() - Code Metrics - Inspection of "CI: Scrutinizer use php 8.1.2" - chamilo/chamilo-lms - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 719760...59ab2e )

by Angel Fernando Quiroz

created 2023-12-31 00:00 UTC

Version20231026231100::up() A

↳ Parent: Version20231026231100

Complexity

Conditions	4
Paths	4

Size

Total Lines	62
Code Lines	44

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
cc	4
eloc	44
c	2
b	0
f	0
nc	4
nop	1
dl	0
loc	62
rs	9.216

How to fix Long Method

<?php

declare(strict_types=1);

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

namespace Chamilo\CoreBundle\Migrations\Schema\V200;

use Chamilo\CoreBundle\Entity\SocialPost;
use Chamilo\CoreBundle\Entity\SocialPostAttachment;
use Chamilo\CoreBundle\Migrations\AbstractMigrationChamilo;
use Chamilo\CoreBundle\Repository\Node\SocialPostAttachmentRepository;
use Chamilo\CoreBundle\Repository\Node\UserRepository;
use DateTime;
use DateTimeZone;
use DirectoryIterator;
use Doctrine\DBAL\Connection;
use Doctrine\DBAL\Schema\Schema;
use Symfony\Component\HttpFoundation\File\UploadedFile;

final class Version20231026231100 extends AbstractMigrationChamilo
{
    public function getDescription(): string
    {
        return 'Migrate message_attachments to social_post_attachments';
    }

    public function up(Schema $schema): void
    {
        $container = $this->getContainer();
        $em = $this->getEntityManager();

        /** @var Connection $connection */
        $connection = $em->getConnection();

        $kernel = $container->get('kernel');
        $rootPath = $kernel->getProjectDir();

        $repo = $container->get(SocialPostAttachmentRepository::class);
        $userRepo = $container->get(UserRepository::class);
        $admin = $this->getAdmin();

        $sub = $em->createQueryBuilder();
        $sub->select('sp.id')
            ->from('Chamilo\CoreBundle\Entity\SocialPost', 'sp')
        ;

        $qb = $em->createQueryBuilder();
        $qb->select('ma')
            ->from('Chamilo\CoreBundle\Entity\MessageAttachment', 'ma')
            ->where($qb->expr()->in('ma.message', $sub->getDQL()))
        ;

        $query = $qb->getQuery();
        $messageAttachments = $query->getResult();

        foreach ($messageAttachments as $attachment) {
            $message = $attachment->getMessage();
            if ($message) {
                $messageId = $message->getId();
                $filename = $attachment->getFilename();
                $rootDir = $rootPath.'/app/upload/users';
                $targetFile = $attachment->getPath();
                $foundFilePath = $this->findFileRecursively($rootDir, $targetFile);
                $sender = $message->getSender();

                if ($foundFilePath) {
                    error_log("File found in $foundFilePath");

                    $mimeType = mime_content_type($foundFilePath);
                    $uploadFile = new UploadedFile($foundFilePath, $filename, $mimeType, null, true);

                    $socialPost = $em->getRepository(SocialPost::class)->find($messageId);

                    $attachment = new SocialPostAttachment();
                    $attachment->setSocialPost($socialPost);
                    $attachment->setPath(uniqid('social_post', true));
                    $attachment->setFilename($uploadFile->getClientOriginalName());
                    $attachment->setSize($uploadFile->getSize());
                    $attachment->setInsertUserId($sender->getId());
                    $attachment->setInsertDateTime(new DateTime('now', new DateTimeZone('UTC')));
                    $attachment->setParent($sender);
                    $attachment->addUserLink($sender);
                    $attachment->setCreator($sender);

                    $em->persist($attachment);
                    $em->flush();

                    $repo->addFile($attachment, $uploadFile);
                }
            }
        }
    }

    private function findFileRecursively(string $directory, string $targetFile): ?string
    {
        if (!is_dir($directory)) {
            return null;
        }

        foreach (new DirectoryIterator($directory) as $fileInfo) {
            if ($fileInfo->isDot()) {
                continue;
            }

            $filePath = $fileInfo->getPathname();

            if ($fileInfo->isDir()) {
                $result = $this->findFileRecursively($filePath, $targetFile);
                if (null !== $result) {
                    return $result;
                }
            } else {
                if (str_contains($fileInfo->getFilename(), $targetFile)) {
                    return $filePath;
                }
            }
        }

        return null;
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			/* For licensing terms, see /license.txt */
6
7			namespace Chamilo\CoreBundle\Migrations\Schema\V200;
8
9			use Chamilo\CoreBundle\Entity\SocialPost;
10			use Chamilo\CoreBundle\Entity\SocialPostAttachment;
11			use Chamilo\CoreBundle\Migrations\AbstractMigrationChamilo;
12			use Chamilo\CoreBundle\Repository\Node\SocialPostAttachmentRepository;
13			use Chamilo\CoreBundle\Repository\Node\UserRepository;
14			use DateTime;
15			use DateTimeZone;
16			use DirectoryIterator;
17			use Doctrine\DBAL\Connection;
18			use Doctrine\DBAL\Schema\Schema;
19			use Symfony\Component\HttpFoundation\File\UploadedFile;
20
21			final class Version20231026231100 extends AbstractMigrationChamilo
22			{
23			public function getDescription(): string
24			{
25			return 'Migrate message_attachments to social_post_attachments';
26			}
27
28			public function up(Schema $schema): void
29			{
30			$container = $this->getContainer();
31			$em = $this->getEntityManager();
32
33			/** @var Connection $connection */
34			$connection = $em->getConnection();
35
36			$kernel = $container->get('kernel');
37			$rootPath = $kernel->getProjectDir();
38
39			$repo = $container->get(SocialPostAttachmentRepository::class);
40			$userRepo = $container->get(UserRepository::class);
41			$admin = $this->getAdmin();
42
43			$sub = $em->createQueryBuilder();
44			$sub->select('sp.id')
45			->from('Chamilo\CoreBundle\Entity\SocialPost', 'sp')
46			;
47
48			$qb = $em->createQueryBuilder();
49			$qb->select('ma')
50			->from('Chamilo\CoreBundle\Entity\MessageAttachment', 'ma')
51			->where($qb->expr()->in('ma.message', $sub->getDQL()))
52			;
53
54			$query = $qb->getQuery();
55			$messageAttachments = $query->getResult();
56
57			foreach ($messageAttachments as $attachment) {
58			$message = $attachment->getMessage();
59			if ($message) {
60			$messageId = $message->getId();
61			$filename = $attachment->getFilename();
62			$rootDir = $rootPath.'/app/upload/users';
63			$targetFile = $attachment->getPath();
64			$foundFilePath = $this->findFileRecursively($rootDir, $targetFile);
65			$sender = $message->getSender();
66
67			if ($foundFilePath) {
68			error_log("File found in $foundFilePath");
69
70			$mimeType = mime_content_type($foundFilePath);
71			$uploadFile = new UploadedFile($foundFilePath, $filename, $mimeType, null, true);
72
73			$socialPost = $em->getRepository(SocialPost::class)->find($messageId);
74
75			$attachment = new SocialPostAttachment();
76			$attachment->setSocialPost($socialPost);
77			$attachment->setPath(uniqid('social_post', true));
78			$attachment->setFilename($uploadFile->getClientOriginalName());
79			$attachment->setSize($uploadFile->getSize());
80			$attachment->setInsertUserId($sender->getId());
81			$attachment->setInsertDateTime(new DateTime('now', new DateTimeZone('UTC')));
82			$attachment->setParent($sender);
83			$attachment->addUserLink($sender);
84			$attachment->setCreator($sender);
85
86			$em->persist($attachment);
87			$em->flush();
88
89			$repo->addFile($attachment, $uploadFile);
90			}
91			}
92			}
93			}
94
95			private function findFileRecursively(string $directory, string $targetFile): ?string
96			{
97			if (!is_dir($directory)) {
98			return null;
99			}
100
101			foreach (new DirectoryIterator($directory) as $fileInfo) {
102			if ($fileInfo->isDot()) {
103			continue;
104			}
105
106			$filePath = $fileInfo->getPathname();
107
108			if ($fileInfo->isDir()) {
109			$result = $this->findFileRecursively($filePath, $targetFile);
110			if (null !== $result) {
111			return $result;
112			}
113			} else {
114			if (str_contains($fileInfo->getFilename(), $targetFile)) {
115			return $filePath;
116			}
117			}
118			}
119
120			return null;
121			}
122			}
123

chamilo / chamilo-lms

Push — master ( 719760...59ab2e )

Version20231026231100::up() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like