WanderDeleteListener::preRemove() - Code Metrics - Inspection of "Add featured image feature" - gothick/omm - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Pull Request — master (#85)

by Matt

created 2021-05-31 20:41 UTC

WanderDeleteListener::preRemove() A

↳ Parent: WanderDeleteListener

Complexity

Conditions	2
Paths	2

Size

Total Lines	11
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	2
eloc	3
c	1
b	0
f	0
nc	2
nop	2
dl	0
loc	11
rs	10

<?php

namespace App\EventListener;

use App\Entity\Wander;
use App\Service\GpxService;
use Psr\Log\LoggerInterface;
use Doctrine\Persistence\Event\LifecycleEventArgs;

class WanderDeleteListener
{
    /** @var LoggerInterface $logger */
    private $logger;

    /** @var GpxService */
    private $gpxService;

    public function __construct(LoggerInterface $logger, GpxService $gpxService)
    {
        $this->logger = $logger;
        $this->gpxService = $gpxService;
    }

    public function preRemove(
            Wander $wander,
            /** @scrutinizer ignore-unused */ LifecycleEventArgs $event
        ): void
    {
        // If we're about to delete a wander, we want to remove it as a featuring
        // wander from the related Image first, otherwise we'll break referential
        // integrity.
        $image = $wander->getFeaturedImage();
        if ($image !== null) {
            $image->setFeaturingWander(null);
        }
    }

    public function postRemove(
            Wander $wander,
            /** @scrutinizer ignore-unused */ LifecycleEventArgs $event
        ): void
    {
        $path = $this->gpxService->getFullGpxFilePathFromWander($wander);
        if (!file_exists($path)) {
            $this->logger->debug("Could not find GPX file " . $path . " to remove");
            return;
        }

        $this->logger->debug("Removing GPX file " . $path . " on deletion of wander " . $wander->getId());
        unlink($path);
    }
}

1			<?php
2
3			namespace App\EventListener;
4
5			use App\Entity\Wander;
6			use App\Service\GpxService;
7			use Psr\Log\LoggerInterface;
8			use Doctrine\Persistence\Event\LifecycleEventArgs;
9
10			class WanderDeleteListener
11			{
12			/** @var LoggerInterface $logger */
13			private $logger;
14
15			/** @var GpxService */
16			private $gpxService;
17
18			public function __construct(LoggerInterface $logger, GpxService $gpxService)
19			{
20			$this->logger = $logger;
21			$this->gpxService = $gpxService;
22			}
23
24			public function preRemove(
25			Wander $wander,
26			/** @scrutinizer ignore-unused */ LifecycleEventArgs $event
27			): void
28			{
29			// If we're about to delete a wander, we want to remove it as a featuring
30			// wander from the related Image first, otherwise we'll break referential
31			// integrity.
32			$image = $wander->getFeaturedImage();
33			if ($image !== null) {
34			$image->setFeaturingWander(null);
35			}
36			}
37
38			public function postRemove(
39			Wander $wander,
40			/** @scrutinizer ignore-unused */ LifecycleEventArgs $event
41			): void
42			{
43			$path = $this->gpxService->getFullGpxFilePathFromWander($wander);
44			if (!file_exists($path)) {
45			$this->logger->debug("Could not find GPX file " . $path . " to remove");
46			return;
47			}
48
49			$this->logger->debug("Removing GPX file " . $path . " on deletion of wander " . $wander->getId());
50			unlink($path);
51			}
52			}

gothick / omm

Pull Request — master (#85)

WanderDeleteListener::preRemove() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like