Serie

Complexity

Total Complexity

10

Size/Duplication

Total Lines	55
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

96.3%

Importance

Changes	1
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	4	4	1
A	getUri()	3	3	1
A	getEntryId()	3	3	1
A	getCount()	4	4	1
A	getSerieByBookId()	10	10	2
A	getSerieById()	8	8	2
A	getAllSeries()	3	3	1
A	getAllSeriesByQuery()	3	3	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 Serie extends Base {

This class seems to be duplicated in your project.

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

    const ALL_SERIES_ID = "cops:series";

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

    const SERIES_COLUMNS = "series.id as id, series.name as name, series.sort as sort, count(*) as count";

The string literal series.id as id, series.name as name, series.sort as sort, count(*) as count does not require double quotes, as per coding-style, please use single quotes.

    const SQL_ALL_SERIES = "select {0} from series, books_series_link where series.id = series group by series.id, series.name, series.sort order by series.sort";

The string literal select {0} from series, books_series_link where series.id = series group by series.id, series.name, series.sort order by series.sort does not require double quotes, as per coding-style, please use single quotes.

    const SQL_SERIES_FOR_SEARCH = "select {0} from series, books_series_link where series.id = series and upper (series.name) like ? group by series.id, series.name, series.sort order by series.sort";

The string literal select {0} from series, books_series_link where series.id = series and upper (series.name) like ? group by series.id, series.name, series.sort order by series.sort does not require double quotes, as per coding-style, please use single quotes.

    public $id;
    public $name;

    public function __construct($post) {
        $this->id = $post->id;
        $this->name = $post->name;
    }

    public function getUri () {
        return "?page=".parent::PAGE_SERIE_DETAIL."&id=$this->id";

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

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

    }

    public function getEntryId () {
        return self::ALL_SERIES_ID.":".$this->id;

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

    }

    public static function getCount() {
        // str_format (localize("series.alphabetical", count(array))

50% of this comment could be valid code. Did you maybe forget this after debugging?

        return parent::getCountGeneric ("series", self::ALL_SERIES_ID, parent::PAGE_ALL_SERIES);

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

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

    }

    public static function getSerieByBookId ($bookId) {
        $result = parent::getDb ()->prepare('select  series.id as id, name

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

from books_series_link, series
where series.id = series and book = ?');
        $result->execute (array ($bookId));
        if ($post = $result->fetchObject ()) {
            return new Serie ($post);
        }
        return NULL;
    }

    public static function getSerieById ($serieId) {
        $result = parent::getDb ()->prepare('select id, name  from series where id = ?');

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

        $result->execute (array ($serieId));
        if ($post = $result->fetchObject ()) {
            return new Serie ($post);
        }
        return NULL;
    }

    public static function getAllSeries() {
        return Base::getEntryArrayWithBookNumber (self::SQL_ALL_SERIES, self::SERIES_COLUMNS, array (), "Serie");

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

    }

    public static function getAllSeriesByQuery($query) {
        return Base::getEntryArrayWithBookNumber (self::SQL_SERIES_FOR_SEARCH, self::SERIES_COLUMNS, array ('%' . $query . '%'), "Serie");

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

    }
}