ShippingCategoryContext::getShippingCategoryByName() - Code Metrics - Inspection of "Merge pull request #6824 from tuka217/shipping-met..." - pamil/Sylius - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( e6e3d2...83da08 )

by Kamil

created 2016-11-23 13:59 UTC

getShippingCategoryByName() A

↳ Parent: ShippingCategoryContext

Complexity

Conditions	1
Paths	1

Size

Total Lines	12
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	12
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	7
nc	1
nop	1

<?php

/*
 * This file is part of the Sylius package.
 *
 * (c) Paweł Jędrzejewski
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Sylius\Behat\Context\Transform;

use Behat\Behat\Context\Context;
use Sylius\Component\Resource\Repository\RepositoryInterface;
use Webmozart\Assert\Assert;

/**
 * @author Anna Walasek <[email protected]>
 */
class ShippingCategoryContext implements Context
{
    /**
     * @var RepositoryInterface
     */
    private $shippingCategoryRepository;

    /**
     * @param RepositoryInterface $shippingCategoryRepository
     */
    public function __construct(RepositoryInterface $shippingCategoryRepository)

    {
        $this->shippingCategoryRepository = $shippingCategoryRepository;
    }

    /**
     * @Transform /^"([^"]+)" shipping category/
     */
    public function getShippingCategoryByName($shippingCategoryName)

    {
        $shippingCategories = $this->shippingCategoryRepository->findBy(['name' => $shippingCategoryName]);

        Assert::eq(
            1,
            count($shippingCategories),
            sprintf('%d shipping category has been found with name "%s".', count($shippingCategories), $shippingCategoryName)
        );

        return $shippingCategories[0];
    }
}


1			<?php
2
3			/*
4			* This file is part of the Sylius package.
5			*
6			* (c) Paweł Jędrzejewski
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			namespace Sylius\Behat\Context\Transform;
13
14			use Behat\Behat\Context\Context;
15			use Sylius\Component\Resource\Repository\RepositoryInterface;
16			use Webmozart\Assert\Assert;
17
18			/**
19			* @author Anna Walasek <[email protected]>
20			*/
21			class ShippingCategoryContext implements Context
22			{
23			/**
24			* @var RepositoryInterface
25			*/
26			private $shippingCategoryRepository;
27
28			/**
29			* @param RepositoryInterface $shippingCategoryRepository
30			*/
31			public function __construct(RepositoryInterface $shippingCategoryRepository)
			0 ignored issues – show Comprehensibility Naming introduced 2016-11-23 14:20 UTC by Report Bug Copy Issue Report The variable name `$shippingCategoryRepository` exceeds the maximum configured length of `20`. Very long variable names usually make code harder to read. It is therefore recommended not to make variable names too verbose. Loading history...
32			{
33			$this->shippingCategoryRepository = $shippingCategoryRepository;
34			}
35
36			/**
37			* @Transform /^"([^"]+)" shipping category/
38			*/
39			public function getShippingCategoryByName($shippingCategoryName)
			0 ignored issues – show Documentation introduced 2016-11-23 14:20 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
40			{
41			$shippingCategories = $this->shippingCategoryRepository->findBy(['name' => $shippingCategoryName]);
42
43			Assert::eq(
44			1,
45			count($shippingCategories),
46			sprintf('%d shipping category has been found with name "%s".', count($shippingCategories), $shippingCategoryName)
47			);
48
49			return $shippingCategories[0];
50			}
51			}
52

pamil / Sylius

Push — master ( e6e3d2...83da08 )

getShippingCategoryByName() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like