WebpackPageControllerExtension

Complexity

Total Complexity

13

Size/Duplication

Total Lines	126
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	IsNotWebpackDevServer()	0	4	2
A	IsWebpackDevServer()	0	17	4
A	WebpackBaseURL()	0	12	3
A	WebpackDistributionFolderExtension()	0	4	1
A	WebpackFileHash()	0	14	3

<?php


class WebpackPageControllerExtension extends Extension

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{

    /**
     * you only need to set this if you have some themes that are enabled and some themes
     * that do not run webpack
     * @var {Array}
     */
    private static $webpack_enabled_themes = [];

The property $webpack_enabled_themes is not used and could be removed.

    /**
     * override webpack server for custom set ups
     * set to true to make this class believe you are always running
     * the webpack server
     * @see IsWebpackDevServer
     * @var bool
     */
    private static $is_webpack_server = false;

The property $is_webpack_server is not used and could be removed.

    /**
     * override webpack server for custom set ups
     * this is the server used for checking if the webpack server is running
     * @see IsWebpackDevServer
     * @var bool
     */
    private static $webpack_socket_server = 'localhost';

The property $webpack_socket_server is not used and could be removed.

    /**
     * usually this is set to current domain
     * only set if you need an alternative
     * @see: WebpackBaseURL
     * @var string
     */
    private static $webpack_server = '';

The property $webpack_server is not used and could be removed.

    /**
     *
     * @var int
     */
    private static $webpack_port = 3000;

The property $webpack_port is not used and could be removed.

    /**
     * this is the folder where the distilled files are placed.
     * If your theme is foo then you will find the distilled files in themes/foo_dist
     * @var string
     */
    private static $webpack_distribution_folder_extension = 'dist';

The property $webpack_distribution_folder_extension is not used and could be removed.

    /**
     *
     * @return bool
     */
    public function IsNotWebpackDevServer()
    {
        return $this->IsWebpackDevServer() ? false : true;
    }


    /**
     *
     * @return bool
     */
    public function IsWebpackDevServer()
    {
        $override = $this->owner->Config()->get('is_webpack_server');
        if ($override) {
            return $override;
        }
        if (Director::isDev()) {
            $socket = @fsockopen(
                $this->owner->Config()->get('webpack_socket_server'),
                $this->owner->Config()->get('webpack_port'),
                $errno,
                $errstr,
                1
            );
            return ! $socket ? false : true;
        }
    }


    /**
     *
     * @return string
     */
    public function WebpackBaseURL()
    {
        $webpackServer = $this->owner->Config()->get('webpack_server');
        if (! $webpackServer) {
            $webpackServer = Director::AbsoluteBaseURL('/');

The call to Director::absoluteBaseURL() has too many arguments starting with '/'.

        }
        if ($this->IsWebpackDevServer()) {
            $webpackServer = rtrim($webpackServer, '/') .':'.$this->owner->Config()->get('webpack_port').'/';
        }

        return $webpackServer;
    }

    /**
     * @return string
     */
    public function WebpackDistributionFolderExtension()
    {
        return $this->owner->Config()->get('webpack_distribution_folder_extension');
    }

    /**
     *
     * @param string $type should be set to JS or CSS
     * @return string
     */
    public function WebpackFileHash($type = 'JS')
    {
        $base = Director::baseFolder();
        if ($type === 'JS') {
            $file = 'bundle.js';
        } elseif ($type === 'CSS') {
            $file = 'style.css';
        } else {
            user_error('Please specify JS or CSS, '.$type.' specified.');
        }
        $fullFile = $base.'/'.THEMES_DIR . "/" . Config::inst()->get('SSViewer', 'theme').'_'.$this->WebpackDistributionFolderExtension().'/'.$file;

The variable $file does not seem to be defined for all execution paths leading up to this point.

        return @filemtime($fullFile);
    }
}