librenms/librenms

View on GitHub
LibreNMS/Snmptrap/Handlers/AdvaNetworkElementAlmTrap.php

Summary

Maintainability
A
0 mins
Test Coverage

Avoid unused parameters such as '$device'.
Open

    public function handle(Device $device, Trap $trap)

UnusedFormalParameter

Since: 0.2

Avoid passing parameters to methods or constructors and then not using those parameters.

Example

class Foo
{
    private function bar($howdy)
    {
        // $howdy is not used
    }
}

Source https://phpmd.org/rules/unusedcode.html#unusedformalparameter

Line indented incorrectly; expected at least 12 spaces, found 8
Open

        $almObjName = $trap->getOidData($trap->findOid('CM-ALARM-MIB::cmNetworkElementAlmObjectName'));

Line indented incorrectly; expected 8 spaces, found 12
Open

            default => Severity::Info,

Line indented incorrectly; expected at least 12 spaces, found 8
Open

        $almDescr = $trap->getOidData($trap->findOid('CM-ALARM-MIB::cmNetworkElementAlmDescr'));

Space before opening parenthesis of function call prohibited
Open

        $logSeverity = match ($alSeverity) {

Line indented incorrectly; expected at least 12 spaces, found 8
Open

        $trap->log("Alarming Element: $almObjName Description: $almDescr Severity: $alSeverity", $logSeverity);

There are no issues that match your filters.

Category
Status