ZfTable\Decorator\Condition\AbstractCondition

Complexity

Total Complexity

3

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getDecorator()	0	4	1
A	setDecorator()	0	5	1
A	getActulRow()	0	4	1

<?php
/**
 * ZfTable ( Module for Zend Framework 2)
 *
 * @copyright Copyright (c) 2013 Piotr Duda dudapiotrek@gmail.com
 * @license   MIT License
 */

namespace ZfTable\Decorator\Condition;

use ZfTable\Decorator\AbstractDecorator;
use ZfTable\Decorator\DataAccessInterface;

abstract class AbstractCondition implements ConditionInterface
{

    /**
     * Decorator
     * @var AbstractDecorator
     */
    protected $decorator;

    /**
     * Get decorator
     * @return AbstractDecorator
     */
    public function getDecorator()
    {
        return $this->decorator;
    }

    /**
     * Set decorator
     *
     * @param DataAccessInterface $decorator
     * @return $this
     */
    public function setDecorator($decorator)
    {
        $this->decorator = $decorator;

It seems like $decorator of type object<ZfTable\Decorator\DataAccessInterface> is incompatible with the declared type object<ZfTable\Decorator\AbstractDecorator> of property $decorator.

        return $this;
    }

    /**
     *
     * @return type
     */
    public function getActulRow()
    {
        return $this->decorator->getActualRow();

It seems like you code against a specific sub-type and not the parent class ZfTable\Decorator\AbstractDecorator as the method getActualRow() does only exist in the following sub-classes of ZfTable\Decorator\AbstractDecorator: ZfTable\Decorator\Cell\AbstractCellDecorator, ZfTable\Decorator\Cell\AttrDecorator, ZfTable\Decorator\Cell\CallableDecorator, ZfTable\Decorator\Cell\ClassDecorator, ZfTable\Decorator\Cell\Editable, ZfTable\Decorator\Cell\Icon, ZfTable\Decorator\Cell\Link, ZfTable\Decorator\Cell\Mapper, ZfTable\Decorator\Cell\Template, ZfTable\Decorator\Cell\VarAttrDecorator, ZfTable\Decorator\Row\AbstractRowDecorator, ZfTable\Decorator\Row\ClassDecorator, ZfTable\Decorator\Row\Separatable, ZfTable\Decorator\Row\VarAttr. Maybe you want to instanceof check for one of these explicitly?

    }
}