DPD\Interconnector\Request\LabelsRequest

Complexity

Total Complexity

4

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	getTrackingNumbers()	0	3	1
A	getCountry()	0	3	1
A	toArray()	0	5	1

<?php

namespace DPD\Interconnector\Request;

use DPD\Interconnector\Authentication;

class LabelsRequest implements RequestInterface
{
    private $authentication;

    private $trackingNumbers = [];

    private $printInfo = [];

    public function __construct(Authentication $authentication, array $trackingNumbers, string $printType, string $printFormat)
    {
        $this->authentication = $authentication;
        $this->trackingNumbers = $trackingNumbers;
        $this->printInfo = [
            'type' => $printType,
            'format' => $printFormat
        ];
    }

    public function toArray(): array
    {
        return array_merge(
            $this->authentication->toArray(),
            ['parcels' => $this->getTrackingNumbers(), 'printType' => $this->printInfo['type'], 'printFormat' => $this->printInfo['format']]
        );
    }

    public function getTrackingNumbers(): string
    {
        return implode('|', $this->trackingNumbers);
    }

    public function getCountry(): string
    {
        $this->authentication->country;

In this branch, the function will implicitly return null which is incompatible with the type-hinted return string. Consider adding a return statement or allowing null as return value.

    }
}