BaseWidget::addHtmlComments() - Code Metrics - Inspection of "- debug and code refactoring" - imanghafoori1/laravel-widgetize - Measure and Improve Code Quality continuously with Scrutinizer

Completed
Push — master ( e7c634...2c7ea9 )

by Iman
created 2017-03-27 22:24 UTC
BaseWidget::addHtmlComments() A

↳ Parent: BaseWidget
Complexity

Conditions	1
Paths	1
Size

Total Lines	4
Code Lines	2
Duplication

Lines	0
Ratio	0 %
Importance

Changes
Metric	Value
dl	0
loc	4
rs	10
c	0
b	0
f	0
cc	1
eloc	2
nc	1
nop	0
<?php


namespace Imanghafoori\Widgets;

abstract class BaseWidget
{
    protected $template = null;
    protected $minifyOutput = true;
    protected $cacheLifeTime = 'env_default';
    protected $contextAs = '$data';
    protected $presenter = 'default';
    protected $controller = null;
    protected $cacheTags = null;
    private $html;
    private $viewData;

    /**
     * BaseWidget constructor.
     */
    public function __construct()
    {
        $this->normalizeControllerMethod();
        $this->normalizePresenterName();
        $this->normalizeTemplateName();
        $this->normalizeContextAs();
        $this->normalizeCacheLifeTime();
        $this->normalizeCacheTags();
    }

    /**
     * Figures out which method should be called as the controller.
     * @return null
     */
    private function normalizeControllerMethod()
    {
        // If the user has explicitly declared controller class path on the sub-class
        if ($this->controller === null) {
            if (!method_exists($this, 'data')) {
                throw new \BadMethodCallException("'data' method not found on " . get_called_class());
            }
            // We decide to call data method on widget object.
            $this->controller = [$this, 'data'];
        } else {
            // If the user has specified the controller class path
            if (!class_exists($this->controller)) {
                throw new \InvalidArgumentException("Controller class: [{$this->controller}] not found.");
            }

            // we decide to call data method on that.
            $this->controller = ($this->controller) . '@data';
        }
    }

    /**
     * Figures out which method should be called as the presenter
     * @return null
     */
    private function normalizePresenterName()
    {
        if ($this->presenter === 'default') {
            $presenter = get_called_class() . 'Presenter';

            if (class_exists($presenter)) {
                $this->presenter = $presenter . '@presenter';
            } else {
                $this->presenter = null;
            }
        } else {
            if (class_exists($this->presenter) === false) {
                throw new \InvalidArgumentException("Presenter Class [{$this->presenter}] not found.");
            }
            $this->presenter = $this->presenter . '@present';
        }

    }

    /**
     * Figures out which template to render.
     * @return null
     */
    private function normalizeTemplateName()
    {
        // class name without namespace.
        $className = str_replace('App\\Widgets\\', '', get_called_class());
        // replace slashes with dots
        $className = str_replace(['\\', '/'], '.', $className);

        if ($this->template === null) {
            $this->template = 'Widgets::' . $className . 'View';
        }

        if (!view()->exists($this->template)) {
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            throw new \InvalidArgumentException("View file [{$className}View] not found by: '" . get_called_class() . " '");
        }
    }

    /**
     * Figures out what the variable name should be in view file.
     * @return null
     */
    private function normalizeContextAs()
    {
        // removes the $ sign.
        $this->contextAs = str_replace('$', '', (string)$this->contextAs);
    }

    /**
     * ّFigures out how long the cache life time should be.
     * @return null
     */
    private function normalizeCacheLifeTime()
    {
        if ($this->cacheLifeTime === 'env_default') {
            $this->cacheLifeTime = (int)(env('WIDGET_DEFAULT_CACHE_LIFETIME', 0));
$answer = 42;

$correct = false;

$correct = (bool) $answer;
        };
    }

    /**
     * ّFigures out what the cache tags should be.
     * @return null
     */
    private function normalizeCacheTags()
    {
        if ($this->cacheShouldBeTagged()) {
            if (is_string($this->cacheTags)) {
                $this->cacheTags = [$this->cacheTags];
            }

            if (!is_array($this->cacheTags)) {
                throw new \InvalidArgumentException('Cache Tags should be of type String or Array.');
            }
        }
    }

    /**
     * Determine whether cache tags should be applied or not
     * @return bool
     */
    private function cacheShouldBeTagged()
    {
        return !in_array(env('CACHE_DRIVER', 'file'), ['file', 'database']) and $this->cacheTags;
// Logical operators have lower precedence:
$f = false or true;

// is executed like this:
($f = false) or true;


// Boolean operators have higher precedence:
$f = false || true;

// is executed like this:
$f = (false || true);
    }

    /**
     * This method is called when you try to invoke the object like a function in blade files.
     * like this : {!! $myWidgetObj('param1') !!}
     * @param array $args
     * @return string
     */
    public function __invoke(...$args)
    {
        return $this->renderWidget(...$args);
    }

    /**
     * @param array $args
     * @return string
     */
    private function renderWidget(...$args)
    {
        try {
            $html = $this->generateHtml(...$args);
        } catch (\Exception $e) {
            return app()->make(ExceptionHandler::class)->render(app('request'), $e)->send();
        }
        return $html;
    }

    /**
     * It tries to get the html from cache if possible, otherwise generates it.
     * @param array ...$args
     * @return string
     */
    private function generateHtml(...$args)
    {
        // Everything inside this function is executed only when the cache is not available.
        $expensivePhpCode = function () use ($args) {
            $this->prepareDataForView($args);

            // render the template with the resulting data.
            return $this->renderTemplate();
        };

        $key = $this->makeCacheKey($args);

        // We first try to get the output from the cache before trying to run the expensive $expensivePhpCode...
        if ($this->widgetShouldUseCache()) {
            return $this->cacheResult($key, $expensivePhpCode);
        }

        return $expensivePhpCode();
    }

