CustomColumn

Complexity

Total Complexity

16

Size/Duplication

Total Lines	110
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

96.23%

13 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getUri()	0	3	1
A	getEntryId()	0	3	1
A	getTableName()	0	3	1
A	getTableLinkName()	0	3	1
A	getTableLinkColumn()	0	3	1
A	getAllCustomsId()	0	3	1
A	getUriAllCustoms()	0	3	1
A	getAllTitle()	0	6	1
A	getCustomId()	0	8	2
A	getCount()	0	12	1
A	getCustomById()	0	8	2
A	getAllCustoms()	0	17	2

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

require_once('base.php');

class CustomColumn extends Base {
    const ALL_CUSTOMS_ID = "cops:custom";
    
    const SQL_COUNT_CUSTOM_VALUES = 
    	"select count(distinct {0}.id) as count
    	 from {0}
    		inner join {1} as link on {0}.id = link.{2}
    		inner join ({3}) as filter on filter.id = link.book";
    const SQL_ALL_CUSTOMS = 
    	"select {0}.id as id, {0}.value as name, count(*) as count
    	 from {0}
    		inner join {1} as link on {0}.id = link.{2}
    		inner join ({3}) as filter on filter.id = link.book
    	 group by {0}.id, {0}.value
		 order by {0}.value";

    public $id;
    public $name;
    public $customId;

    public function __construct($pid, $pname, $pcustomId) {
        $this->id = $pid;
        $this->name = $pname;
        $this->customId = $pcustomId;
    }

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

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

    public static function getTableName ($customId) {
        return "custom_column_{$customId}";
    }

    public static function getTableLinkName ($customId) {
        return "books_custom_column_{$customId}_link";
    }

    public static function getTableLinkColumn ($customId) {

The parameter $customId is not used and could be removed.

        return "value";
    }

    public static function getAllCustomsId ($customId) {
        return self::ALL_CUSTOMS_ID . ":" . $customId;
    }

    public static function getUriAllCustoms ($customId) {
        return "?page=" . parent::PAGE_ALL_CUSTOMS . "&custom={$customId}";
    }

    public static function getAllTitle ($customId) {
        $result = parent::getDb ()->prepare('select name from custom_columns where id = ?');

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

        $result->execute (array ($customId));
        $post = $result->fetchObject ();
        return $post->name;
    }

    public static function getCustomId ($lookup) {
        $result = parent::getDb ()->prepare('select id from custom_columns where label = ?');

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

        $result->execute (array ($lookup));
        if ($post = $result->fetchObject ()) {
            return $post->id;
        }
        return NULL;
    }

    public static function getCount($customId) {
    	$sql = str_format(self::SQL_COUNT_CUSTOM_VALUES, 
    			self::getTableName ($customId), 
    			self::getTableLinkName($customId),
    			self::getTableLinkColumn($customId),
    			'{0}');
        $nCustoms = parent::executeFilteredQuerySingle ($sql);

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

        $entry = new Entry (self::getAllTitle ($customId), self::getAllCustomsId ($customId),
            str_format (localize("tags.alphabetical", $nCustoms), $nCustoms), "text",
            array ( new LinkNavigation (self::getUriAllCustoms ($customId))), "", $nCustoms);
        return $entry;
    }

    public static function getCustomById ($customId, $id) {
        $result = parent::getDb ()->prepare('select id, value as name from ' . self::getTableName ($customId) . ' where id = ?');

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

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

    public static function getAllCustoms($customId) {
    	$sql = str_format(self::SQL_ALL_CUSTOMS,
    			self::getTableName ($customId),
    			self::getTableLinkName($customId),
    			self::getTableLinkColumn($customId),
    			'{0}');
    	list (, $result) = parent::executeFilteredQuery($sql, array(), -1);

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

        $entryArray = array();
        while ($post = $result->fetchObject ())
        {
            $customColumn = new CustomColumn ($post->id, $post->name, $customId);
            array_push ($entryArray, new Entry ($customColumn->name, $customColumn->getEntryId (),
                str_format (localize("bookword", $post->count), $post->count), "text",
                array ( new LinkNavigation ($customColumn->getUri ())), "", $post->count));
        }
        return $entryArray;
    }
}