File ThumcnoServer.php has 1131 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php

namespace Tacnoman;

//These are now disabled by default because the file sizes of PNGs (and GIFs) are much smaller than we used to generate.

Function processImageAndWriteToCache has a Cognitive Complexity of 85 (exceeds 5 allowed). Consider refactoring.

Open

    protected function processImageAndWriteToCache($localImage)
    {
        $config = Config::getInstance();
        $sData = getimagesize($localImage);
        $origType = $sData[2];

• Read up
  Read up

Method processImageAndWriteToCache has 249 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected function processImageAndWriteToCache($localImage)
    {
        $config = Config::getInstance();
        $sData = getimagesize($localImage);
        $origType = $sData[2];

ThumcnoServer has 40 functions (exceeds 20 allowed). Consider refactoring.

Open

class ThumcnoServer
{
    public $src = '';
    public $is404 = false;
    public $docRoot = '';

Function getLocalImagePath has a Cognitive Complexity of 24 (exceeds 5 allowed). Consider refactoring.

Open

    protected function getLocalImagePath($src)
    {
        $src = ltrim($src, '/'); //strip off the leading '/'
        if (!$this->docRoot) {
            Logger::error('We have no document root set, so as a last resort, lets check if the image is in the current dir and serve that.');

• Read up
  Read up

Function getURL has a Cognitive Complexity of 23 (exceeds 5 allowed). Consider refactoring.

Open

    protected function getURL($url, $tempfile)
    {
        $config = Config::getInstance();
        $this->lastURLError = false;
        $url = preg_replace('/ /', '%20', $url);

• Read up
  Read up

Method getURL has 59 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected function getURL($url, $tempfile)
    {
        $config = Config::getInstance();
        $this->lastURLError = false;
        $url = preg_replace('/ /', '%20', $url);

Function validateExternalImage has a Cognitive Complexity of 17 (exceeds 5 allowed). Consider refactoring.

Open

    public function validateExternalImage()
    {
        $config = Config::getInstance();
        if ($this->isURL && (!$config->appConfigs['allow_external'])) {
            $this->error('You are not allowed to fetch images from an external website.');

• Read up
  Read up

Function cleanCache has a Cognitive Complexity of 17 (exceeds 5 allowed). Consider refactoring.

Open

    protected function cleanCache()
    {
        $config = Config::getInstance();
        if ($config->appConfigs['file_cache_time_between_cleans'] < 0) {
            return;

• Read up
  Read up

Method getLocalImagePath has 52 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected function getLocalImagePath($src)
    {
        $src = ltrim($src, '/'); //strip off the leading '/'
        if (!$this->docRoot) {
            Logger::error('We have no document root set, so as a last resort, lets check if the image is in the current dir and serve that.');

Function tryServerCache has a Cognitive Complexity of 15 (exceeds 5 allowed). Consider refactoring.

Open

    protected function tryServerCache()
    {
        $config = Config::getInstance();
        Logger::info('Trying server cache');
        if (file_exists($this->cachefile)) {

• Read up
  Read up

Method serveWebshot has 46 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected function serveWebshot()
    {
        Logger::notice('Starting serveWebshot');
        $instr = 'Please follow the instructions at http://code.google.com/p/timthumb/ to set your server up for taking website screenshots.';
        if (!is_file(WEBSHOT_CUTYCAPT)) {

Function handleErrors has a Cognitive Complexity of 14 (exceeds 5 allowed). Consider refactoring.

Open

    protected function handleErrors()
    {
        if ($this->haveErrors()) {
            $config = Config::getInstance();
            if ($config->appConfigs['not_found_image'] && $this->is404()) {

• Read up
  Read up

Function tryBrowserCache has a Cognitive Complexity of 14 (exceeds 5 allowed). Consider refactoring.

Open

    protected function tryBrowserCache()
    {
        $config = Config::getInstance();
        if ($config->appConfigs['browser_cache_disable']) {
            Logger::debug('Browser caching is disabled');

• Read up
  Read up

Method tryBrowserCache has 38 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected function tryBrowserCache()
    {
        $config = Config::getInstance();
        if ($config->appConfigs['browser_cache_disable']) {
            Logger::debug('Browser caching is disabled');

Function getIP has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.

Open

    protected function getIP()
    {
        $rem = @$_SERVER['REMOTE_ADDR'];
        $ff = @$_SERVER['HTTP_X_FORWARDED_FOR'];
        $ci = @$_SERVER['HTTP_CLIENT_IP'];

• Read up
  Read up

Method serveCacheFile has 33 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected function serveCacheFile()
    {
        Logger::notice("Serving {$this->cachefile}");
        if (!is_file($this->cachefile)) {
            $this->error("serveCacheFile called in thumcno but we couldn't find the cached file.");

Method cleanCache has 33 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected function cleanCache()
    {
        $config = Config::getInstance();
        if ($config->appConfigs['file_cache_time_between_cleans'] < 0) {
            return;

Function run has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.

Open

    public function run()
    {
        $config = Config::getInstance();
        if ($this->isURL) {
            if (!$config->appConfigs['allow_external']) {

• Read up
  Read up

Method tryServerCache has 31 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected function tryServerCache()
    {
        $config = Config::getInstance();
        Logger::info('Trying server cache');
        if (file_exists($this->cachefile)) {

Method serveExternalImage has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected function serveExternalImage()
    {
        if (!preg_match('/^https?:\/\/[a-zA-Z0-9\-\.]+/i', $this->src)) {
            $this->error('Invalid URL supplied.');

Consider simplifying this complex logical expression.

Open

        if ($imgType == 'png' && OPTIPNG_ENABLED && OPTIPNG_PATH && @is_file(OPTIPNG_PATH)) {
            $exec = OPTIPNG_PATH;
            Logger::debug("optipng'ing $tempfile");
            $presize = filesize($tempfile);
            $out = `$exec -o1 $tempfile`; //you can use up to -o7 but it really slows things down

Function calcDocRoot has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    protected function calcDocRoot()
    {
        $docRoot = @$_SERVER['DOCUMENT_ROOT'];
        if (defined('LOCAL_FILE_BASE_DIRECTORY')) {
            $docRoot = LOCAL_FILE_BASE_DIRECTORY;

• Read up
  Read up

Function setCacheDirectory has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    protected function setCacheDirectory()
    {
        $config = Config::getInstance();
        if ($config->appConfigs['file_cache_directory']) {
            if (!is_dir($config->appConfigs['file_cache_directory'])) {

• Read up
  Read up

Function serveWebshot has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    protected function serveWebshot()
    {
        Logger::notice('Starting serveWebshot');
        $instr = 'Please follow the instructions at http://code.google.com/p/timthumb/ to set your server up for taking website screenshots.';
        if (!is_file(WEBSHOT_CUTYCAPT)) {

• Read up
  Read up

Avoid too many return statements within this method.

Open

                    return $real;

Avoid too many return statements within this method.

Open

                return false;

Avoid too many return statements within this method.

Open

        return false;

Avoid too many return statements within this method.

Open

            return $this->error('Could not get a lock for writing.');

Avoid too many return statements within this method.

Open

        return false;

Avoid too many return statements within this method.

Open

                return false;

Avoid too many return statements within this method.

Open

                return true;

Avoid too many return statements within this method.

Open

            return true;

Avoid too many return statements within this method.

Open

            return $this->serveCacheFile();

Avoid too many return statements within this method.

Open

                return $ci;

Avoid too many return statements within this method.

Open

            return 'UNKNOWN';

Avoid too many return statements within this method.

Open

            return $this->error('Could not open the lockfile for writing an image.');

Avoid too many return statements within this method.

Open

        return true;

Avoid too many return statements within this method.

Open

                return false;

Avoid too many return statements within this method.

Open

            return false;

Avoid too many return statements within this method.

Open

            return false;

Avoid too many return statements within this method.

Open

            return false;

Avoid too many return statements within this method.

Open

            return true;

Avoid too many return statements within this method.

Open

            return false;

Avoid too many return statements within this method.

Open

                return $ff;

Function serveCacheFile has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

    protected function serveCacheFile()
    {
        Logger::notice("Serving {$this->cachefile}");
        if (!is_file($this->cachefile)) {
            $this->error("serveCacheFile called in thumcno but we couldn't find the cached file.");

• Read up
  Read up