CLinkImageController::__invoke() - Code Metrics - Inspection of "Course: Add custom image support for course links..." - chamilo/chamilo-lms - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#6039)

unknown

created 2025-01-21 05:05 UTC

CLinkImageController::__invoke() B

↳ Parent: CLinkImageController

Complexity

Conditions	8
Paths	58

Size

Total Lines	52
Code Lines	33

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	8
eloc	33
nc	58
nop	2
dl	0
loc	52
rs	8.1475
c	1
b	0
f	0

How to fix Long Method

<?php

declare(strict_types=1);

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

namespace Chamilo\CoreBundle\Controller\Api;

use Chamilo\CoreBundle\Entity\Asset;
use Chamilo\CourseBundle\Entity\CLink;
use Doctrine\ORM\EntityManagerInterface;
use Symfony\Component\HttpFoundation\File\File;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;

class CLinkImageController
{
    private EntityManagerInterface $entityManager;

    public function __construct(EntityManagerInterface $entityManager)
    {
        $this->entityManager = $entityManager;
    }

    public function __invoke(CLink $link, Request $request): Response
    {
        $removeImage = $request->request->getBoolean('removeImage', false);
        $file = $request->files->get('customImage');

        if ($removeImage) {
            if ($link->getCustomImage()) {
                $this->entityManager->remove($link->getCustomImage());
                $link->setCustomImage(null);
                $this->entityManager->persist($link);
                $this->entityManager->flush();

                if (!$file) {
                    return new Response('Image removed successfully', Response::HTTP_OK);
                }
            }
        }

        if (!$file || !$file->isValid()) {
            return new Response('Invalid or missing file', Response::HTTP_BAD_REQUEST);
        }

        try {
            $asset = new Asset();
            $asset->setFile($file)
                ->setCategory(Asset::LINK)
                ->setTitle($file->getClientOriginalName());

            $this->entityManager->persist($asset);
            $this->entityManager->flush();

            $uploadedFilePath = $file->getPathname();

            $croppedFilePath = $this->cropImage($uploadedFilePath);

            if (!file_exists($croppedFilePath)) {
                @unlink($uploadedFilePath);
                /** @scrutinizer ignore-unhandled */ @unlink($uploadedFilePath);
                return new Response('Error creating cropped image', Response::HTTP_INTERNAL_SERVER_ERROR);
            }

            $asset->setFile(new File($croppedFilePath));
            $this->entityManager->persist($asset);
            $this->entityManager->flush();

            $link->setCustomImage($asset);
            $this->entityManager->persist($link);
            $this->entityManager->flush();

            return new Response('Image uploaded and linked successfully', Response::HTTP_OK);

        } catch (\Exception $e) {
            return new Response('Error processing image: ' . $e->getMessage(), Response::HTTP_INTERNAL_SERVER_ERROR);
        }
    }

    private function cropImage(string $filePath): string
    {
        [$originalWidth, $originalHeight, $imageType] = getimagesize($filePath);

        if (!$originalWidth || !$originalHeight) {
            throw new \RuntimeException('Invalid image file');
        }

        switch ($imageType) {
            case IMAGETYPE_JPEG:
                $sourceImage = imagecreatefromjpeg($filePath);
                break;
            case IMAGETYPE_PNG:
                $sourceImage = imagecreatefrompng($filePath);
                break;
            case IMAGETYPE_GIF:
                $sourceImage = imagecreatefromgif($filePath);
                break;
            default:
                throw new \RuntimeException('Unsupported image type');
        }

        $croppedImage = imagecreatetruecolor(120, 120);

        $cropWidth = min($originalWidth, $originalHeight);
        $cropHeight = $cropWidth;
        $srcX = (int) (($originalWidth - $cropWidth) / 2);
        $srcY = (int) (($originalHeight - $cropHeight) / 2);

        imagecopyresampled(
            $croppedImage,
            $sourceImage,
            0, 0,
            $srcX, $srcY,
            $cropWidth, $cropHeight,
            120, 120
        );

        $croppedFilePath = sys_get_temp_dir() . '/' . uniqid('cropped_', true) . '.png';
        imagepng($croppedImage, $croppedFilePath);

        imagedestroy($sourceImage);
        imagedestroy($croppedImage);

        return $croppedFilePath;
    }
}


chamilo / chamilo-lms

Pull Request — master (#6039)

CLinkImageController::__invoke() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like