detain/myadmin-quickservers-module

Method loadProcessing has 54 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 '161', column '27').

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 '105', column '22').

Open

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

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

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

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

Open

                $smarty = new \TFSmarty();

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

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

Avoid unused local variables such as '$server_info'.

Open

        $server_info = $serviceInfo['server_info'];

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

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

Avoid unused local variables such as '$class'.

Open

                $class = '\\MyAdmin\\Orm\\'.get_orm_class_from_table($settings['TABLE']);

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

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

The variable $server_info is not named in camelCase.

Open

    public static function getQueue(GenericEvent $event)
    {
        //if (in_array($event['type'], [get_service_define('KVM_LINUX'), get_service_define('KVM_WINDOWS'), get_service_define('CLOUD_KVM_LINUX'), get_service_define('CLOUD_KVM_WINDOWS')])) {
        $settings = get_module_settings(self::$module);
        $serviceInfo = $event->getSubject();

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

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}