Connection::prepare() - Code Metrics - Inspection of "Run selenium tests on Github Actions" - OpenConext/Stepup-Gateway - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — feature/acceptance-tests (#192)

unknown

created 2020-04-28 20:27 UTC

Connection::prepare() A

↳ Parent: Connection

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	4
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	1

<?php

namespace Surfnet\StepupGateway\Behat\Repository;

use Doctrine\DBAL\Connection as DBALConnection;
use PDO;
use PDOStatement;

class Connection
{
    /**
     * @var PDO
     */
    private $connection;

    /**
     * @param DBALConnection $connection
     * @throws \Exception
     */
    public function __construct(DBALConnection $connection)
    {
        var_dump([

            'host' => $connection->getHost(),
            'port' => $connection->getPort(),
            'dbname' => $connection->getDatabase(),
            'user' => $connection->getUsername(),
            'pass' => $connection->getPassword(),
        ]);
        $connection->connect();
        $conn = $connection->getWrappedConnection();
        if (!$conn instanceof PDO) {
            throw new \Exception('DBAL Connection should be wrapped around PDO connection');
        }

        $this->connection = $conn;
    }

    /**
     * @param string $statement
     * @return bool|PDOStatement
     */
    public function prepare($statement)
    {
        return $this->connection->prepare($statement);
    }
}


1			<?php
2
3			namespace Surfnet\StepupGateway\Behat\Repository;
4
5			use Doctrine\DBAL\Connection as DBALConnection;
6			use PDO;
7			use PDOStatement;
8
9			class Connection
10			{
11			/**
12			* @var PDO
13			*/
14			private $connection;
15
16			/**
17			* @param DBALConnection $connection
18			* @throws \Exception
19			*/
20			public function __construct(DBALConnection $connection)
21			{
22			var_dump([
			0 ignored issues – show Security Debugging Code introduced 2020-04-28 20:23 UTC by Report Bug Copy Issue Report `var_dump(array('host' =>...ction->getPassword()));` looks like debug code. Are you sure you do not want to remove it? This might expose sensitive data. Loading history...
23			'host' => $connection->getHost(),
24			'port' => $connection->getPort(),
25			'dbname' => $connection->getDatabase(),
26			'user' => $connection->getUsername(),
27			'pass' => $connection->getPassword(),
28			]);
29			$connection->connect();
30			$conn = $connection->getWrappedConnection();
31			if (!$conn instanceof PDO) {
32			throw new \Exception('DBAL Connection should be wrapped around PDO connection');
33			}
34
35			$this->connection = $conn;
36			}
37
38			/**
39			* @param string $statement
40			* @return bool\|PDOStatement
41			*/
42			public function prepare($statement)
43			{
44			return $this->connection->prepare($statement);
45			}
46			}
47

OpenConext / Stepup-Gateway

Pull Request — feature/acceptance-tests (#192)

Connection::prepare() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like