Book - Code Metrics - Inspection of "Epub loader upgrade to cops refactoring" - seblucas/cops - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Pull Request — master (#424)

unknown

created 2019-03-22 14:38 UTC

Book F

↳ Parent: Project

Complexity

Total Complexity

122

Size/Duplication

Total Lines	689
Duplicated Lines	4.35 %

Coupling/Cohesion

Components	3
Dependencies	13

Test Coverage

Coverage

89.66%

Importance

Changes

Metric	Value
dl	30
loc	689
ccs	260
cts	290
cp	0.8966
rs	1.911
c	0
b	0
f	0
wmc	122
lcom	3
cbo	13

46 Methods

Rating	Name	Duplication	Size	Complexity
F	__construct()	12	68	17
A	getBookColumns()	0	11	2
A	getEntryId()	0	3	1
A	getEntryIdByLetter()	0	3	1
A	getUri()	0	3	1
A	getDetailUrl()	1	5	2
A	getTitle()	0	3	1
A	getAuthors()	0	6	2
A	getAuthorsName()	0	3	1
A	getAuthorsSort()	0	3	1
A	getPublisher()	0	6	2
A	getSerie()	0	6	2
A	getLanguages()	0	14	2
A	getTags()	0	17	3
A	getTagsName()	0	3	1
A	getDatas()	0	7	2
A	getFilterString()	0	18	4
A	GetMostInterestingDataToSendToKindle()	0	14	4
A	getDataById()	0	7	1
A	getRating()	0	13	5
A	getPubDate()	0	10	3
A	getComment()	0	15	4
A	getDataFormat()	0	6	1
B	getFilePath()	0	42	6
B	getUpdatedEpub()	0	39	6
B	getThumbnail()	0	36	7
B	getLinkArray()	0	39	7
A	getEntry()	0	5	1
A	getBookCount()	0	3	1
A	getCount()	0	18	2
A	getBooksByAuthor()	0	3	1
A	getBooksByRating()	0	3	1
A	getBooksByPublisher()	0	3	1
A	getBooksBySeries()	0	3	1
A	getBooksByTag()	0	3	1
A	getBooksByLanguage()	0	3	1
A	getBooksByCustom()	0	5	1
A	getBookById()	0	12	2
A	getBookByDataId()	0	15	2
B	getBooksByQuery()	0	23	6
A	getBooks()	0	4	1
A	getAllBooks()	17	17	2
A	getBooksByStartingLetter()	0	3	1
A	getEntryArray()	0	13	2
A	getAllRecentBooks()	0	5	1
A	getCustomColumnValues()	0	19	5

How to fix Duplicated Code Complexity

Duplicated Code

Duplicate code is one of the most pungent code smells. A rule that is often used is to re-structure code once it is duplicated in three or more places.

Common duplication problems, and corresponding solutions are:

If you have the same expression in different places: Extract expression to a method
If you have the same method in different sub-classes: Extract method, and pull up field to the parent class
If you have the same code in unrelated classes: Consider extracting the code to a new class, and injecting that class

Complex Class

Tip: Before tackling complexity, make sure that you eliminate any duplication first. This often can reduce the size of classes significantly.

Complex classes like Book often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.

Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.

While breaking up the class, it is a good idea to analyze how other classes use Book, and based on these observations, apply Extract Interface, too.

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

// Silly thing because PHP forbid string concatenation in class const
define ('SQL_BOOKS_LEFT_JOIN', 'left outer join comments on comments.book = books.id
                                left outer join books_ratings_link on books_ratings_link.book = books.id
                                left outer join ratings on books_ratings_link.rating = ratings.id ');
define ('SQL_BOOKS_ALL', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . ' order by books.sort ');
define ('SQL_BOOKS_BY_PUBLISHER', 'select {0} from books_publishers_link, books ' . SQL_BOOKS_LEFT_JOIN . '
                                                    where books_publishers_link.book = books.id and publisher = ? {1} order by publisher');
define ('SQL_BOOKS_BY_FIRST_LETTER', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
                                                    where upper (books.sort) like ? order by books.sort');
define ('SQL_BOOKS_BY_AUTHOR', 'select {0} from books_authors_link, books ' . SQL_BOOKS_LEFT_JOIN . '
                                                    left outer join books_series_link on books_series_link.book = books.id
                                                    where books_authors_link.book = books.id and author = ? {1} order by series desc, series_index asc, pubdate asc');
define ('SQL_BOOKS_BY_SERIE', 'select {0} from books_series_link, books ' . SQL_BOOKS_LEFT_JOIN . '
                                                    where books_series_link.book = books.id and series = ? {1} order by series_index');
define ('SQL_BOOKS_BY_TAG', 'select {0} from books_tags_link, books ' . SQL_BOOKS_LEFT_JOIN . '
                                                    where books_tags_link.book = books.id and tag = ? {1} order by sort');
define ('SQL_BOOKS_BY_LANGUAGE', 'select {0} from books_languages_link, books ' . SQL_BOOKS_LEFT_JOIN . '
                                                    where books_languages_link.book = books.id and lang_code = ? {1} order by sort');
define ('SQL_BOOKS_BY_CUSTOM', 'select {0} from {2}, books ' . SQL_BOOKS_LEFT_JOIN . '
                                                    where {2}.book = books.id and {2}.{3} = ? {1} order by sort');
define ('SQL_BOOKS_BY_CUSTOM_BOOL_TRUE', 'select {0} from {2}, books ' . SQL_BOOKS_LEFT_JOIN . '
                                                    where {2}.book = books.id and {2}.value = 1 {1} order by sort');
define ('SQL_BOOKS_BY_CUSTOM_BOOL_FALSE', 'select {0} from {2}, books ' . SQL_BOOKS_LEFT_JOIN . '
                                                    where {2}.book = books.id and {2}.value = 0 {1} order by sort');
define ('SQL_BOOKS_BY_CUSTOM_BOOL_NULL', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
                                                    where books.id not in (select book from {2}) {1} order by sort');
define ('SQL_BOOKS_BY_CUSTOM_RATING', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
                                                    left join {2} on {2}.book = books.id
                                                    left join {3} on {3}.id = {2}.{4}
                                                    where {3}.value = ?  order by sort');
define ('SQL_BOOKS_BY_CUSTOM_RATING_NULL', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
								                    left join {2} on {2}.book = books.id
								                    left join {3} on {3}.id = {2}.{4}
                                                    where ((books.id not in (select {2}.book from {2})) or ({3}.value = 0)) {1} order by sort');
define ('SQL_BOOKS_BY_CUSTOM_DATE', 'select {0} from {2}, books ' . SQL_BOOKS_LEFT_JOIN . '
                                                    where {2}.book = books.id and date({2}.value) = ? {1} order by sort');
define ('SQL_BOOKS_BY_CUSTOM_DIRECT', 'select {0} from {2}, books ' . SQL_BOOKS_LEFT_JOIN . '
                                                    where {2}.book = books.id and {2}.value = ? {1} order by sort');
define ('SQL_BOOKS_BY_CUSTOM_DIRECT_ID', 'select {0} from {2}, books ' . SQL_BOOKS_LEFT_JOIN . '
                                                    where {2}.book = books.id and {2}.id = ? {1} order by sort');
define ('SQL_BOOKS_QUERY', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
                                                    where (
                                                    exists (select null from authors, books_authors_link where book = books.id and author = authors.id and authors.name like ?) or
                                                    exists (select null from tags, books_tags_link where book = books.id and tag = tags.id and tags.name like ?) or
                                                    exists (select null from series, books_series_link on book = books.id and books_series_link.series = series.id and series.name like ?) or
                                                    exists (select null from publishers, books_publishers_link where book = books.id and books_publishers_link.publisher = publishers.id and publishers.name like ?) or
                                                    title like ?) {1} order by books.sort');
define ('SQL_BOOKS_RECENT', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
                                                    where 1=1 {1} order by timestamp desc limit ');
define ('SQL_BOOKS_BY_RATING', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
                                                    where books_ratings_link.book = books.id and ratings.id = ? {1} order by sort');

class Book extends Base
namespace YourVendor;

class YourClass { }
{
    const ALL_BOOKS_UUID = 'urn:uuid';
    const ALL_BOOKS_ID = 'cops:books';
    const ALL_RECENT_BOOKS_ID = 'cops:recentbooks';
    const BOOK_COLUMNS = 'books.id as id, books.title as title, text as comment, path, timestamp, pubdate, series_index, uuid, has_cover, ratings.rating';

    const SQL_BOOKS_LEFT_JOIN = SQL_BOOKS_LEFT_JOIN;
    const SQL_BOOKS_ALL = SQL_BOOKS_ALL;
    const SQL_BOOKS_BY_PUBLISHER = SQL_BOOKS_BY_PUBLISHER;
    const SQL_BOOKS_BY_FIRST_LETTER = SQL_BOOKS_BY_FIRST_LETTER;
    const SQL_BOOKS_BY_AUTHOR = SQL_BOOKS_BY_AUTHOR;
    const SQL_BOOKS_BY_SERIE = SQL_BOOKS_BY_SERIE;
    const SQL_BOOKS_BY_TAG = SQL_BOOKS_BY_TAG;
    const SQL_BOOKS_BY_LANGUAGE = SQL_BOOKS_BY_LANGUAGE;
    const SQL_BOOKS_BY_CUSTOM = SQL_BOOKS_BY_CUSTOM;
    const SQL_BOOKS_BY_CUSTOM_BOOL_TRUE = SQL_BOOKS_BY_CUSTOM_BOOL_TRUE;
    const SQL_BOOKS_BY_CUSTOM_BOOL_FALSE = SQL_BOOKS_BY_CUSTOM_BOOL_FALSE;
    const SQL_BOOKS_BY_CUSTOM_BOOL_NULL = SQL_BOOKS_BY_CUSTOM_BOOL_NULL;
    const SQL_BOOKS_BY_CUSTOM_RATING = SQL_BOOKS_BY_CUSTOM_RATING;
    const SQL_BOOKS_BY_CUSTOM_RATING_NULL = SQL_BOOKS_BY_CUSTOM_RATING_NULL;
    const SQL_BOOKS_BY_CUSTOM_DATE = SQL_BOOKS_BY_CUSTOM_DATE;
    const SQL_BOOKS_BY_CUSTOM_DIRECT = SQL_BOOKS_BY_CUSTOM_DIRECT;
    const SQL_BOOKS_BY_CUSTOM_DIRECT_ID = SQL_BOOKS_BY_CUSTOM_DIRECT_ID;
    const SQL_BOOKS_QUERY = SQL_BOOKS_QUERY;
    const SQL_BOOKS_RECENT = SQL_BOOKS_RECENT;
    const SQL_BOOKS_BY_RATING = SQL_BOOKS_BY_RATING;

    const BAD_SEARCH = 'QQQQQ';

    public $id;
    public $title;
    public $timestamp;
    public $pubdate;
    public $path;
    public $uuid;
    public $hasCover;
    public $relativePath;
    public $seriesIndex;
    public $comment;
    public $rating;
    public $datas = NULL;
    public $authors = NULL;
    public $publisher = NULL;
    public $serie = NULL;
    public $tags = NULL;
    public $languages = NULL;
    public $format = array ();
    private $coverFileName = NULL;

    public function __construct($line) {
    	global $config;
function myFunction($a, $b) {
    // Do something
}

        $this->id = $line->id;
        $this->title = $line->title;
        $this->timestamp = strtotime($line->timestamp);
        $this->pubdate = $line->pubdate;
        //$this->path = Base::getDbDirectory() . $line->path;

        //$this->relativePath = $line->path;

        // -DC- Init relative or full path
        $this->path = $line->path;
        if (!is_dir($this->path)) {
        	$this->path = Base::getDbDirectory() . $line->path;
        }
        $this->seriesIndex = $line->series_index;
        $this->comment = $line->comment;
        $this->uuid = $line->uuid;
        $this->hasCover = $line->has_cover;
        // -DC- Use cover file name
        //if (!file_exists($this->getFilePath('jpg'))) {

        //    // double check
        //    $this->hasCover = 0;

        //}
        if ($this->hasCover) {
        	if (!empty($config['calibre_database_field_cover'])) {
        		$imgDirectory = Base::getImgDirectory();
        		$this->coverFileName = $line->cover;
        		if (!file_exists($this->coverFileName)) {
        			$this->coverFileName = NULL;
        		}
        		if (empty($this->coverFileName)) {

        			$this->coverFileName = sprintf('%s%s', $imgDirectory, $line->cover);
        			if (!file_exists($this->coverFileName)) {
        				$this->coverFileName = NULL;
        			}
        		}
        		if (empty($this->coverFileName)) {
        			$data = $this->getDataById($this->id);
        			if ($data) {
        				$this->coverFileName = sprintf('%s%s/%s', $imgDirectory, $data->name, $line->cover);
        				if (!file_exists($this->coverFileName)) {
        					$this->coverFileName = NULL;
        				}
        				if (empty($this->coverFileName)) {

        					$this->coverFileName = sprintf('%s%s.jpg', $imgDirectory, $data->name);
        					if (!file_exists($this->coverFileName)) {
        						$this->coverFileName = NULL;
        					}
        				}
        			}
        		}
        	}
        	// Else try with default cover file name
        	if (empty($this->coverFileName)) {
        		$cover = $this->getFilePath("jpg");
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        		if ($cover === false || !file_exists($cover)) {
        			$cover = $this->getFilePath("png");
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        		}
        		if ($cover === false || !file_exists($cover)) {
        			$this->hasCover = 0;
        		}
        		else {
        			$this->coverFileName = $cover;
        		}
        	}
        }
        $this->rating = $line->rating;
    }

    // -DC- Get customisable book columns
    private static function getBookColumns()
    {
    	global $config;
function myFunction($a, $b) {
    // Do something
}

    	$res = self::BOOK_COLUMNS;
    	if (!empty($config['calibre_database_field_cover'])) {
    		$res = str_replace('has_cover,', 'has_cover, ' . $config['calibre_database_field_cover'] . ',', $res);
    	}

    	return $res;
    }

    public function getEntryId() {
        return self::ALL_BOOKS_UUID.':'.$this->uuid;
    }

    public static function getEntryIdByLetter ($startingLetter) {
        return self::ALL_BOOKS_ID.':letter:'.$startingLetter;
    }

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

    public function getDetailUrl () {
        $urlParam = $this->getUri();
        if (!is_null(GetUrlParam(DB))) $urlParam = addURLParameter($urlParam, DB, GetUrlParam (DB));

        return 'index.php' . $urlParam;
    }

    public function getTitle () {
        return $this->title;
    }

    /* Other class (author, series, tag, ...) initialization and accessors */

    /**
     * @return Author[]
     */
    public function getAuthors () {
        if (is_null($this->authors)) {
            $this->authors = Author::getAuthorByBookId($this->id);
        }
        return $this->authors;
    }

    public function getAuthorsName () {
        return implode(', ', array_map(function ($author) { return $author->name; }, $this->getAuthors()));
// Bad
$a = 5; $b = 6; $c = 7;

// Good
$a = 5;
$b = 6;
$c = 7;
    }

    public function getAuthorsSort () {
        return implode(', ', array_map(function ($author) { return $author->sort; }, $this->getAuthors()));
// Bad
$a = 5; $b = 6; $c = 7;

// Good
$a = 5;
$b = 6;
$c = 7;
    }

    public function getPublisher () {
        if (is_null($this->publisher)) {
            $this->publisher = Publisher::getPublisherByBookId($this->id);
        }
        return $this->publisher;
    }

    /**
     * @return Serie
     */
    public function getSerie() {
        if (is_null($this->serie)) {
            $this->serie = Serie::getSerieByBookId($this->id);
        }
        return $this->serie;
    }

    /**
     * @return string
     */
    public function getLanguages() {
        $lang = array();
        $result = parent::getDb()->prepare('select languages.lang_code
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

    protected function getSurName()
    {
        return "Gudjohnsen";
    }
}

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}
                from books_languages_link, languages
                where books_languages_link.lang_code = languages.id
                and book = ?
                order by item_order');
        $result->execute(array($this->id));
        while ($post = $result->fetchObject())
        {
            array_push($lang, Language::getLanguageString($post->lang_code));
        }
        return implode(', ', $lang);
    }

    /**
     * @return Tag[]
     */
    public function getTags() {
        if (is_null ($this->tags)) {
            $this->tags = array();

            $result = parent::getDb()->prepare('select tags.id as id, name
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

    protected function getSurName()
    {
        return "Gudjohnsen";
    }
}

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}
                from books_tags_link, tags
                where tag = tags.id
                and book = ?
                order by name');
            $result->execute(array($this->id));
            while ($post = $result->fetchObject())
            {
                array_push($this->tags, new Tag($post));
            }
        }
        return $this->tags;
    }

    public function getTagsName() {
        return implode(', ', array_map(function ($tag) { return $tag->name; }, $this->getTags()));
// Bad
$a = 5; $b = 6; $c = 7;

// Good
$a = 5;
$b = 6;
$c = 7;
    }

    /**
     * @return Data[]
     */
    public function getDatas()
    {
        if (is_null($this->datas)) {
            $this->datas = Data::getDataByBook($this);
        }
        return $this->datas;
    }

    /* End of other class (author, series, tag, ...) initialization and accessors */

    public static function getFilterString() {
        $filter = getURLParam('tag', NULL);
        if (empty($filter)) return '';

        $exists = true;
        if (preg_match("/^!(.*)$/", $filter, $matches)) {
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
            $exists = false;
            $filter = $matches[1];
        }

        $result = 'exists (select null from books_tags_link, tags where books_tags_link.book = books.id and books_tags_link.tag = tags.id and tags.name = "' . $filter . '")';

        if (!$exists) {
            $result = 'not ' . $result;
        }

        return 'and ' . $result;
    }

    public function GetMostInterestingDataToSendToKindle()

    {
        $bestFormatForKindle = array('EPUB', 'PDF', 'AZW3', 'MOBI');
        $bestRank = -1;
        $bestData = NULL;
        foreach ($this->getDatas() as $data) {
            $key = array_search($data->format, $bestFormatForKindle);
            if ($key !== false && $key > $bestRank) {
                $bestRank = $key;
                $bestData = $data;
            }
        }
        return $bestData;
    }

    public function getDataById($idData)
    {
        $reduced = array_filter($this->getDatas(), function ($data) use ($idData) {
            return $data->id == $idData;
        });
        return reset($reduced);
    }

    public function getRating() {
        if (is_null($this->rating) || $this->rating == 0) {
            return '';
        }
        $retour = '';
        for ($i = 0; $i < $this->rating / 2; $i++) {
            $retour .= '&#9733;';
        }
        for ($i = 0; $i < 5 - $this->rating / 2; $i++) {
            $retour .= '&#9734;';
        }
        return $retour;
    }

    public function getPubDate() {
        if (empty ($this->pubdate)) {
            return '';
        }
        $dateY = (int) substr($this->pubdate, 0, 4);
        if ($dateY > 102) {
            return str_pad($dateY, 4, '0', STR_PAD_LEFT);
        }
        return '';
    }

    public function getComment($withSerie = true) {
        $addition = '';
        $se = $this->getSerie ();
        if (!is_null ($se) && $withSerie) {
            $addition = $addition . '<strong>' . localize('content.series') . '</strong>' . str_format(localize('content.series.data'), $this->seriesIndex, htmlspecialchars($se->name)) . "<br />\n";
        }
        if (preg_match('/<\/(div|p|a|span)>/', $this->comment))
        {
            return $addition . html2xhtml($this->comment);
        }
        else
        {
            return $addition . htmlspecialchars($this->comment);
        }
    }

    public function getDataFormat($format) {
        $reduced = array_filter($this->getDatas(), function ($data) use ($format) {
            return $data->format == $format;
        });
        return reset($reduced);
    }

    public function getFilePath($extension, $idData = NULL, $relative = false)

    {
        /*if ($extension == 'jpg')

        {
            $file = 'cover.jpg';
        }
        else
        {
            $data = $this->getDataById($idData);
            if (!$data) return NULL;
            $file = $data->name . '.' . strtolower($data->format);
        }

        if ($relative)
        {
            return $this->relativePath.'/'.$file;
        }
        else
        {
            return $this->path.'/'.$file;
        }*/
    	if ($extension == "jpg" || $extension == "png") {
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
    		if (empty($this->coverFileName)) {
    			return $this->path . '/cover.' . $extension;
    		}
    		else {
    			$ext = strtolower(pathinfo($this->coverFileName, PATHINFO_EXTENSION));
    			if ($ext == $extension) {
    				return $this->coverFileName;
    			}
    		}
    		return false;
    	}
    	else {
    		$data = $this->getDataById($idData);
    		if (!$data) {
    			return NULL;
    		}
    		$file = $data->name . "." . strtolower($data->format);
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
    		return $this->path . '/' . $file;
    	}
    }

    public function getUpdatedEpub($idData)
    {
        global $config;
function myFunction($a, $b) {
    // Do something
}
        $data = $this->getDataById($idData);

        try
        {
            $epub = new EPub($data->getLocalPath());

            $epub->Title($this->title);
            $authorArray = array();
            foreach ($this->getAuthors() as $author) {
                $authorArray[$author->sort] = $author->name;
            }
            $epub->Authors($authorArray);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $epub->Language($this->getLanguages());
            $epub->Description ($this->getComment(false));
            $epub->Subjects($this->getTagsName());
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            // -DC- Use cover file name
            // $epub->Cover2($this->getFilePath('jpg'), 'image/jpeg');

            $epub->Cover2($this->coverFileName, 'image/jpeg');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $epub->Calibre($this->uuid);
            $se = $this->getSerie();
            if (!is_null($se)) {
                $epub->Serie($se->name);
                $epub->SerieIndex($this->seriesIndex);
            }
            $filename = $data->getUpdatedFilenameEpub();
            if ($config['cops_provide_kepub'] == '1'  && preg_match('/Kobo/', $_SERVER['HTTP_USER_AGENT'])) {
                $epub->updateForKepub();
                $filename = $data->getUpdatedFilenameKepub();
            }
            $epub->download($filename);
        }
        catch (Exception $e)
        {
            echo 'Exception : ' . $e->getMessage();
        }
    }

    public function getThumbnail($width, $height, $outputfile = NULL) {
        if (is_null($width) && is_null($height)) {
            return false;
        }

        // -DC- Use cover file name
        //$file = $this->getFilePath('jpg');

        $file = $this->coverFileName;
        // get image size
        if ($size = GetImageSize($file)) {
            $w = $size[0];
            $h = $size[1];
            //set new size
            if (!is_null($width)) {
                $nw = $width;
                if ($nw >= $w) { return false; }
                $nh = ($nw*$h)/$w;
            } else {
                $nh = $height;
                if ($nh >= $h) { return false; }
                $nw = ($nh*$w)/$h;
            }
        } else {
            return false;
        }

        //draw the image
        $src_img = imagecreatefromjpeg($file);
        $dst_img = imagecreatetruecolor($nw,$nh);
        imagecopyresampled($dst_img, $src_img, 0, 0, 0, 0, $nw, $nh, $w, $h);//resizing the image
        imagejpeg($dst_img,$outputfile,80);
        imagedestroy($src_img);
        imagedestroy($dst_img);

        return true;
    }

    public function getLinkArray ()
    {
        $linkArray = array();

        if ($this->hasCover) {
        	// -DC- Use cover file name
        	//array_push($linkArray, Data::getLink($this, 'jpg', 'image/jpeg', Link::OPDS_IMAGE_TYPE, 'cover.jpg', NULL));

        	//array_push($linkArray, Data::getLink($this, 'jpg', 'image/jpeg', Link::OPDS_THUMBNAIL_TYPE, 'cover.jpg', NULL));

        	$ext = strtolower(pathinfo($this->coverFileName, PATHINFO_EXTENSION));
        	if ($ext == 'png') {
        		array_push($linkArray, Data::getLink($this, "png", "image/png", Link::OPDS_IMAGE_TYPE, "cover.png", NULL));
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        		array_push($linkArray, Data::getLink($this, "png", "image/png", Link::OPDS_THUMBNAIL_TYPE, "cover.png", NULL));
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        	}
        	else {
        		array_push($linkArray, Data::getLink($this, 'jpg', 'image/jpeg', Link::OPDS_IMAGE_TYPE, 'cover.jpg', NULL));
        		array_push($linkArray, Data::getLink($this, "jpg", "image/jpeg", Link::OPDS_THUMBNAIL_TYPE, "cover.jpg", NULL));
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        	}
        }

        foreach ($this->getDatas() as $data)
        {
            if ($data->isKnownType())
            {
                array_push($linkArray, $data->getDataLink(Link::OPDS_ACQUISITION_TYPE, $data->format));
            }
        }

        foreach ($this->getAuthors() as $author) {
            /* @var $author Author */
            array_push($linkArray, new LinkNavigation($author->getUri(), 'related', str_format(localize('bookentry.author'), localize('splitByLetter.book.other'), $author->name)));
        }

        $serie = $this->getSerie();
        if (!is_null ($serie)) {
            array_push($linkArray, new LinkNavigation($serie->getUri(), 'related', str_format(localize('content.series.data'), $this->seriesIndex, $serie->name)));
        }

        return $linkArray;
    }


    public function getEntry() {
        return new EntryBook($this->getTitle(), $this->getEntryId(),
            $this->getComment(), 'text/html',
            $this->getLinkArray(), $this);
    }

    public static function getBookCount($database = NULL) {
        return parent::executeQuerySingle('select count(*) from books', $database);
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

    protected function getSurName()
    {
        return "Gudjohnsen";
    }
}

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}
    }

    public static function getCount() {
        global $config;
function myFunction($a, $b) {
    // Do something
}
        $nBooks = parent::executeQuerySingle('select count(*) from books');
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

    protected function getSurName()
    {
        return "Gudjohnsen";
    }
}

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}
        $result = array();
        $entry = new Entry(localize('allbooks.title'),
                          self::ALL_BOOKS_ID,
                          str_format(localize('allbooks.alphabetical', $nBooks), $nBooks), 'text',
                          array(new LinkNavigation('?page='.parent::PAGE_ALL_BOOKS)), '', $nBooks);
        array_push($result, $entry);
        if ($config['cops_recentbooks_limit'] > 0) {
            $entry = new Entry(localize('recent.title'),
                              self::ALL_RECENT_BOOKS_ID,
                              str_format(localize('recent.list'), $config['cops_recentbooks_limit']), 'text',
                              array ( new LinkNavigation ('?page='.parent::PAGE_ALL_RECENT_BOOKS)), '', $config['cops_recentbooks_limit']);
            array_push($result, $entry);
        }
        return $result;
    }

    public static function getBooksByAuthor($authorId, $n) {
        return self::getEntryArray(self::SQL_BOOKS_BY_AUTHOR, array($authorId), $n);
    }

    public static function getBooksByRating($ratingId, $n) {
        return self::getEntryArray(self::SQL_BOOKS_BY_RATING, array($ratingId), $n);
    }

    public static function getBooksByPublisher($publisherId, $n) {
        return self::getEntryArray(self::SQL_BOOKS_BY_PUBLISHER, array($publisherId), $n);
    }

    public static function getBooksBySeries($serieId, $n) {
        return self::getEntryArray(self::SQL_BOOKS_BY_SERIE, array($serieId), $n);
    }

    public static function getBooksByTag($tagId, $n) {
        return self::getEntryArray(self::SQL_BOOKS_BY_TAG, array($tagId), $n);
    }

    public static function getBooksByLanguage($languageId, $n) {
        return self::getEntryArray(self::SQL_BOOKS_BY_LANGUAGE, array($languageId), $n);
    }

    /**
     * @param $customColumn CustomColumn
     * @param $id integer
     * @param $n integer
     * @return array
     */
    public static function getBooksByCustom($customColumn, $id, $n) {
        list($query, $params) = $customColumn->getQuery($id);

        return self::getEntryArray($query, $params, $n);
    }

    public static function getBookById($bookId) {
    	$result = parent::getDb()->prepare('select ' . self::getBookColumns() . '
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

    protected function getSurName()
    {
        return "Gudjohnsen";
    }
}

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}
from books ' . self::SQL_BOOKS_LEFT_JOIN . '
where books.id = ?');
        $result->execute(array($bookId));
        while ($post = $result->fetchObject())
        {
            $book = new Book($post);
            return $book;
        }
        return NULL;
    }

    public static function getBookByDataId($dataId) {
    	$result = parent::getDb()->prepare('select ' . self::getBookColumns() . ', data.name, data.format
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

    protected function getSurName()
    {
        return "Gudjohnsen";
    }
}

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}
from data, books ' . self::SQL_BOOKS_LEFT_JOIN . '
where data.book = books.id and data.id = ?');
        $result->execute(array($dataId));
        while ($post = $result->fetchObject())
        {
            $book = new Book($post);
            $data = new Data($post, $book);
            $data->id = $dataId;
            $book->datas = array($data);
            return $book;
        }
        return NULL;
    }

    public static function getBooksByQuery($query, $n, $database = NULL, $numberPerPage = NULL) {
        $i = 0;
        $critArray = array();
        foreach (array(PageQueryResult::SCOPE_AUTHOR,
                       PageQueryResult::SCOPE_TAG,
                       PageQueryResult::SCOPE_SERIES,
                       PageQueryResult::SCOPE_PUBLISHER,
                       PageQueryResult::SCOPE_BOOK) as $key) {
            if (in_array($key, getCurrentOption('ignored_categories')) ||
                (!array_key_exists($key, $query) && !array_key_exists('all', $query))) {
                $critArray[$i] = self::BAD_SEARCH;
            }
            else {
                if (array_key_exists($key, $query)) {
                    $critArray[$i] = $query[$key];
                } else {
                    $critArray[$i] = $query["all"];
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
                }
            }
            $i++;
        }
        return self::getEntryArray(self::SQL_BOOKS_QUERY, $critArray, $n, $database, $numberPerPage);
    }

    public static function getBooks($n) {
        list ($entryArray, $totalNumber) = self::getEntryArray(self::SQL_BOOKS_ALL , array (), $n);
        return array($entryArray, $totalNumber);
    }

    public static function getAllBooks() {

        /* @var $result PDOStatement */

        list (, $result) = parent::executeQuery('select {0}
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

    protected function getSurName()
    {
        return "Gudjohnsen";
    }
}

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}
from books
group by substr (upper (sort), 1, 1)
order by substr (upper (sort), 1, 1)', 'substr (upper (sort), 1, 1) as title, count(*) as count', self::getFilterString(), array(), -1);

        $entryArray = array();
        while ($post = $result->fetchObject())
        {
            array_push($entryArray, new Entry($post->title, Book::getEntryIdByLetter($post->title),
                str_format(localize('bookword', $post->count), $post->count), 'text',
                array(new LinkNavigation('?page='.parent::PAGE_ALL_BOOKS_LETTER.'&id='. rawurlencode($post->title))), '', $post->count));
        }
        return $entryArray;
    }

    public static function getBooksByStartingLetter($letter, $n, $database = NULL, $numberPerPage = NULL) {
        return self::getEntryArray(self::SQL_BOOKS_BY_FIRST_LETTER, array($letter . '%'), $n, $database, $numberPerPage);
    }

    public static function getEntryArray($query, $params, $n, $database = NULL, $numberPerPage = NULL) {
        /* @var $totalNumber integer */
        /* @var $result PDOStatement */
    	list($totalNumber, $result) = parent::executeQuery($query, self::getBookColumns(), self::getFilterString(), $params, $n, $database, $numberPerPage);
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

    protected function getSurName()
    {
        return "Gudjohnsen";
    }
}

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}

        $entryArray = array();
        while ($post = $result->fetchObject())
        {
            $book = new Book($post);
            array_push($entryArray, $book->getEntry());
        }
        return array($entryArray, $totalNumber);
    }

    public static function getAllRecentBooks() {
        global $config;
function myFunction($a, $b) {
    // Do something
}
        list ($entryArray, ) = self::getEntryArray(self::SQL_BOOKS_RECENT . $config['cops_recentbooks_limit'], array(), -1);
        return $entryArray;
    }

    /**
     * The values of all the specified columns
     *
     * @param string[] $columns
     * @return CustomColumn[]
     */
    public function getCustomColumnValues($columns, $asArray = false) {
        $result = array();

        foreach ($columns as $lookup) {
            $col = CustomColumnType::createByLookup($lookup);
            if (!is_null($col)) {
                $cust = $col->getCustomByBook($this);
                if (!is_null($cust)) {
                    if ($asArray) {
                        array_push($result, $cust->toArray());
                    } else {
                        array_push($result, $cust);
                    }
                }
            }
        }

        return $result;
    }
}


Pull Request — master (#424)

Book F

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

46 Methods

How to fix Duplicated Code Complexity

Duplicated Code

Complex Class

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1			<?php
2			/**
3			* COPS (Calibre OPDS PHP Server) class file
4			*
5			* @license GPL 2 (http://www.gnu.org/licenses/gpl.html)
6			* @author Sébastien Lucas <[email protected]>
7			*/
8
9			// Silly thing because PHP forbid string concatenation in class const
10			define ('SQL_BOOKS_LEFT_JOIN', 'left outer join comments on comments.book = books.id
11			left outer join books_ratings_link on books_ratings_link.book = books.id
12			left outer join ratings on books_ratings_link.rating = ratings.id ');
13			define ('SQL_BOOKS_ALL', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . ' order by books.sort ');
14			define ('SQL_BOOKS_BY_PUBLISHER', 'select {0} from books_publishers_link, books ' . SQL_BOOKS_LEFT_JOIN . '
15			where books_publishers_link.book = books.id and publisher = ? {1} order by publisher');
16			define ('SQL_BOOKS_BY_FIRST_LETTER', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
17			where upper (books.sort) like ? order by books.sort');
18			define ('SQL_BOOKS_BY_AUTHOR', 'select {0} from books_authors_link, books ' . SQL_BOOKS_LEFT_JOIN . '
19			left outer join books_series_link on books_series_link.book = books.id
20			where books_authors_link.book = books.id and author = ? {1} order by series desc, series_index asc, pubdate asc');
21			define ('SQL_BOOKS_BY_SERIE', 'select {0} from books_series_link, books ' . SQL_BOOKS_LEFT_JOIN . '
22			where books_series_link.book = books.id and series = ? {1} order by series_index');
23			define ('SQL_BOOKS_BY_TAG', 'select {0} from books_tags_link, books ' . SQL_BOOKS_LEFT_JOIN . '
24			where books_tags_link.book = books.id and tag = ? {1} order by sort');
25			define ('SQL_BOOKS_BY_LANGUAGE', 'select {0} from books_languages_link, books ' . SQL_BOOKS_LEFT_JOIN . '
26			where books_languages_link.book = books.id and lang_code = ? {1} order by sort');
27			define ('SQL_BOOKS_BY_CUSTOM', 'select {0} from {2}, books ' . SQL_BOOKS_LEFT_JOIN . '
28			where {2}.book = books.id and {2}.{3} = ? {1} order by sort');
29			define ('SQL_BOOKS_BY_CUSTOM_BOOL_TRUE', 'select {0} from {2}, books ' . SQL_BOOKS_LEFT_JOIN . '
30			where {2}.book = books.id and {2}.value = 1 {1} order by sort');
31			define ('SQL_BOOKS_BY_CUSTOM_BOOL_FALSE', 'select {0} from {2}, books ' . SQL_BOOKS_LEFT_JOIN . '
32			where {2}.book = books.id and {2}.value = 0 {1} order by sort');
33			define ('SQL_BOOKS_BY_CUSTOM_BOOL_NULL', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
34			where books.id not in (select book from {2}) {1} order by sort');
35			define ('SQL_BOOKS_BY_CUSTOM_RATING', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
36			left join {2} on {2}.book = books.id
37			left join {3} on {3}.id = {2}.{4}
38			where {3}.value = ? order by sort');
39			define ('SQL_BOOKS_BY_CUSTOM_RATING_NULL', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
40			left join {2} on {2}.book = books.id

seblucas / cops

Pull Request — master (#424)

Book F

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

46 Methods

How to fix Duplicated Code Complexity

Duplicated Code

Complex Class

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

Duplication Side-by-Side

Filter issues like