Shlinkio\Shlink\Core\Util\CocurSymfonySluggerBridge

Complexity

Total Complexity

2

Size/Duplication

Total Lines	12
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	slug()	0	3	1
A	__construct()	0	3	1

<?php

declare(strict_types=1);

namespace Shlinkio\Shlink\Core\Util;

use Cocur\Slugify\SlugifyInterface;
use Symfony\Component\String\AbstractUnicodeString;
use Symfony\Component\String\Slugger\SluggerInterface;

use function Symfony\Component\String\s;

class CocurSymfonySluggerBridge implements SluggerInterface
{
    private SlugifyInterface $slugger;

    public function __construct(SlugifyInterface $slugger)
    {
        $this->slugger = $slugger;
    }

    public function slug(string $string, string $separator = '-', ?string $locale = null): AbstractUnicodeString
    {
        return s($this->slugger->slugify($string, $separator));

The expression return s($this->slugger->slugify($string, $separator)) could return the type Symfony\Component\String\ByteString which is incompatible with the type-hinted return Symfony\Component\String\AbstractUnicodeString. Consider adding an additional type-check to rule them out.

    }
}