Box\Spout\Writer\Common\Manager\RowManager

Complexity

Total Complexity

6

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

66.67%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	applyStyle()	0	5	1
A	__construct()	0	4	1
A	hasCells()	0	4	1
A	isEmpty()	0	6	3

<?php

namespace Box\Spout\Writer\Common\Manager;

use Box\Spout\Writer\Common\Entity\Row;
use Box\Spout\Writer\Common\Entity\Style\Style;
use Box\Spout\Writer\Common\Manager\Style\StyleMerger;

class RowManager
{
    /**
     * @var StyleMerger
     */
    protected $styleMerger;

    /**
     * @param StyleMerger $styleMerger
     */
    public function __construct(StyleMerger $styleMerger)
    {
        $this->styleMerger = $styleMerger;
    }

    /**
     * @param Row $row
     * @param Style|null $style
     * @return $this
     */
    public function applyStyle(Row $row, Style $style)
    {
        $mergedStyle = $this->styleMerger->merge($row->getStyle(), $style);

It seems like $row->getStyle() can be null; however, merge() does not accept null, maybe add an additional type check?

        $row->setStyle($mergedStyle);
    }

    /**
     * Returns whether a row has cells
     *
     * @param Row $row
     * @return bool
     */
    public function hasCells(Row $row)
    {
        return count($row->getCells()) !== 0;
    }

    /**
     * Detect whether a row is considered empty.
     * An empty row has either no cells at all - or only one empty cell
     *
     * @param Row $row
     * @return bool
     */
    public function isEmpty(Row $row)
    {
        $cells = $row->getCells();

        return count($cells) === 0 || (count($cells) === 1 && $cells[0]->isEmpty());
    }
}