Foo\Grid\Cell\Cell

Complexity

Total Complexity

2

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

71.43%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	getGroup()	0	4	1

<?php

declare(strict_types = 1);

namespace Foo\Grid\Cell;

use Foo\Grid\Component\Component;
use Foo\Grid\Component\IComponent;

class Cell extends Component implements ICell
{
    const HEAD = 'th';
    const BODY = 'td';

    /** @var string */
    protected $group = '';

    /**
     * @param string|IComponent $content
     * @param string            $group
     * @param array             $attributes
     * @param string            $tag
     */
    public function __construct(string $content, string $group, array $attributes = [], string $tag = self::BODY)
    {
        $this->group = $group;

        parent::__construct($content, $tag, $attributes);

It seems like $content defined by parameter $content on line 24 can also be of type object<Foo\Grid\Component\IComponent>; however, Foo\Grid\Component\Component::__construct() does only seem to accept string, maybe add an additional type check?

        $this->appendToAttribute(Component::ATTRIBUTE_CLASS, $tag . '-' . $group);
    }

    /**
     * @return string
     */
    public function getGroup(): string
    {
        return $this->group;
    }
}