DailyChargeBackCommand::execute() - Code Metrics - getparthenon/parthenon - Measure and Improve Code Quality continuously with Scrutinizer

DailyChargeBackCommand::execute() A
last analyzed 2025-01-23 10:56 UTC

↳ Parent: DailyChargeBackCommand

Complexity

Conditions	2
Paths	2

Size

Total Lines	8
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

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

<?php

declare(strict_types=1);

/*
 * Copyright (C) 2020-2025 Iain Cambridge
 *
 * This program is free software: you can redistribute it and/or modify
 * it under the terms of the GNU LESSER GENERAL PUBLIC LICENSE as published by
 * the Free Software Foundation, either version 2.1 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 Lesser 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\Command;

use Obol\Provider\ProviderInterface;
use Parthenon\Billing\ChargeBack\ChargeBackSyncerInterface;
use Symfony\Component\Console\Attribute\AsCommand;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

#[AsCommand(name: 'parthenon:billing:daily:charge-back', description: 'Sync charge backs for the past day', aliases: ['p:b:d:c'])]
class DailyChargeBackCommand extends Command
{
    public function __construct(
        private ProviderInterface $provider,
        private ChargeBackSyncerInterface $syncer,
    ) {
        parent::__construct(null);
    }

    protected function execute(InputInterface $input, OutputInterface $output): int
    {
        $output->writeln('Starting daily charge back command');

        $chargeBacks = $this->provider->chargeBacks()->createdSinceYesterday();

        foreach ($chargeBacks as $obolChargeBack) {
            $chargeBack = $this->syncer->sync($obolChargeBack);

        }
interface ReturnsInt {
    public function returnsIntHinted(): int;
}

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


1			<?php
2
3			declare(strict_types=1);
4
5			/*
6			* Copyright (C) 2020-2025 Iain Cambridge
7			*
8			* This program is free software: you can redistribute it and/or modify
9			* it under the terms of the GNU LESSER GENERAL PUBLIC LICENSE as published by
10			* the Free Software Foundation, either version 2.1 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 Lesser 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\Command;
23
24			use Obol\Provider\ProviderInterface;
25			use Parthenon\Billing\ChargeBack\ChargeBackSyncerInterface;
26			use Symfony\Component\Console\Attribute\AsCommand;
27			use Symfony\Component\Console\Command\Command;
28			use Symfony\Component\Console\Input\InputInterface;
29			use Symfony\Component\Console\Output\OutputInterface;
30
31			#[AsCommand(name: 'parthenon:billing:daily:charge-back', description: 'Sync charge backs for the past day', aliases: ['p:b:d:c'])]
32			class DailyChargeBackCommand extends Command
33			{
34			public function __construct(
35			private ProviderInterface $provider,
36			private ChargeBackSyncerInterface $syncer,
37			) {
38			parent::__construct(null);
39			}
40
41			protected function execute(InputInterface $input, OutputInterface $output): int
42			{
43			$output->writeln('Starting daily charge back command');
44
45			$chargeBacks = $this->provider->chargeBacks()->createdSinceYesterday();
46
47			foreach ($chargeBacks as $obolChargeBack) {
48			$chargeBack = $this->syncer->sync($obolChargeBack);
			0 ignored issues – show Unused Code introduced 2023-04-23 09:00 UTC by Report Bug Copy Issue Report The assignment to `$chargeBack` is dead and can be removed. Loading history...
49			}
			0 ignored issues – show Bug Best Practice introduced 2024-02-22 22:11 UTC by Report Bug Copy Issue Report In this branch, the function will implicitly return `null` which is incompatible with the type-hinted return `integer`. 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...
50			}
51			}
52

getparthenon / parthenon

DailyChargeBackCommand::execute() A last analyzed 2025-01-23 10:56 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

DailyChargeBackCommand::execute() A
last analyzed 2025-01-23 10:56 UTC