Sign upLogin

pluf/elearn

View on GitHub
Star

Showing 910 of 910 total issues

Opening brace should be on a new line
Open

    private static function validatePart($request, $match, $id){
Severity: Minor
Found in src/ELearn/Views/Part.php by phpcodesniffer

Opening brace should be on a new line
Open

    public static function listAll($request, $match, $p){
Severity: Minor
Found in src/ELearn/Views.php by phpcodesniffer

Whitespace found at end of line
Open

     * @param Pluf_HTTP_Request $request
Severity: Minor
Found in src/ELearn/Views/Lesson.php by phpcodesniffer

Whitespace found at end of line
Open

     * @param Pluf_HTTP_Request $request
Severity: Minor
Found in src/ELearn/Views/Topic.php by phpcodesniffer

Multi-line function call not indented correctly; expected 4 spaces but found 8
Open

        ));
Severity: Minor
Found in src/ELearn/Shortcuts.php by phpcodesniffer

Closing parenthesis of a multi-line function call must be on a line by itself
Open

                'more than one Part exist with the name $s', $name));
Severity: Minor
Found in src/ELearn/Shortcuts.php by phpcodesniffer

Whitespace found at end of line
Open

     * @param array $p
Severity: Minor
Found in src/ELearn/Views/Part.php by phpcodesniffer

Whitespace found at end of line
Open

     * @param array $p
Severity: Minor
Found in src/ELearn/Views/Course.php by phpcodesniffer

Expected 1 newline at end of file; 0 found
Open

}
Severity: Minor
Found in src/ELearn/Views/Course.php by phpcodesniffer

Whitespace found at end of line
Open

     * @param array $match
Severity: Minor
Found in src/ELearn/Views/Lesson.php by phpcodesniffer

Usage of ELSE IF is discouraged; use ELSEIF instead
Open

        } else if (isset($request->REQUEST['course'])) {
Severity: Minor
Found in src/ELearn/Views/Lesson.php by phpcodesniffer

Whitespace found at end of line
Open

     * @param array $p
Severity: Minor
Found in src/ELearn/Views/Topic.php by phpcodesniffer

Multi-line function call not indented correctly; expected 12 spaces but found 16
Open

                'The model class was not provided in the parameters.');
Severity: Minor
Found in src/ELearn/Views.php by phpcodesniffer

The variable $search_fields is not named in camelCase.
Open

    function configure($search_fields = array(), $sort_fields = array())
    {
        if (is_array($search_fields)) {
            $this->search_fields = $search_fields;
        }
Severity: Minor
Found in src/ELearn/Searcher.php by phpmd

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $sort_fields is not named in camelCase.
Open

    public static function find($request, $match)
    {
        // check for topic
        if (isset($match['topicId'])) {
            $topicId = $match['topicId'];
Severity: Minor
Found in src/ELearn/Views/Course.php by phpmd

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $search_fields is not named in camelCase.
Open

    public static function find($request, $match)
    {
        // check for lesson
        if (isset($match['lessonId'])) {
            $lessonId = $match['lessonId'];
Severity: Minor
Found in src/ELearn/Views/Part.php by phpmd

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $sort_fields is not named in camelCase.
Open

    public static function find($request, $match)
    {
        // check for topic
        if (isset($match['topicId'])) {
            $topicId = $match['topicId'];
Severity: Minor
Found in src/ELearn/Views/Course.php by phpmd

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $search_fields is not named in camelCase.
Open

    function __construct($model = null, $search_fields = array())
    {
        $this->model = $model;
        $this->configure($search_fields);
    }
Severity: Minor
Found in src/ELearn/Searcher.php by phpmd

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $search_fields is not named in camelCase.
Open

    public static function find($request, $match)
    {
        // check for lesson
        if (isset($match['lessonId'])) {
            $lessonId = $match['lessonId'];
Severity: Minor
Found in src/ELearn/Views/Part.php by phpmd

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $sort_fields is not named in camelCase.
Open

    public static function find($request, $match)
    {
        // check for lesson
        if (isset($match['lessonId'])) {
            $lessonId = $match['lessonId'];
Severity: Minor
Found in src/ELearn/Views/Part.php by phpmd

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

Severity
Category
Status
Source
Language