CustomColumnTypeInteger

Complexity

Total Complexity

12

Size/Duplication

Total Lines	70
Duplicated Lines	24.29 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

100%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getTableName()	0	4	1
A	getQuery()	0	5	1
A	getCustom()	0	4	1
A	getAllCustomValuesFromDatabase()	17	17	2
A	getDescription()	0	6	3
A	getCustomByBook()	0	11	2
A	isSearchable()	0	4	1

<?php
/**
 * COPS (Calibre OPDS PHP Server) class file
 *
 * @license    GPL 2 (http://www.gnu.org/licenses/gpl.html)
 * @author     Sébastien Lucas <sebastien@slucas.fr>
 */

class CustomColumnTypeInteger extends CustomColumnType

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    protected function __construct($pcustomId)
    {
        parent::__construct($pcustomId, self::CUSTOM_TYPE_INT);
    }

    /**
     * Get the name of the sqlite table for this column
     *
     * @return string|null
     */
    private function getTableName()
    {
        return "custom_column_{$this->customId}";

As per coding-style, please use concatenation or sprintf for the variable $this instead of interpolation.

    }

    public function getQuery($id)
    {
        $query = str_format(Book::SQL_BOOKS_BY_CUSTOM_DIRECT, "{0}", "{1}", $this->getTableName());

The string literal {0} does not require double quotes, as per coding-style, please use single quotes.

The string literal {1} does not require double quotes, as per coding-style, please use single quotes.

        return array($query, array($id));
    }

    public function getCustom($id)
    {
        return new CustomColumn($id, $id, $this);
    }

    protected function getAllCustomValuesFromDatabase()

This method seems to be duplicated in your project.

    {
        $queryFormat = "SELECT value AS id, count(*) AS count FROM {0} GROUP BY value";

The string literal SELECT value AS id, count(*) AS count FROM {0} GROUP BY value does not require double quotes, as per coding-style, please use single quotes.

        $query = str_format($queryFormat, $this->getTableName());

        $result = Base::getDb()->query($query);
        $entryArray = array();
        while ($post = $result->fetchObject()) {
            $entryPContent = str_format(localize("bookword", $post->count), $post->count);

The string literal bookword does not require double quotes, as per coding-style, please use single quotes.

            $entryPLinkArray = array(new LinkNavigation($this->getUri($post->id)));

            $entry = new Entry($post->id, $this->getEntryId($post->id), $entryPContent, $this->datatype, $entryPLinkArray, "", $post->count);

The string literal does not require double quotes, as per coding-style, please use single quotes.

            array_push($entryArray, $entry);
        }
        return $entryArray;
    }

    public function getDescription()
    {
        $desc = $this->getDatabaseDescription();
        if ($desc === NULL || empty($desc)) $desc = str_format(localize("customcolumn.description"), $this->getTitle());

The string literal customcolumn.description does not require double quotes, as per coding-style, please use single quotes.

        return $desc;
    }

    public function getCustomByBook($book)
    {
        $queryFormat = "SELECT {0}.value AS value FROM {0} WHERE {0}.book = {1}";

The string literal SELECT {0}.value AS value FROM {0} WHERE {0}.book = {1} does not require double quotes, as per coding-style, please use single quotes.

        $query = str_format($queryFormat, $this->getTableName(), $book->id);

        $result = Base::getDb()->query($query);
        if ($post = $result->fetchObject()) {
            return new CustomColumn($post->value, $post->value, $this);
        }
        return new CustomColumn(NULL, localize("customcolumn.int.unknown"), $this);

The string literal customcolumn.int.unknown does not require double quotes, as per coding-style, please use single quotes.

    }

    public function isSearchable()
    {
        return true;
    }
}