chamilo/chamilo-lms

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

Summary

Maintainability
A
0 mins
Test Coverage

The method createNewSentWork() has an NPath complexity of 351. The configured NPath complexity threshold is 200.
Open

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

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 '476', column '25').
Open

            $temp = new Dropbox_Work($res['file_id']);

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 '493', column '37').
Open

            $this->sentWork[] = new Dropbox_SentWork($res['iid']);

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

Avoid assigning values to variables in if clauses and the like (line '370', column '18').
Open

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

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

        } else {
            $this->upload_date = $this->last_upload_date;
            $params = [
                'c_id' => $course_id,
                'uploader_id' => $this->uploader_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 __construct uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
Open

        } else {
            $this->createExistingSentWork($arg1);
        }

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

        } else {
            $this->uploader_id = $uploader_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

Avoid assigning values to variables in if clauses and the like (line '373', column '18').
Open

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

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

        } else {
            $this->createExistingWork($arg1);
        }

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

            } else {
                $this->recipients[] = [
                    'id' => $dest_user_id,
                    'name' => $user_info['complete_name'],
                    'user_id' => $dest_user_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 property $last_upload_date is not named in camelCase.
Open

class Dropbox_Work
{
    public $id;
    public $uploader_id;
    public $filename;

CamelCasePropertyName

Since: 0.2

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

Example

class ClassName {
    protected $property_name;
}

Source

The class Dropbox_Work is not named in CamelCase.
Open

class Dropbox_Work
{
    public $id;
    public $uploader_id;
    public $filename;

CamelCaseClassName

Since: 0.2

It is considered best practice to use the CamelCase notation to name classes.

Example

class class_name {
}

Source

The class Dropbox_SentWork is not named in CamelCase.
Open

class Dropbox_SentWork extends Dropbox_Work
{
    public $recipients; //array of ['id']['name'] arrays

    /**

CamelCaseClassName

Since: 0.2

It is considered best practice to use the CamelCase notation to name classes.

Example

class class_name {
}

Source

The class Dropbox_Person is not named in CamelCase.
Open

class Dropbox_Person
{
    // The receivedWork and the sentWork arrays are sorted.
    public $receivedWork; // an array of Dropbox_Work objects
    public $sentWork; // an array of Dropbox_SentWork objects

CamelCaseClassName

Since: 0.2

It is considered best practice to use the CamelCase notation to name classes.

Example

class class_name {
}

Source

The parameter $uploader_id is not named in camelCase.
Open

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

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

class Dropbox_Work
{
    public $id;
    public $uploader_id;
    public $filename;

CamelCasePropertyName

Since: 0.2

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

Example

class ClassName {
    protected $property_name;
}

Source

The parameter $uploader_id is not named in camelCase.
Open

    public function createNewWork($uploader_id, $title, $description, $author, $filename, $filesize)
    {
        // Fill in the properties
        $this->uploader_id = (int) $uploader_id;
        $this->filename = $filename;

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

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

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

class Dropbox_Work
{
    public $id;
    public $uploader_id;
    public $filename;

CamelCasePropertyName

Since: 0.2

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

Example

class ClassName {
    protected $property_name;
}

Source

The property $uploader_id is not named in camelCase.
Open

class Dropbox_Work
{
    public $id;
    public $uploader_id;
    public $filename;

CamelCasePropertyName

Since: 0.2

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

Example

class ClassName {
    protected $property_name;
}

Source

The property $_orderBy is not named in camelCase.
Open

class Dropbox_Person
{
    // The receivedWork and the sentWork arrays are sorted.
    public $receivedWork; // an array of Dropbox_Work objects
    public $sentWork; // an array of Dropbox_SentWork objects

CamelCasePropertyName

Since: 0.2

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

Example

class ClassName {
    protected $property_name;
}

Source

Each class must be in a file by itself
Open

class Dropbox_Person

Each class must be in a file by itself
Open

class Dropbox_SentWork extends Dropbox_Work

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

                'uploader_id' => $this->uploader_id,

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

                'last_upload_date' => $this->last_upload_date,

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

            'upload_date' => $this->upload_date,

Multiple classes defined in a single file
Open

class Dropbox_SentWork extends Dropbox_Work

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

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)

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

                    VALUES ($course_id, $file_id, $user_id, $session_id, '$now', 0)";

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

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

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

                WHERE c_id = $course_id AND user_id='".$this->userId."' AND file_id='".$id."'";

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

        $this->last_upload_date = api_get_utc_datetime();

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

                    c_id = $course_id AND

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

        $uploader_id = stripslashes($res['uploader_id']);

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

            while ($row_feedback = Database::fetch_array($result)) {

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

            'last_upload_date' => $this->last_upload_date,

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

        foreach ($recipient_ids as $rec) {

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

                        VALUES ($course_id, $file_id, $user_id)";

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

                    'name' => $user_info['complete_name'],

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

                INNER JOIN $person_tbl p

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

                    $condition_session

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

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

Member variable "last_upload_date" is not in valid camel caps format
Open

    public $last_upload_date;

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

            'uploader_id' => $this->uploader_id,

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

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

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

        $session_id = api_get_session_id();

Property name "$_orderBy" should not be prefixed with an underscore to indicate visibility
Open

    public $_orderBy = ''; // private property that determines by which field

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

        $post_tbl = Database::get_course_table(TABLE_DROPBOX_POST);

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

                FROM $post_tbl r

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

        $course_id = api_get_course_int_id();

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

        $this->upload_date = stripslashes($res['upload_date']);

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

            $file_id = (int) $this->id;

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

            if (!$user_info) {

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

                     r.c_id = $course_id AND

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

        $this->uploader_id = (int) $uploader_id;

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

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

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

                ['c_id = ? AND id = ?' => [$course_id, $this->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

                    VALUES ($course_id, ".intval($this->id).' , '.intval($this->uploader_id).')';

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

            if (($ownerid = $this->uploader_id) > $mailId) {

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

        $file_tbl = Database::get_course_table(TABLE_DROPBOX_FILE);

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

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

Member variable "upload_date" is not in valid camel caps format
Open

    public $upload_date;

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

            $this->upload_date = $this->last_upload_date;

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

        if (empty($this->id) || empty($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

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

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

                $row_feedback['feedback'] = Security::remove_XSS($row_feedback['feedback']);

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

                    VALUES ($course_id, $file_id, $user_id, $session_id, '$now', 0)";

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

            if ($user_id != $user) {

Multiple classes defined in a single file
Open

class Dropbox_Person

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

        $course_id = api_get_course_int_id();

Member variable "feedback_date" is not in valid camel caps format
Open

    public $feedback_date;

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

                        VALUES ($course_id, $file_id, $user_id)";

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

        $course_id = api_get_course_int_id();

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

            $user_info = api_get_user_info($dest_user_id);

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

        $session_id = api_get_session_id();

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

        $condition_session = api_get_session_condition($session_id);

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

                WHERE c_id = $course_id AND file_id = ".intval($this->id).' AND user_id = '.$this->uploader_id;

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

                WHERE c_id = $course_id AND file_id = ".intval($this->id).' AND user_id = '.$this->uploader_id;

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

                    VALUES ($course_id, ".intval($this->id).' , '.intval($this->uploader_id).')';

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

            $this->uploader_id = $uploader_id;

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

            $dest_user_id = $res['dest_user_id'];

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

            $user_info = api_get_user_info($dest_user_id);

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

                    'id' => $dest_user_id,

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

                     r.dest_user_id = '.intval($this->userId)." $condition_session ";

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

            $this->uploader_id = -1;

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

        $uploader_id = (int) $uploader_id;

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

        if (is_int($recipient_ids)) {

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

        $table_post = Database::get_course_table(TABLE_DROPBOX_POST);

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

        $table_person = Database::get_course_table(TABLE_DROPBOX_PERSON);

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

            $sql = "INSERT INTO $table_post (c_id, file_id, dest_user_id, session_id, feedback_date, cat_id)

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

                $sql = "INSERT INTO $table_person (c_id, file_id, user_id)

Missing class doc comment
Open

class Dropbox_Person

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

        $person_tbl = Database::get_course_table(TABLE_DROPBOX_PERSON);

Member variable "uploader_id" is not in valid camel caps format
Open

    public $uploader_id;

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

    public function createNewWork($uploader_id, $title, $description, $author, $filename, $filesize)

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

                'upload_date' => $this->upload_date,

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

        $course_id = api_get_course_int_id();

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

            $recipient_ids = [$uploader_id];

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

        $course_id = api_get_course_int_id();

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

                INNER JOIN $person_tbl p

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

                    f.c_id = $course_id AND

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

                'c_id' => $course_id,

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

            $this->uploader_id = $uploader_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

            ['c_id = ? AND id = ?' => [$course_id, $this->id]]

Missing class doc comment
Open

class Dropbox_SentWork extends Dropbox_Work

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

            $recipient_ids = [$uploader_id];

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

                    'user_id' => $dest_user_id,

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

        $condition_session = api_get_session_condition($session_id);

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

                WHERE c_id = $course_id AND user_id = '".$this->userId."' AND file_id ='".$id."'";

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

        $course_id = api_get_course_int_id();

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

                $row_feedback['feedback'] = Security::remove_XSS($row_feedback['feedback']);

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

                $feedback2[] = $row_feedback;

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

            $recipient_ids = [$recipient_ids + $this->id];

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

                    VALUES ($course_id, $file_id, $user_id, $session_id, '$now', 0)";

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

        $this->uploader_id = (int) $uploader_id;

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

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)

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

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

Public member variable "_orderBy" must not contain a leading underscore
Open

    public $_orderBy = ''; // private property that determines by which field

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

            $this->upload_date = $res['upload_date'];

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

        $this->last_upload_date = stripslashes($res['last_upload_date']);

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

            $uploader_id,

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

        $uploader_id = (int) $uploader_id;

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

            $recipient_ids = [$recipient_ids + $this->id];

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

        } elseif (0 == count($recipient_ids)) {

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

            $user_id = (int) $rec['id'];

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

                    VALUES ($course_id, $file_id, $user_id, $session_id, '$now', 0)";

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

        $course_id = api_get_course_int_id();

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

                'last_upload_date' => $this->last_upload_date,

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

            $this->upload_date = $this->last_upload_date;

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

        $userInfo = api_get_user_info($uploader_id);

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

                        VALUES ($course_id, $file_id, $user_id)";

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

                FROM $file_tbl f

Class name "Dropbox_SentWork" is not in camel caps format
Open

class Dropbox_SentWork extends Dropbox_Work

Class name "Dropbox_Work" is not in camel caps format
Open

class Dropbox_Work

Class name "Dropbox_Person" is not in camel caps format
Open

class Dropbox_Person

The variable $course_id is not named in camelCase.
Open

    public function updateFile()
    {
        $course_id = api_get_course_int_id();
        if (empty($this->id) || empty($course_id)) {
            return false;

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $user_id is not named in camelCase.
Open

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_id is not named in camelCase.
Open

    public function createNewWork($uploader_id, $title, $description, $author, $filename, $filesize)
    {
        // Fill in the properties
        $this->uploader_id = (int) $uploader_id;
        $this->filename = $filename;

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $uploader_id is not named in camelCase.
Open

    public function createExistingWork($id)
    {
        $course_id = api_get_course_int_id();
        $action = isset($_GET['action']) ? $_GET['action'] : null;

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $dest_user_id is not named in camelCase.
Open

    public function createExistingSentWork($id)
    {
        $id = (int) $id;
        $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 $row_feedback is not named in camelCase.
Open

    public function createExistingWork($id)
    {
        $course_id = api_get_course_int_id();
        $action = isset($_GET['action']) ? $_GET['action'] : null;

CamelCaseVariableName

Since: 0.2

It is 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

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_id is not named in camelCase.
Open

    public function createNewWork($uploader_id, $title, $description, $author, $filename, $filesize)
    {
        // Fill in the properties
        $this->uploader_id = (int) $uploader_id;
        $this->filename = $filename;

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_id is not named in camelCase.
Open

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

It is 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

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_id is not named in camelCase.
Open

    public function deleteReceivedWorkFolder($id)
    {
        $course_id = api_get_course_int_id();

        $id = intval($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 $uploader_id is not named in camelCase.
Open

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $user_id is not named in camelCase.
Open

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_id is not named in camelCase.
Open

    public function createExistingSentWork($id)
    {
        $id = (int) $id;
        $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 $dest_user_id is not named in camelCase.
Open

    public function createExistingSentWork($id)
    {
        $id = (int) $id;
        $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

    public function deleteSentWork($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 $table_person is not named in camelCase.
Open

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $user_id is not named in camelCase.
Open

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $dest_user_id is not named in camelCase.
Open

    public function createExistingSentWork($id)
    {
        $id = (int) $id;
        $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 $dest_user_id is not named in camelCase.
Open

    public function createExistingSentWork($id)
    {
        $id = (int) $id;
        $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

    public function createNewWork($uploader_id, $title, $description, $author, $filename, $filesize)
    {
        // Fill in the properties
        $this->uploader_id = (int) $uploader_id;
        $this->filename = $filename;

CamelCaseVariableName

Since: 0.2

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

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

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

    public function createExistingSentWork($id)
    {
        $id = (int) $id;
        $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 $person_tbl is not named in camelCase.
Open

    public function __construct($userId, $isCourseAdmin, $isCourseTutor)
    {
        $course_id = api_get_course_int_id();

        // Fill in properties

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_id is not named in camelCase.
Open

    public function createExistingWork($id)
    {
        $course_id = api_get_course_int_id();
        $action = isset($_GET['action']) ? $_GET['action'] : null;

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_id is not named in camelCase.
Open

    public function createExistingWork($id)
    {
        $course_id = api_get_course_int_id();
        $action = isset($_GET['action']) ? $_GET['action'] : null;

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $row_feedback is not named in camelCase.
Open

    public function createExistingWork($id)
    {
        $course_id = api_get_course_int_id();
        $action = isset($_GET['action']) ? $_GET['action'] : null;

CamelCaseVariableName

Since: 0.2

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

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

It is 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

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

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

    public function __construct($userId, $isCourseAdmin, $isCourseTutor)
    {
        $course_id = api_get_course_int_id();

        // Fill in properties

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_id is not named in camelCase.
Open

    public function deleteReceivedWorkFolder($id)
    {
        $course_id = api_get_course_int_id();

        $id = intval($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 $uploader_id is not named in camelCase.
Open

    public function createNewWork($uploader_id, $title, $description, $author, $filename, $filesize)
    {
        // Fill in the properties
        $this->uploader_id = (int) $uploader_id;
        $this->filename = $filename;

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_id is not named in camelCase.
Open

    public function createNewWork($uploader_id, $title, $description, $author, $filename, $filesize)
    {
        // Fill in the properties
        $this->uploader_id = (int) $uploader_id;
        $this->filename = $filename;

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $row_feedback is not named in camelCase.
Open

    public function createExistingWork($id)
    {
        $course_id = api_get_course_int_id();
        $action = isset($_GET['action']) ? $_GET['action'] : null;

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $uploader_id is not named in camelCase.
Open

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

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

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

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

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

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

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

It is 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

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

It is 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

    public function __construct($userId, $isCourseAdmin, $isCourseTutor)
    {
        $course_id = api_get_course_int_id();

        // Fill in properties

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_id is not named in camelCase.
Open

    public function __construct($userId, $isCourseAdmin, $isCourseTutor)
    {
        $course_id = api_get_course_int_id();

        // Fill in properties

CamelCaseVariableName

Since: 0.2

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

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $uploader_id is not named in camelCase.
Open

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

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

    public function createExistingSentWork($id)
    {
        $id = (int) $id;
        $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

    public function __construct($userId, $isCourseAdmin, $isCourseTutor)
    {
        $course_id = api_get_course_int_id();

        // Fill in properties

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_id is not named in camelCase.
Open

    public function __construct($userId, $isCourseAdmin, $isCourseTutor)
    {
        $course_id = api_get_course_int_id();

        // Fill in properties

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $post_tbl is not named in camelCase.
Open

    public function __construct($userId, $isCourseAdmin, $isCourseTutor)
    {
        $course_id = api_get_course_int_id();

        // Fill in properties

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_id is not named in camelCase.
Open

    public function __construct($userId, $isCourseAdmin, $isCourseTutor)
    {
        $course_id = api_get_course_int_id();

        // Fill in properties

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_id is not named in camelCase.
Open

    public function deleteReceivedWorkFolder($id)
    {
        $course_id = api_get_course_int_id();

        $id = intval($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 $uploader_id is not named in camelCase.
Open

    public function createExistingWork($id)
    {
        $course_id = api_get_course_int_id();
        $action = isset($_GET['action']) ? $_GET['action'] : null;

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $row_feedback is not named in camelCase.
Open

    public function createExistingWork($id)
    {
        $course_id = api_get_course_int_id();
        $action = isset($_GET['action']) ? $_GET['action'] : null;

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $uploader_id is not named in camelCase.
Open

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_id is not named in camelCase.
Open

    public function createExistingSentWork($id)
    {
        $id = (int) $id;
        $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

    public function createNewWork($uploader_id, $title, $description, $author, $filename, $filesize)
    {
        // Fill in the properties
        $this->uploader_id = (int) $uploader_id;
        $this->filename = $filename;

CamelCaseVariableName

Since: 0.2

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

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_id is not named in camelCase.
Open

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

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

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

It is 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

    public function __construct($userId, $isCourseAdmin, $isCourseTutor)
    {
        $course_id = api_get_course_int_id();

        // Fill in properties

CamelCaseVariableName

Since: 0.2

It is 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

    public function __construct($userId, $isCourseAdmin, $isCourseTutor)
    {
        $course_id = api_get_course_int_id();

        // Fill in properties

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_id is not named in camelCase.
Open

    public function deleteReceivedWorkFolder($id)
    {
        $course_id = api_get_course_int_id();

        $id = intval($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

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

    public function deleteReceivedWork($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 $user_info is not named in camelCase.
Open

    public function createExistingSentWork($id)
    {
        $id = (int) $id;
        $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 $person_tbl is not named in camelCase.
Open

    public function __construct($userId, $isCourseAdmin, $isCourseTutor)
    {
        $course_id = api_get_course_int_id();

        // Fill in properties

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_id is not named in camelCase.
Open

    public function deleteSentWork($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 $uploader_id is not named in camelCase.
Open

    public function createExistingWork($id)
    {
        $course_id = api_get_course_int_id();
        $action = isset($_GET['action']) ? $_GET['action'] : null;

CamelCaseVariableName

Since: 0.2

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

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

It is 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

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

It is 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

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $post_tbl is not named in camelCase.
Open

    public function __construct($userId, $isCourseAdmin, $isCourseTutor)
    {
        $course_id = api_get_course_int_id();

        // Fill in properties

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_id is not named in camelCase.
Open

    public function createNewWork($uploader_id, $title, $description, $author, $filename, $filesize)
    {
        // Fill in the properties
        $this->uploader_id = (int) $uploader_id;
        $this->filename = $filename;

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_id is not named in camelCase.
Open

    public function createExistingWork($id)
    {
        $course_id = api_get_course_int_id();
        $action = isset($_GET['action']) ? $_GET['action'] : null;

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_id is not named in camelCase.
Open

    public function updateFile()
    {
        $course_id = api_get_course_int_id();
        if (empty($this->id) || empty($course_id)) {
            return false;

CamelCaseVariableName

Since: 0.2

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

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $user_id is not named in camelCase.
Open

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

It is 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

    public function __construct($userId, $isCourseAdmin, $isCourseTutor)
    {
        $course_id = api_get_course_int_id();

        // Fill in properties

CamelCaseVariableName

Since: 0.2

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

    public function __construct($userId, $isCourseAdmin, $isCourseTutor)
    {
        $course_id = api_get_course_int_id();

        // Fill in properties

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_id is not named in camelCase.
Open

    public function updateFile()
    {
        $course_id = api_get_course_int_id();
        if (empty($this->id) || empty($course_id)) {
            return false;

CamelCaseVariableName

Since: 0.2

It is 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

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

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

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_id is not named in camelCase.
Open

    public function createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
    {
        $_course = api_get_course_info();

        // Call constructor of Dropbox_Work object

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $person_tbl is not named in camelCase.
Open

    public function __construct($userId, $isCourseAdmin, $isCourseTutor)
    {
        $course_id = api_get_course_int_id();

        // Fill in properties

CamelCaseVariableName

Since: 0.2

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