phug-php/watcher

View on GitHub
src/Phug/PhugDevServer.php

Summary

Maintainability
A
1 hr
Test Coverage

Method listen has 26 lines of code (exceeds 25 allowed). Consider refactoring.
Open

    public function listen()
    {
        $browserReloadPort = $this->browserReloadPort;
        $serverPort = intval($this->server);
        if ($serverPort) {
Severity: Minor
Found in src/Phug/PhugDevServer.php - About 1 hr to fix

    Missing class import via use statement (line '80', column '23').
    Open

                throw new \RuntimeException('No watcher program found in the vendor/bin directory.');
    Severity: Minor
    Found in src/Phug/PhugDevServer.php by phpmd

    MissingImport

    Since: 2.7.0

    Importing all external classes in a file through use statements makes them clearly visible.

    Example

    function make() {
        return new \stdClass();
    }

    Source http://phpmd.org/rules/cleancode.html#MissingImport

    The variable $__server is not named in camelCase.
    Open

        public function __construct($server, $file, $browserReloadPort)
        {
            if (ctype_digit(strval($file)) && !file_exists($file)) {
                $__server = $server;
                $server = $file;
    Severity: Minor
    Found in src/Phug/PhugDevServer.php by phpmd

    CamelCaseVariableName

    Since: 0.2

    It is considered best practice to use the camelCase notation to name variables.

    Example

    class ClassName {
        public function doSomething() {
            $data_module = new DataModule();
        }
    }

    Source

    The variable $__server is not named in camelCase.
    Open

        public function __construct($server, $file, $browserReloadPort)
        {
            if (ctype_digit(strval($file)) && !file_exists($file)) {
                $__server = $server;
                $server = $file;
    Severity: Minor
    Found in src/Phug/PhugDevServer.php by phpmd

    CamelCaseVariableName

    Since: 0.2

    It is considered best practice to use the camelCase notation to name variables.

    Example

    class ClassName {
        public function doSomething() {
            $data_module = new DataModule();
        }
    }

    Source

    There are no issues that match your filters.

    Category
    Status