PhpCfdi\Finkok\Services\Stamping\StampService

Complexity

Total Complexity

6

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Test Coverage

Coverage

81.25%

Importance

Changes	3
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	stamp()	0	17	4
A	settings()	0	3	1

<?php

declare(strict_types=1);

namespace PhpCfdi\Finkok\Services\Stamping;

use PhpCfdi\Finkok\Definitions\Services;
use PhpCfdi\Finkok\FinkokSettings;

class StampService
{
    /** @var FinkokSettings */
    private $settings;

    public function __construct(FinkokSettings $settings)
    {
        $this->settings = $settings;
    }

    public function settings(): FinkokSettings
    {
        return $this->settings;
    }

    public function stamp(StampingCommand $command): StampingResult
    {
        $soapCaller = $this->settings()->createCallerForService(Services::stamping());
        // Finkok, repeat to fix bad webservice behavior of remote stamp method
        // This will not be fixed according to Finkok
        do {
            $rawResponse = $soapCaller->call('stamp', [
                'xml' => $command->xml(),
            ]);
            $result = new StampingResult('stampResult', $rawResponse);
            if (null !== $result->alerts()->findByErrorCode('307') && '' === $result->uuid()) {

Are you sure the usage of $result->alerts()->findByErrorCode('307') targeting PhpCfdi\Finkok\Services\Stamping\StampingAlerts::findByErrorCode() seems to always return null.

                usleep(200000); // 0.2 seconds
                continue;
            }
            break;
        } while (true);
        return $result;
    }
}