chamilo/chamilo-lms

View on GitHub
public/main/inc/lib/SkillRelUserModel.php

Summary

Maintainability
A
0 mins
Test Coverage

The parameter $skill_list is not named in camelCase.
Open

    public function getUserBySkills($skill_list)
    {
        $users = [];
        if (!empty($skill_list)) {
            $skill_list = array_map('intval', $skill_list);

CamelCaseParameterName

Since: 0.2

It is considered best practice to use the camelCase notation to name parameters.

Example

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

Source

Variable "skill_list" is not in valid camel caps format
Open

            $skill_list = implode("', '", $skill_list);

Variable "skill_list" is not in valid camel caps format
Open

    public function getUserBySkills($skill_list)

Variable "skill_list" is not in valid camel caps format
Open

        if (!empty($skill_list)) {

Variable "skill_list" is not in valid camel caps format
Open

            $skill_list = array_map('intval', $skill_list);

Variable "skill_list" is not in valid camel caps format
Open

            $skill_list = implode("', '", $skill_list);

Variable "skill_list" is not in valid camel caps format
Open

            $skill_list = array_map('intval', $skill_list);

Missing class doc comment
Open

class SkillRelUserModel extends Model

Missing function doc comment
Open

    public function __construct()

Variable "skill_list" is not in valid camel caps format
Open

                    WHERE skill_id IN ('$skill_list') ";

The variable $skill_list is not named in camelCase.
Open

    public function getUserBySkills($skill_list)
    {
        $users = [];
        if (!empty($skill_list)) {
            $skill_list = array_map('intval', $skill_list);

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

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

Source

The variable $skill_list is not named in camelCase.
Open

    public function getUserBySkills($skill_list)
    {
        $users = [];
        if (!empty($skill_list)) {
            $skill_list = array_map('intval', $skill_list);

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

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

Source

The variable $skill_list is not named in camelCase.
Open

    public function getUserBySkills($skill_list)
    {
        $users = [];
        if (!empty($skill_list)) {
            $skill_list = array_map('intval', $skill_list);

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

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

Source

The variable $skill_list is not named in camelCase.
Open

    public function getUserBySkills($skill_list)
    {
        $users = [];
        if (!empty($skill_list)) {
            $skill_list = array_map('intval', $skill_list);

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

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

Source

The variable $skill_list is not named in camelCase.
Open

    public function getUserBySkills($skill_list)
    {
        $users = [];
        if (!empty($skill_list)) {
            $skill_list = array_map('intval', $skill_list);

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

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

Source

The variable $skill_list is not named in camelCase.
Open

    public function getUserBySkills($skill_list)
    {
        $users = [];
        if (!empty($skill_list)) {
            $skill_list = array_map('intval', $skill_list);

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

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

Source

There are no issues that match your filters.

Category
Status