chamilo/chamilo-lms

View on GitHub
public/plugin/ims_lti/src/ContentItem/LtiResourceLink.php

Summary

Maintainability
A
0 mins
Test Coverage

The method LtiResourceLink::validateItemData() calls the typical debug function print_r() which is mostly only used during development.
Open

            throw new Exception(sprintf("Iframe size is wrong in data form content item: %s", print_r($itemData, true)));

DevelopmentCodeFragment

Since: 2.3.0

Functions like vardump(), printr() etc. are normally only used during development and therefore such calls in production code are a good indicator that they were just forgotten.

Example

class SuspectCode {

    public function doSomething(array $items)
    {
        foreach ($items as $i => $item) {
            // …

            if ('qafoo' == $item) var_dump($i);

            // …
        }
    }
}

Source https://phpmd.org/rules/design.html#developmentcodefragment

The method LtiResourceLink::validateItemData() calls the typical debug function print_r() which is mostly only used during development.
Open

            throw new Exception(sprintf("Submission properties are missing in data form content item: %s", print_r($itemData, true)));

DevelopmentCodeFragment

Since: 2.3.0

Functions like vardump(), printr() etc. are normally only used during development and therefore such calls in production code are a good indicator that they were just forgotten.

Example

class SuspectCode {

    public function doSomething(array $items)
    {
        foreach ($items as $i => $item) {
            // …

            if ('qafoo' == $item) var_dump($i);

            // …
        }
    }
}

Source https://phpmd.org/rules/design.html#developmentcodefragment

The method LtiResourceLink::validateItemData() calls the typical debug function print_r() which is mostly only used during development.
Open

            throw new Exception(sprintf("Icon properties are missing in data form content item: %s", print_r($itemData, true)));

DevelopmentCodeFragment

Since: 2.3.0

Functions like vardump(), printr() etc. are normally only used during development and therefore such calls in production code are a good indicator that they were just forgotten.

Example

class SuspectCode {

    public function doSomething(array $items)
    {
        foreach ($items as $i => $item) {
            // …

            if ('qafoo' == $item) var_dump($i);

            // …
        }
    }
}

Source https://phpmd.org/rules/design.html#developmentcodefragment

The method LtiResourceLink::validateItemData() calls the typical debug function print_r() which is mostly only used during development.
Open

            throw new Exception(sprintf("LineItem properties are missing in data form content item: %s", print_r($itemData, true)));

DevelopmentCodeFragment

Since: 2.3.0

Functions like vardump(), printr() etc. are normally only used during development and therefore such calls in production code are a good indicator that they were just forgotten.

Example

class SuspectCode {

    public function doSomething(array $items)
    {
        foreach ($items as $i => $item) {
            // …

            if ('qafoo' == $item) var_dump($i);

            // …
        }
    }
}

Source https://phpmd.org/rules/design.html#developmentcodefragment

The method LtiResourceLink::validateItemData() calls the typical debug function print_r() which is mostly only used during development.
Open

            throw new Exception(sprintf("LineItem properties are missing in data form content item: %s", print_r($itemData, true)));

DevelopmentCodeFragment

Since: 2.3.0

Functions like vardump(), printr() etc. are normally only used during development and therefore such calls in production code are a good indicator that they were just forgotten.

Example

class SuspectCode {

    public function doSomething(array $items)
    {
        foreach ($items as $i => $item) {
            // …

            if ('qafoo' == $item) var_dump($i);

            // …
        }
    }
}

Source https://phpmd.org/rules/design.html#developmentcodefragment

The method LtiResourceLink::validateItemData() calls the typical debug function print_r() which is mostly only used during development.
Open

            throw new Exception(sprintf("Thumbnail URL is missing in data form content item: %s", print_r($itemData, true)));

DevelopmentCodeFragment

Since: 2.3.0

Functions like vardump(), printr() etc. are normally only used during development and therefore such calls in production code are a good indicator that they were just forgotten.

Example

class SuspectCode {

    public function doSomething(array $items)
    {
        foreach ($items as $i => $item) {
            // …

            if ('qafoo' == $item) var_dump($i);

            // …
        }
    }
}

Source https://phpmd.org/rules/design.html#developmentcodefragment

The method validateItemData() has an NPath complexity of 4915200. The configured NPath complexity threshold is 200.
Open

    protected function validateItemData(stdClass $itemData)
    {
        $this->url = empty($itemData->url) ? '' : $itemData->url;
        $this->title = empty($itemData->title) ? '' : $itemData->title;
        $this->text = empty($itemData->text) ? '' : $itemData->text;

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 '86', column '23').
Open

            throw new Exception(sprintf("Icon properties are missing in data form content item: %s", print_r($itemData, true)));

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 '106', column '23').
Open

            throw new Exception(sprintf("LineItem properties are missing in data form content item: %s", print_r($itemData, true)));

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 '94', column '23').
Open

            throw new Exception(sprintf("Thumbnail URL is missing in data form content item: %s", print_r($itemData, true)));

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 '118', column '23').
Open

            throw new Exception(sprintf("Submission properties are missing in data form content item: %s", print_r($itemData, true)));

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 '100', column '23').
Open

            throw new Exception(sprintf("Iframe size is wrong in data form content item: %s", print_r($itemData, true)));

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 '112', column '23').
Open

            throw new Exception(sprintf("LineItem properties are missing in data form content item: %s", print_r($itemData, true)));

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

There are no issues that match your filters.

Category
Status