Germania\Addresses\PdoAddressFactory

Complexity

Total Complexity

4

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	11	2
A	__invoke()	0	7	2

<?php
namespace Germania\Addresses;

class PdoAddressFactory
{
	/**
	 * @var AddressFactory
	 */
	public $address_factory; 	

There is some trailing whitespace on this line which should be avoided as per coding-style.

	/**
	 * @var PDOStmt
	 */
	public $stmt; 

	public static $DB_FIELDS = array(
		'id',
		'type',
		'street1',
		'street2',
		'zip',
		'location',
		'country'
	);

	/**
	 * @param \PDO                $pdo
	 * @param string              $table_name
	 * @param AddressFactory|null $address_factory
	 */
	public function __construct( \PDO $pdo, string $table_name, AddressFactory $address_factory = null )
	{
		$this->address_factory = $address_factory ?: new AddressFactory;

		$sql = "SELECT " . implode(",", static::$DB_FIELDS) ."
		FROM {$table_name}
		WHERE id = :id";

		$this->stmt = $pdo->prepare( $sql );

It seems like $pdo->prepare($sql) of type object<PDOStatement> is incompatible with the declared type object<Germania\Addresses\PDOStmt> of property $stmt.

		$this->stmt->setFetchMode(\PDO::FETCH_CLASS, PdoAddress::class);
	}


	public function __invoke( int $id ) : AddressInterface
	{
		if (!$this->stmt->execute(['id' => $id]))
			throw new \RuntimeException("Could not execute SELECT Address PDOStatement");

		return $this->stmt->fetch();
	}
}