YetiForceCompany/YetiForceCRM

View on GitHub
modules/OSSEmployees/models/Relation.php

Summary

Maintainability
A
0 mins
Test Coverage
F
0%

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

class OSSEmployees_Relation_Model extends Vtiger_Relation_Model

The class OSSEmployees_Relation_Model is not named in CamelCase.
Open

class OSSEmployees_Relation_Model extends Vtiger_Relation_Model
{
    /** {@inheritdoc} */
    public function privilegeToDelete(Vtiger_Record_Model $recordModel = null, int $recordId = null): bool
    {

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 function privilegeToDelete(Vtiger_Record_Model $recordModel = null, int $recordId = null): bool

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 'OSSTimeControl' !== $this->getRelationModuleName() && parent::privilegeToDelete($recordModel, $recordId);

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

    }

Line exceeds 120 characters; contains 121 characters
Open

        return 'OSSTimeControl' !== $this->getRelationModuleName() && parent::privilegeToDelete($recordModel, $recordId);

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

class OSSEmployees_Relation_Model extends Vtiger_Relation_Model

There are no issues that match your filters.

Category
Status