Imanghafoori\Widgets\Utils\Policies

Complexity

Total Complexity

7

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	3
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	widgetShouldHaveDebugInfo()	0	3	2
A	widgetShouldUseCache()	0	3	2
A	widgetShouldBeMinified()	0	5	2
A	widgetShouldBeExtracted()	0	4	1

<?php

namespace Imanghafoori\Widgets\Utils;

class Policies
{
    /**
     * Detects the widget Should Have Debug Info.
     *
     * @return bool
     */
    public function widgetShouldHaveDebugInfo(): bool
    {
        return config('widgetize.debug_info') && !app()->environment('production');

The method environment() does not exist on Illuminate\Container\Container. Are you sure you never get this type here, but always one of the subclasses?

    }

    /**
     * The caching is turned off when:
     * 1 - we are running tests
     * 2 - have disabled it in config file.
     *
     * @return bool
     */
    public function widgetShouldUseCache(): bool
    {
        return config('widgetize.enable_cache') && (!app()->environment('testing'));
    }

    /**
     * Widget Should Be Minified or Not.
     *
     * @param $widget
     *
     * @return bool
     */
    public function widgetShouldBeMinified($widget): bool
    {
        $conf = (config('widgetize.minify_html') || app()->environment('production'));

        return $widget->minifyOutput ?? $conf;
    }

    public function widgetShouldBeExtracted($widget): bool

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

    {
        $conf = config('widgetize.extract_data');
        return /* $widget->extract_data ?? */ $conf;
    }
}