library\search\SearchDbConnected

Complexity

Total Complexity

7

Size/Duplication

Total Lines	56
Duplicated Lines	14.29 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	initializeDb()	0	6	2
A	configureDatabase()	0	7	1
A	isDatabaseConfigured()	0	7	1
A	getSearchDbHandle()	8	8	2

<?php
/**
 * User: jensk
 * Date: 21-2-2017
 * Time: 17:05
 */

namespace library\search;


use library\storage\JsonStorage;

/**
 * Abstract Class SearchDbConnected
 * Handles connection with the search index database
 * @package library\search
 */
abstract class SearchDbConnected
{
	/**
	 * @var resource
	 */
	protected $searchDbHandle;

	/**
	 * @var \library\storage\JsonStorage
	 */
	protected $storage;

	/**
	 * Indexer constructor.
	 *
	 * @param \library\storage\JsonStorage $storage
	 */
	public function __construct(JsonStorage $storage)
	{
		$this->storageDir = $storage->getStorageDir();

The property storageDir does not seem to exist. Did you mean storage?

		$this->storage = $storage;
		$this->initializeDb();
	}

	protected function initializeDb()
	{
		if (!$this->isDatabaseConfigured()) {
			$this->configureDatabase();
		}
	}

	protected function configureDatabase()
	{
		$db = $this->getSearchDbHandle();
		$sqlPath = __DIR__ . DIRECTORY_SEPARATOR . '../cc/install/search.sql';
		$searchSql = file_get_contents($sqlPath);
		$db->exec($searchSql);
	}

	protected function isDatabaseConfigured()
	{
		$db = $this->getSearchDbHandle();
		$stmt = $db->query('SELECT name FROM sqlite_master WHERE type=\'table\' AND name=\'term_count\';');
		$result = $stmt->fetchAll();
		return !empty($result);
	}

	protected function getSearchDbHandle()

This method seems to be duplicated in your project.

	{
		if ($this->searchDbHandle === null) {
			$path = __DIR__ . DIRECTORY_SEPARATOR . '..' . DIRECTORY_SEPARATOR . '..' . DIRECTORY_SEPARATOR . $this->storageDir . DIRECTORY_SEPARATOR;

The property storageDir does not seem to exist. Did you mean storage?

			$this->searchDbHandle = new \PDO('sqlite:' . $path . 'search.db');

It seems like new \PDO('sqlite:' . $path . 'search.db') of type object<PDO> is incompatible with the declared type resource of property $searchDbHandle.

		}
		return $this->searchDbHandle;
	}
}