chamilo/chamilo-lms

View on GitHub
public/main/dropbox/dropbox_functions.inc.php

Summary

Maintainability
A
0 mins
Test Coverage

The function display_addcategory_form() has an NPath complexity of 288. The configured NPath complexity threshold is 200.
Open

function display_addcategory_form($category_name = '', $id = 0, $action = '')
{
    $course_id = api_get_course_int_id();
    $title = get_lang('Add a new folder');

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

function handle_multiple_actions()
{
    $_user = api_get_user_info();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

Missing class import via use statement (line '67', column '28').
Open

        $dropboxfile = new Dropbox_Person($_user['user_id'], $is_courseAdmin, $is_courseTutor);

MissingImport

Since: 2.7.0

Importing all external classes in a file through use statements makes them clearly visible.

Example

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

Source http://phpmd.org/rules/cleancode.html#MissingImport

Missing class import via use statement (line '199', column '17').
Open

    $form = new FormValidator(

MissingImport

Since: 2.7.0

Importing all external classes in a file through use statements makes them clearly visible.

Example

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

Source http://phpmd.org/rules/cleancode.html#MissingImport

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

        $dropbox_person = new Dropbox_Person(

MissingImport

Since: 2.7.0

Importing all external classes in a file through use statements makes them clearly visible.

Example

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

Source http://phpmd.org/rules/cleancode.html#MissingImport

Missing class import via use statement (line '519', column '27').
Open

    $dropbox_person = new Dropbox_Person(

MissingImport

Since: 2.7.0

Importing all external classes in a file through use statements makes them clearly visible.

Example

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

Source http://phpmd.org/rules/cleancode.html#MissingImport

Remove error control operator '@' on line 1059.
Open

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

ErrorControlOperator

Error suppression should be avoided if possible as it doesn't just suppress the error, that you are trying to stop, but will also suppress errors that you didn't predict would ever occur. Consider changing error_reporting() level and/or setting up your own error handler.

Example

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

Source http://phpmd.org/rules/cleancode.html#errorcontroloperator

Missing class import via use statement (line '473', column '17').
Open

    $form = new FormValidator(

MissingImport

Since: 2.7.0

Importing all external classes in a file through use statements makes them clearly visible.

Example

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

Source http://phpmd.org/rules/cleancode.html#MissingImport

Missing class import via use statement (line '528', column '17').
Open

    $form = new FormValidator(

MissingImport

Since: 2.7.0

Importing all external classes in a file through use statements makes them clearly visible.

Example

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

Source http://phpmd.org/rules/cleancode.html#MissingImport

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

        $result = new Dropbox_SentWork(

MissingImport

Since: 2.7.0

Importing all external classes in a file through use statements makes them clearly visible.

Example

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

Source http://phpmd.org/rules/cleancode.html#MissingImport

Missing class import via use statement (line '655', column '22').
Open

    $userGroup = new UserGroupModel();

MissingImport

Since: 2.7.0

Importing all external classes in a file through use statements makes them clearly visible.

Example

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

Source http://phpmd.org/rules/cleancode.html#MissingImport

Missing class import via use statement (line '171', column '28').
Open

        $dropboxfile = new Dropbox_Person($user_id, $is_courseAdmin, $is_courseTutor);

MissingImport

Since: 2.7.0

Importing all external classes in a file through use statements makes them clearly visible.

Example

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

Source http://phpmd.org/rules/cleancode.html#MissingImport

Missing class import via use statement (line '725', column '25').
Open

    $multipleForm = new FormValidator(

MissingImport

Since: 2.7.0

Importing all external classes in a file through use statements makes them clearly visible.

Example

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

Source http://phpmd.org/rules/cleancode.html#MissingImport

Remove error control operator '@' on line 789.
Open

function removeUnusedFiles()
{
    $_course = api_get_course_info();
    $course_id = $_course['real_id'];

ErrorControlOperator

Error suppression should be avoided if possible as it doesn't just suppress the error, that you are trying to stop, but will also suppress errors that you didn't predict would ever occur. Consider changing error_reporting() level and/or setting up your own error handler.

Example

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

Source http://phpmd.org/rules/cleancode.html#errorcontroloperator

Remove error control operator '@' on line 1349.
Open

function zip_download($fileList)
{
    $_course = api_get_course_info();
    $course_id = api_get_course_int_id();
    $fileList = array_map('intval', $fileList);

ErrorControlOperator

Error suppression should be avoided if possible as it doesn't just suppress the error, that you are trying to stop, but will also suppress errors that you didn't predict would ever occur. Consider changing error_reporting() level and/or setting up your own error handler.

Example

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

Source http://phpmd.org/rules/cleancode.html#errorcontroloperator

Missing class import via use statement (line '1334', column '16').
Open

    $zip = new PclZip($temp_zip_file);

MissingImport

Since: 2.7.0

Importing all external classes in a file through use statements makes them clearly visible.

Example

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

Source http://phpmd.org/rules/cleancode.html#MissingImport

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

        } else {
            $part = 'sent';
            $to_cat_id = str_replace('move_sent_', '', $_POST['action']);
        }

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

    } else {
        return get_lang('Error');
    }

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 assigning values to variables in if clauses and the like (line '850', column '9').
Open

function removeMoreIfMailing($file_id)
{
    $course_id = api_get_course_int_id();
    // when deleting a mailing zip-file (posted to mailingPseudoId):
    // 1. the detail window is no longer reachable, so

IfStatementAssignment

Since: 2.7.0

Assignments in if clauses and the like are considered a code smell. Assignments in PHP return the right operand as their result. In many cases, this is an expected behavior, but can lead to many difficult to spot bugs, especially when the right operand could result in zero, null or an empty string and the like.

Example

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

Source http://phpmd.org/rules/cleancode.html#ifstatementassignment

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

        } else {
            // Check if all the recipients are valid
            $thisIsAMailing = false;
            $thisIsJustUpload = false;

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

    } else {
        $params = [
            'title' => $_POST['category_name'],
            'received' => $received,
            'sent' => $sent,

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

        } else {
            $complete_user_list_for_dropbox = CourseManager::get_teacher_list_from_course_code(
                $course_info['code'],
                false
            );

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 assigning values to variables in if clauses and the like (line '819', column '11').
Open

function getUserOwningThisMailing($mailingPseudoId, $owner = 0, $or_die = '')
{
    $course_id = api_get_course_int_id();

    $mailingPseudoId = (int) $mailingPseudoId;

IfStatementAssignment

Since: 2.7.0

Assignments in if clauses and the like are considered a code smell. Assignments in PHP return the right operand as their result. In many cases, this is an expected behavior, but can lead to many difficult to spot bugs, especially when the right operand could result in zero, null or an empty string and the like.

Example

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

Source http://phpmd.org/rules/cleancode.html#ifstatementassignment

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

    } else {
        $return_message = get_lang('The file(s) can not be moved.');
    }

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

    } else {
        foreach ($_POST as $key => $value) {
            if (strstr($value, $part.'_') && 'view_received_category' != $key && 'view_sent_category' != $key) {
                $checked_files = true;
                $checked_file_ids[] = intval(substr($value, strrpos($value, '_')));

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

    } else {
        // Update
        $work->title = $dropbox_title;
        $work->filename = $dropbox_filename;
        $work->filesize = $dropbox_filesize;

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

    } else {
        $return .= get_lang('All users have deleted the file so nobody will see the feedback you are adding.');
    }

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

            } else {
                $courseCoachList[] = $coach;
            }

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

    } else {
        return get_lang('Error');
    }

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

    } else {  // rename file to login_filename_uniqueId format
        $dropbox_filename = $_user['username']."_".$dropbox_filename."_".uniqid('');
    }

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

        } else {
            return ['type' => 'error', 'message' => get_lang('This category already exists, please use a different name')];
        }

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

    } else {
        if ('true' == api_get_setting('dropbox_allow_student_to_student') || STUDENT != $_user['status']) {
            $complete_user_list_for_dropbox = CourseManager::get_user_list_from_course_code(
                $course_info['code'],
                api_get_session_id(),

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

        } else {
            return 0;
        }

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

    } else {
        $table = Database::get_course_table(TABLE_DROPBOX_FEEDBACK);
        $params = [
            'c_id' => $course_id,
            'file_id' => $_GET['id'],

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

                } else {
                    $return .= $value[$column_value];
                }

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

    } else {
        return 0;
    }

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 '$found'.
Open

                $found = true; // note: do we still need this?

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

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

                $checked_files = true;

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

Avoid unused parameters such as '$p_event'.
Open

function my_pre_add_callback($p_event, &$p_header)

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 '$column_key'.
Open

        foreach ($column as $column_key => $column_value) {

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

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

            foreach ($columns_array as $columns_array_key => $columns_array_value) {

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

function getUserOwningThisMailing($mailingPseudoId, $owner = 0, $or_die = '')
{
    $course_id = api_get_course_int_id();

    $mailingPseudoId = (int) $mailingPseudoId;

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

function user_can_download_file($id, $user_id)
{
    $course_id = api_get_course_int_id();
    $id = (int) $id;
    $user_id = (int) $user_id;

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

function delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function display_addcategory_form($category_name = '', $id = 0, $action = '')
{
    $course_id = api_get_course_int_id();
    $title = get_lang('Add a new folder');

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

function display_move_form(
    $part,
    $id,
    $target = [],
    $extra_params = [],

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

function removeMoreIfMailing($file_id)
{
    $course_id = api_get_course_int_id();
    // when deleting a mailing zip-file (posted to mailingPseudoId):
    // 1. the detail window is no longer reachable, so

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

function my_pre_add_callback($p_event, &$p_header)
{
    $files = Session::read('dropbox_files_to_download');
    $p_header['stored_filename'] = $files[$p_header['stored_filename']]['title'];

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

function my_pre_add_callback($p_event, &$p_header)
{
    $files = Session::read('dropbox_files_to_download');
    $p_header['stored_filename'] = $files[$p_header['stored_filename']]['title'];

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

function get_last_tool_access($tool, $courseId = null, $user_id = null)
{
    // The default values of the parameters
    if (empty($courseId)) {
        $courseId = api_get_course_int_id();

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

function generate_html_overview($files, $dont_show_columns = [], $make_link = [])
{
    $return = '<!DOCTYPE html'."\n";
    $return .= "\t".'PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"'."\n";
    $return .= "\t".'"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">'."\n";

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

function generate_html_overview($files, $dont_show_columns = [], $make_link = [])
{
    $return = '<!DOCTYPE html'."\n";
    $return .= "\t".'PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"'."\n";
    $return .= "\t".'"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">'."\n";

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

    $is_courseAdmin = api_is_course_admin();

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

        $checked_file_ids = $_POST['id'];

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

    $is_courseTutor = api_is_course_tutor();

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

                $checked_file_ids[] = intval(substr($value, strrpos($value, '_')));

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

                $checked_files = true;

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

function handle_multiple_actions()

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

    $_user = api_get_user_info();

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

$this_section = SECTION_COURSES;

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

    $checked_file_ids = $_POST['id'];

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

    if (!is_array($checked_file_ids) || 0 == count($checked_file_ids)) {

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

        $dropboxfile = new Dropbox_Person($_user['user_id'], $is_courseAdmin, $is_courseTutor);

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

                        c_id = $course_id AND

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

            WHERE c_id = $course_id AND user_id='".$_user['user_id']."' $condition_session";

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

    return $return_array;

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

    $course_id = api_get_course_int_id();

Missing parameter name
Open

 * @param $view

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

            $userId = $current_user['user_id'];

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

                $current_user_id = $current_user['user_id'];

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

                $options['user_'.$current_user_id] = $full_name;

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

    if (($dropbox_person->isCourseTutor || $dropbox_person->isCourseAdmin)

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

        if (count($complete_group_list_for_dropbox) > 0) {

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

        foreach ($checked_file_ids as $key => $value) {

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

    if ($cat['user_id'] != $user_id && !api_is_platform_admin($user_id)) {

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

        $return_message = get_lang('The folder has been deleted');

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

                'user_id' => $_user['user_id'],

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

        $complete_user_list_for_dropbox = [];

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

                $options['user_'.$current_user_id] = $full_name;

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

    if (!is_array($checked_file_ids) || 0 == count($checked_file_ids)) {

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

        zip_download($checked_file_ids);

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

    $is_courseAdmin = api_is_course_admin();

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

        $return_message = get_lang('The folder has been deleted');

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

    $course_id = api_get_course_int_id();

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

    $course_id = api_get_course_int_id();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)

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

        $id_field = 'file_id';

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

        $entries_table = Database::get_course_table(TABLE_DROPBOX_FILE);

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

        api_get_self().'?view_received_category='.$viewReceivedCategory.'&view_sent_category='.$viewSentCategory.'&view='.$view.'&'.$extra_params

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

            WHERE c_id = $course_id AND user_id='".$_user['user_id']."' $condition_session";

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

function display_addcategory_form($category_name = '', $id = 0, $action = '')

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

        $complete_user_list_for_dropbox = TableSort::sort_table($complete_user_list_for_dropbox, 'lastcommafirst');

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

        $dropboxfile = new Dropbox_Person($_user['user_id'], $is_courseAdmin, $is_courseTutor);

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

                    WHERE c_id = $course_id AND dest_user_id = ".intval($_user['user_id'])."

Missing parameter name
Open

 * @param $viewReceivedCategory

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

    $is_courseTutor = api_is_course_tutor();

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

            $course_info['code'],

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

        if ('true' == api_get_setting('dropbox_allow_student_to_student') || STUDENT != $_user['status']) {

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

        foreach ($complete_user_list_for_dropbox as $k => $e) {

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

            $complete_user_list_for_dropbox[$k] = $e + [

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

    $current_user_id = '';

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

                    $options['group_'.$current_group['iid']] = 'G: '.$current_group['name'].' - '.$current_group['number_of_members'].' '.get_lang('Users');

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

function removeUnusedFiles()

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

        $dropboxfile = new Dropbox_Person($_user['user_id'], $is_courseAdmin, $is_courseTutor);

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

        $dropboxfile = new Dropbox_Person($user_id, $is_courseAdmin, $is_courseTutor);

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

    return $return_message;

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

    $extra_params = [],

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

            $return_array[$row['cat_id']] = $row;

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

    $_user = api_get_user_info();

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

                $course_info['code'],

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

        $complete_user_list2 = CourseManager::get_coach_list_from_course_code(

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

                $complete_user_list_for_dropbox,

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

                || 5 != $current_user['status']                         // Always allow teachers.

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

                if ($current_group['number_of_members'] > 0) {

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

            store_move($value, $to_cat_id, $part);

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

            WHERE c_id = $course_id AND cat_id='".intval($id)."' AND $sentreceived='1'";

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

        $return_message = get_lang('The file(s) can not be moved.');

Missing parameter name
Open

 * @param $filter default '', when we need only the categories of the sent or the received part

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

    $course_id = api_get_course_int_id();

Expected 18 spaces after parameter type; 1 found
Open

 * @param int $id

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

    $dropbox_person = new Dropbox_Person(

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

                $course_info['code'],

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

                $current_user_id = $current_user['user_id'];

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

    if (($dropbox_person->isCourseTutor || $dropbox_person->isCourseAdmin)

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

            foreach ($complete_group_list_for_dropbox as $current_group) {

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

                    $options['group_'.$current_group['iid']] = 'G: '.$current_group['name'].' - '.$current_group['number_of_members'].' '.get_lang('Users');

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

function get_dropbox_category($id)

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

    $_user = api_get_user_info();

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

                $complete_user_list_for_dropbox,

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

            $complete_user_list_for_dropbox = CourseManager::get_user_list_from_course_code(

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

            $complete_user_list_for_dropbox = CourseManager::get_teacher_list_from_course_code(

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

function getDropboxConf()

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

function delete_category($action, $id, $user_id = null)

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

            $dropboxfile->deleteReceivedWork($row[$id_field]);

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

    $session_id = api_get_session_id();

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

            WHERE c_id = $course_id AND user_id='".$_user['user_id']."' $condition_session";

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

                    c_id = $course_id AND

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

                    $course_id,

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

        if (empty($category_name)) {

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

            $category_name = $row['title'];

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

            $dropbox_person->isCourseTutor

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

            foreach ($complete_group_list_for_dropbox as $current_group) {

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

        $user_id = api_get_user_id();

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

    if ($cat['user_id'] != $user_id && !api_is_platform_admin($user_id)) {

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

    $course_id = api_get_course_int_id();

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

    $is_courseAdmin = api_is_course_admin();

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

                $course_info['code'],

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

function delete_category($action, $id, $user_id = null)

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

    if (empty($user_id)) {

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

        $dropboxfile = new Dropbox_Person($user_id, $is_courseAdmin, $is_courseTutor);

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

            $dropboxfile->deleteSentWork($row[$id_field]);

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

    $_user = api_get_user_info();

Missing parameter name
Open

 * @param int The category ID

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

                    session_id='$session_id'";

Missing parameter name
Open

 * @param $viewSentCategory

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

        $is_courseTutor

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

    foreach ($complete_user_list_for_dropbox as $current_user) {

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

                ) && $current_user['user_id'] != $_user['user_id']) {   // Don't include yourself.

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

                    $options['group_'.$current_group['iid']] = 'G: '.$current_group['name'].' - '.$current_group['number_of_members'].' '.get_lang('Users');

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

    $is_courseTutor = api_is_course_tutor();

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

function display_move_form(

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

function get_dropbox_categories($filter = '')

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

                'c_id' => $course_id,

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

        foreach ($complete_user_list2 as $coach) {

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

        $complete_user_list_for_dropbox = TableSort::sort_table($complete_user_list_for_dropbox, 'lastcommafirst');

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

            if ($current_user['user_id'] == $current_user_id) {

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

            $to_cat_id = str_replace('move_sent_', '', $_POST['action']);

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

    $course_id = api_get_course_int_id();

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

            WHERE c_id = $course_id AND cat_id='".intval($id)."'";

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

                    user_id='".$_user['user_id']."' AND

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

                'session_id' => $session_id,

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

    $defaults['category_name'] = Security::remove_XSS($category_name);

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

    $course_info = api_get_course_info();

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

        if ('true' == api_get_setting('dropbox_allow_student_to_student') || STUDENT != $_user['status']) {

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

    foreach ($complete_user_list_for_dropbox as $current_user) {

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

                || $dropbox_person->isCourseAdmin

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

                || 1 == $current_user['is_tutor']                       // Always allow tutors.

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

                $full_name = $userInfo['complete_name'].$groupNameListToString;

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

            $to_cat_id = str_replace('move_received_', '', $_POST['action']);

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

        foreach ($checked_file_ids as $value) {

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

        $dropboxfile = new Dropbox_Person($user_id, $is_courseAdmin, $is_courseTutor);

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

                    WHERE c_id = $course_id AND dest_user_id = ".intval($_user['user_id'])."

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

            $return_message = get_lang('The sent file has been moved');

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

    $_user = api_get_user_info();

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

            WHERE c_id = $course_id AND cat_id='".$id."'";

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

        $session_id = api_get_session_id();

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

function display_addcategory_form($category_name = '', $id = 0, $action = '')

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

            $complete_user_list_for_dropbox = CourseManager:: get_user_list_from_course_code(

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

                ) && $current_user['user_id'] != $_user['user_id']) {   // Don't include yourself.

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

        $id_field = 'id';

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

                        uploader_id = ".intval($_user['user_id'])." AND

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

    $condition_session = api_get_session_condition($session_id);

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

        $entries_table = Database::get_course_table(TABLE_DROPBOX_POST);

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

    $sql = "SELECT * FROM ".$entries_table."

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

function store_move($id, $target, $part)

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

            $return_message = get_lang('The received file has been moved.');

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

    return $return_message;

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

    $return_array = [];

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

    $condition_session = api_get_session_condition($session_id);

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

function store_addcategory()

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

                    $_user['user_id'],

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

                WHERE c_id = $course_id AND cat_id = ".$id;

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

        $is_courseAdmin,

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

            $complete_user_list_for_dropbox = array_merge(

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

            $complete_user_list_for_dropbox = array_merge(

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

    if (!empty($complete_user_list_for_dropbox)) {

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

            if ($current_user['user_id'] == $current_user_id) {

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

        $complete_group_list_for_dropbox = GroupManager::get_group_list(null, $course);

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

        $options['user_'.$_user['user_id']] = get_lang('---Just upload---');

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

                WHERE c_id = $course_id AND id ='".$res['id']."'";

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

    $dropbox_filetype = $file['type'];

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

    $dropbox_filename = php2phps($dropbox_filename);

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

                    if (!in_array($usr['user_id'], $new_work_recipients) && $usr['user_id'] != $_user['user_id']) {

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

        foreach ($new_work_recipients as $recipient_id) {

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

    $course_id = api_get_course_int_id();

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

            ON (person.file_id=file.id AND file.c_id = $course_id AND person.c_id = $course_id)

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

    $temp_zip_file = api_get_path(SYS_ARCHIVE_PATH).api_get_unique_id().".zip";

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

            api_get_path(SYS_COURSE_PATH).$_course['path'].'/dropbox/'.$value['filename'],

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

function generate_html_overview($files, $dont_show_columns = [], $make_link = [])

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

    $_user = api_get_user_info();

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

    $dropbox_filename = api_replace_dangerous_char($dropbox_filename);

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

                $dropbox_filename = $w->filename;

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

                        $new_work_recipients[] = $usr['user_id'];

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

                    $_user['firstName'],

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

                $_user['mail']

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

function my_pre_add_callback($p_event, &$p_header)

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

function getUserOwningThisMailing($mailingPseudoId, $owner = 0, $or_die = '')

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

    $file_id = (int) $file_id;

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

    $_course = api_get_course_info();

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

    $dropbox_filename = php2phps($dropbox_filename);

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

    if (!filter_extension($dropbox_filename)) {

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

    $dropbox_title = $dropbox_filename;

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

        api_get_path(SYS_COURSE_PATH).$_course['path'].'/dropbox/'.$dropbox_filename

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

                    $_user['lastName'],

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

            $_user['user_id'],

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

            $dropbox_filesize,

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

            $new_work_recipients

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

        $work->filesize = $dropbox_filesize;

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

    return $number_users_who_see_file > 0 || $count > 0;

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

    $_course = api_get_course_info();

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

function generate_html_overview($files, $dont_show_columns = [], $make_link = [])

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

                $column[] = $columns_array_value;

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

function store_add_dropbox($file = [], $work = null)

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

                        $_course['code'],

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

    $dropbox_filename = api_replace_dangerous_char($dropbox_filename);

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

function user_can_download_file($id, $user_id)

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

    $course_id = api_get_course_int_id();

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

            WHERE c_id = $course_id AND file_id = ".$id;

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

            ON (post.file_id = file.id AND post.c_id = $course_id AND file.c_id = $course_id)

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

    $result = DocumentManager::file_send_for_download($temp_zip_file, true, $name);

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

function removeMoreIfMailing($file_id)

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

    $upload_ok = process_uploaded_file($file, true);

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

        foreach ($dropbox_person->sentWork as $w) {

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

            $recipent_temp = api_get_user_info($recipient_id);

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

            WHERE c_id = $course_id AND file_id = ".$id;

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

            ON (post.file_id = file.id AND post.c_id = $course_id AND file.c_id = $course_id)

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

function generate_html_overview($files, $dont_show_columns = [], $make_link = [])

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

    $_course = api_get_course_info();

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

            WHERE c_id = $course_id AND p.file_id = '".$file_id."'";

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

                    WHERE c_id = $course_id AND uploader_id='".$mailingPseudoId."'";

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

    if ($dropbox_filesize <= 0 || $dropbox_filesize > $maxFileSize) {

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

            $recipent_temp = api_get_user_info($recipient_id);

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

                )."\n".get_lang('e-mail')." : ".$_user['mail'],

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

            $dropbox_filename,

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

    $number_users_who_see_file = check_if_file_exist($_GET['id']);

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

            WHERE c_id = $course_id AND user_id = $user_id AND file_id = ".$id;

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

    $number_users_who_see_file = Database::num_rows($result);

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

function my_pre_add_callback($p_event, &$p_header)

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

                    $return .= "\n\t<th>".$columns_array_value."</th>";

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

        @unlink(api_get_path(SYS_COURSE_PATH).$_course['path'].'/dropbox/'.$res['filename']);

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

    $file_id = (int) $file_id;

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

    $dropbox_filename = add_ext_on_mime($dropbox_filename, $dropbox_filetype);

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

        $dropbox_person = new Dropbox_Person(

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

        $dropbox_filename = $_user['username']."_".$dropbox_filename."_".uniqid('');

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

                if (!in_array($columns_array_value, $dont_show_columns)) {

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

    $course_id = $_course['real_id'];

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

                    WHERE c_id = $course_id AND user_id='".$mailingPseudoId."'";

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

        foreach ($new_work_recipients as $recipient_id) {

Missing function doc comment
Open

function user_can_download_file($id, $user_id)

You must use "/**" style comments for a function comment
Open

function check_if_file_exist($id)

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

    $zip = new PclZip($temp_zip_file);

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

            $columns_array = array_keys($value);

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

    $course_id = api_get_course_int_id();

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

        $dropbox_overwrite = true;

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

                if (($w->recipients[0]['id'] == $_user['user_id']) xor $thisIsJustUpload) {

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

                    $recipent_temp['firstname'].' '.$recipent_temp['lastname'],

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

        $work->upload_date = api_get_utc_datetime();

Missing parameter name
Open

 * @param an array containing all the feedback about the given message

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

function user_can_download_file($id, $user_id)

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

    $user_id = (int) $user_id;

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

    $p_header['stored_filename'] = $files[$p_header['stored_filename']]['title'];

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

            ON (f.id = p.file_id AND f.c_id = $course_id AND p.c_id = $course_id)

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

        $work->title = $dropbox_title;

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

    if ($number_users_who_see_file) {

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

    $number_users_who_see_file = Database::num_rows($result);

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

function zip_download($fileList)

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

                p.c_id = $course_id

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

    $dropbox_overwrite = false;

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

        api_get_path(SYS_COURSE_PATH).$_course['path'].'/dropbox/'.$dropbox_filename

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

                    $recipent_temp['firstname'].' '.$recipent_temp['lastname'],

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

            WHERE c_id = $course_id AND dest_user_id = $user_id AND file_id = ".$id;

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

function my_pre_add_callback($p_event, &$p_header)

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

    $dropbox_filesize = $file['size'];

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

    if ($dropbox_filesize <= 0 || $dropbox_filesize > $maxFileSize) {

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

    $dropbox_filename = add_ext_on_mime($dropbox_filename, $dropbox_filetype);

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

            $_user['user_id'],

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

                    $_user['firstName'],

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

            $dropbox_title,

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

function store_feedback()

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

            foreach ($columns_array as $columns_array_key => $columns_array_value) {

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

    $course_id = $_course['real_id'];

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

function getUserOwningThisMailing($mailingPseudoId, $owner = 0, $or_die = '')

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

            ON (f.id = p.file_id AND f.c_id = $course_id AND p.c_id = $course_id)

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

    $course_id = api_get_course_int_id();

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

    $dropbox_filename = add_ext_on_mime($dropbox_filename, $dropbox_filetype);

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

                $new_work_recipients[] = substr($rec, strlen('user_'));

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

                    if (!in_array($usr['user_id'], $new_work_recipients) && $usr['user_id'] != $_user['user_id']) {

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

    $b_send_mail = api_get_course_setting('email_alert_on_new_doc_dropbox');

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

        $work->filename = $dropbox_filename;

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

function format_feedback($feedback)

Missing parameter name
Open

 * @param $url  url string

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

function feedback_form($url)

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

function check_if_file_exist($id)

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

            foreach ($columns_array as $columns_array_key => $columns_array_value) {

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

    exit(get_lang('An error has occured. Please contact your system administrator.').' (code '.$or_die.')');

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

function removeMoreIfMailing($file_id)

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

    $dropbox_filename = $file['name'];

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

    $dropbox_filetmpname = $file['tmp_name'];

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

    if (!is_uploaded_file($dropbox_filetmpname)) { // check user fraud : no clean error msg.

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

    $dropbox_title = $dropbox_filename;

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

    if ($dropbox_overwrite && empty($work)) {

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

            if ($w->title == $dropbox_filename) {

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

        $dropbox_filename = $_user['username']."_".$dropbox_filename."_".uniqid('');

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

                    $_user['lastName'],

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

        $work->uploader_id = api_get_user_id();

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

function feedback($array, $url)

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

    $user_id = (int) $user_id;

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

    return $number_users_who_see_file > 0 || $count > 0;

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

    @unlink($temp_zip_file);

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

            foreach ($columns_array as $columns_array_key => $columns_array_value) {

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

        foreach ($column as $column_key => $column_value) {

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

                $recipent_temp['email'],

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

        $work->last_upload_date = api_get_utc_datetime();

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

            WHERE c_id = $course_id AND dest_user_id = $user_id AND file_id = ".$id;

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

            'c_id' => $course_id,

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

    $course_id = api_get_course_int_id();

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

            ON (person.file_id=file.id AND file.c_id = $course_id AND person.c_id = $course_id)

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

                if (!in_array($columns_array_value, $dont_show_columns)) {

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

                  f.c_id = $course_id

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

                WHERE c_id = $course_id AND file_id = '".$res['id']."'";

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

            WHERE c_id = $course_id AND p.file_id = '".$file_id."'";

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

    if (!$upload_ok) {

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

        $dropbox_filename = $_user['username']."_".$dropbox_filename."_".uniqid('');

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

        $new_work_recipients = [];

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

        $dropbox_filetmpname,

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

    if ($b_send_mail && empty($work)) {

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

            WHERE c_id = $course_id AND user_id = $user_id AND file_id = ".$id;

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

    $course_id = api_get_course_int_id();

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

    $p_header['stored_filename'] = $files[$p_header['stored_filename']]['title'];

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

function get_mail_id_base()

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

        foreach ($column as $column_key => $column_value) {

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

    if (empty($user_id)) {

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

                    $return .= '<a href="'.$value[$column_value].'">'.$value[$column_value].'</a>';

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

                    $return .= $value[$column_value];

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

            if (!in_array($column_value, $dont_show_columns)) {

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

    $course_id = api_get_course_int_id();

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

function get_last_tool_access($tool, $courseId = null, $user_id = null)

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

    $sql = "SELECT access_date FROM $table_last_access

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

            if (!in_array($column_value, $dont_show_columns)) {

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

                    $return .= '<a href="'.$value[$column_value].'">'.$value[$column_value].'</a>';

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

function get_total_number_feedback()

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

            WHERE c_id = $course_id

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

function check_number_feedback($key, $array)

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

    $table_last_access = Database::get_main_table('track_e_lastaccess');

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

                if (in_array($column_value, $make_link)) {

Missing parameter name
Open

 * @param $courseId the course_id

Missing parameter name
Open

 * @param $user_id the id of the user

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

function get_last_tool_access($tool, $courseId = null, $user_id = null)

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

        $user_id = api_get_user_id();

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

                if (in_array($column_value, $make_link)) {

Missing parameter name
Open

 * @param $tool string the tool constant

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

                access_user_id = ".intval($user_id)." AND

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

    $viewReceivedCategory,

The variable $checked_files is not named in camelCase.
Open

function handle_multiple_actions()
{
    $_user = api_get_user_info();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function handle_multiple_actions()
{
    $_user = api_get_user_info();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function handle_multiple_actions()
{
    $_user = api_get_user_info();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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 delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function store_addcategory()
{
    $course_id = api_get_course_int_id();
    $_user = api_get_user_info();

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

function store_addcategory()
{
    $course_id = api_get_course_int_id();
    $_user = api_get_user_info();

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

function store_addcategory()
{
    $course_id = api_get_course_int_id();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function handle_multiple_actions()
{
    $_user = api_get_user_info();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function get_dropbox_category($id)
{
    $course_id = api_get_course_int_id();
    $id = (int) $id;

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

function store_addcategory()
{
    $course_id = api_get_course_int_id();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function handle_multiple_actions()
{
    $_user = api_get_user_info();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function store_move($id, $target, $part)
{
    $_user = api_get_user_info();
    $course_id = api_get_course_int_id();

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

function get_dropbox_categories($filter = '')
{
    $course_id = api_get_course_int_id();
    $_user = api_get_user_info();
    $return_array = [];

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

function store_addcategory()
{
    $course_id = api_get_course_int_id();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function removeUnusedFiles()
{
    $_course = api_get_course_info();
    $course_id = $_course['real_id'];

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

function removeMoreIfMailing($file_id)
{
    $course_id = api_get_course_int_id();
    // when deleting a mailing zip-file (posted to mailingPseudoId):
    // 1. the detail window is no longer reachable, so

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function user_can_download_file($id, $user_id)
{
    $course_id = api_get_course_int_id();
    $id = (int) $id;
    $user_id = (int) $user_id;

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 user_can_download_file($id, $user_id)
{
    $course_id = api_get_course_int_id();
    $id = (int) $id;
    $user_id = (int) $user_id;

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

function delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function display_move_form(
    $part,
    $id,
    $target = [],
    $extra_params = [],

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

function get_dropbox_categories($filter = '')
{
    $course_id = api_get_course_int_id();
    $_user = api_get_user_info();
    $return_array = [];

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function feedback_form($url)
{
    $return = '<div class="feeback-form">';
    $number_users_who_see_file = check_if_file_exist($_GET['id']);
    if ($number_users_who_see_file) {

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

function handle_multiple_actions()
{
    $_user = api_get_user_info();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function store_move($id, $target, $part)
{
    $_user = api_get_user_info();
    $course_id = api_get_course_int_id();

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

function get_dropbox_categories($filter = '')
{
    $course_id = api_get_course_int_id();
    $_user = api_get_user_info();
    $return_array = [];

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

function store_addcategory()
{
    $course_id = api_get_course_int_id();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function getUserOwningThisMailing($mailingPseudoId, $owner = 0, $or_die = '')
{
    $course_id = api_get_course_int_id();

    $mailingPseudoId = (int) $mailingPseudoId;

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

function removeMoreIfMailing($file_id)
{
    $course_id = api_get_course_int_id();
    // when deleting a mailing zip-file (posted to mailingPseudoId):
    // 1. the detail window is no longer reachable, so

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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 user_can_download_file($id, $user_id)
{
    $course_id = api_get_course_int_id();
    $id = (int) $id;
    $user_id = (int) $user_id;

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

function user_can_download_file($id, $user_id)
{
    $course_id = api_get_course_int_id();
    $id = (int) $id;
    $user_id = (int) $user_id;

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

function handle_multiple_actions()
{
    $_user = api_get_user_info();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function handle_multiple_actions()
{
    $_user = api_get_user_info();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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 delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function store_move($id, $target, $part)
{
    $_user = api_get_user_info();
    $course_id = api_get_course_int_id();

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

function store_move($id, $target, $part)
{
    $_user = api_get_user_info();
    $course_id = api_get_course_int_id();

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

function get_dropbox_categories($filter = '')
{
    $course_id = api_get_course_int_id();
    $_user = api_get_user_info();
    $return_array = [];

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function removeMoreIfMailing($file_id)
{
    $course_id = api_get_course_int_id();
    // when deleting a mailing zip-file (posted to mailingPseudoId):
    // 1. the detail window is no longer reachable, so

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function get_dropbox_categories($filter = '')
{
    $course_id = api_get_course_int_id();
    $_user = api_get_user_info();
    $return_array = [];

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

function store_addcategory()
{
    $course_id = api_get_course_int_id();
    $_user = api_get_user_info();

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

function display_addcategory_form($category_name = '', $id = 0, $action = '')
{
    $course_id = api_get_course_int_id();
    $title = get_lang('Add a new folder');

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function removeMoreIfMailing($file_id)
{
    $course_id = api_get_course_int_id();
    // when deleting a mailing zip-file (posted to mailingPseudoId):
    // 1. the detail window is no longer reachable, so

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function handle_multiple_actions()
{
    $_user = api_get_user_info();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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 delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function store_move($id, $target, $part)
{
    $_user = api_get_user_info();
    $course_id = api_get_course_int_id();

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

function get_dropbox_categories($filter = '')
{
    $course_id = api_get_course_int_id();
    $_user = api_get_user_info();
    $return_array = [];

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_move($id, $target, $part)
{
    $_user = api_get_user_info();
    $course_id = api_get_course_int_id();

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

function store_addcategory()
{
    $course_id = api_get_course_int_id();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function getUserOwningThisMailing($mailingPseudoId, $owner = 0, $or_die = '')
{
    $course_id = api_get_course_int_id();

    $mailingPseudoId = (int) $mailingPseudoId;

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

function getUserOwningThisMailing($mailingPseudoId, $owner = 0, $or_die = '')
{
    $course_id = api_get_course_int_id();

    $mailingPseudoId = (int) $mailingPseudoId;

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function user_can_download_file($id, $user_id)
{
    $course_id = api_get_course_int_id();
    $id = (int) $id;
    $user_id = (int) $user_id;

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

function delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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 delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function get_dropbox_category($id)
{
    $course_id = api_get_course_int_id();
    $id = (int) $id;

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function removeUnusedFiles()
{
    $_course = api_get_course_info();
    $course_id = $_course['real_id'];

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

function getUserOwningThisMailing($mailingPseudoId, $owner = 0, $or_die = '')
{
    $course_id = api_get_course_int_id();

    $mailingPseudoId = (int) $mailingPseudoId;

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function handle_multiple_actions()
{
    $_user = api_get_user_info();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function handle_multiple_actions()
{
    $_user = api_get_user_info();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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 delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function store_move($id, $target, $part)
{
    $_user = api_get_user_info();
    $course_id = api_get_course_int_id();

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

function get_dropbox_categories($filter = '')
{
    $course_id = api_get_course_int_id();
    $_user = api_get_user_info();
    $return_array = [];

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

function get_dropbox_categories($filter = '')
{
    $course_id = api_get_course_int_id();
    $_user = api_get_user_info();
    $return_array = [];

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

function get_dropbox_categories($filter = '')
{
    $course_id = api_get_course_int_id();
    $_user = api_get_user_info();
    $return_array = [];

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function handle_multiple_actions()
{
    $_user = api_get_user_info();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function get_dropbox_categories($filter = '')
{
    $course_id = api_get_course_int_id();
    $_user = api_get_user_info();
    $return_array = [];

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

function store_addcategory()
{
    $course_id = api_get_course_int_id();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function removeUnusedFiles()
{
    $_course = api_get_course_info();
    $course_id = $_course['real_id'];

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

function removeMoreIfMailing($file_id)
{
    $course_id = api_get_course_int_id();
    // when deleting a mailing zip-file (posted to mailingPseudoId):
    // 1. the detail window is no longer reachable, so

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function handle_multiple_actions()
{
    $_user = api_get_user_info();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function handle_multiple_actions()
{
    $_user = api_get_user_info();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function store_move($id, $target, $part)
{
    $_user = api_get_user_info();
    $course_id = api_get_course_int_id();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function removeUnusedFiles()
{
    $_course = api_get_course_info();
    $course_id = $_course['real_id'];

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

function removeUnusedFiles()
{
    $_course = api_get_course_info();
    $course_id = $_course['real_id'];

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

function getUserOwningThisMailing($mailingPseudoId, $owner = 0, $or_die = '')
{
    $course_id = api_get_course_int_id();

    $mailingPseudoId = (int) $mailingPseudoId;

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

function removeMoreIfMailing($file_id)
{
    $course_id = api_get_course_int_id();
    // when deleting a mailing zip-file (posted to mailingPseudoId):
    // 1. the detail window is no longer reachable, so

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function handle_multiple_actions()
{
    $_user = api_get_user_info();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function delete_category($action, $id, $user_id = null)
{
    $course_id = api_get_course_int_id();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function store_move($id, $target, $part)
{
    $_user = api_get_user_info();
    $course_id = api_get_course_int_id();

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

function display_addcategory_form($category_name = '', $id = 0, $action = '')
{
    $course_id = api_get_course_int_id();
    $title = get_lang('Add a new folder');

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

function display_addcategory_form($category_name = '', $id = 0, $action = '')
{
    $course_id = api_get_course_int_id();
    $title = get_lang('Add a new folder');

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function handle_multiple_actions()
{
    $_user = api_get_user_info();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function store_move($id, $target, $part)
{
    $_user = api_get_user_info();
    $course_id = api_get_course_int_id();

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

function get_dropbox_categories($filter = '')
{
    $course_id = api_get_course_int_id();
    $_user = api_get_user_info();
    $return_array = [];

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

function store_addcategory()
{
    $course_id = api_get_course_int_id();
    $_user = api_get_user_info();

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

function display_addcategory_form($category_name = '', $id = 0, $action = '')
{
    $course_id = api_get_course_int_id();
    $title = get_lang('Add a new folder');

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

function display_addcategory_form($category_name = '', $id = 0, $action = '')
{
    $course_id = api_get_course_int_id();
    $title = get_lang('Add a new folder');

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function handle_multiple_actions()
{
    $_user = api_get_user_info();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function handle_multiple_actions()
{
    $_user = api_get_user_info();
    $is_courseAdmin = api_is_course_admin();
    $is_courseTutor = api_is_course_tutor();

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

function store_addcategory()
{
    $course_id = api_get_course_int_id();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function feedback_form($url)
{
    $return = '<div class="feeback-form">';
    $number_users_who_see_file = check_if_file_exist($_GET['id']);
    if ($number_users_who_see_file) {

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function removeUnusedFiles()
{
    $_course = api_get_course_info();
    $course_id = $_course['real_id'];

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

function removeMoreIfMailing($file_id)
{
    $course_id = api_get_course_int_id();
    // when deleting a mailing zip-file (posted to mailingPseudoId):
    // 1. the detail window is no longer reachable, so

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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 user_can_download_file($id, $user_id)
{
    $course_id = api_get_course_int_id();
    $id = (int) $id;
    $user_id = (int) $user_id;

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

function generate_html_overview($files, $dont_show_columns = [], $make_link = [])
{
    $return = '<!DOCTYPE html'."\n";
    $return .= "\t".'PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"'."\n";
    $return .= "\t".'"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">'."\n";

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

function generate_html_overview($files, $dont_show_columns = [], $make_link = [])
{
    $return = '<!DOCTYPE html'."\n";
    $return .= "\t".'PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"'."\n";
    $return .= "\t".'"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">'."\n";

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

function generate_html_overview($files, $dont_show_columns = [], $make_link = [])
{
    $return = '<!DOCTYPE html'."\n";
    $return .= "\t".'PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"'."\n";
    $return .= "\t".'"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">'."\n";

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

function get_total_number_feedback()
{
    $course_id = api_get_course_int_id();
    $sql = "SELECT COUNT(feedback_id) AS total, file_id
            FROM ".Database::get_course_table(TABLE_DROPBOX_FEEDBACK)."

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id = 0)
{
    $course_info = api_get_course_info();
    $course = api_get_course_entity();
    $_user = api_get_user_info();

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

function removeUnusedFiles()
{
    $_course = api_get_course_info();
    $course_id = $_course['real_id'];

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function store_add_dropbox($file = [], $work = null)
{
    $_course = api_get_course_info();
    $_user = api_get_user_info();

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

function zip_download($fileList)
{
    $_course = api_get_course_info();
    $course_id = api_get_course_int_id();
    $fileList = array_map('intval', $fileList);

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

function zip_download($fileList)
{
    $_course = api_get_course_info();
    $course_id = api_get_course_int_id();
    $fileList = array_map('intval', $fileList);

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

function generate_html_overview($files, $dont_show_columns = [], $make_link = [])
{
    $return = '<!DOCTYPE html'."\n";
    $return .= "\t".'PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"'."\n";
    $return .= "\t".'"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">'."\n";

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

function generate_html_overview($files, $dont_show_columns = [], $make_link = [])
{
    $return = '<!DOCTYPE html'."\n";
    $return .= "\t".'PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"'."\n";
    $return .= "\t".'"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">'."\n";

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

function generate_html_overview($files, $dont_show_columns = [], $make_link = [])
{
    $return = '<!DOCTYPE html'."\n";
    $return .= "\t".'PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"'."\n";
    $return .= "\t".'"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">'."\n";

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

function zip_download($fileList)
{
    $_course = api_get_course_info();
    $course_id = api_get_course_int_id();
    $fileList = array_map('intval', $fileList);

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

function zip_download($fileList)
{
    $_course = api_get_course_info();
    $course_id = api_get_course_int_id();
    $fileList = array_map('intval', $fileList);

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

function generate_html_overview($files, $dont_show_columns = [], $make_link = [])
{
    $return = '<!DOCTYPE html'."\n";
    $return .= "\t".'PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"'."\n";
    $return .= "\t".'"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">'."\n";

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

function zip_download($fileList)
{
    $_course = api_get_course_info();
    $course_id = api_get_course_int_id();
    $fileList = array_map('intval', $fileList);

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

function generate_html_overview($files, $dont_show_columns = [], $make_link = [])
{
    $return = '<!DOCTYPE html'."\n";
    $return .= "\t".'PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"'."\n";
    $return .= "\t".'"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">'."\n";

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

function user_can_download_file($id, $user_id)
{
    $course_id = api_get_course_int_id();
    $id = (int) $id;
    $user_id = (int) $user_id;

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

function zip_download($fileList)
{
    $_course = api_get_course_info();
    $course_id = api_get_course_int_id();
    $fileList = array_map('intval', $fileList);

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

function my_pre_add_callback($p_event, &$p_header)
{
    $files = Session::read('dropbox_files_to_download');
    $p_header['stored_filename'] = $files[$p_header['stored_filename']]['title'];

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

function generate_html_overview($files, $dont_show_columns = [], $make_link = [])
{
    $return = '<!DOCTYPE html'."\n";
    $return .= "\t".'PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"'."\n";
    $return .= "\t".'"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">'."\n";

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

function generate_html_overview($files, $dont_show_columns = [], $make_link = [])
{
    $return = '<!DOCTYPE html'."\n";
    $return .= "\t".'PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"'."\n";
    $return .= "\t".'"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">'."\n";

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

function check_if_file_exist($id)
{
    $id = (int) $id;
    $course_id = api_get_course_int_id();
    $sql = "SELECT file_id

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

function zip_download($fileList)
{
    $_course = api_get_course_info();
    $course_id = api_get_course_int_id();
    $fileList = array_map('intval', $fileList);

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

function generate_html_overview($files, $dont_show_columns = [], $make_link = [])
{
    $return = '<!DOCTYPE html'."\n";
    $return .= "\t".'PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"'."\n";
    $return .= "\t".'"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">'."\n";

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

function generate_html_overview($files, $dont_show_columns = [], $make_link = [])
{
    $return = '<!DOCTYPE html'."\n";
    $return .= "\t".'PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"'."\n";
    $return .= "\t".'"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">'."\n";

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

function get_last_tool_access($tool, $courseId = null, $user_id = null)
{
    // The default values of the parameters
    if (empty($courseId)) {
        $courseId = api_get_course_int_id();

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 user_can_download_file($id, $user_id)
{
    $course_id = api_get_course_int_id();
    $id = (int) $id;
    $user_id = (int) $user_id;

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

function check_if_file_exist($id)
{
    $id = (int) $id;
    $course_id = api_get_course_int_id();
    $sql = "SELECT file_id

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

function zip_download($fileList)
{
    $_course = api_get_course_info();
    $course_id = api_get_course_int_id();
    $fileList = array_map('intval', $fileList);

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

function zip_download($fileList)
{
    $_course = api_get_course_info();
    $course_id = api_get_course_int_id();
    $fileList = array_map('intval', $fileList);

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 get_last_tool_access($tool, $courseId = null, $user_id = null)
{
    // The default values of the parameters
    if (empty($courseId)) {
        $courseId = api_get_course_int_id();

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

function generate_html_overview($files, $dont_show_columns = [], $make_link = [])
{
    $return = '<!DOCTYPE html'."\n";
    $return .= "\t".'PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"'."\n";
    $return .= "\t".'"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">'."\n";

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

function generate_html_overview($files, $dont_show_columns = [], $make_link = [])
{
    $return = '<!DOCTYPE html'."\n";
    $return .= "\t".'PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"'."\n";
    $return .= "\t".'"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">'."\n";

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

function user_can_download_file($id, $user_id)
{
    $course_id = api_get_course_int_id();
    $id = (int) $id;
    $user_id = (int) $user_id;

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

function check_if_file_exist($id)
{
    $id = (int) $id;
    $course_id = api_get_course_int_id();
    $sql = "SELECT file_id

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

function check_if_file_exist($id)
{
    $id = (int) $id;
    $course_id = api_get_course_int_id();
    $sql = "SELECT file_id

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

function zip_download($fileList)
{
    $_course = api_get_course_info();
    $course_id = api_get_course_int_id();
    $fileList = array_map('intval', $fileList);

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

function get_total_number_feedback()
{
    $course_id = api_get_course_int_id();
    $sql = "SELECT COUNT(feedback_id) AS total, file_id
            FROM ".Database::get_course_table(TABLE_DROPBOX_FEEDBACK)."

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

function zip_download($fileList)
{
    $_course = api_get_course_info();
    $course_id = api_get_course_int_id();
    $fileList = array_map('intval', $fileList);

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

function store_feedback()
{
    if (!is_numeric($_GET['id'])) {
        return get_lang('Feedback error');
    }

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

function generate_html_overview($files, $dont_show_columns = [], $make_link = [])
{
    $return = '<!DOCTYPE html'."\n";
    $return .= "\t".'PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"'."\n";
    $return .= "\t".'"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">'."\n";

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

function get_last_tool_access($tool, $courseId = null, $user_id = null)
{
    // The default values of the parameters
    if (empty($courseId)) {
        $courseId = api_get_course_int_id();

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 get_last_tool_access($tool, $courseId = null, $user_id = null)
{
    // The default values of the parameters
    if (empty($courseId)) {
        $courseId = api_get_course_int_id();

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

function check_if_file_exist($id)
{
    $id = (int) $id;
    $course_id = api_get_course_int_id();
    $sql = "SELECT file_id

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

function my_pre_add_callback($p_event, &$p_header)
{
    $files = Session::read('dropbox_files_to_download');
    $p_header['stored_filename'] = $files[$p_header['stored_filename']]['title'];

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 get_last_tool_access($tool, $courseId = null, $user_id = null)
{
    // The default values of the parameters
    if (empty($courseId)) {
        $courseId = api_get_course_int_id();

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

function store_feedback()
{
    if (!is_numeric($_GET['id'])) {
        return get_lang('Feedback error');
    }

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

function generate_html_overview($files, $dont_show_columns = [], $make_link = [])
{
    $return = '<!DOCTYPE html'."\n";
    $return .= "\t".'PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"'."\n";
    $return .= "\t".'"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">'."\n";

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

function generate_html_overview($files, $dont_show_columns = [], $make_link = [])
{
    $return = '<!DOCTYPE html'."\n";
    $return .= "\t".'PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"'."\n";
    $return .= "\t".'"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">'."\n";

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