VPA\Console\Glyphs\Text::render()

Complexity

Conditions	6
Paths	32

Size

Total Lines	28
Code Lines	23

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	23
CRAP Score	6

Importance

Changes	3
Bugs	0	Features	0

<?php
    public function render(): GlyphInline
    {
        $batches = $this->splitText();
        $lengths = array_map(function ($it) {
            return strlen($it);
        }, $batches);
        $this->height = count($batches);
        $maxLength = max($lengths);
        $parent = $this->getParent();
        $contentWidth = $parent && $parent instanceof GlyphBlock ? $parent->getContentWidth() : 0;
        $this->width = $contentWidth > 0 ? $contentWidth : min($maxLength, $this->__get('maxWidth'));
        $this->__set('maxWidth', max($this->width, $this->__get('maxWidth')));
        $padding = match ($this->__get('textAlign')) {
            default => STR_PAD_RIGHT,
            'right' => STR_PAD_LEFT,
            'center' => STR_PAD_BOTH,
        };
        $resultString = '';
        foreach ($batches as $batch) {
            $resultString .= str_pad(ltrim($batch), $this->getWidth(), ' ', $padding);
        }
        $symbols = str_split($resultString);

        $this->renderMap = $this->getWidth() ? array_chunk(array_map(function ($value) {
            return new Symbol($value);
        }, $symbols), $this->getWidth()) : [];

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

        $this->height = count($this->renderMap);
        return $this;
    }