chamilo/chamilo-lms

View on GitHub
src/CourseBundle/Component/CourseCopy/Resources/Document.php

Summary

Maintainability
A
0 mins
Test Coverage

The parameter $file_type is not named in camelCase.
Open

    public function __construct($id, $path, $comment, $title, $file_type, $size)
    {
        parent::__construct($id, RESOURCE_DOCUMENT);
        $this->path = 'document'.$path;
        $this->comment = $comment;

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

class Document extends Resource
{
    public $path;
    public $comment;
    public $file_type;

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

    public function __construct($id, $path, $comment, $title, $file_type, $size)
    {
        parent::__construct($id, RESOURCE_DOCUMENT);
        $this->path = 'document'.$path;
        $this->comment = $comment;

CamelCaseVariableName

Since: 0.2

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