Devhelp\Piwik\Api\Param\Segment\Segment

Complexity

Total Complexity

7

Size/Duplication

Total Lines	65
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getQuery()	0	8	2
A	where()	0	7	1
A	andWhere()	0	4	1
A	orWhere()	0	4	1
A	append()	0	10	2

<?php

namespace Devhelp\Piwik\Api\Param\Segment;

use Devhelp\Piwik\Api\Param\Segment\Operator\Operator;

/**
 * Represents piwik segment value. Can be used to build
 * valid segment query by adding segment assertions
 */
class Segment
{
    private $queryParts = array();

    /**
     * builds the query and returns it as a string
     *
     * @return string
     */
    public function getQuery()
    {
        $queryParts = array_map(function ($value) {
            return ($value instanceof Operator) ? $value->expression() : $value;
        }, $this->queryParts);

        return implode('', $queryParts);
    }

    /**
     * starts segment query with operator
     *
     * @param Operator $operator
     * @return self
     */
    public function where(Operator $operator)
    {
        $this->queryParts = array();
        $this->queryParts[] = $operator;

        return $this;
    }

    /**
     * adds new operator using with AND. Starts new segment query if was not started yet
     *
     * @param Operator $operator
     * @return self
     */
    public function andWhere(Operator $operator)
    {
        return $this->append($operator, ';');
    }

    /**
     * adds new operator using with OR. Starts new segment query if was not started yet
     *
     * @param Operator $operator
     * @return self
     */
    public function orWhere(Operator $operator)
    {
        return $this->append($operator, ',');
    }

    private function append(Operator $operator, $sign)
    {
        if ($this->queryParts) {

The expression $this->queryParts 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.

            $this->queryParts[] = $sign;
        }

        $this->queryParts[] = $operator;

        return $this;
    }
}