Aggregate::__construct() - Code Metrics - Inspection of "Update the schema version requirements" - vanilla/garden-db - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#10)

by Todd

created 2018-09-08 18:03 UTC

Aggregate::__construct() A

↳ Parent: Aggregate

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	2.032

Importance

Changes

Metric	Value
dl	0
loc	13
ccs	8
cts	10
cp	0.8
rs	9.8333
c	0
b	0
f	0
cc	2
nc	2
nop	3
crap	2.032

<?php
/**
 * @author Todd Burry <[email protected]>
 * @copyright 2009-2017 Vanilla Forums Inc.
 * @license MIT
 */

namespace Garden\Db;

/**
 * Represents an aggregate function call.
 *
 * Use an aggregate by passing an instance as a value in the "columns" option on a call to {@link Db::get()}.
 */
class Aggregate extends Literal {
    const AVG = 'avg';
    const COUNT = 'count';
    const COUNT_DISTINCT = 'count-distinct';
    const MAX = 'max';
    const MIN = 'min';
    const SUM = 'sum';

    private $func;
    private $column;
    private $alias;

    /**
     * Construct an {@link Aggregate} object.
     *
     * @param string $func The name of the aggregate function. Use one of the **Aggregate::*** constants.
     * @param string $column The name of the column to aggregate.
     * @param string $alias The alias of the aggregate. If left out then the function name will be used as the alias.
     */
    public function __construct($func, $column, $alias = '') {
        $format = '%1$s(%2$s) as %3$s';
        if ($func === static::COUNT_DISTINCT) {
            $format = '%1$s(distinct %2$s) as %3$s';
            $func = static::COUNT;
        }

        parent::__construct($format);

        $this->func = $func;
        $this->column = $column;
        $this->alias = $alias;
    }

    /**
     * {@inheritdoc}
     */
    public function getValue(Db $db, ...$args) {
        return parent::getValue(
            $db,
            $this->func,
            $this->column === '*' ? '*' : $db->escape($this->column),
            $db->escape($this->alias ?: $this->func)
        );
    }
}


1		<?php
2		/**
3		* @author Todd Burry <[email protected]>
4		* @copyright 2009-2017 Vanilla Forums Inc.
5		* @license MIT
6		*/
7
8		namespace Garden\Db;
9
10		/**
11		* Represents an aggregate function call.
12		*
13		* Use an aggregate by passing an instance as a value in the "columns" option on a call to {@link Db::get()}.
14		*/
15		class Aggregate extends Literal {
16		const AVG = 'avg';
17		const COUNT = 'count';
18		const COUNT_DISTINCT = 'count-distinct';
19		const MAX = 'max';
20		const MIN = 'min';
21		const SUM = 'sum';
22
23		private $func;
24		private $column;
25		private $alias;
26
27		/**
28		* Construct an {@link Aggregate} object.
29		*
30		* @param string $func The name of the aggregate function. Use one of the Aggregate::* constants.
31		* @param string $column The name of the column to aggregate.
32		* @param string $alias The alias of the aggregate. If left out then the function name will be used as the alias.
33		*/
34	2	public function __construct($func, $column, $alias = '') {
35	2	$format = '%1$s(%2$s) as %3$s';
36	2	if ($func === static::COUNT_DISTINCT) {
37		$format = '%1$s(distinct %2$s) as %3$s';
38		$func = static::COUNT;
39		}
40
41	2	parent::__construct($format);
42
43	2	$this->func = $func;
44	2	$this->column = $column;
45	2	$this->alias = $alias;
46	2	}
47
48		/**
49		* {@inheritdoc}
50		*/
51	2	public function getValue(Db $db, ...$args) {
52	2	return parent::getValue(
53	2	$db,
54	2	$this->func,
55	2	$this->column === '' ? '' : $db->escape($this->column),
56	2	$db->escape($this->alias ?: $this->func)
57		);
58		}
59		}
60

vanilla / garden-db

Pull Request — master (#10)

Aggregate::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like