florinp\messenger\libs\database

Complexity

Total Complexity

10

Size/Duplication

Total Lines	97
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	0

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	select()	0	10	2
A	insert()	0	15	2
A	update()	0	18	3
A	delete()	0	4	1
A	find()	0	12	1

<?php

namespace florinp\messenger\libs;

use PDO;

class database extends \PDO
{
	protected $extension_manager;

	public function __construct()
	{
		global $phpbb_root_path;

Use of global functionality is not recommended; it makes your code harder to test, and less reusable.

		$database = $phpbb_root_path.'store/messenger.db';
		parent::__construct('sqlite:'.$database);
		parent::setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);

It seems like you call parent on a different method (setAttribute() instead of __construct()). Are you sure this is correct? If so, you might want to change this to $this->setAttribute().

	}

	/**
	 * @param string $sql
	 * @return array
	 */
	public function select($sql, $array = array(), $fetchMode = PDO::FETCH_ASSOC)
	{
		$sth = $this->prepare($sql);
		foreach ($array as $key => $value) {
			$sth->bindValue("$key", $value);
		}

		$sth->execute();
		return $sth->fetchAll($fetchMode);
	}

	/**
	 * @param string $table
	 * @return bool
	 */
	public function insert($table, $data)
	{
		//ksort($data);

		$fieldNames = implode('`, `', array_keys($data));
		$fieldValues = ':'.implode(', :', array_keys($data));

		$sth = $this->prepare("INSERT INTO $table (`$fieldNames`) VALUES ($fieldValues)");

		foreach ($data as $key => $value) {
			$sth->bindValue(":$key", $value);
		}

		return $sth->execute();
	}

	/**
	 * @param string $table
	 * @param array $data
	 * @param string $where
	 * @return bool
	 */
	public function update($table, $data, $where)
	{
		ksort($data);

		$fieldDetails = NULL;
		foreach ($data as $key => $value) {
			$fieldDetails .= "`$key`=:$key,";
		}
		$fieldDetails = rtrim($fieldDetails, ',');

		$sth = $this->prepare("UPDATE $table SET $fieldDetails WHERE $where");

		foreach ($data as $key => $value) {
			$sth->bindValue(":$key", $value);
		}

		return $sth->execute();
	}

	/**
	 * @param string $table
	 * @param string $where
	 * @param int $limit
	 * @return int
	 */
	public function delete($table, $where, $limit = 1)
	{
		return $this->exec("DELETE FROM $table WHERE $where LIMIT $limit");
	}

	protected function find($path, $prefix, $suffix)
	{
		$finder = $this->extension_manager->get_finder();

		return $finder
			->set_extensions(array('phpbb/messenger'))
			->prefix($prefix)
			->suffix($suffix)
			->core_path("$path/")
			->extension_directory("/$path")
			->find();
	}
}