PDOStatement::fetchColumn() - Code Metrics - Inspection of "#2648 fix last insert id consistency" - doctrine/dbal - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Pull Request — master (#2765)

by Marco

created 2017-07-02 11:41 UTC

PDOStatement::fetchColumn() A

↳ Parent: PDOStatement

Complexity

Conditions	2
Paths	2

Size

Total Lines	8
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	2.5

Importance

Changes

Metric	Value
dl	0
loc	8
ccs	2
cts	4
cp	0.5
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	5
nc	2
nop	1
crap	2.5

<?php
/*
 * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
 * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
 * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
 * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
 * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
 * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
 * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
 * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
 * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 *
 * This software consists of voluntary contributions made by many individuals
 * and is licensed under the MIT license. For more information, see
 * <http://www.doctrine-project.org>.
 */

namespace Doctrine\DBAL\Driver;

/**
 * The PDO implementation of the Statement interface.
 * Used by all PDO-based drivers.
 *
 * @since 2.0
 */
class PDOStatement extends \PDOStatement implements Statement
{
    /**
     * @var PDOConnection|null
     */
    private $connection;

    /**
     * Protected constructor.
     *
     * @param PDOConnection|null $connection
     *
     * @todo do we need to keep BC here? Or can we make $connection mandatory?
     */
    protected function __construct(PDOConnection $connection = null)
    {
        $this->connection = $connection;
    }

    /**
     * {@inheritdoc}
     */
    public function setFetchMode($fetchMode, $arg2 = null, $arg3 = null)
    {
        // This thin wrapper is necessary to shield against the weird signature
        // of PDOStatement::setFetchMode(): even if the second and third
        // parameters are optional, PHP will not let us remove it from this
        // declaration.
        try {
            if ($arg2 === null && $arg3 === null) {
                return parent::setFetchMode($fetchMode);
            }

            if ($arg3 === null) {
                return parent::setFetchMode($fetchMode, $arg2);
            }

            return parent::setFetchMode($fetchMode, $arg2, $arg3);
        } catch (\PDOException $exception) {
            throw new PDOException($exception);
        }
    }

    /**
     * {@inheritdoc}
     */
    public function bindValue($param, $value, $type = \PDO::PARAM_STR)
    {
        try {
            return parent::bindValue($param, $value, $type);
        } catch (\PDOException $exception) {
            throw new PDOException($exception);
        }
    }

    /**
     * {@inheritdoc}
     */
    public function bindParam($column, &$variable, $type = \PDO::PARAM_STR, $length = null, $driverOptions = null)
    {
        try {
            return parent::bindParam($column, $variable, $type, $length, $driverOptions);
        } catch (\PDOException $exception) {
            throw new PDOException($exception);
        }
    }

    /**
     * {@inheritdoc}
     */
    public function closeCursor()
    {
        try {
            return parent::closeCursor();
        } catch (\PDOException $exception) {
            // Exceptions not allowed by the interface.
            // In case driver implementations do not adhere to the interface, silence exceptions here.
            return true;
        }
    }

    /**
     * {@inheritdoc}
     */
    public function execute($params = null)
    {
        try {
            $result = parent::execute($params);

            if (! $this->connection) {
                $this->connection->trackLastInsertId();

            }

            return $result;
        } catch (\PDOException $exception) {
            throw new PDOException($exception);
        }
    }

    /**
     * {@inheritdoc}
     */
    public function fetch($fetchMode = null, $cursorOrientation = \PDO::FETCH_ORI_NEXT, $cursorOffset = 0)
    {
        try {
            if ($fetchMode === null && \PDO::FETCH_ORI_NEXT === $cursorOrientation && 0 === $cursorOffset) {
                return parent::fetch();
            }

            if (\PDO::FETCH_ORI_NEXT === $cursorOrientation && 0 === $cursorOffset) {
                return parent::fetch($fetchMode);
            }

            if (0 === $cursorOffset) {
                return parent::fetch($fetchMode, $cursorOrientation);
            }

            return parent::fetch($fetchMode, $cursorOrientation, $cursorOffset);
        } catch (\PDOException $exception) {
            throw new PDOException($exception);
        }
    }

    /**
     * {@inheritdoc}
     */
    public function fetchAll($fetchMode = null, $fetchArgument = null, $ctorArgs = null)
    {
        try {
            if ($fetchMode === null && null === $fetchArgument && null === $ctorArgs) {
                return parent::fetchAll();
            }

            if (null === $fetchArgument && null === $ctorArgs) {
                return parent::fetchAll($fetchMode);
            }

            if (null === $ctorArgs) {
                return parent::fetchAll($fetchMode, $fetchArgument);
            }

            return parent::fetchAll($fetchMode, $fetchArgument, $ctorArgs);
        } catch (\PDOException $exception) {
            throw new PDOException($exception);
        }
    }

    /**
     * {@inheritdoc}
     */
    public function fetchColumn($columnIndex = 0)
    {
        try {
            return parent::fetchColumn($columnIndex);
        } catch (\PDOException $exception) {
            throw new PDOException($exception);
        }
    }
}


1		<?php
2		/*
3		* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
4		* "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
5		* LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
6		* A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
7		* OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
8		* SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
9		* LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
10		* DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
11		* THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
12		* (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
13		* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
14		*
15		* This software consists of voluntary contributions made by many individuals
16		* and is licensed under the MIT license. For more information, see
17		* <http://www.doctrine-project.org>.
18		*/
19
20		namespace Doctrine\DBAL\Driver;
21
22		/**
23		* The PDO implementation of the Statement interface.
24		* Used by all PDO-based drivers.
25		*
26		* @since 2.0
27		*/
28		class PDOStatement extends \PDOStatement implements Statement
29		{
30		/**
31		* @var PDOConnection\|null
32		*/
33		private $connection;
34
35		/**
36		* Protected constructor.
37		*
38		* @param PDOConnection\|null $connection
39		*
40		* @todo do we need to keep BC here? Or can we make $connection mandatory?
41		*/
42	243	protected function __construct(PDOConnection $connection = null)
43		{
44	243	$this->connection = $connection;
45	243	}
46
47		/**
48		* {@inheritdoc}
49		*/
50	213	public function setFetchMode($fetchMode, $arg2 = null, $arg3 = null)
51		{
52		// This thin wrapper is necessary to shield against the weird signature
53		// of PDOStatement::setFetchMode(): even if the second and third
54		// parameters are optional, PHP will not let us remove it from this
55		// declaration.
56		try {
57	213	if ($arg2 === null && $arg3 === null) {
58	213	return parent::setFetchMode($fetchMode);
59		}
60
61	2	if ($arg3 === null) {
62	2	return parent::setFetchMode($fetchMode, $arg2);
63		}
64
65		return parent::setFetchMode($fetchMode, $arg2, $arg3);
66		} catch (\PDOException $exception) {
67		throw new PDOException($exception);
68		}
69		}
70
71		/**
72		* {@inheritdoc}
73		*/
74	33	public function bindValue($param, $value, $type = \PDO::PARAM_STR)
75		{
76		try {
77	33	return parent::bindValue($param, $value, $type);
78		} catch (\PDOException $exception) {
79		throw new PDOException($exception);
80		}
81		}
82
83		/**
84		* {@inheritdoc}
85		*/
86	7	public function bindParam($column, &$variable, $type = \PDO::PARAM_STR, $length = null, $driverOptions = null)
87		{
88		try {
89	7	return parent::bindParam($column, $variable, $type, $length, $driverOptions);
90		} catch (\PDOException $exception) {
91		throw new PDOException($exception);
92		}
93		}
94
95		/**
96		* {@inheritdoc}
97		*/
98	20	public function closeCursor()
99		{
100		try {
101	20	return parent::closeCursor();
102		} catch (\PDOException $exception) {
103		// Exceptions not allowed by the interface.
104		// In case driver implementations do not adhere to the interface, silence exceptions here.
105		return true;
106		}
107		}
108
109		/**
110		* {@inheritdoc}
111		*/
112	132	public function execute($params = null)
113		{
114		try {
115	132	$result = parent::execute($params);
116
117	127	if (! $this->connection) {
118		$this->connection->trackLastInsertId();
		0 ignored issues – show Bug introduced 2017-07-02 11:45 UTC by Report Bug Copy Issue Report The method `trackLastInsertId` cannot be called on `$this->connection` (of type `null`). Methods can only be called on objects. This check looks for methods being called on variables that have been inferred to never be objects. Loading history...
119		}
120
121	127	return $result;
122	7	} catch (\PDOException $exception) {
123	3	throw new PDOException($exception);
124		}
125		}
126
127		/**
128		* {@inheritdoc}
129		*/
130	67	public function fetch($fetchMode = null, $cursorOrientation = \PDO::FETCH_ORI_NEXT, $cursorOffset = 0)
131		{
132		try {
133	67	if ($fetchMode === null && \PDO::FETCH_ORI_NEXT === $cursorOrientation && 0 === $cursorOffset) {
134	9	return parent::fetch();
135		}
136
137	58	if (\PDO::FETCH_ORI_NEXT === $cursorOrientation && 0 === $cursorOffset) {
138	58	return parent::fetch($fetchMode);
139		}
140
141		if (0 === $cursorOffset) {
142		return parent::fetch($fetchMode, $cursorOrientation);
143		}
144
145		return parent::fetch($fetchMode, $cursorOrientation, $cursorOffset);
146		} catch (\PDOException $exception) {
147		throw new PDOException($exception);
148		}
149		}
150
151		/**
152		* {@inheritdoc}
153		*/
154	89	public function fetchAll($fetchMode = null, $fetchArgument = null, $ctorArgs = null)
155		{
156		try {
157	89	if ($fetchMode === null && null === $fetchArgument && null === $ctorArgs) {
158	76	return parent::fetchAll();
159		}
160
161	52	if (null === $fetchArgument && null === $ctorArgs) {
162	51	return parent::fetchAll($fetchMode);
163		}
164
165	1	if (null === $ctorArgs) {
166	1	return parent::fetchAll($fetchMode, $fetchArgument);
167		}
168
169		return parent::fetchAll($fetchMode, $fetchArgument, $ctorArgs);
170		} catch (\PDOException $exception) {
171		throw new PDOException($exception);
172		}
173		}
174
175		/**
176		* {@inheritdoc}
177		*/
178	42	public function fetchColumn($columnIndex = 0)
179		{
180		try {
181	42	return parent::fetchColumn($columnIndex);
182		} catch (\PDOException $exception) {
183		throw new PDOException($exception);
184		}
185		}
186		}
187

doctrine / dbal

Pull Request — master (#2765)

PDOStatement::fetchColumn() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like