SQLParser\Node\WhenConditions

Complexity

Total Complexity

7

Size/Duplication

Total Lines	60
Duplicated Lines	5 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	2
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getValue()	0	4	1
A	setValue()	0	4	1
A	toSql()	3	17	4
A	getOperatorSymbol()	0	4	1

<?php

namespace SQLParser\Node;

use Mouf\Utils\Common\ConditionInterface\ConditionTrait;
use Doctrine\DBAL\Connection;
use Mouf\MoufManager;
use Mouf\MoufInstanceDescriptor;
use SQLParser\Node\Traverser\NodeTraverser;
use SQLParser\Node\Traverser\VisitorInterface;

/**
 * This class represents a set of ... WHEN ... THEN ... construct (inside a CASE).
 *
 * @author David Négrier <d.negrier@thecodingmachine.com>
 */
class WhenConditions extends AbstractManyInstancesOperator
{

    private $value;

    public function getValue()
    {
        return $this->value;
    }

    /**
     * Sets the value.
     *
     * @Important
     *
     * @param NodeInterface|NodeInterface[]|string $value
     */
    public function setValue($value)
    {
        $this->value = $value;
    }

    /**
     * Renders the object as a SQL string.
     *
     * @param Connection $dbConnection
     * @param array      $parameters
     * @param number     $indent
     * @param int        $conditionsMode
     *
     * @return string
     */
    public function toSql(array $parameters = array(), Connection $dbConnection = null, $indent = 0, $conditionsMode = self::CONDITION_APPLY)
    {
        $fullSql = '';

        if ($this->value) {

This code seems to be duplicated across your project.

            $fullSql = NodeFactory::toSql($this->value, $dbConnection, $parameters, ' ', false, $indent, $conditionsMode);
        }

        foreach ($this->getOperands() as $operand) {
            $sql = NodeFactory::toSql($operand, $dbConnection, $parameters, ' ', false, $indent, $conditionsMode);
            if ($sql != null) {

It seems like you are loosely comparing $sql of type null|string against null; this is ambiguous if the string can be empty. Consider using a strict comparison !== instead.

                $fullSql .= "\n".str_repeat(' ', $indent).'WHEN '.$sql;
            }
        }

        return $fullSql;
    }

    /**
     * Returns the symbol for this operator.
     *
     * @return string
     */
    protected function getOperatorSymbol()
    {
        return 'WHEN';
    }
}