PaymentCardRepository::findById() - Code Metrics - Inspection of "[Billing] Read subscriptions from BillaBear" - getparthenon/parthenon - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — main ( 241451...172c2c )

by Iain

created 2024-06-07 16:31 UTC

PaymentCardRepository::findById() A

↳ Parent: PaymentCardRepository

Complexity

Conditions	1
Paths	1

Size

Total Lines	5
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

declare(strict_types=1);

/*
 * Copyright (C) 2020-2024 Iain Cambridge
 *
 *     This program is free software: you can redistribute it and/or modify
 *     it under the terms of the GNU General Public License as published by
 *     the Free Software Foundation, either version 3 of the License, or
 *     (at your option) any later version.
 *
 *     This program is distributed in the hope that it will be useful,
 *     but WITHOUT ANY WARRANTY; without even the implied warranty of
 *     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 *     GNU General Public License for more details.
 *
 *     You should have received a copy of the GNU General Public License
 *     along with this program.  If not, see <https://www.gnu.org/licenses/>.
 */

namespace Parthenon\Billing\BillaBear\Repository;

use BillaBear\Model\PaymentDetails;
filter:
    dependency_paths: ["lib/*"]
use Parthenon\Billing\BillaBear\SdkFactory;
use Parthenon\Billing\Entity\CustomerInterface;
use Parthenon\Billing\Entity\PaymentCard;
use Parthenon\Billing\Repository\PaymentCardRepositoryInterface;

class PaymentCardRepository implements PaymentCardRepositoryInterface
{
    public function __construct(private SdkFactory $sdkFactory)
    {
    }

    public function getPaymentCardForCustomer(CustomerInterface $customer): array
    {
        $cardResponse = $this->sdkFactory->createPaymentDetails()->listPaymentDetails($customer->getExternalCustomerReference());

        $output = [];

        foreach ($cardResponse->getData() as $paymentDetails) {
            $output[] = $this->buildPaymentCard($paymentDetails);
        }

        return $output;
    }

    public function markAllCustomerCardsAsNotDefault(CustomerInterface $customer): void
    {
        // TODO: Implement markAllCustomerCardsAsNotDefault() method.
    }

    public function getDefaultPaymentCardForCustomer(CustomerInterface $customer): PaymentCard
    {
        // TODO: Implement getDefaultPaymentCardForCustomer() method.
    }
interface ReturnsInt {
    public function returnsIntHinted(): int;
}

class MyClass implements ReturnsInt {
    public function returnsIntHinted(): int
    {
        if (foo()) {
            return 123;
        }
        // here: null is implicitly returned
    }
}

    public function findById($id)
    {
        $paymentCard = $this->sdkFactory->createPaymentDetails()->getPaymentDetails($id);

        return $this->buildPaymentCard($paymentCard);
    }

    public function save($entity)
    {
        throw new \Exception("SHouldn't be used. ISP failure.");
    }

