PDOQuery::__construct() - Code Metrics - Inspection of "Merge pull request #6518 from sminnee/generators" - silverstripe/silverstripe-framework - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 288a96...964b09 )

by Daniel

created 2018-03-17 15:05 UTC

PDOQuery::__construct() A

↳ Parent: PDOQuery

Complexity

Conditions	1
Paths	1

Size

Total Lines	8
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	3
nc	1
nop	1
dl	0
loc	8
rs	9.4285
c	0
b	0
f	0

<?php

namespace SilverStripe\ORM\Connect;

use PDOStatement;
use PDO;
use ArrayIterator;

/**
 * A result-set from a PDO database.
 */
class PDOQuery extends Query
{
    /**
     * The internal MySQL handle that points to the result set.
     * @var PDOStatement
     */
    protected $statement = null;

    protected $results = null;

    /**
     * Hook the result-set given into a Query class, suitable for use by SilverStripe.
     * @param PDOStatement $statement The internal PDOStatement containing the results
     */
    public function __construct(PDOStatement $statement)
    {
        $this->statement = $statement;
        // Since no more than one PDOStatement for any one connection can be safely
        // traversed, each statement simply requests all rows at once for safety.
        // This could be re-engineered to call fetchAll on an as-needed basis
        $this->results = $statement->fetchAll(PDO::FETCH_ASSOC);
        $statement->closeCursor();
    }

    public function getIterator()
    {
        return new ArrayIterator($this->results);
    }

    public function numRecords()
    {
        return count($this->results);
    }
}


1			<?php
2
3			namespace SilverStripe\ORM\Connect;
4
5			use PDOStatement;
6			use PDO;
7			use ArrayIterator;
8
9			/**
10			* A result-set from a PDO database.
11			*/
12			class PDOQuery extends Query
13			{
14			/**
15			* The internal MySQL handle that points to the result set.
16			* @var PDOStatement
17			*/
18			protected $statement = null;
19
20			protected $results = null;
21
22			/**
23			* Hook the result-set given into a Query class, suitable for use by SilverStripe.
24			* @param PDOStatement $statement The internal PDOStatement containing the results
25			*/
26			public function __construct(PDOStatement $statement)
27			{
28			$this->statement = $statement;
29			// Since no more than one PDOStatement for any one connection can be safely
30			// traversed, each statement simply requests all rows at once for safety.
31			// This could be re-engineered to call fetchAll on an as-needed basis
32			$this->results = $statement->fetchAll(PDO::FETCH_ASSOC);
33			$statement->closeCursor();
34			}
35
36			public function getIterator()
37			{
38			return new ArrayIterator($this->results);
39			}
40
41			public function numRecords()
42			{
43			return count($this->results);
44			}
45			}
46

silverstripe / silverstripe-framework

Push — master ( 288a96...964b09 )

PDOQuery::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like