Sign upLogin

chamilo/chamilo-lms

View on GitHub
Star
public/main/exercise/exercise_result.class.php

Summary

Maintainability
A
0 mins
Test Coverage

The method getExercisesReporting() has an NPath complexity of 1874880. The configured NPath complexity threshold is 200.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

NPathComplexity

Since: 0.1

The NPath complexity of a method is the number of acyclic execution paths through that method. A threshold of 200 is generally considered the point where measures should be taken to reduce complexity.

Example

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

Source https://phpmd.org/rules/codesize.html#npathcomplexity

The method exportCompleteReportXLS() has an NPath complexity of 3720. The configured NPath complexity threshold is 200.
Open

    public function exportCompleteReportXLS(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

NPathComplexity

Since: 0.1

The NPath complexity of a method is the number of acyclic execution paths through that method. A threshold of 200 is generally considered the point where measures should be taken to reduce complexity.

Example

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

Source https://phpmd.org/rules/codesize.html#npathcomplexity

The method exportCompleteReportCSV() has an NPath complexity of 16896. The configured NPath complexity threshold is 200.
Open

    public function exportCompleteReportCSV(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

NPathComplexity

Since: 0.1

The NPath complexity of a method is the number of acyclic execution paths through that method. A threshold of 200 is generally considered the point where measures should be taken to reduce complexity.

Example

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

Source https://phpmd.org/rules/codesize.html#npathcomplexity

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

                } else {
                    //revised or not
                    $sql_exe = "SELECT exe_id
                                FROM $tblTrackAttemptQualify
                                WHERE
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

ElseExpression

Since: 1.4.0

An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

Example

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

Source https://phpmd.org/rules/cleancode.html#elseexpression

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

        } else {
            header('Content-Disposition: attachment; filename= '.$filename);
        }
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

ElseExpression

Since: 1.4.0

An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

Example

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

Source https://phpmd.org/rules/cleancode.html#elseexpression

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

        } else {
            $user_id_and = ' AND te.exe_user_id = '.api_get_user_id().' ';
            $orderBy = 'lastname';
            if (api_is_western_name_order()) {
                $orderBy = 'firstname';
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

ElseExpression

Since: 1.4.0

An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

Example

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

Source https://phpmd.org/rules/cleancode.html#elseexpression

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

                } else {
                    $list[$column][] = api_html_entity_decode(
                        strip_tags($row['last_name']),
                        ENT_QUOTES,
                        $charset
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

ElseExpression

Since: 1.4.0

An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

Example

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

Source https://phpmd.org/rules/cleancode.html#elseexpression

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

            } else {
                $results[] = $rowx;
            }
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

ElseExpression

Since: 1.4.0

An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

Example

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

Source https://phpmd.org/rules/cleancode.html#elseexpression

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

        } else {
            if (!empty($this->results[0]['lastname'])) {
                $data .= get_lang('Last name').';';
            }
            if (!empty($this->results[0]['firstname'])) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

ElseExpression

Since: 1.4.0

An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

Example

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

Source https://phpmd.org/rules/cleancode.html#elseexpression

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

                } else {
                    if ($rowx['exresult'] > $bestAttemptPerUser[$rowx['excruid']]['exresult']) {
                        $bestAttemptPerUser[$rowx['excruid']] = $rowx;
                    }
                }
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

ElseExpression

Since: 1.4.0

An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

Example

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

Source https://phpmd.org/rules/cleancode.html#elseexpression

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

            } else {
                $list[0][] = get_lang('Last name');
                $list[0][] = get_lang('First name');
            }
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

ElseExpression

Since: 1.4.0

An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

Example

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

Source https://phpmd.org/rules/cleancode.html#elseexpression

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

        } else {
            $students = CourseManager::get_user_list_from_course_code($cid, $sessionId);
        }
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

ElseExpression

Since: 1.4.0

An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

Example

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

Source https://phpmd.org/rules/cleancode.html#elseexpression

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

                } else {
                    $return[$i]['is_user_subscribed'] = get_lang('No');
                }
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

ElseExpression

Since: 1.4.0

An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

Example

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

Source https://phpmd.org/rules/cleancode.html#elseexpression

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

            } else {
                $data .= str_replace("\r\n", '  ', api_html_entity_decode(strip_tags($row['lastname']), ENT_QUOTES, $charset)).';';
                $data .= str_replace("\r\n", '  ', api_html_entity_decode(strip_tags($row['firstname']), ENT_QUOTES, $charset)).';';
            }
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

ElseExpression

Since: 1.4.0

An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

Example

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

Source https://phpmd.org/rules/cleancode.html#elseexpression

Avoid unused parameters such as '$document_path'.
Open

        $document_path = '',
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

UnusedFormalParameter

Since: 0.2

Avoid passing parameters to methods or constructors and then not using those parameters.

