Deamon\LoggerExtraBundle\Services\DeamonLoggerExtraContext

Complexity

Total Complexity

5

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Test Coverage

Coverage

66.67%

Importance

Changes	2
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getLocale()	0	3	1
A	__construct()	0	7	2
A	getApplicationVersion()	0	3	1
A	getApplicationName()	0	3	1

<?php

namespace Deamon\LoggerExtraBundle\Services;

class DeamonLoggerExtraContext
{
    public function __construct(
        private ?string $applicationName,
        private ?string $locale = null,
        private ?string $applicationVersion = null)
    {
        if(null !== $applicationVersion){
            $this->applicationVersion = trim($this->applicationVersion);

It seems like $this->applicationVersion can also be of type null; however, parameter $string of trim() does only seem to accept string, maybe add an additional type check?

        }
    }

    public function getLocale(): ?string
    {
        return $this->locale;
    }

    public function getApplicationName(): ?string
    {
        return $this->applicationName;
    }

    public function getApplicationVersion(): ?string
    {
        return $this->applicationVersion;
    }
}