Sauls\Component\Helper\Operation\StringOperation\TruncateSentences::execute()

Complexity

Conditions	2
Paths	2

Size

Total Lines	14
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function execute(string $value, int $count, string $suffix = '..')
    {
        $value = \strip_tags($value);

        $sentences = \preg_split('/(?<!\w\.\w.)(?<![A-Z][a-z]\.)(?<=\.|\?)\s/um', \trim($value), null, PREG_SPLIT_DELIM_CAPTURE);

        $sentences = array_map(function($value) {
            return trim($value);
        }, $sentences);

It seems like $sentences can also be of type false; however, parameter $arr1 of array_map() does only seem to accept array, maybe add an additional type check?

        if (\count($sentences) > $count) {
            return \implode(' ', \array_slice($sentences, 0, $count)) . $suffix;
        }
        return $value;
    }