GeoLocation

Complexity

Total Complexity

12

Size/Duplication

Total Lines	88
Duplicated Lines	30.68 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	setAddress()	0	3	1
A	getData()	0	3	1
A	setData()	0	3	1
A	save()	0	24	4
A	getAddress()	0	3	1
A	getByAddress()	0	18	3
A	getCreation()	0	3	1

<?php

/**
 * Geolocation data object
 *
 * This is a cache.
 */
class GeoLocation extends DataObject
{
	private $address;
	private $data;
	private $creation;

	/**
	 * @param string $address
	 * @param PdoDatabase $database
	 * @param bool $forUpdate
	 * @return GeoLocation
	 */
	public static function getByAddress($address, PdoDatabase $database, $forUpdate = false)
	{
		$lockMode = $forUpdate ? ' FOR UPDATE' : '';
		$sql = "SELECT * FROM geolocation WHERE address = :id LIMIT 1" . $lockMode;

		$statement = $database->prepare($sql);
		$statement->bindValue(":id", $address);

		$statement->execute();

		$resultObject = $statement->fetchObject(get_called_class());

		if ($resultObject != false) {
			$resultObject->isNew = false;
			$resultObject->setDatabase($database);
		}

		return $resultObject;
	}

	public function save()
	{
		if ($this->isNew) {
// insert
			$statement = $this->dbObject->prepare("INSERT INTO `geolocation` (address, data) VALUES (:address, :data) ON DUPLICATE KEY UPDATE address = address;");
			$statement->bindValue(":address", $this->address);
			$statement->bindValue(":data", $this->data);
			if ($statement->execute()) {
				$this->isNew = false;
				$this->id = $this->dbObject->lastInsertId();

The property $id was declared of type integer, but $this->dbObject->lastInsertId() is of type string. Maybe add a type cast?

			}
			else {
				throw new Exception($statement->errorInfo());

$statement->errorInfo() of type array is incompatible with the type string expected by parameter $message of Exception::__construct().

			}
		}
		else {
// update
			$statement = $this->dbObject->prepare("UPDATE `geolocation` SET address = :address, data = :data WHERE id = :id;");
			$statement->bindValue(":address", $this->address);
			$statement->bindValue(":id", $this->id);
			$statement->bindValue(":data", $this->data);

			if (!$statement->execute()) {
				throw new Exception($statement->errorInfo());
			}
		}
	}

	public function getAddress()
	{
		return $this->address;
	}

	/**
	 * @param string $address
	 */
	public function setAddress($address)
	{
		$this->address = $address;
	}

	public function getData()
	{
		return unserialize($this->data);
	}

	public function setData($data)
	{
		$this->data = serialize($data);
	}

	public function getCreation()
	{
		return $this->creation;
	}
}