Log

Complexity

Total Complexity

20

Size/Duplication

Total Lines	151
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

15 Methods

Rating	Name	Duplication	Size	Complexity
A	setObjectType()	0	3	1
A	getUserObject()	0	3	1
A	getObjectId()	0	3	1
A	delete()	0	3	1
A	setUser()	0	7	2
A	getUser()	0	3	1
A	setComment()	0	3	1
A	setAction()	0	3	1
A	save()	0	25	3
A	setObjectId()	0	3	1
A	getObjectDescription()	0	16	3
A	getTimestamp()	0	3	1
A	getComment()	0	3	1
A	getObjectType()	0	3	1
A	getAction()	0	3	1

<?php

/**
 * Log short summary.
 *
 * Log description.
 *
 * @version 1.0
 * @author stwalkerster
 */
class Log extends DataObject
{
	private $objectid;
	private $objecttype;
	private $user;
	private $action;
	private $timestamp;
	private $comment;

	public function save()
	{
		if ($this->isNew) {
			$statement = $this->dbObject->prepare(<<<SQL
                INSERT INTO log (objectid, objecttype, user, action, timestamp, comment) 
                VALUES (:id, :type, :user, :action, CURRENT_TIMESTAMP(), :comment);
SQL
			);

			$statement->bindValue(":id", $this->objectid);
			$statement->bindValue(":type", $this->objecttype);
			$statement->bindValue(":user", $this->user);
			$statement->bindValue(":action", $this->action);
			$statement->bindValue(":comment", $this->comment);

			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 {
			throw new Exception("Updating logs is not available");
		}
	}

	public function delete()
	{
		throw new Exception("Deleting logs is not available.");
	}

	public function getObjectId()
	{
		return $this->objectid;
	}

	/**
	 * Summary of setObjectId
	 * @param int $objectId
	 */
	public function setObjectId($objectId)
	{
		$this->objectid = $objectId;
	}

	public function getObjectType()
	{
		return $this->objecttype;
	}

	/**
	 * Summary of setObjectType
	 * @param string $objectType
	 */
	public function setObjectType($objectType)
	{
		$this->objecttype = $objectType;
	}

	public function getUser()
	{
		return $this->user;
	}
	
	/**
	 * Summary of getUserObject
	 * @return User|null
	 */
	public function getUserObject()
	{
		return User::getById($this->user, $this->dbObject);
	}

	/**
	 * Summary of setUser
	 * @param User $user
	 */
	public function setUser($user)
	{
		if (is_a($user, "User")) {
			$this->user = $user->getId();   
		}
		else {
			$this->user = $user;   
		}
	}

	public function getAction()
	{
		return $this->action;
	}

	/**
	 * Summary of setAction
	 * @param string $action 
	 */
	public function setAction($action)
	{
		$this->action = $action;
	}

	public function getTimestamp()
	{
		return $this->timestamp;
	}

	public function getComment()
	{
		return $this->comment;
	}

	/**
	 * Summary of setComment
	 * @param string $comment 
	 */
	public function setComment($comment)
	{
		$this->comment = $comment;
	}
	
	/**
	 * Let's be really sneaky here, and fake this to the object description of the logged object.
	 * @return string
	 */
	public function getObjectDescription()
	{
		$type = $this->objecttype;
		
		if ($type == "") {
			return "";
		}

		/** @var DataObject $object */
		$object = $type::getById($this->objectid, $this->dbObject);

		if ($object === false) {

The condition $object === false is always false.

			return '[' . $this->objecttype . " " . $this->objectid . ']';
		}
		
		return $object->getObjectDescription();
	}
}