chamilo/chamilo-lms

View on GitHub
public/main/admin/course_user_import.php

Summary

Maintainability
A
0 mins
Test Coverage

The function save_data() has an NPath complexity of 404. The configured NPath complexity threshold is 200.
Open

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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 function validate_data() has an NPath complexity of 401. The configured NPath complexity threshold is 200.
Open

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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 save_data uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
Open

        } else {
            $courseInfo = $courseListCache[$user_course['CourseCode']];
        }

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 save_data uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
Open

                } else {
                    $courseInfo = api_get_course_info_by_id($courseId);
                }

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 validate_data uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
Open

                } else {
                    $coursecodes[$user_course['CourseCode']] = 1;
                }

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 local variables such as '$key'.
Open

        foreach ($mandatory_fields as $key => $field) {

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

                    $user_course['error'] = get_lang('This code does not exist');

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

                    $coursecodes[$user_course['CourseCode']] = 1;

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

        if (isset($user_course['Status']) && 0 != strlen($user_course['Status'])) {

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

            $courseInfo = $courseListCache[$user_course['CourseCode']];

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

        $to_subscribe = array_diff(array_keys($csv_subscriptions), array_keys($db_subscriptions));

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

        $to_unsubscribe = array_diff(array_keys($db_subscriptions), array_keys($csv_subscriptions));

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

                    $csv_subscriptions[$courseId]

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

    $users_courses = parse_csv_data($_FILES['import_file']['tmp_name']);

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

        $error_message .= $error_course['Code'].' '.$error_course['Title'];

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

        if (isset($user_course['Status']) && 0 != strlen($user_course['Status'])) {

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

    foreach ($users_courses as $user_course) {

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

            foreach ($to_unsubscribe as $courseId) {

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

    $error_message = '<ul>';

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

        $error_message .= $error_course['Code'].' '.$error_course['Title'];

Consider putting global function "validate_data" in a static class
Open

function validate_data($users_courses)

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

        $user_course['line'] = $index + 1;

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

        $mandatory_fields = ['UserName', 'CourseCode', 'Status'];

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

    foreach ($csv_data as $username => $csv_subscriptions) {

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

        $to_subscribe = array_diff(array_keys($csv_subscriptions), array_keys($db_subscriptions));

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

    echo Display::return_message($error_message, 'error', false);

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

            $courseInfo = api_get_course_info($user_course['CourseCode']);

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

    foreach ($csv_data as $username => $csv_subscriptions) {

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

$tool_name = get_lang('Add users to course').' CSV';

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

            if (!isset($coursecodes[$user_course['CourseCode']])) {

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

                    $errors[] = $user_course;

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

            if (UserManager::is_username_available($user_course['UserName'])) {

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

function save_data($users_courses)

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

    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);

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

        $csv_data[$user_course['UserName']][$courseInfo['real_id']] = $user_course['Status'];

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

        if (!empty($inserted_in_course)) {

Consider putting global function "parse_csv_data" in a static class
Open

function parse_csv_data($file)

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

            if (COURSEMANAGER != $user_course['Status'] && STUDENT != $user_course['Status']) {

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

        $sql = "SELECT * FROM $course_user_table cu

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

            if (!isset($user_course[$field]) || 0 == strlen($user_course[$field])) {

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

        $db_subscriptions = [];

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

            foreach ($to_subscribe as $courseId) {

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

                CourseManager::unsubscribe_user($user_id, $courseCode);

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

        $inserted_in_course = save_data($users_courses);

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

function validate_data($users_courses)

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

            if (!isset($user_course[$field]) || 0 == strlen($user_course[$field])) {

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

                $user_course['error'] = get_lang($field.'Mandatory');

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

        if (isset($user_course['UserName']) && 0 != strlen($user_course['UserName'])) {

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

            $db_subscriptions[$obj->c_id] = $obj->status;

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

        $to_subscribe = array_diff(array_keys($csv_subscriptions), array_keys($db_subscriptions));

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

        $inserted_in_course = save_data($users_courses);

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

        $error_message .= '<li>'.get_lang('Line').' '.$error_course['line'].': <strong>'.$error_course['error'].'</strong>: ';

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

                $courseListCache[$user_course['CourseCode']] = $courseInfo;

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

        $error_message .= '<li>'.get_lang('Line').' '.$error_course['line'].': <strong>'.$error_course['error'].'</strong>: ';

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

    $error_message .= '</ul>';

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

        foreach ($mandatory_fields as $key => $field) {

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

                $errors[] = $user_course;

Consider putting global function "save_data" in a static class
Open

function save_data($users_courses)

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

        $to_unsubscribe = array_diff(array_keys($db_subscriptions), array_keys($csv_subscriptions));

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

                    $user_id,

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

                    $inserted_in_course[$courseInfo['code']] = $courseInfo['title'];

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

    foreach ($errors as $index => $error_course) {

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

        $error_message .= '<li>'.get_lang('Line').' '.$error_course['line'].': <strong>'.$error_course['error'].'</strong>: ';

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

        if (isset($user_course['CourseCode']) && 0 != strlen($user_course['CourseCode'])) {

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

        if (isset($user_course['UserName']) && 0 != strlen($user_course['UserName'])) {

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

        $csv_data[$user_course['UserName']][$courseInfo['real_id']] = $user_course['Status'];

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

                WHERE cu.user_id = $user_id AND cu.relation_type <> ".COURSE_RELATION_TYPE_RRHH.' ';

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

    $errors = validate_data($users_courses);

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

                $courseInfo = api_get_course_info($user_course['CourseCode']);

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

            if (COURSEMANAGER != $user_course['Status'] && STUDENT != $user_course['Status']) {

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

        $csv_data[$user_course['UserName']][$courseInfo['real_id']] = $user_course['Status'];

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

    return $inserted_in_course;

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

        $error_message .= $error_course['Code'].' '.$error_course['Title'];

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

                $user_course['error'] = get_lang('Unknown status');

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

                $errors[] = $user_course;

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

    $inserted_in_course = [];

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

            $db_subscriptions[$obj->c_id] = $obj->status;

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

$this_section = SECTION_PLATFORM_ADMIN;

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

$form->addElement('header', '', $tool_name);

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

        $error_message .= '</li>';

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

    foreach ($users_courses as $index => $user_course) {

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

    foreach ($users_courses as $index => $user_course) {

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

        if (isset($user_course['CourseCode']) && 0 != strlen($user_course['CourseCode'])) {

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

        $to_unsubscribe = array_diff(array_keys($db_subscriptions), array_keys($csv_subscriptions));

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

Display :: display_header($tool_name);

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

                $errors[] = $user_course;

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

                $user_course['error'] = get_lang('Unknown user');

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

    $csv_data = [];

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

    foreach ($users_courses as $user_course) {

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

        if (!in_array($user_course['CourseCode'], array_keys($courseListCache))) {

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

        $user_id = $userInfo['user_id'];

The variable $csv_data is not named in camelCase.
Open

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function validate_data($users_courses)
{
    $errors = [];
    $coursecodes = [];
    foreach ($users_courses as $index => $user_course) {

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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

function save_data($users_courses)
{
    $course_user_table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
    $csv_data = [];
    $inserted_in_course = [];

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