Example

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

Source https://phpmd.org/rules/unusedcode.html#unusedformalparameter

Avoid unused parameters such as '$document_path'.
Open

        $document_path = '',
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

UnusedFormalParameter

Since: 0.2

Avoid passing parameters to methods or constructors and then not using those parameters.

Example

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

Source https://phpmd.org/rules/unusedcode.html#unusedformalparameter

Avoid unused local variables such as '$isWestern'.
Open

                        $isWestern = api_is_western_name_order();
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

UnusedLocalVariable

Since: 0.2

Detects when a local variable is declared and/or assigned, but not used.

Example

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

Source https://phpmd.org/rules/unusedcode.html#unusedlocalvariable

The parameter $export_user_fields is not named in camelCase.
Open

    public function exportCompleteReportCSV(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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

The parameter $exercise_id is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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

The parameter $user_id is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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

The parameter $exercise_id is not named in camelCase.
Open

    public function exportCompleteReportCSV(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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

The parameter $exercise_id is not named in camelCase.
Open

    public function exportCompleteReportXLS(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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

The parameter $export_filter is not named in camelCase.
Open

    public function exportCompleteReportCSV(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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

The parameter $export_user_fields is not named in camelCase.
Open

    public function exportCompleteReportXLS(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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

The parameter $export_filter is not named in camelCase.
Open

    public function exportCompleteReportXLS(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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

The parameter $hotpotato_name is not named in camelCase.
Open

    public function exportCompleteReportXLS(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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

The parameter $document_path is not named in camelCase.
Open

    public function exportCompleteReportXLS(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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

The parameter $user_id is not named in camelCase.
Open

    public function exportCompleteReportCSV(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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

The parameter $document_path is not named in camelCase.
Open

    public function exportCompleteReportCSV(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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

The parameter $user_id is not named in camelCase.
Open

    public function exportCompleteReportXLS(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 "TBL_EXERCISES" is not in valid camel caps format
Open

                FROM $TBL_EXERCISES AS ce
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                        te.c_id = ce.c_id $user_id_and $session_id_and AND
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                    $filter_by_not_revised = true;
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                        if (empty($user_id)) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

            if (!empty($extra_user_fields)) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                foreach ($extra_user_fields as $field) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

            $hotpotato_name
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $exercise_id = 0
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                    LEFT JOIN $TBL_TABLE_LP_MAIN AS tlm
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $filter_by_not_revised = false;
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                    $query = Database::query($sql_exe);
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

            if ($export_user_fields) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                    foreach ($user_fields_values as $value) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $document_path = '',
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

            $exercise_id,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $user_id = null,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $user_id = (int) $user_id;
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $user_id = (int) $user_id;
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                LEFT JOIN $TBL_TABLE_LP_MAIN AS tlm
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $course_id = api_get_course_int_id();
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $exercise_id = (int) $exercise_id;
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

            $session_id_and .= " AND exe_exo_id = $exercise_id ";
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

            $user_id_and = ' AND te.exe_user_id = '.api_get_user_id().' ';
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $user_id = null,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

            $user_id,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        if ($export_user_fields) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        if (!empty($exercise_id)) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

            $filename = 'exercise_results_user_'.$user_id.'_'.$now;
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

            if ($export_user_fields) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                    INNER JOIN $TBL_USER AS user
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $filter_by_revised = false;
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $TBL_TRACK_EXERCISES = Database::get_main_table(TABLE_STATISTIC_TRACK_E_EXERCISES);
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        if (empty($user_id)) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                INNER JOIN $TBL_TRACK_EXERCISES AS te
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $exercise_id = 0
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        if (!empty($user_id)) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $exercise_id = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        if (!empty($user_id)) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

            $extra_user_fields = UserManager::get_extra_fields(
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $user_id = null,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $session_id_and = $sessionCondition;
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                    FROM $TBL_EXERCISES  AS ce
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                    $filter_by_revised = true;
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

            $filename = 'exercise_results_user_'.$user_id.'_'.$now.'.csv';
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                $user_fields_values = UserManager::get_extra_user_data(
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

            foreach ($extra_user_fields as $field) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $TBL_TABLE_LP_MAIN = Database::get_course_table(TABLE_LP_MAIN);
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        if ($export_user_fields) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $export_user_fields = false,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                if ($filter_by_not_revised && 1 === $revised) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                if ($filter_by_revised && $revised < 1) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $export_user_fields = false,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                if (!empty($user_fields_values)) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                    te.c_id = ce.c_id $user_id_and  $session_id_and AND
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                    INNER JOIN $TBL_TRACK_EXERCISES AS te
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $document_path = '',
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

            $export_filter,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $exercise_id = (int) $exercise_id;
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

            $user_id_and = null;
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                    te.c_id = ce.c_id $user_id_and  $session_id_and AND
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

            $export_filter,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

            $exercise_id
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $hotpotato_name = null
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $TBL_USER = Database::get_main_table(TABLE_MAIN_USER);
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

            $session_id_and .= " AND exe_exo_id = $exercise_id ";
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                    ce.c_id = $course_id AND
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

        $TBL_EXERCISES = Database::get_course_table(TABLE_QUIZ_TEST);
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                    $sql_exe = "SELECT exe_id
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                INNER JOIN $TBL_USER AS user
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                        ce.c_id = $course_id AND
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                        te.c_id = ce.c_id $user_id_and $session_id_and AND
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

                if (empty($user_id)) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

            $user_id,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

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

            $extra_user_fields = UserManager::get_extra_fields(
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpcodesniffer

The 'getExercisesReporting()' method which returns a boolean should be named 'is...()' or 'has...()'
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

BooleanGetMethodName

Since: 0.2

Looks for methods named 'getX()' with 'boolean' as the return type. The convention is to name these methods 'isX()' or 'hasX()'.

Example

class Foo {
    /**
     * @return boolean
     */
    public function getFoo() {} // bad
    /**
     * @return bool
     */
    public function isFoo(); // ok
    /**
     * @return boolean
     */
    public function getFoo($bar); // ok, unless checkParameterizedMethods=true
}

Source https://phpmd.org/rules/naming.html#booleangetmethodname

The variable $user_id is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $exercise_id is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $TBL_EXERCISES is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $exercise_id is not named in camelCase.
Open

    public function exportCompleteReportCSV(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $hotpotato_name is not named in camelCase.
Open

    public function exportCompleteReportXLS(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $export_user_fields is not named in camelCase.
Open

    public function exportCompleteReportXLS(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $TBL_TABLE_LP_MAIN is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $exercise_id is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $course_id is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $session_id_and is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $sql_exe is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $user_id is not named in camelCase.
Open

    public function exportCompleteReportXLS(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $TBL_TABLE_LP_MAIN is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $TBL_USER is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $user_id_and is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $filter_by_not_revised is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $sql_exe is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $user_id is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $user_id is not named in camelCase.
Open

    public function exportCompleteReportXLS(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $export_user_fields is not named in camelCase.
Open

    public function exportCompleteReportXLS(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $TBL_USER is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $user_id_and is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $TBL_TRACK_EXERCISES is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $TBL_TABLE_LP_MAIN is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $user_id is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $TBL_TRACK_EXERCISES is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $TBL_USER is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $TBL_EXERCISES is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $user_fields_values is not named in camelCase.
Open

    public function exportCompleteReportCSV(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $user_id is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $user_id is not named in camelCase.
Open

    public function exportCompleteReportCSV(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $user_fields_values is not named in camelCase.
Open

    public function exportCompleteReportCSV(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $extra_user_fields is not named in camelCase.
Open

    public function exportCompleteReportXLS(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $course_id is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $course_id is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $filter_by_revised is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $extra_user_fields is not named in camelCase.
Open

    public function exportCompleteReportCSV(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $user_id is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $user_id_and is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $TBL_EXERCISES is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $session_id_and is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $export_user_fields is not named in camelCase.
Open

    public function exportCompleteReportCSV(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $export_user_fields is not named in camelCase.
Open

    public function exportCompleteReportCSV(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $session_id_and is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $filter_by_revised is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $extra_user_fields is not named in camelCase.
Open

    public function exportCompleteReportXLS(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $exercise_id is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $user_id is not named in camelCase.
Open

    public function exportCompleteReportXLS(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $exercise_id is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $extra_user_fields is not named in camelCase.
Open

    public function exportCompleteReportCSV(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $extra_user_fields is not named in camelCase.
Open

    public function exportCompleteReportCSV(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $filter_by_revised is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $filter_by_not_revised is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $export_filter is not named in camelCase.
Open

    public function exportCompleteReportCSV(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $user_fields_values is not named in camelCase.
Open

    public function exportCompleteReportCSV(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $export_filter is not named in camelCase.
Open

    public function exportCompleteReportXLS(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $user_id_and is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $filter_by_not_revised is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $session_id_and is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $TBL_TRACK_EXERCISES is not named in camelCase.
Open

    public function getExercisesReporting(
        $user_id = null,
        $filter = 0,
        $exercise_id = 0
    ) {
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $user_id is not named in camelCase.
Open

    public function exportCompleteReportCSV(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $user_id is not named in camelCase.
Open

    public function exportCompleteReportCSV(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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 $exercise_id is not named in camelCase.
Open

    public function exportCompleteReportXLS(
        $document_path = '',
        $user_id = null,
        $export_user_fields = false,
        $export_filter = 0,
Severity: Minor
Found in public/main/exercise/exercise_result.class.php by phpmd

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