chamilo/chamilo-lms

View on GitHub
public/main/inc/lib/search/IndexableChunk.class.php

Summary

Maintainability
A
0 mins
Test Coverage

The class _IndexableChunk is not named in CamelCase.
Open

abstract class _IndexableChunk
{
    /* struct (array)
     * {
     *     string title; <- nombre de archivo/elemento

CamelCaseClassName

Since: 0.2

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

Example

class class_name {
}

Source

The parameter $tool_id is not named in camelCase.
Open

    public function addToolId($tool_id)
    {
        $this->addTerm($tool_id, XAPIAN_PREFIX_TOOLID);
    }

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

    public function addCourseId($course_id)
    {
        $this->addTerm($course_id, XAPIAN_PREFIX_COURSEID);
    }

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

abstract class _IndexableChunk
{
    /* struct (array)
     * {
     *     string title; <- nombre de archivo/elemento

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 IndexableChunk extends _IndexableChunk

Missing parameter name
Open

     * @param string Term

Missing parameter name
Open

     * @param  string  Key

Multiple classes defined in a single file
Open

class IndexableChunk extends _IndexableChunk

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

        $this->addTerm($course_id, XAPIAN_PREFIX_COURSEID);

Missing parameter name
Open

     * @param  string  Value

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

    public function addToolId($tool_id)

Abstract class name is not prefixed with "Abstract"
Open

abstract class _IndexableChunk

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

    public $xapian_data;

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

        $this->addTerm($tool_id, XAPIAN_PREFIX_TOOLID);

Missing parameter name
Open

     * @param string Flag (one character)

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

    public function addCourseId($course_id)

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

abstract class _IndexableChunk

The variable $course_id is not named in camelCase.
Open

    public function addCourseId($course_id)
    {
        $this->addTerm($course_id, XAPIAN_PREFIX_COURSEID);
    }

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $tool_id is not named in camelCase.
Open

    public function addToolId($tool_id)
    {
        $this->addTerm($tool_id, XAPIAN_PREFIX_TOOLID);
    }

CamelCaseVariableName

Since: 0.2

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