detain/myadmin-servers-module

Method loadProcessing has 34 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public static function loadProcessing(GenericEvent $event)
    {
        /**
         * @var \ServiceHandler $service
         */

Missing class import via use statement (line '110', column '31').

Open

                $smarty = new \TFSmarty();

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

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

Missing class import via use statement (line '114', column '22').

Open

                (new \MyAdmin\Mail())->adminMail($subject, $email, false, 'admin/server_reactivated.tpl');

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

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

Avoid unused local variables such as '$serviceTypes'.

Open

        $serviceTypes = run_event('get_service_types', false, self::$module);

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

class Foo {
    public function doSomething()
    {
        $i = 5; // Unused
    }
}