chamilo/chamilo-lms

The function display_requirements() has an NPath complexity of 1404. The configured NPath complexity threshold is 200.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

class Foo {
    function bar() {
        // lots of complicated code
    }
}

The function display_configuration_settings_form has 15 parameters. Consider reducing the number of parameters to less than 10.

Open

function display_configuration_settings_form(
    string $installType,
    string $urlForm,
    string $languageForm,
    string $emailForm,

• Exclude checks
  • Disable engine
  • Disable check

The function installSettings has 10 parameters. Consider reducing the number of parameters to less than 10.

Open

function installSettings(
    $organizationName,
    $organizationUrl,
    $siteName,
    $adminEmail,

• Exclude checks
  • Disable engine
  • Disable check

The function finishInstallationWithContainer has 17 parameters. Consider reducing the number of parameters to less than 10.

Open

function finishInstallationWithContainer(
    $container,
    $sysPath,
    $encryptPassForm,
    $passForm,

• Exclude checks
  • Disable engine
  • Disable check

The function installProfileSettings() has an NPath complexity of 576. The configured NPath complexity threshold is 200.

Open

function installProfileSettings($installationProfile = '')
{
    error_log('installProfileSettings');
    if (empty($installationProfile)) {
        return false;

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

class Foo {
    function bar() {
        // lots of complicated code
    }
}

Avoid using undefined variables such as '$_configuration' which will lead to PHP notices.

Open

        $config = new Laminas\Config\Config($_configuration);

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Remove error control operator '@' on line 451.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

Missing class import via use statement (line '1899', column '23').

Open

        $kernel = new Chamilo\Kernel($env, false);

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

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

Missing class import via use statement (line '293', column '23').

Open

        $config = new Laminas\Config\Config($_configuration);

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

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

Remove error control operator '@' on line 1615.

Open

function rrmdir($dir)
{
    if (is_dir($dir)) {
        $objects = scandir($dir);
        foreach ($objects as $object) {

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

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

Remove error control operator '@' on line 250.

Open

function set_file_folder_permissions()
{
    @chmod('.', 0755); //set permissions on install dir
    @chmod('..', 0755); //set permissions on parent dir of install dir
}

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

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

Remove error control operator '@' on line 1613.

Open

function rrmdir($dir)
{
    if (is_dir($dir)) {
        $objects = scandir($dir);
        foreach ($objects as $object) {

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

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

Remove error control operator '@' on line 459.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

Remove error control operator '@' on line 743.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

Remove error control operator '@' on line 249.

Open

function set_file_folder_permissions()
{
    @chmod('.', 0755); //set permissions on install dir
    @chmod('..', 0755); //set permissions on parent dir of install dir
}

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

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

Missing class import via use statement (line '1839', column '24').

Open

            $now = new DateTime();

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

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

Missing class import via use statement (line '1301', column '18').

Open

    $input = new Symfony\Component\Console\Input\StringInput($result);

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

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

Remove error control operator '@' on line 436.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

Remove error control operator '@' on line 790.

Open

function display_license_agreement(): array
{
    $license = api_htmlentities(@file_get_contents(api_get_path(SYMFONY_SYS_PATH).'public/documentation/license.txt'));

    $activtiesList = [

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

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

Missing class import via use statement (line '1375', column '23').

Open

            throw new \Exception("The parameter $requirement is needed in order to edit the .env file");

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

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

Missing class import via use statement (line '1838', column '40').

Open

            $latestMigrationDate = new DateTime($result[0]['executed_at']);

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

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

Remove error control operator '@' on line 460.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

Remove error control operator '@' on line 1705.

Open

function checkCanCreateFile(string $file): bool
{
    if (file_exists($file)) {
        return is_writable($file);
    }

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

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

The method checkExtension uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            } else {
                if ($optional) {
                    return [
                        'severity' => 'warning',
                        'message' => get_lang('Extension installed but not enabled'),

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid assigning values to variables in if clauses and the like (line '324', column '20').

Open

function get_config_param_from_db($param = '')
{
    $param = Database::escape_string($param);

    $schemaManager = Database::getConnection()->createSchemaManager();

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

class Foo
{
    public function bar($flag)
    {
        if ($foo = 'bar') { // possible typo
            // ...
        }
        if ($baz = 0) { // always false
            // ...
        }
    }
}

The method displayDatabaseParameter uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

    } else {
        $inputType = 'dbPassForm' === $formFieldName ? 'password' : 'text';
        //Slightly limit the length of the database prefix to avoid having to cut down the databases names later on
        $maxLength = 'dbPrefixForm' === $formFieldName ? '15' : MAX_FORM_FIELD_LENGTH;
        if (INSTALL_TYPE_UPDATE == $installType) {

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method migrateSwitch uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            } else {
                error_log('There was an error during running migrations. Check error.log');
                exit;
            }

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method executeMigration uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            $resultStatus['message'] = 'Migration completed with errors.';
            $resultStatus['progress_percentage'] = 0;
        }

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method check_writable uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

    } else {
        return Display::label(get_lang('Not writable'), 'important');
    }

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method checkReadable uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            return Display::label(get_lang('Not readable'), 'important');
        }

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method checkExtension uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

    } else {
        if ($optional) {
            return [
                'severity' => 'warning',
                'message' => $returnFailure,

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method checkReadable uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

    } else {
        if ($suggestion) {
            return Display::label(get_lang('Not readable'), 'info');
        } else {
            return Display::label(get_lang('Not readable'), 'important');

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method rrmdir uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

                } else {
                    @unlink($dir.'/'.$object);
                }

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method get_config_param_from_db uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

    } else {
        return null;
    }

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method display_database_settings_form uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            connectToDatabase(
                $dbHostForm,
                $dbUsernameForm,
                $dbPassForm,

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method displayDatabaseParameter uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            echo '<input
                        type="'.$inputType.'"
                        class="form-control"
                        size="'.DATABASE_FORM_FIELD_DISPLAY_LENGTH.'"

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method get_config_param uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            // Give up recovering.
            return null;
        }

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method display_configuration_parameter uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

    } else {
        $html .= '<div class="col-sm-6">
                    <input
                        class="form-control"
                        type="text"

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method installSchemas uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

    } else {
        error_log('Install settings');
        // Installing schemas (filling settings table)
        $settingsManager->installSchemas($accessUrl);
    }

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid unused parameters such as '$encryptPassForm'.

Open

    $encryptPassForm,

• 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 '$upgrade'.

Open

function installTools($container, $manager, $upgrade = false)

• 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 '$sysPath'.

Open

    $sysPath,

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

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

Avoid unused local variables such as '$_configuration'.

Open

        $config = new Laminas\Config\Config($_configuration);

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

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

Avoid unused local variables such as '$tableCreationWorks'.

Open

            $tableCreationWorks = false;

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

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

Avoid unused parameters such as '$upgradeFromVersion'.

Open

    array $upgradeFromVersion = []

• 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 '$suggestion'.

Open

function check_writable($folder, $suggestion = false)

• 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 '$name'.

Open

function display_language_selection_box($name = 'language_list', $default_language = 'en_US')

• 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 '$manager'.

Open

function installTools($container, $manager, $upgrade = false)

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

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

Avoid unused local variables such as '$id'.

Open

    foreach ($settings as $id => $param) {

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

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

Avoid unused local variables such as '$tableDropWorks'.

Open

            $tableDropWorks = false;

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

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

Avoid unused local variables such as '$to'.

Open

    $to = null; // if $to == null then schema will be migrated to latest version

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

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

The parameter $current_value is not named in camelCase.

Open

function compare_setting_values(string $current_value, string $wanted_value): array
{
    $current_value_string = $current_value;
    $current_value = (float) $current_value;
    $wanted_value = (float) $wanted_value;

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The parameter $extra_notice is not named in camelCase.

Open

function displayDatabaseParameter(
    $installType,
    $parameterName,
    $formFieldName,
    $parameterValue,

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The parameter $default_language is not named in camelCase.

Open

function display_language_selection_box($name = 'language_list', $default_language = 'en_US')
{
    // Displaying the box.
    return Display::select(
        'language_list',

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The parameter $wanted_value is not named in camelCase.

Open

function compare_setting_values(string $current_value, string $wanted_value): array
{
    $current_value_string = $current_value;
    $current_value = (float) $current_value;
    $wanted_value = (float) $wanted_value;

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

class ClassName {
    public function doSomething($user_name) {
    }
}

Variable "current_config_file" is not in valid camel caps format

Open

    if (!file_exists($current_config_file)) {

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "checkExtension" in a static class

Open

function checkExtension(

• Exclude checks
  • Disable engine
  • Disable check

Variable "_configuration" is not in valid camel caps format

Open

    global $new_version, $_configuration;

• Exclude checks
  • Disable engine
  • Disable check

Variable "current_version" is not in valid camel caps format

Open

        $current_version = trim($_configuration['system_version']);

• Exclude checks
  • Disable engine
  • Disable check

Variable "current_config_file" is not in valid camel caps format

Open

    $current_config_file = api_get_path(CONFIGURATION_PATH).'configuration.php';

• Exclude checks
  • Disable engine
  • Disable check

Variable "current_version" is not in valid camel caps format

Open

    return empty($current_version) ? false : version_compare($current_version, $new_version, '>=');

• Exclude checks
  • Disable engine
  • Disable check

Variable "new_version" is not in valid camel caps format

Open

    global $new_version, $_configuration;

• Exclude checks
  • Disable engine
  • Disable check

Variable "_configuration" is not in valid camel caps format

Open

        $current_version = trim($_configuration['system_version']);

• Exclude checks
  • Disable engine
  • Disable check

Variable "_configuration" is not in valid camel caps format

Open

    if (isset($_configuration['system_version'])) {

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "isAlreadyInstalledSystem" in a static class

Open

function isAlreadyInstalledSystem()

• Exclude checks
  • Disable engine
  • Disable check

Variable "new_version" is not in valid camel caps format

Open

    if (empty($new_version)) {

• Exclude checks
  • Disable engine
  • Disable check

Variable "current_version" is not in valid camel caps format

Open

    $current_version = null;

• Exclude checks
  • Disable engine
  • Disable check

Variable "new_version" is not in valid camel caps format

Open

    return empty($current_version) ? false : version_compare($current_version, $new_version, '>=');

• Exclude checks
  • Disable engine
  • Disable check

Variable "current_version" is not in valid camel caps format

Open

    return empty($current_version) ? false : version_compare($current_version, $new_version, '>=');

• Exclude checks
  • Disable engine
  • Disable check

Variable "current_config_file" is not in valid camel caps format

Open

    require $current_config_file;

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "getPhpSetting" in a static class

Open

function getPhpSetting($val)

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "display_requirements" in a static class

Open

function display_requirements(

• Exclude checks
  • Disable engine
  • Disable check

Variable "_setting" is not in valid camel caps format

Open

    global $_setting, $originalMemoryLimit;

• Exclude checks
  • Disable engine
  • Disable check

Missing parameter name

Open

 * @param   string  Type of install

• Exclude checks
  • Disable engine
  • Disable check

Missing parameter name

Open

 * @param   string  Name of parameter

• Exclude checks
  • Disable engine
  • Disable check

Variable "extra_notice" is not in valid camel caps format

Open

    $extra_notice,

• Exclude checks
  • Disable engine
  • Disable check

Variable "a_countries" is not in valid camel caps format

Open

    return $a_countries;

• Exclude checks
  • Disable engine
  • Disable check

Variable "current_value_string" is not in valid camel caps format

Open

        : ['severity' => 'danger', 'value' => $current_value_string];

• Exclude checks
  • Disable engine
  • Disable check

Variable "progress_percentage" is not in valid camel caps format

Open

        $progress_percentage = ($executedMigrations / $totalMigrations) * 100;

• Exclude checks
  • Disable engine
  • Disable check

Variable "current_step" is not in valid camel caps format

Open

    global $current_step;

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "display_step_sequence" in a static class

Open

function display_step_sequence()

• Exclude checks
  • Disable engine
  • Disable check

Doc comment for parameter $upgradeFromVersion does not match actual variable name $badUpdatePath

Open

 * @param array  $upgradeFromVersion The different subversions from version 1.9

• Exclude checks
  • Disable engine
  • Disable check

Variable "current_step" is not in valid camel caps format

Open

    return get_lang('Step'.$current_step).' – ';

• Exclude checks
  • Disable engine
  • Disable check

Variable "fil_perm_verified" is not in valid camel caps format

Open

    $fil_perm_verified = 0666;

• Exclude checks
  • Disable engine
  • Disable check

Variable "default_language" is not in valid camel caps format

Open

function display_language_selection_box($name = 'language_list', $default_language = 'en_US')

• Exclude checks
  • Disable engine
  • Disable check

Variable "fil_perm_verified" is not in valid camel caps format

Open

    $filePerm = '0'.decoct($fil_perm_verified);

• Exclude checks
  • Disable engine
  • Disable check

Variable "course_test_was_created" is not in valid camel caps format

Open

        if (!$course_test_was_created) {

• Exclude checks
  • Disable engine
  • Disable check

Missing parameter name

Open

 * @param   string  Field value

• Exclude checks
  • Disable engine
  • Disable check

Variable "perm_file" is not in valid camel caps format

Open

            @chmod($checked_writable, $perm_file);

• Exclude checks
  • Disable engine
  • Disable check

Missing parameter name

Open

 * @param   bool Whether to display in update mode

• Exclude checks
  • Disable engine
  • Disable check

Variable "permissions_for_new_files" is not in valid camel caps format

Open

        ['selected_value' => '0'.decoct($permissions_for_new_files)],

• Exclude checks
  • Disable engine
  • Disable check

Variable "a_countries" is not in valid camel caps format

Open

    $options = array_combine($a_countries, $a_countries);

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "compare_setting_values" in a static class

Open

function compare_setting_values(string $current_value, string $wanted_value): array

• Exclude checks
  • Disable engine
  • Disable check

Variable "wanted_value" is not in valid camel caps format

Open

    $wanted_value = (float) $wanted_value;

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "executeLexikKeyPair" in a static class

Open

function executeLexikKeyPair(\Chamilo\Kernel $kernel): void

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "updateEnvFile" in a static class

Open

function updateEnvFile($distFile, $envFile, $params)

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "finishInstallationWithContainer" in a static class

Open

function finishInstallationWithContainer(

• Exclude checks
  • Disable engine
  • Disable check

Variable "progress_percentage" is not in valid camel caps format

Open

        'progress_percentage' => ceil($progress_percentage),

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "initializeEntityManager" in a static class

Open

function initializeEntityManager(): EntityManager

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "get_config_param_from_db" in a static class

Open

function get_config_param_from_db($param = '')

• Exclude checks
  • Disable engine
  • Disable check

Variable "perms_fil" is not in valid camel caps format

Open

    $perms_fil = [0666, 0644, 0664, 0660, 0640, 0600];

• Exclude checks
  • Disable engine
  • Disable check

Variable "fil_perm_verified" is not in valid camel caps format

Open

                $fil_perm_verified = $perm;

• Exclude checks
  • Disable engine
  • Disable check

Variable "perm_file" is not in valid camel caps format

Open

        $perm_file = octdec('0666');

• Exclude checks
  • Disable engine
  • Disable check

Variable "a_countries" is not in valid camel caps format

Open

    $a_countries = [

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "migrate" in a static class

Open

function migrate(EntityManager $manager)

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "installTools" in a static class

Open

function installTools($container, $manager, $upgrade = false)

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "checkPhpSetting" in a static class

Open

function checkPhpSetting(

• Exclude checks
  • Disable engine
  • Disable check

Variable "default_language" is not in valid camel caps format

Open

        $default_language,

• Exclude checks
  • Disable engine
  • Disable check

Variable "checked_writable" is not in valid camel caps format

Open

            @chmod($checked_writable, $perm_file);

• Exclude checks
  • Disable engine
  • Disable check

Missing parameter name

Open

 * @param   string  Extra notice (to show on the right side)

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "displayDatabaseParameter" in a static class

Open

function displayDatabaseParameter(

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "get_countries_list_from_array" in a static class

Open

function get_countries_list_from_array($combo = false)

• Exclude checks
  • Disable engine
  • Disable check

Variable "current_value" is not in valid camel caps format

Open

    $current_value = (float) $current_value;

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "upgradeWithContainer" in a static class

Open

function upgradeWithContainer($container)

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "installProfileSettings" in a static class

Open

function installProfileSettings($installationProfile = '')

• Exclude checks
  • Disable engine
  • Disable check

Variable "progress_percentage" is not in valid camel caps format

Open

    $progress_percentage = 0;

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "executeMigration" in a static class

Open

function executeMigration(): array

• Exclude checks
  • Disable engine
  • Disable check

Variable "dir_perm_verified" is not in valid camel caps format

Open

            $dir_perm_verified = $perm;

• Exclude checks
  • Disable engine
  • Disable check

Variable "file_course_test_was_created" is not in valid camel caps format

Open

                $file_course_test_was_created = true;

• Exclude checks
  • Disable engine
  • Disable check

Variable "_setting" is not in valid camel caps format

Open

    $_SESSION['permissions_for_new_directories'] = $_setting['permissions_for_new_directories'] = $dir_perm_verified;

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "display_database_settings_form" in a static class

Open

function display_database_settings_form(

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "display_configuration_parameter" in a static class

Open

function display_configuration_parameter(

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "lockSettings" in a static class

Open

function lockSettings()

• Exclude checks
  • Disable engine
  • Disable check

Variable "permissions_for_new_directories" is not in valid camel caps format

Open

    $permissions_for_new_directories = isset($_SESSION['permissions_for_new_directories']) ? $_SESSION['permissions_for_new_directories'] : 0770;

• Exclude checks
  • Disable engine
  • Disable check

Variable "current_value" is not in valid camel caps format

Open

    return $current_value >= $wanted_value

• Exclude checks
  • Disable engine
  • Disable check

Variable "current_migration" is not in valid camel caps format

Open

        'current_migration' => $current_migration,

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "trueFalse" in a static class

Open

function trueFalse($var)

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "check_writable" in a static class

Open

function check_writable($folder, $suggestion = false)

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "get_config_param" in a static class

Open

function get_config_param($param, $updatePath = '')

• Exclude checks
  • Disable engine
  • Disable check

Missing blank line before return statement

Open

            return $row['selected_value'];

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "step_active" in a static class

Open

function step_active($param)

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "display_language_selection_box" in a static class

Open

function display_language_selection_box($name = 'language_list', $default_language = 'en_US')

• Exclude checks
  • Disable engine
  • Disable check

Variable "dir_perm_verified" is not in valid camel caps format

Open

    $dir_perm_verified = 0777;

• Exclude checks
  • Disable engine
  • Disable check

Variable "perms_dir" is not in valid camel caps format

Open

    foreach ($perms_dir as $perm) {

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "display_license_agreement" in a static class

Open

function display_license_agreement(): array

• Exclude checks
  • Disable engine
  • Disable check

Space before opening parenthesis of function call prohibited

Open

        fn ($country) => [$country],

• Exclude checks
  • Disable engine
  • Disable check

Variable "current_value" is not in valid camel caps format

Open

    $current_value = (float) $current_value;

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "installSchemas" in a static class

Open

function installSchemas($container, $upgrade = false)

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "migrateSwitch" in a static class

Open

function migrateSwitch($fromVersion, $manager, $processFiles = true)

• Exclude checks
  • Disable engine
  • Disable check

Expected 4 spaces after parameter type; 1 found

Open

 * @param int $port

• Exclude checks
  • Disable engine
  • Disable check

Variable "course_test_was_created" is not in valid camel caps format

Open

    $course_test_was_created = false;

• Exclude checks
  • Disable engine
  • Disable check

Variable "_setting" is not in valid camel caps format

Open

    $_SESSION['permissions_for_new_files'] = $_setting['permissions_for_new_files'] = $fil_perm_verified;

• Exclude checks
  • Disable engine
  • Disable check

Variable "checked_writable" is not in valid camel caps format

Open

        $checked_writable = api_get_path(CONFIGURATION_PATH).'configuration.php';

• Exclude checks
  • Disable engine
  • Disable check

Variable "wanted_value" is not in valid camel caps format

Open

    $wanted_value = (float) $wanted_value;

• Exclude checks
  • Disable engine
  • Disable check

Variable "current_migration" is not in valid camel caps format

Open

    $current_migration = $connection->createQueryBuilder()

• Exclude checks
  • Disable engine
  • Disable check

Variable "_configuration" is not in valid camel caps format

Open

        $config = new Laminas\Config\Config($_configuration);

• Exclude checks
  • Disable engine
  • Disable check

The closing parenthesis and the opening brace of a multi-line function declaration must be on the same line

Open

{

• Exclude checks
  • Disable engine
  • Disable check

Variable "checked_writable" is not in valid camel caps format

Open

        if (file_exists($checked_writable) && !is_writable($checked_writable)) {

• Exclude checks
  • Disable engine
  • Disable check

Variable "permissions_for_new_files" is not in valid camel caps format

Open

    $permissions_for_new_files = isset($_SESSION['permissions_for_new_files']) ? $_SESSION['permissions_for_new_files'] : 0660;

• Exclude checks
  • Disable engine
  • Disable check

Variable "wanted_value" is not in valid camel caps format

Open

function compare_setting_values(string $current_value, string $wanted_value): array

• Exclude checks
  • Disable engine
  • Disable check

Variable "current_value_string" is not in valid camel caps format

Open

        ? ['severity' => 'success', 'value' => $current_value_string]

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "installSettings" in a static class

Open

function installSettings(

• Exclude checks
  • Disable engine
  • Disable check

Variable "current_step" is not in valid camel caps format

Open

    global $current_step;

• Exclude checks
  • Disable engine
  • Disable check

Variable "file_course_test_was_created" is not in valid camel caps format

Open

            if ($file_course_test_was_created) {

• Exclude checks
  • Disable engine
  • Disable check

Variable "a_countries" is not in valid camel caps format

Open

    $options = array_combine($a_countries, $a_countries);

• Exclude checks
  • Disable engine
  • Disable check

Variable "permissions_for_new_directories" is not in valid camel caps format

Open

        ['selected_value' => '0'.decoct($permissions_for_new_directories)],

• Exclude checks
  • Disable engine
  • Disable check

Missing function doc comment

Open

function compare_setting_values(string $current_value, string $wanted_value): array

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "set_file_folder_permissions" in a static class

Open

function set_file_folder_permissions()

• Exclude checks
  • Disable engine
  • Disable check

Variable "file_course_test_was_created" is not in valid camel caps format

Open

    $file_course_test_was_created = false;

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "generateRandomToken" in a static class

Open

function generateRandomToken()

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "checkCanCreateFile" in a static class

Open

function checkCanCreateFile(string $file): bool

• Exclude checks
  • Disable engine
  • Disable check

Missing blank line before return statement

Open

    return $result['version'] ?? '';

• Exclude checks
  • Disable engine
  • Disable check

Blank line found at end of control structure

Open

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "connectToDatabase" in a static class

Open

function connectToDatabase(

• Exclude checks
  • Disable engine
  • Disable check

Variable "perms_fil" is not in valid camel caps format

Open

        foreach ($perms_fil as $perm) {

• Exclude checks
  • Disable engine
  • Disable check

Variable "checked_writable" is not in valid camel caps format

Open

        if (file_exists($checked_writable) && !is_writable($checked_writable)) {

• Exclude checks
  • Disable engine
  • Disable check

Variable "extra_notice" is not in valid camel caps format

Open

                    '.$extra_notice.'

• Exclude checks
  • Disable engine
  • Disable check

Variable "current_value" is not in valid camel caps format

Open

    $current_value_string = $current_value;

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "rrmdir" in a static class

Open

function rrmdir($dir)

• Exclude checks
  • Disable engine
  • Disable check

Variable "perms_dir" is not in valid camel caps format

Open

    $perms_dir = [0777, 0755, 0775, 0770, 0750, 0700];

• Exclude checks
  • Disable engine
  • Disable check

Variable "dir_perm_verified" is not in valid camel caps format

Open

    $_SESSION['permissions_for_new_directories'] = $_setting['permissions_for_new_directories'] = $dir_perm_verified;

• Exclude checks
  • Disable engine
  • Disable check

Variable "dir_perm_verified" is not in valid camel caps format

Open

    $dirPerm = '0'.decoct($dir_perm_verified);

• Exclude checks
  • Disable engine
  • Disable check

Missing parameter name

Open

 * @param   string  Additional attribute for the <tr> element

• Exclude checks
  • Disable engine
  • Disable check

Missing parameter name

Open

 * @param $dir

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "isVersionTableValid" in a static class

Open

function isVersionTableValid($connection): bool

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "getLastExecutedMigration" in a static class

Open

function getLastExecutedMigration(Connection $connection): string

• Exclude checks
  • Disable engine
  • Disable check

Variable "checked_writable" is not in valid camel caps format

Open

            $notWritable[] = $checked_writable;

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "display_configuration_settings_form" in a static class

Open

function display_configuration_settings_form(

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "updateDirAndFilesPermissions" in a static class

Open

function updateDirAndFilesPermissions()

• Exclude checks
  • Disable engine
  • Disable check

Variable "wanted_value" is not in valid camel caps format

Open

    return $current_value >= $wanted_value

• Exclude checks
  • Disable engine
  • Disable check

Missing function doc comment

Open

function installTools($container, $manager, $upgrade = false)

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "checkMigrationStatus" in a static class

Open

function checkMigrationStatus(): array

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "checkPhpSettingExists" in a static class

Open

function checkPhpSettingExists($phpSetting)

• Exclude checks
  • Disable engine
  • Disable check

Doc comment for parameter $updatePath does not match actual variable name $installType

Open

 * @param string $updatePath         The updatePath given (if given)

• Exclude checks
  • Disable engine
  • Disable check

Variable "fil_perm_verified" is not in valid camel caps format

Open

    $_SESSION['permissions_for_new_files'] = $_setting['permissions_for_new_files'] = $fil_perm_verified;

• Exclude checks
  • Disable engine
  • Disable check

Variable "current_value" is not in valid camel caps format

Open

function compare_setting_values(string $current_value, string $wanted_value): array

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "checkReadable" in a static class

Open

function checkReadable($folder, $suggestion = false)

• Exclude checks
  • Disable engine
  • Disable check

Variable "current_step" is not in valid camel caps format

Open

    if ($param == $current_step) {

• Exclude checks
  • Disable engine
  • Disable check

Variable "course_test_was_created" is not in valid camel caps format

Open

            $course_test_was_created = true;

• Exclude checks
  • Disable engine
  • Disable check

Missing parameter name

Open

 * @param   string  Field name (in the HTML form)

• Exclude checks
  • Disable engine
  • Disable check

Variable "current_value_string" is not in valid camel caps format

Open

    $current_value_string = $current_value;

• Exclude checks
  • Disable engine
  • Disable check

Variable "selected_value" is not in valid camel caps format

Open

            ['selected_value' => $param->selected_value],

• Exclude checks
  • Disable engine
  • Disable check

Consider putting global function "isUpdateAvailable" in a static class

Open

function isUpdateAvailable(): bool

• Exclude checks
  • Disable engine
  • Disable check

Arguments with default values must be at the end of the argument list

Open

    \Chamilo\Kernel $kernel

• Exclude checks
  • Disable engine
  • Disable check

Space before opening parenthesis of function call prohibited

Open

        fn ($country) => [$country],

• Exclude checks
  • Disable engine
  • Disable check

The variable $_configuration is not named in camelCase.

Open

function isAlreadyInstalledSystem()
{
    global $new_version, $_configuration;

    if (empty($new_version)) {

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

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

The variable $_configuration is not named in camelCase.

Open

function isAlreadyInstalledSystem()
{
    global $new_version, $_configuration;

    if (empty($new_version)) {

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

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

The variable $new_version is not named in camelCase.

Open

function isAlreadyInstalledSystem()
{
    global $new_version, $_configuration;

    if (empty($new_version)) {

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

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

The variable $new_version is not named in camelCase.

Open

function isAlreadyInstalledSystem()
{
    global $new_version, $_configuration;

    if (empty($new_version)) {

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

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

The variable $current_config_file is not named in camelCase.

Open

function isAlreadyInstalledSystem()
{
    global $new_version, $_configuration;

    if (empty($new_version)) {

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

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

The variable $current_config_file is not named in camelCase.

Open

function isAlreadyInstalledSystem()
{
    global $new_version, $_configuration;

    if (empty($new_version)) {

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

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

The variable $current_version is not named in camelCase.

Open

function isAlreadyInstalledSystem()
{
    global $new_version, $_configuration;

    if (empty($new_version)) {

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

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

The variable $current_config_file is not named in camelCase.

Open

function isAlreadyInstalledSystem()
{
    global $new_version, $_configuration;

    if (empty($new_version)) {

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

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

The variable $checked_writable is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $perms_fil is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $a_countries is not named in camelCase.

Open

function get_countries_list_from_array($combo = false)
{
    $a_countries = [
        'Afghanistan', 'Albania', 'Algeria', 'Andorra', 'Angola', 'Antigua and Barbuda', 'Argentina', 'Armenia', 'Australia', 'Austria', 'Azerbaijan',
        'Bahamas', 'Bahrain', 'Bangladesh', 'Barbados', 'Belarus', 'Belgium', 'Belize', 'Benin', 'Bhutan', 'Bolivia', 'Bosnia and Herzegovina', 'Botswana', 'Brazil', 'Brunei', 'Bulgaria', 'Burkina Faso', 'Burundi',

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

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

The variable $current_migration is not named in camelCase.

Open

function checkMigrationStatus(): array
{
    Database::setManager(initializeEntityManager());
    $manager = Database::getManager();
    $connection = $manager->getConnection();

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

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

The variable $fil_perm_verified is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $current_version is not named in camelCase.

Open

function isAlreadyInstalledSystem()
{
    global $new_version, $_configuration;

    if (empty($new_version)) {

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

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

The variable $perms_dir is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $fil_perm_verified is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $extra_notice is not named in camelCase.

Open

function displayDatabaseParameter(
    $installType,
    $parameterName,
    $formFieldName,
    $parameterValue,

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

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

The variable $permissions_for_new_directories is not named in camelCase.

Open

function updateDirAndFilesPermissions()
{
    $table = Database::get_main_table(TABLE_MAIN_SETTINGS);
    $permissions_for_new_directories = isset($_SESSION['permissions_for_new_directories']) ? $_SESSION['permissions_for_new_directories'] : 0770;
    $permissions_for_new_files = isset($_SESSION['permissions_for_new_files']) ? $_SESSION['permissions_for_new_files'] : 0660;

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

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

The variable $wanted_value is not named in camelCase.

Open

function compare_setting_values(string $current_value, string $wanted_value): array
{
    $current_value_string = $current_value;
    $current_value = (float) $current_value;
    $wanted_value = (float) $wanted_value;

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

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

The variable $current_step is not named in camelCase.

Open

function display_step_sequence()
{
    global $current_step;

    return get_lang('Step'.$current_step).' – ';

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

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

The variable $perms_dir is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $dir_perm_verified is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $checked_writable is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $a_countries is not named in camelCase.

Open

function get_countries_list_from_array($combo = false)
{
    $a_countries = [
        'Afghanistan', 'Albania', 'Algeria', 'Andorra', 'Angola', 'Antigua and Barbuda', 'Argentina', 'Armenia', 'Australia', 'Austria', 'Azerbaijan',
        'Bahamas', 'Bahrain', 'Bangladesh', 'Barbados', 'Belarus', 'Belgium', 'Belize', 'Benin', 'Bhutan', 'Bolivia', 'Bosnia and Herzegovina', 'Botswana', 'Brazil', 'Brunei', 'Bulgaria', 'Burkina Faso', 'Burundi',

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

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

The variable $current_value_string is not named in camelCase.

Open

function compare_setting_values(string $current_value, string $wanted_value): array
{
    $current_value_string = $current_value;
    $current_value = (float) $current_value;
    $wanted_value = (float) $wanted_value;

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

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

The variable $progress_percentage is not named in camelCase.

Open

function checkMigrationStatus(): array
{
    Database::setManager(initializeEntityManager());
    $manager = Database::getManager();
    $connection = $manager->getConnection();

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

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

The variable $fil_perm_verified is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $_setting is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $permissions_for_new_directories is not named in camelCase.

Open

function updateDirAndFilesPermissions()
{
    $table = Database::get_main_table(TABLE_MAIN_SETTINGS);
    $permissions_for_new_directories = isset($_SESSION['permissions_for_new_directories']) ? $_SESSION['permissions_for_new_directories'] : 0770;
    $permissions_for_new_files = isset($_SESSION['permissions_for_new_files']) ? $_SESSION['permissions_for_new_files'] : 0660;

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

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

The variable $current_version is not named in camelCase.

Open

function isAlreadyInstalledSystem()
{
    global $new_version, $_configuration;

    if (empty($new_version)) {

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

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

The variable $current_step is not named in camelCase.

Open

function step_active($param)
{
    global $current_step;
    if ($param == $current_step) {
        echo 'install-steps__step--active';

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

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

The variable $dir_perm_verified is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $file_course_test_was_created is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $dir_perm_verified is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $file_course_test_was_created is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $_setting is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $fil_perm_verified is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $course_test_was_created is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $a_countries is not named in camelCase.

Open

function get_countries_list_from_array($combo = false)
{
    $a_countries = [
        'Afghanistan', 'Albania', 'Algeria', 'Andorra', 'Angola', 'Antigua and Barbuda', 'Argentina', 'Armenia', 'Australia', 'Austria', 'Azerbaijan',
        'Bahamas', 'Bahrain', 'Bangladesh', 'Barbados', 'Belarus', 'Belgium', 'Belize', 'Benin', 'Bhutan', 'Bolivia', 'Bosnia and Herzegovina', 'Botswana', 'Brazil', 'Brunei', 'Bulgaria', 'Burkina Faso', 'Burundi',

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

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

The variable $checked_writable is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $_setting is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $dir_perm_verified is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $permissions_for_new_files is not named in camelCase.

Open

function updateDirAndFilesPermissions()
{
    $table = Database::get_main_table(TABLE_MAIN_SETTINGS);
    $permissions_for_new_directories = isset($_SESSION['permissions_for_new_directories']) ? $_SESSION['permissions_for_new_directories'] : 0770;
    $permissions_for_new_files = isset($_SESSION['permissions_for_new_files']) ? $_SESSION['permissions_for_new_files'] : 0660;

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

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

The variable $new_version is not named in camelCase.

Open

function isAlreadyInstalledSystem()
{
    global $new_version, $_configuration;

    if (empty($new_version)) {

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

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

The variable $current_step is not named in camelCase.

Open

function step_active($param)
{
    global $current_step;
    if ($param == $current_step) {
        echo 'install-steps__step--active';

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

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

The variable $current_step is not named in camelCase.

Open

function display_step_sequence()
{
    global $current_step;

    return get_lang('Step'.$current_step).' – ';

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

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

The variable $course_test_was_created is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $file_course_test_was_created is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $checked_writable is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $a_countries is not named in camelCase.

Open

function get_countries_list_from_array($combo = false)
{
    $a_countries = [
        'Afghanistan', 'Albania', 'Algeria', 'Andorra', 'Angola', 'Antigua and Barbuda', 'Argentina', 'Armenia', 'Australia', 'Austria', 'Azerbaijan',
        'Bahamas', 'Bahrain', 'Bangladesh', 'Barbados', 'Belarus', 'Belgium', 'Belize', 'Benin', 'Bhutan', 'Bolivia', 'Bosnia and Herzegovina', 'Botswana', 'Brazil', 'Brunei', 'Bulgaria', 'Burkina Faso', 'Burundi',

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

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

The variable $current_value is not named in camelCase.

Open

function compare_setting_values(string $current_value, string $wanted_value): array
{
    $current_value_string = $current_value;
    $current_value = (float) $current_value;
    $wanted_value = (float) $wanted_value;

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

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

The variable $current_value is not named in camelCase.

Open

function compare_setting_values(string $current_value, string $wanted_value): array
{
    $current_value_string = $current_value;
    $current_value = (float) $current_value;
    $wanted_value = (float) $wanted_value;

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

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

The variable $current_value_string is not named in camelCase.

Open

function compare_setting_values(string $current_value, string $wanted_value): array
{
    $current_value_string = $current_value;
    $current_value = (float) $current_value;
    $wanted_value = (float) $wanted_value;

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

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

The variable $current_version is not named in camelCase.

Open

function isAlreadyInstalledSystem()
{
    global $new_version, $_configuration;

    if (empty($new_version)) {

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

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

The variable $perms_fil is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $permissions_for_new_files is not named in camelCase.

Open

function updateDirAndFilesPermissions()
{
    $table = Database::get_main_table(TABLE_MAIN_SETTINGS);
    $permissions_for_new_directories = isset($_SESSION['permissions_for_new_directories']) ? $_SESSION['permissions_for_new_directories'] : 0770;
    $permissions_for_new_files = isset($_SESSION['permissions_for_new_files']) ? $_SESSION['permissions_for_new_files'] : 0660;

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

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

The variable $progress_percentage is not named in camelCase.

Open

function checkMigrationStatus(): array
{
    Database::setManager(initializeEntityManager());
    $manager = Database::getManager();
    $connection = $manager->getConnection();

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

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

The variable $_configuration is not named in camelCase.

Open

function get_config_param($param, $updatePath = '')
{
    global $updateFromConfigFile;
    if (empty($updatePath) && !empty($_POST['updatePath'])) {
        $updatePath = $_POST['updatePath'];

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

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

The variable $checked_writable is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $wanted_value is not named in camelCase.

Open

function compare_setting_values(string $current_value, string $wanted_value): array
{
    $current_value_string = $current_value;
    $current_value = (float) $current_value;
    $wanted_value = (float) $wanted_value;

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

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

The variable $current_value_string is not named in camelCase.

Open

function compare_setting_values(string $current_value, string $wanted_value): array
{
    $current_value_string = $current_value;
    $current_value = (float) $current_value;
    $wanted_value = (float) $wanted_value;

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

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

The variable $default_language is not named in camelCase.

Open

function display_language_selection_box($name = 'language_list', $default_language = 'en_US')
{
    // Displaying the box.
    return Display::select(
        'language_list',

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

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

The variable $perm_file is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $wanted_value is not named in camelCase.

Open

function compare_setting_values(string $current_value, string $wanted_value): array
{
    $current_value_string = $current_value;
    $current_value = (float) $current_value;
    $wanted_value = (float) $wanted_value;

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

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

The variable $current_migration is not named in camelCase.

Open

function checkMigrationStatus(): array
{
    Database::setManager(initializeEntityManager());
    $manager = Database::getManager();
    $connection = $manager->getConnection();

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

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

The variable $_configuration is not named in camelCase.

Open

function isAlreadyInstalledSystem()
{
    global $new_version, $_configuration;

    if (empty($new_version)) {

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

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

The variable $course_test_was_created is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $perm_file is not named in camelCase.

Open

function display_requirements(
    string $installType,
    bool $badUpdatePath,
    string $updatePath = '',
    array $upgradeFromVersion = []

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

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

The variable $current_value is not named in camelCase.

Open

function compare_setting_values(string $current_value, string $wanted_value): array
{
    $current_value_string = $current_value;
    $current_value = (float) $current_value;
    $wanted_value = (float) $wanted_value;

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

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

The variable $current_value is not named in camelCase.

Open

function compare_setting_values(string $current_value, string $wanted_value): array
{
    $current_value_string = $current_value;
    $current_value = (float) $current_value;
    $wanted_value = (float) $wanted_value;

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

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

The variable $progress_percentage is not named in camelCase.

Open

function checkMigrationStatus(): array
{
    Database::setManager(initializeEntityManager());
    $manager = Database::getManager();
    $connection = $manager->getConnection();

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

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