src/Phug/PhugDevServer.php
Method listen
has 26 lines of code (exceeds 25 allowed). Consider refactoring. Open
Open
public function listen()
{
$browserReloadPort = $this->browserReloadPort;
$serverPort = intval($this->server);
if ($serverPort) {
Missing class import via use statement (line '80', column '23'). Open
Open
throw new \RuntimeException('No watcher program found in the vendor/bin directory.');
- Read upRead up
- Exclude checks
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
Open
public function __construct($server, $file, $browserReloadPort)
{
if (ctype_digit(strval($file)) && !file_exists($file)) {
$__server = $server;
$server = $file;
- Read upRead up
- Exclude checks
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
Open
public function __construct($server, $file, $browserReloadPort)
{
if (ctype_digit(strval($file)) && !file_exists($file)) {
$__server = $server;
$server = $file;
- Read upRead up
- Exclude checks
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();
}
}