YetiForceCompany/YetiForceCRM

View on GitHub
modules/Vtiger/widgets/PreviewMail.php

Summary

Maintainability
A
0 mins
Test Coverage
F
0%

Suspicious type false of a variable or expression used to build a string. (Expected type to be able to cast to a string)
Open

        return 'module=' . $this->Module . '&view=Sview&noloadlibs=true&record=' . $this->Record;
Severity: Minor
Found in modules/Vtiger/widgets/PreviewMail.php by phan

Each class must be in a namespace of at least one level (a top-level vendor name)
Open

class Vtiger_PreviewMail_Widget extends Vtiger_Basic_Widget

The class Vtiger_PreviewMail_Widget is not named in CamelCase.
Open

class Vtiger_PreviewMail_Widget extends Vtiger_Basic_Widget
{
    /** {@inheritdoc} */
    public $allowedModules = ['OSSMailView'];
    public $dbParams = [];

CamelCaseClassName

Since: 0.2

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

Example

class class_name {
}

Source

Spaces must be used to indent lines; tabs are not allowed
Open

    public $allowedModules = ['OSSMailView'];

Spaces must be used to indent lines; tabs are not allowed
Open

    {

Spaces must be used to indent lines; tabs are not allowed
Open

    /**

Spaces must be used to indent lines; tabs are not allowed
Open

        $this->Config['tpl'] = 'PreviewMail.tpl';

Spaces must be used to indent lines; tabs are not allowed
Open

    public function getUrl()

Spaces must be used to indent lines; tabs are not allowed
Open

    {

Spaces must be used to indent lines; tabs are not allowed
Open

    /** {@inheritdoc} */

Spaces must be used to indent lines; tabs are not allowed
Open

    {

Spaces must be used to indent lines; tabs are not allowed
Open

        return 'module=' . $this->Module . '&view=Sview&noloadlibs=true&record=' . $this->Record;

Spaces must be used to indent lines; tabs are not allowed
Open

    /** {@inheritdoc} */

Spaces must be used to indent lines; tabs are not allowed
Open

    }

Spaces must be used to indent lines; tabs are not allowed
Open

    public function getConfigTplName()

Spaces must be used to indent lines; tabs are not allowed
Open

    /** {@inheritdoc} */

Spaces must be used to indent lines; tabs are not allowed
Open

    public $dbParams = [];

Spaces must be used to indent lines; tabs are not allowed
Open

        $this->Config['url'] = $this->getUrl();

Spaces must be used to indent lines; tabs are not allowed
Open

        return 'PreviewMailConfig';

Spaces must be used to indent lines; tabs are not allowed
Open

    }

Spaces must be used to indent lines; tabs are not allowed
Open

     * Get URL.

Spaces must be used to indent lines; tabs are not allowed
Open

     *

Spaces must be used to indent lines; tabs are not allowed
Open

     * @return string

Spaces must be used to indent lines; tabs are not allowed
Open

    }

Spaces must be used to indent lines; tabs are not allowed
Open

    public function getWidget()

Spaces must be used to indent lines; tabs are not allowed
Open

        return $this->Config;

Spaces must be used to indent lines; tabs are not allowed
Open

     */

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

class Vtiger_PreviewMail_Widget extends Vtiger_Basic_Widget

There are no issues that match your filters.

Category
Status