MetaDataHelper::setDynamicMetaData() - Code Metrics - Inspection of "[POC] start to replace our custom parameters with..." - zikula/core - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#4433)

by Craig

created 2020-08-05 14:01 UTC

MetaDataHelper::setDynamicMetaData() A

↳ Parent: MetaDataHelper

Complexity

Conditions	3
Paths	3

Size

Total Lines	13
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

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

<?php

declare(strict_types=1);

/*
 * This file is part of the Zikula package.
 *
 * Copyright Zikula - https://ziku.la/
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Zikula\ExtensionsModule\Helper;

use Zikula\Bundle\CoreBundle\Composer\MetaData;
use Zikula\Bundle\CoreBundle\HttpKernel\ZikulaKernel;
use Zikula\ExtensionsModule\Entity\Repository\ExtensionRepository;
use Zikula\ExtensionsModule\Entity\RepositoryInterface\ExtensionRepositoryInterface;

class MetaDataHelper
{
    /**
     * @var ExtensionRepository
     */
    private $extensionRepository;

    /**
     * @var bool
     */
    private $installed;

    public function __construct(
        ExtensionRepositoryInterface $extensionRepository,
        string $installed
    ) {
        $this->extensionRepository = $extensionRepository;
class Alien {}

class Dalek extends Alien {}

class Plot
{
    /** @var  Dalek */
    public $villain;
}

$alien = new Alien();
$plot = new Plot();
if ($alien instanceof Dalek) {
    $plot->villain = $alien;
}
        $this->installed = ZikulaKernel::VERSION === $installed;
    }

    /**
     * overwrite composer.json settings with dynamic values from extension repository
     */
    public function setDynamicMetaData(MetaData $metaData): MetaData
    {
        if ($this->installed) {
            $extensionEntity = $this->extensionRepository->get($metaData->getShortName());
            if (null !== $extensionEntity) {
                $metaData->setUrl($extensionEntity->getUrl());
                $metaData->setDisplayName($extensionEntity->getDisplayname());
                $metaData->setDescription($extensionEntity->getDescription());
                $metaData->setIcon($extensionEntity->getIcon());
            }
        }

        return $metaData;
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			/*
6			* This file is part of the Zikula package.
7			*
8			* Copyright Zikula - https://ziku.la/
9			*
10			* For the full copyright and license information, please view the LICENSE
11			* file that was distributed with this source code.
12			*/
13
14			namespace Zikula\ExtensionsModule\Helper;
15
16			use Zikula\Bundle\CoreBundle\Composer\MetaData;
17			use Zikula\Bundle\CoreBundle\HttpKernel\ZikulaKernel;
18			use Zikula\ExtensionsModule\Entity\Repository\ExtensionRepository;
19			use Zikula\ExtensionsModule\Entity\RepositoryInterface\ExtensionRepositoryInterface;
20
21			class MetaDataHelper
22			{
23			/**
24			* @var ExtensionRepository
25			*/
26			private $extensionRepository;
27
28			/**
29			* @var bool
30			*/
31			private $installed;
32
33			public function __construct(
34			ExtensionRepositoryInterface $extensionRepository,
35			string $installed
36			) {
37			$this->extensionRepository = $extensionRepository;
			0 ignored issues – show Documentation Bug introduced 2020-08-03 23:00 UTC by Report Bug Copy Issue Report `$extensionRepository` is of type `Zikula\ExtensionsModule\...sionRepositoryInterface`, but the property `$extensionRepository` was declared to be of type `Zikula\ExtensionsModule\...ory\ExtensionRepository`. Are you sure that you always receive this specific sub-class here, or does it make sense to add an `instanceof` check? Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a given class or a super-class is assigned to a property that is type hinted more strictly. Either this assignment is in error or an instanceof check should be added for that assignment. class Alien {} class Dalek extends Alien {} class Plot { /** @var Dalek */ public $villain; } $alien = new Alien(); $plot = new Plot(); if ($alien instanceof Dalek) { $plot->villain = $alien; } Loading history...
38			$this->installed = ZikulaKernel::VERSION === $installed;
39			}
40
41			/**
42			* overwrite composer.json settings with dynamic values from extension repository
43			*/
44			public function setDynamicMetaData(MetaData $metaData): MetaData
45			{
46			if ($this->installed) {
47			$extensionEntity = $this->extensionRepository->get($metaData->getShortName());
48			if (null !== $extensionEntity) {
49			$metaData->setUrl($extensionEntity->getUrl());
50			$metaData->setDisplayName($extensionEntity->getDisplayname());
51			$metaData->setDescription($extensionEntity->getDescription());
52			$metaData->setIcon($extensionEntity->getIcon());
53			}
54			}
55
56			return $metaData;
57			}
58			}
59

zikula / core

Pull Request — master (#4433)

MetaDataHelper::setDynamicMetaData() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like