    private function buildPaymentCard(PaymentDetails $paymentDetails): PaymentCard
    {
        $paymentCard = new PaymentCard();
        $paymentCard->setId($paymentDetails->getId());
        $paymentCard->setName($paymentDetails->getName());
        $paymentCard->setDefaultPaymentOption($paymentDetails->getDefault());
        $paymentCard->setBrand($paymentDetails->getBrand());
        $paymentCard->setLastFour($paymentDetails->getLastFour());
        $paymentCard->setExpiryMonth($paymentDetails->getExpiryMonth());
        $paymentCard->setExpiryYear($paymentDetails->getExpiryYear());
        $paymentCard->setCreatedAt(new \DateTime($paymentDetails->getCreatedAt()));

        return $paymentCard;
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			/*
6			* Copyright (C) 2020-2024 Iain Cambridge
7			*
8			* This program is free software: you can redistribute it and/or modify
9			* it under the terms of the GNU General Public License as published by
10			* the Free Software Foundation, either version 3 of the License, or
11			* (at your option) any later version.
12			*
13			* This program is distributed in the hope that it will be useful,
14			* but WITHOUT ANY WARRANTY; without even the implied warranty of
15			* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
16			* GNU General Public License for more details.
17			*
18			* You should have received a copy of the GNU General Public License
19			* along with this program. If not, see <https://www.gnu.org/licenses/>.
20			*/
21
22			namespace Parthenon\Billing\BillaBear\Repository;
23
24			use BillaBear\Model\PaymentDetails;
			0 ignored issues – show Bug introduced 2024-06-07 02:46 UTC by Report Bug Copy Issue Report The type `BillaBear\Model\PaymentDetails` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
25			use Parthenon\Billing\BillaBear\SdkFactory;
26			use Parthenon\Billing\Entity\CustomerInterface;
27			use Parthenon\Billing\Entity\PaymentCard;
28			use Parthenon\Billing\Repository\PaymentCardRepositoryInterface;
29
30			class PaymentCardRepository implements PaymentCardRepositoryInterface
31			{
32			public function __construct(private SdkFactory $sdkFactory)
33			{
34			}
35
36			public function getPaymentCardForCustomer(CustomerInterface $customer): array
37			{
38			$cardResponse = $this->sdkFactory->createPaymentDetails()->listPaymentDetails($customer->getExternalCustomerReference());
39
40			$output = [];
41
42			foreach ($cardResponse->getData() as $paymentDetails) {
43			$output[] = $this->buildPaymentCard($paymentDetails);
44			}
45
46			return $output;
47			}
48
49			public function markAllCustomerCardsAsNotDefault(CustomerInterface $customer): void
50			{
51			// TODO: Implement markAllCustomerCardsAsNotDefault() method.
52			}
53
54			public function getDefaultPaymentCardForCustomer(CustomerInterface $customer): PaymentCard
55			{
56			// TODO: Implement getDefaultPaymentCardForCustomer() method.
57			}
			0 ignored issues – show Bug Best Practice introduced 2024-06-07 02:46 UTC by Report Bug Copy Issue Report In this branch, the function will implicitly return `null` which is incompatible with the type-hinted return `Parthenon\Billing\Entity\PaymentCard`. Consider adding a return statement or allowing `null` as return value. For hinted functions/methods where all return statements with the correct type are only reachable via conditions, ?null? gets implicitly returned which may be incompatible with the hinted type. Let?s take a look at an example: interface ReturnsInt { public function returnsIntHinted(): int; } class MyClass implements ReturnsInt { public function returnsIntHinted(): int { if (foo()) { return 123; } // here: null is implicitly returned } } Loading history...
58
59			public function findById($id)
60			{
61			$paymentCard = $this->sdkFactory->createPaymentDetails()->getPaymentDetails($id);
62
63			return $this->buildPaymentCard($paymentCard);
64			}
65
66			public function save($entity)
67			{
68			throw new \Exception("SHouldn't be used. ISP failure.");
69			}
70
71			private function buildPaymentCard(PaymentDetails $paymentDetails): PaymentCard
72			{
73			$paymentCard = new PaymentCard();
74			$paymentCard->setId($paymentDetails->getId());
75			$paymentCard->setName($paymentDetails->getName());
76			$paymentCard->setDefaultPaymentOption($paymentDetails->getDefault());
77			$paymentCard->setBrand($paymentDetails->getBrand());
78			$paymentCard->setLastFour($paymentDetails->getLastFour());
79			$paymentCard->setExpiryMonth($paymentDetails->getExpiryMonth());
80			$paymentCard->setExpiryYear($paymentDetails->getExpiryYear());
81			$paymentCard->setCreatedAt(new \DateTime($paymentDetails->getCreatedAt()));
82
83			return $paymentCard;
84			}
85			}
86

getparthenon / parthenon

Push — main ( 241451...172c2c )

PaymentCardRepository::findById() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like