rtens\domin\delivery\web\renderers\tables\TableRenderer

Complexity

Total Complexity

14

Size/Duplication

Total Lines	88
Duplicated Lines	10.23 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes	3
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	getHeaders()	0	3	1
A	handles()	0	3	1
A	headElements()	6	12	4
A	__construct()	0	4	1
A	renderHeaders()	0	7	2
A	renderRows()	3	12	3
A	render()	0	11	2

<?php
namespace rtens\domin\delivery\web\renderers\tables;

use rtens\domin\delivery\RendererRegistry;
use rtens\domin\delivery\web\Element;
use rtens\domin\delivery\web\renderers\link\LinkPrinter;
use rtens\domin\delivery\web\WebRenderer;
use watoki\reflect\Property;

class TableRenderer implements WebRenderer {

    /** @var RendererRegistry */
    private $renderers;

    /** @var LinkPrinter */
    private $printer;

    public function __construct(RendererRegistry $renderers, LinkPrinter $printer) {
        $this->renderers = $renderers;
        $this->printer = $printer;
    }

    /**
     * @param mixed $value
     * @return bool
     */
    public function handles($value) {
        return $value instanceof Table;
    }

    /**
     * @param Table $value
     * @return mixed
     */
    public function render($value) {
        $rows = $this->renderRows($value);

        if (!$rows) {

The expression $rows of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            return null;
        };

        return (string)new Element('table', ['class' => 'table table-striped'], array_merge([
            new Element('thead', [], [new Element('tr', [], $this->renderHeaders($value))])
        ], $rows));
    }

    private function renderHeaders($table) {
        $headers = [new Element('th', ['width' => '1'])];
        foreach ($this->getHeaders($table) as $caption) {
            $headers[] = new Element('th', [], [$caption]);
        }
        return $headers;
    }

    /**
     * @param Table $table
     * @return array
     * @throws \Exception
     */
    private function renderRows($table) {
        $rows = [];
        foreach ($table->getItems() as $item) {
            $row = [new Element('td', [], $this->printer->createDropDown($item))];
            foreach ($table->getCells($item) as $cell) {

This code seems to be duplicated across your project.

                $row[] = new Element('td', [], [$this->renderers->getRenderer($cell)->render($cell)]);
            }

            $rows[] = new Element('tr', [], $row);
        }
        return $rows;
    }

    /**
     * @param Table $value
     * @return array|Element[]
     */
    public function headElements($value) {
        $elements = [];
        foreach ($value->getItems() as $item) {
            foreach ($value->getCells($item) as $cell) {

This code seems to be duplicated across your project.

                $renderer = $this->renderers->getRenderer($cell);
                if ($renderer instanceof WebRenderer) {
                    $elements = array_merge($elements, $renderer->headElements($cell));
                }
            }
        }
        return $elements;
    }

    /**
     * @param Table $table
     * @return mixed
     */
    protected function getHeaders($table) {
        return $table->getHeaders();
    }
}