PdoAddressDeleter::__invoke() - Code Metrics - Inspection of "Merge branch 'release/1.2.3'" - GermaniaKG/Addresses - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 982641...a62c86 )

by Carsten

created 2019-10-07 10:58 UTC

PdoAddressDeleter::__invoke() A

↳ Parent: PdoAddressDeleter

Complexity

Conditions	3
Paths	3

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	3

Importance

Changes

Metric	Value
dl	0
loc	16
ccs	9
cts	9
cp	1
rs	9.7333
c	0
b	0
f	0
cc	3
nc	3
nop	1
crap	3

<?php
namespace Germania\Addresses;

class PdoAddressDeleter
{
	/**
	 * @var PDOStmt
	 */
	public $stmt; 


	/**
	 * @param \PDO                $pdo
	 * @param string              $table_name
	 */
	public function __construct( \PDO $pdo, string $table_name )
	{

		$sql = "DELETE FROM {$table_name} 
		WHERE id = :id
		LIMIT 1";

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

	}



	public function __invoke( PdoAddressInterface $address ) : int
	{
		if (!$id = $address->getId())
			throw new \UnexpectedValueException("Addess object 'getId' did not return primary key value");

		$result = $this->stmt->execute([
			'id'       => $id
		]);

		if (!$result) {
			$msg = sprintf("Could not execute PDOStatement for address ID '%s'", $id);
			throw new \RuntimeException($msg);
		}

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

1		<?php
2		namespace Germania\Addresses;
3
4		class PdoAddressDeleter
5		{
6		/**
7		* @var PDOStmt
8		*/
9		public $stmt;
10
11
12		/**
13		* @param \PDO $pdo
14		* @param string $table_name
15		*/
16	6	public function __construct( \PDO $pdo, string $table_name )
17		{
18
19	6	$sql = "DELETE FROM {$table_name}
20		WHERE id = :id
21		LIMIT 1";
22
23	6	$this->stmt = $pdo->prepare( $sql );
		0 ignored issues – show Documentation Bug introduced 2019-10-07 09:03 UTC by Report Bug Copy Issue Report It seems like `$pdo->prepare($sql)` of type `object<PDOStatement>` is incompatible with the declared type `object<Germania\Addresses\PDOStmt>` of property `$stmt`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
24	6	}
25
26
27
28	6	public function __invoke( PdoAddressInterface $address ) : int
29		{
30	6	if (!$id = $address->getId())
31	2	throw new \UnexpectedValueException("Addess object 'getId' did not return primary key value");
32
33	4	$result = $this->stmt->execute([
34	4	'id' => $id
35		]);
36
37	4	if (!$result) {
38	2	$msg = sprintf("Could not execute PDOStatement for address ID '%s'", $id);
39	2	throw new \RuntimeException($msg);
40		}
41
42	2	return $this->stmt->rowCount();
43		}
44		}

GermaniaKG / Addresses

Push — master ( 982641...a62c86 )

PdoAddressDeleter::__invoke() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like