librenms/librenms

Method contactDiscord has 38 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function contactDiscord($obj, $discord_opts)
    {
        $host = $discord_opts['url'];
        $curl = curl_init();
        $discord_title = '#' . $obj['uid'] . ' ' . $obj['title'];

Avoid unused parameters such as '$opts'.

Open

    public function deliverAlert($obj, $opts)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

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

Avoid unused parameters such as '$discord_opts'.

Open

    public function createDiscordFields($obj, $discord_opts)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

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

FIXME found

Open

            var_dump("API '$host' returned Error"); //FIXME: propper debuging

• Exclude checks
  • Disable engine
  • Disable check

FIXME found

Open

            var_dump('Params: ' . $alert_message); //FIXME: propper debuging

• Exclude checks
  • Disable engine
  • Disable check

FIXME found

Open

            var_dump('Return: ' . $ret); //FIXME: propper debuging

• Exclude checks
  • Disable engine
  • Disable check