ZfcDatagrid\Column\Style\AbstractStyle

Complexity

Total Complexity

18

Size/Duplication

Total Lines	112
Duplicated Lines	39.29 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	setByValueOperator()	8	8	3
A	getByValueOperator()	0	4	1
A	addByValue()	0	8	1
A	getByValues()	0	4	1
A	hasByValues()	0	8	2
D	isApply()	36	36	10

<?php

namespace ZfcDatagrid\Column\Style;

use ZfcDatagrid\Column\AbstractColumn;
use ZfcDatagrid\Filter;

abstract class AbstractStyle
{
    protected $byValueOperator = 'OR';

    /**
     * @var array
     */
    private $byValues = [];

    /**
     * Display the values with AND or OR (if multiple showOnValues are defined).
     *
     * @param string $operator
     */
    public function setByValueOperator($operator = 'OR')

This method seems to be duplicated in your project.

    {
        if ($operator != 'AND' && $operator != 'OR') {
            throw new \InvalidArgumentException('not allowed operator: "'.$operator.'" (AND / OR is allowed)');
        }

        $this->byValueOperator = (string) $operator;
    }

    /**
     * Get the show on value operator, e.g.
     * OR, AND.
     *
     * @return string
     */
    public function getByValueOperator()
    {
        return $this->byValueOperator;
    }

    /**
     * Set the style value based and not general.
     *
     * @param AbstractColumn $column
     * @param mixed          $value
     * @param string         $operator
     */
    public function addByValue(AbstractColumn $column, $value, $operator = Filter::EQUAL)
    {
        $this->byValues[] = [
            'column' => $column,
            'value' => $value,
            'operator' => $operator,
        ];
    }

    /**
     * @return array
     */
    public function getByValues()
    {
        return $this->byValues;
    }

    /**
     * @return bool
     */
    public function hasByValues()
    {
        if (count($this->byValues) > 0) {
            return true;
        }

        return false;
    }

    /**
     * @param array $row
     *
     * @return bool
     */
    public function isApply(array $row)

This method seems to be duplicated in your project.

    {
        if ($this->hasByValues() === false) {
            return true;
        }

        $isApply = false;
        foreach ($this->getByValues() as $rule) {
            $value = '';
            if (isset($row[$rule['column']->getUniqueId()])) {
                $value = $row[$rule['column']->getUniqueId()];
            }

            if ($rule['value'] instanceof AbstractColumn) {
                if (isset($row[$rule['value']->getUniqueId()])) {
                    $ruleValue = $row[$rule['value']->getUniqueId()];
                } else {
                    $ruleValue = '';
                }
            } else {
                $ruleValue = $rule['value'];
            }

            $isApplyMatch = Filter::isApply($value, $ruleValue, $rule['operator']);
            if ($this->getByValueOperator() == 'OR' && true === $isApplyMatch) {
                // For OR one match is enough
                return true;
            } elseif ($this->getByValueOperator() == 'AND' && false === $isApplyMatch) {
                return false;
            } else {
                $isApply = $isApplyMatch;
            }
        }

        return $isApply;
    }
}