    /**
     * @param $args
     * @return null
     */
    private function prepareDataForView($args)
    {
        // Here we call the data method on the widget class.
        $viewData = \App::call($this->controller, $args);

        if (($this->presenter)) {
            // We make an object and call the `present` method on it.
            // Piping the data through the presenter before sending it to view.
            $viewData = \App::call($this->presenter, [$viewData]);
        }

        $this->viewData = $viewData;
    }

    private function renderTemplate()
    {
        // Here we render the view file to raw html.
        $this->html = view($this->template, [$this->contextAs => $this->viewData])->render();
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}

        // We try to minify the html before storing it in cache to save space.
        if ($this->widgetShouldBeMinified()) {
            $this->minifyHtml();
        }

        // We add some HTML comments before and after the widget output
        // So then, we will be able to easily identify the widget in browser's developer tool.
        if (env('WIDGET_IDENTIFIER', true) and env('APP_ENV', 'production') === 'local') {
// Logical operators have lower precedence:
$f = false or true;

// is executed like this:
($f = false) or true;


// Boolean operators have higher precedence:
$f = false || true;

// is executed like this:
$f = (false || true);
            $this->addIdentifierToHtml();
        }

        return $this->html;
    }

    /**
     * @return bool
     */
    private function widgetShouldBeMinified()
    {
        return env('WIDGET_MINIFICATION', false) or app()->environment('production');
// Logical operators have lower precedence:
$f = false or true;

// is executed like this:
($f = false) or true;


// Boolean operators have higher precedence:
$f = false || true;

// is executed like this:
$f = (false || true);
    }

    /**
     * @return null
     */
    private function minifyHtml()
    {
        $replace = [
            '<!--(.*?)-->' => '', //remove comments
            "/<\?php/" => '<?php ',
            "/\n([\S])/" => '$1',
            "/\r/" => '', // remove carrage return
            "/\n/" => '', // remove new lines
            "/\t/" => '', // remove tab
            "/\s+/" => ' ', // remove spaces
        ];

        $this->html = preg_replace(array_keys($replace), array_values($replace), $this->html);

    }

    private function addIdentifierToHtml()
    {
        $this->addDebugInfo();
        $this->addHtmlComments();
    }

    /**
     * Generates a string of current cache configurations.
     * @return string
     */
    private function cacheState()
    {
        if ($this->widgetShouldUseCache()) {
            return " || cache: {$this->cacheLifeTime}(min)' ";
        }
        return " || cache : off";
    }

    /**
     * @return bool
     */
    private function widgetShouldUseCache()
    {
        /*
         * ================================== *
         |  The caching is turned off when:   |
         |  1- we are running tests           |
         |  2- have disabled it in .env file  |
         |  3- have set the time to 0 minutes |
         * ================================== *
        */
        return ((env('WIDGET_CACHE', false) !== false) and (!app()->environment('testing')) and ($this->cacheLifeTime !== 0));
// Logical operators have lower precedence:
$f = false or true;

// is executed like this:
($f = false) or true;


// Boolean operators have higher precedence:
$f = false || true;

// is executed like this:
$f = (false || true);
    }

    /**
     * @param $arg
     * @return string
     */
    private function makeCacheKey($arg)
    {
        return md5(json_encode($arg, JSON_FORCE_OBJECT) . $this->template . get_called_class());
    }

    private function cacheResult($key, $phpCode)
    {
        $cache = app()->make('cache');

        if ($this->cacheTags) {

            $cache = $cache->tags($this->cacheTags);
        }

        if ($this->cacheLifeTime > 0) {
            return $cache->remember($key, $this->cacheLifeTime, $phpCode);
        }

        if ($this->cacheLifeTime == 'forever' or $this->cacheLifeTime < 0) {
// Logical operators have lower precedence:
$f = false or true;

// is executed like this:
($f = false) or true;


// Boolean operators have higher precedence:
$f = false || true;

// is executed like this:
$f = (false || true);
            return $cache->rememberForever($key, $phpCode);
        }
    }

    /**
     * This method is called when you try to print the object like an string in blade files.
     * like this : {!! $myWidgetObj !!}
     */
    public function __toString()
    {
        return $this->renderWidget();
    }

    private function addDebugInfo()
    {
        $this->html = "<div title='" . get_called_class() . "::class || template : {$this->template}" . $this->cacheState() . "' style='box-shadow: 0px 0px 15px 5px #00c62b inset'>" . $this->html . "</div>";
    }

    private function addHtmlComments()
    {
        $this->html = "<!-- '{$this->friendlyName}' Widget Start -->" . $this->html . "<!-- '{$this->friendlyName}' Widget End -->";
class MyClass { }

$x = new MyClass();
$x->foo = true;
    }

}

Push — master ( e7c634...2c7ea9 )

BaseWidget::addHtmlComments() A

Complexity

Size

Duplication

Importance

Available Fixes

Logical Operators are used for Control-Flow

Available Fixes

Logical Operators are used for Control-Flow

Logical Operators are used for Control-Flow

Logical Operators are used for Control-Flow

Logical Operators are used for Control-Flow

imanghafoori1 / laravel-widgetize

Push — master ( e7c634...2c7ea9 )

BaseWidget::addHtmlComments() A

Complexity

Size

Duplication

Importance

Available Fixes

Logical Operators are used for Control-Flow

Available Fixes

Logical Operators are used for Control-Flow

Logical Operators are used for Control-Flow

Logical Operators are used for Control-Flow

Logical Operators are used for Control-Flow

Duplication Side-by-Side

Filter issues like
