CustomColumnTypeSeries

Complexity

Total Complexity

13

Size/Duplication

Total Lines	94
Duplicated Lines	18.09 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

97.56%

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getTableName()	0	4	1
A	getTableLinkName()	0	4	1
A	getTableLinkColumn()	0	4	1
A	getQuery()	0	5	1
A	getCustom()	0	9	2
A	getAllCustomValuesFromDatabase()	17	17	2
A	getDescription()	0	4	1
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 CustomColumnTypeSeries 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_SERIES);
    }

    /**
     * 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.

    }

    /**
     * Get the name of the linking sqlite table for this column
     * (or NULL if there is no linktable)
     *
     * @return string|null
     */
    private function getTableLinkName()
    {
        return "books_custom_column_{$this->customId}_link";

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

    }

    /**
     * Get the name of the linking column in the linktable
     *
     * @return string|null
     */
    private function getTableLinkColumn()
    {
        return "value";

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

    }

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

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)
    {
        $result = Base::getDb()->prepare(str_format("SELECT id, value AS name FROM {0} WHERE id = ?", $this->getTableName()));

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

        $result->execute(array($id));
        if ($post = $result->fetchObject()) {
            return new CustomColumn($id, $post->name, $this);
        }
        return NULL;
    }

    protected function getAllCustomValuesFromDatabase()

This method seems to be duplicated in your project.

    {
        $queryFormat = "SELECT {0}.id AS id, {0}.value AS name, count(*) AS count FROM {0}, {1} WHERE {0}.id = {1}.{2} GROUP BY {0}.id, {0}.value ORDER BY {0}.value";

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

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

        $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->name, $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()
    {
        return str_format(localize("customcolumn.description.series", $this->getDistinctValueCount()), $this->getDistinctValueCount());

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

    }

    public function getCustomByBook($book)
    {
        $queryFormat = "SELECT {0}.id AS id, {1}.{2} AS name, {1}.extra AS extra FROM {0}, {1} WHERE {0}.id = {1}.{2} AND {1}.book = {3}";

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

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

        $result = Base::getDb()->query($query);
        if ($post = $result->fetchObject()) {
            return new CustomColumn($post->id, $post->name . " [" . $post->extra . "]", $this);

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

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

        }
        return new CustomColumn(NULL, "", $this);

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

    }

    public function isSearchable()
    {
        return true;
    }
}