Sign upLogin

librenms/librenms

View on GitHub
Star
LibreNMS/Exceptions/RrdGraphException.php

Summary

Maintainability
A
45 mins
Test Coverage

Method __construct has 6 arguments (exceeds 4 allowed). Consider refactoring.
Open

    public function __construct($error, $short_text = null, $width = null, $height = null, $exit_code = 0, $image_output = '')
Severity: Minor
Found in LibreNMS/Exceptions/RrdGraphException.php - About 45 mins to fix

    The property $short_text is not named in camelCase.
    Open

    class RrdGraphException extends Exception
{
    /** @var string */
    protected $image_output;
    /** @var string|null */
    Severity: Minor
    Found in LibreNMS/Exceptions/RrdGraphException.php by phpmd

    CamelCasePropertyName

    Since: 0.2

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

    Example

    class ClassName {
    protected $property_name;
}

    Source

    The property $image_output is not named in camelCase.
    Open

    class RrdGraphException extends Exception
{
    /** @var string */
    protected $image_output;
    /** @var string|null */
    Severity: Minor
    Found in LibreNMS/Exceptions/RrdGraphException.php by phpmd

    CamelCasePropertyName

    Since: 0.2

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

    Example

    class ClassName {
    protected $property_name;
}

    Source

    There are no issues that match your filters.

    Category
    Status