File RelationListView.php has 543 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php

 /* +***********************************************************************************
 * The contents of this file are subject to the vtiger CRM Public License Version 1.0
 * ("License"); You may not use this file except in compliance with the License

Vtiger_RelationListView_Model has 33 functions (exceeds 20 allowed). Consider refactoring.

Open

class Vtiger_RelationListView_Model extends \App\Base
{
    /**
     * Relation model instance.
     *

Function getHeaders has a Cognitive Complexity of 27 (exceeds 5 allowed). Consider refactoring.

Open

    public function getHeaders()
    {
        $fields = [];
        if ($this->get('viewId')) {
            $moduleModel = $this->getRelationModel()->getRelationModuleModel();

• Read up
  Read up

The class Vtiger_RelationListView_Model has an overall complexity of 125 which is very high. The configured complexity threshold is 50.

Open

class Vtiger_RelationListView_Model extends \App\Base
{
    /**
     * Relation model instance.
     *

• Exclude checks
  • Disable engine
  • Disable check

Method getLinks has 79 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function getLinks(): array
    {
        $parentRecordModel = $this->getParentRecordModel();
        $relationModelInstance = $this->getRelationModel();
        $relatedModuleModel = $relationModelInstance->getRelationModuleModel();

Function loadSearchLockedFields has a Cognitive Complexity of 19 (exceeds 5 allowed). Consider refactoring.

Open

    public function loadSearchLockedFields(App\Request $request): void
    {
        $moduleModel = $this->getRelationModel()->getRelationModuleModel();
        if (!$request->isEmpty('lockedFields')) {
            foreach ($request->getArray('lockedFields') as $value) {

• Read up
  Read up

Function getLinks has a Cognitive Complexity of 15 (exceeds 5 allowed). Consider refactoring.

Open

    public function getLinks(): array
    {
        $parentRecordModel = $this->getParentRecordModel();
        $relationModelInstance = $this->getRelationModel();
        $relatedModuleModel = $relationModelInstance->getRelationModuleModel();

• Read up
  Read up

Function getRecordsFromArray has a Cognitive Complexity of 14 (exceeds 5 allowed). Consider refactoring.

Open

    public function getRecordsFromArray(array $rows)
    {
        $listViewRecordModels = $relatedFields = [];
        $moduleModel = $this->getRelationModel()->getRelationModuleModel();
        $recordId = $this->getParentRecordModel()->getId();

• Read up
  Read up

Method getHeaders has 43 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function getHeaders()
    {
        $fields = [];
        if ($this->get('viewId')) {
            $moduleModel = $this->getRelationModel()->getRelationModuleModel();

Method getTreeEntries has 34 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function getTreeEntries()
    {
        $relModuleName = $this->getRelatedModuleModel()->getName();
        $relationModelInstance = $this->getRelationModel();
        $template = $this->getTreeViewModel()->getTemplate();

Method getAddRelationLinks has 31 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function getAddRelationLinks(): array
    {
        $relationModelInstance = $this->getRelationModel();
        if (!$relationModelInstance->isAddActionSupported() || $this->getParentRecordModel()->isReadOnly()) {
            return [];

Method getRecordsFromArray has 29 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function getRecordsFromArray(array $rows)
    {
        $listViewRecordModels = $relatedFields = [];
        $moduleModel = $this->getRelationModel()->getRelationModuleModel();
        $recordId = $this->getParentRecordModel()->getId();

Function getWidgets has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

    public function getWidgets(int $recordId): array
    {
        $widgets = [];
        $moduleModel = $this->getRelatedModuleModel();
        foreach ($this->getWidgetsList() as $widgetCol) {

• Read up
  Read up

Function getRelationQuery has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

    public function getRelationQuery($returnQueryGenerator = false)
    {
        if ($this->has('Query')) {
            return $this->get('Query');
        }

• Read up
  Read up

Function getTreeEntries has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    public function getTreeEntries()
    {
        $relModuleName = $this->getRelatedModuleModel()->getName();
        $relationModelInstance = $this->getRelationModel();
        $template = $this->getTreeViewModel()->getTemplate();

• Read up
  Read up

Function loadOrderBy has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    public function loadOrderBy()
    {
        $orderBy = $this->get('orderby');
        if (!empty($orderBy) && \is_array($orderBy)) {
            foreach ($orderBy as $fieldName => $sortFlag) {

• Read up
  Read up

Function getCreateViewUrl has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    public function getCreateViewUrl(bool $fullView = false)
    {
        $createViewUrl = $this->getRelationModel()->getCreateViewUrl($fullView);
        if (!empty(Config\Relation::$addSearchParamsToCreateView) && ($searchParams = $this->getArray('search_params')) && isset($searchParams['and']) && \is_array($searchParams['and'])) {
            foreach ($searchParams['and'] as $row) {

• Read up
  Read up

Function loadCondition has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

    public function loadCondition()
    {
        $relatedModuleName = $this->getRelatedModuleModel()->getName();
        $queryGenerator = $this->getRelationModel()->getQueryGenerator();
        if ($entityState = $this->get('entityState')) {

• Read up
  Read up

The method getHeaders() has an NPath complexity of 720. The configured NPath complexity threshold is 200.

Open

    public function getHeaders()
    {
        $fields = [];
        if ($this->get('viewId')) {
            $moduleModel = $this->getRelationModel()->getRelationModuleModel();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    function bar() {
        // lots of complicated code
    }
}

The method loadSearchLockedFields() has a Cyclomatic Complexity of 10. The configured cyclomatic complexity threshold is 10.

Open

    public function loadSearchLockedFields(App\Request $request): void
    {
        $moduleModel = $this->getRelationModel()->getRelationModuleModel();
        if (!$request->isEmpty('lockedFields')) {
            foreach ($request->getArray('lockedFields') as $value) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

The method getHeaders() has a Cyclomatic Complexity of 16. The configured cyclomatic complexity threshold is 10.

Open

    public function getHeaders()
    {
        $fields = [];
        if ($this->get('viewId')) {
            $moduleModel = $this->getRelationModel()->getRelationModuleModel();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

Refactor this function to reduce its Cognitive Complexity from 26 to the 15 allowed.

Open

    public function getHeaders()

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

Class "Vtiger_RelationListView_Model" has 33 methods, which is greater than 20 authorized. Split it into smaller classes.

Open

class Vtiger_RelationListView_Model extends \App\Base

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

Refactor this function to reduce its Cognitive Complexity from 19 to the 15 allowed.

Open

    public function loadSearchLockedFields(App\Request $request): void

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

The class Vtiger_RelationListView_Model has a coupling between objects value of 26. Consider to reduce the number of dependencies under 13.

Open

class Vtiger_RelationListView_Model extends \App\Base
{
    /**
     * Relation model instance.
     *

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    /**
     * @var \foo\bar\X
     */
    private $x = null;

    /**
     * @var \foo\bar\Y
     */
    private $y = null;

    /**
     * @var \foo\bar\Z
     */
    private $z = null;

    public function setFoo(\Foo $foo) {}
    public function setBar(\Bar $bar) {}
    public function setBaz(\Baz $baz) {}

    /**
     * @return \SplObjectStorage
     * @throws \OutOfRangeException
     * @throws \InvalidArgumentException
     * @throws \ErrorException
     */
    public function process(\Iterator $it) {}

    // ...
}

Missing class import via use statement (line '663', column '15').

Open

        return (new App\Db\Query())->select(['relcrmid'])->from('u_#__favorites')

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

The method getInstance has a boolean flag argument $relationId, which is a certain sign of a Single Responsibility Principle violation.

Open

    public static function getInstance(Vtiger_Record_Model $parentRecordModel, string $relationModuleName, $relationId = false, $cvId = 0)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function bar($flag = true) {
    }
}

The method getCreateViewUrl has a boolean flag argument $fullView, which is a certain sign of a Single Responsibility Principle violation.

Open

    public function getCreateViewUrl(bool $fullView = false)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function bar($flag = true) {
    }
}

Missing class import via use statement (line '198', column '13').

Open

        throw new \App\Exceptions\AppException('>>> No relationModel instance, requires verification 2 <<<');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '157', column '25').

Open

        $queryGenerator = new \App\QueryGenerator($relationModelInstance->getRelationModuleModel()->getName());

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

The method getRelationQuery has a boolean flag argument $returnQueryGenerator, which is a certain sign of a Single Responsibility Principle violation.

Open

    public function getRelationQuery($returnQueryGenerator = false)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function bar($flag = true) {
    }
}

Missing class import via use statement (line '733', column '17').

Open

        $query = (new App\Db\Query())->from('a_#__relatedlists_widgets')->where(['relation_id' => $relationId]);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '485', column '15').

Open

        return (new \App\Db\Query())->from('vtiger_trees_templates_data ttd')->innerJoin('u_#__crmentity_rel_tree rel', 'rel.tree = ttd.tree')

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '586', column '23').

Open

        $eventHandler = new App\EventHandler();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

The method getInstance uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            $relationModelInstance = Vtiger_Relation_Model::getInstance($parentModuleModel, Vtiger_Module_Model::getInstance($relationModuleName), $relationId);
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid using static access to class 'Vtiger_Relation_Model' in method 'getInstance'.

Open

            $relationModelInstance = Vtiger_Relation_Model::getInstance($parentModuleModel, Vtiger_Module_Model::getInstance($relationModuleName), $relationId);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid assigning values to variables in if clauses and the like (line '208', column '7').

Open

    public function loadCondition()
    {
        $relatedModuleName = $this->getRelatedModuleModel()->getName();
        $queryGenerator = $this->getRelationModel()->getQueryGenerator();
        if ($entityState = $this->get('entityState')) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($foo = 'bar') { // possible typo
            // ...
        }
        if ($baz = 0) { // always false
            // ...
        }
    }
}

Avoid using static access to class 'App\Language' in method 'getTreeEntries'.

Open

                $parentName = '(' . App\Language::translate($treeDetail['name'], $relModuleName) . ') ';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'Vtiger_Link_Model' in method 'getLinks'.

Open

                $relatedLink['RELATEDLIST_MASSACTIONS'][] = Vtiger_Link_Model::getInstanceFromValues([
                    'linktype' => 'RELATEDLIST_MASSACTIONS',
                    'linklabel' => 'LBL_MASS_SEND_SMS',
                    'linkdata' => ['url' => $smsNotifierModel->getMassSMSUrlForModule($relatedModuleModel->getName()), 'type' => 'modal'],
                    'linkicon' => 'fas fa-comment-sms',

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class '\App\Language' in method 'getSelectRelationLinks'.

Open

                'linklabel' => \App\Language::translate('LBL_SELECT_RELATION', $relatedModel->getName()),

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'App\User' in method 'getFavoriteRecords'.

Open

                'userid' => App\User::getCurrentUserId(),

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'Vtiger_Module_Model' in method 'getInstance'.

Open

            $relationModelInstance = Vtiger_Relation_Model::getInstance($parentModuleModel, Vtiger_Module_Model::getInstance($relationModuleName), $relationId);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'Vtiger_Field_Model' in method 'getHeaders'.

Open

                $fieldModel = Vtiger_Field_Model::getInstance($fieldName, Vtiger_Module_Model::getInstance($fieldInfo['module_name']));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class '\App\Cache' in method 'getWidgetsList'.

Open

        if (\App\Cache::has('RelatedModuleWidgets', $relationId)) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid assigning values to variables in if clauses and the like (line '214', column '7').

Open

    public function loadCondition()
    {
        $relatedModuleName = $this->getRelatedModuleModel()->getName();
        $queryGenerator = $this->getRelationModel()->getQueryGenerator();
        if ($entityState = $this->get('entityState')) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($foo = 'bar') { // possible typo
            // ...
        }
        if ($baz = 0) { // always false
            // ...
        }
    }
}

The method getEntries uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            $pagingModel->set('nextPageExists', false);
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid assigning values to variables in if clauses and the like (line '383', column '7').

Open

    public function getHeaders()
    {
        $fields = [];
        if ($this->get('viewId')) {
            $moduleModel = $this->getRelationModel()->getRelationModuleModel();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($foo = 'bar') { // possible typo
            // ...
        }
        if ($baz = 0) { // always false
            // ...
        }
    }
}

Avoid using static access to class '\App\Log' in method 'getHeaders'.

Open

                    \App\Log::warning("The field does not exist: '$fieldName' | Module: " . $this->getRelationModel()->getRelationModuleModel()->getName(), __METHOD__);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'Vtiger_TreeCategoryModal_Model' in method 'getTreeViewModel'.

Open

        return Vtiger_TreeCategoryModal_Model::getInstance($this->getRelatedModuleModel());

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

The method loadCustomView uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            $cvId = array_key_first($this->getRelationModel()->getCustomViewList());
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid using static access to class '\App\Language' in method 'getLinks'.

Open

                        'linkhint' => \App\Language::translate('LBL_EXPORT_PDF'),

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid assigning values to variables in if clauses and the like (line '499', column '65').

Open

    public function getCreateViewUrl(bool $fullView = false)
    {
        $createViewUrl = $this->getRelationModel()->getCreateViewUrl($fullView);
        if (!empty(Config\Relation::$addSearchParamsToCreateView) && ($searchParams = $this->getArray('search_params')) && isset($searchParams['and']) && \is_array($searchParams['and'])) {
            foreach ($searchParams['and'] as $row) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($foo = 'bar') { // possible typo
            // ...
        }
        if ($baz = 0) { // always false
            // ...
        }
    }
}

Avoid using static access to class '\Vtiger_Module_Model' in method 'getLinks'.

Open

            if ($relatedModuleModel->isPermitted('MassSendSMS') && ($smsNotifierModel = \Vtiger_Module_Model::getInstance('SMSNotifier'))->isSMSActiveForModule($relatedModuleModel->getName())) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'Vtiger_Link_Model' in method 'getLinks'.

Open

                    $relatedLink['RELATEDLIST_BASIC'][] = Vtiger_Link_Model::getInstanceFromValues([
                        'linktype' => 'RELATEDLIST_BASIC',
                        'linkdata' => [
                            'type' => 'modal',
                            'url' => "index.php?module={$parentRecordModel->getModuleName()}&view=PDF&fromview=RelatedList",

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'App\Language' in method 'getAddRelationLinks'.

Open

                    'linklabel' => App\Language::translate('LBL_ADD_RELATION', $relatedModel->getName()),

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'Vtiger_Relation_Model' in method 'getInstance'.

Open

            $relationModelInstance = Vtiger_Relation_Model::getInstanceById($relationId);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'App\Language' in method 'getTreeEntries'.

Open

                'name' => $parentName . App\Language::translate($row['name'], $relModuleName),

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'App\Fields\DateTime' in method 'getTreeEntries'.

Open

                $tree['rel_created_time'] = App\Fields\DateTime::formatToDisplay($row['rel_created_time']);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid assigning values to variables in if clauses and the like (line '551', column '60').

Open

    public function getLinks(): array
    {
        $parentRecordModel = $this->getParentRecordModel();
        $relationModelInstance = $this->getRelationModel();
        $relatedModuleModel = $relationModelInstance->getRelationModuleModel();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($foo = 'bar') { // possible typo
            // ...
        }
        if ($baz = 0) { // always false
            // ...
        }
    }
}

Avoid using static access to class 'App\Language' in method 'getAddRelationLinks'.

Open

                'linklabel' => App\Language::translate('LBL_MASS_ADD', 'Documents'),

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'Vtiger_Loader' in method 'getWidgets'.

Open

                $widgetName = Vtiger_Loader::getComponentClassName('Widget', $widget['type'], $moduleModel->getName());

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'App\Cache' in method 'getWidgetsList'.

Open

        App\Cache::save('RelatedModuleWidgets', $relationId, $widgets);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'App\CustomView' in method 'getHeaders'.

Open

            $customView = App\CustomView::getInstance($moduleModel->getName());

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'Vtiger_Link_Model' in method 'getLinks'.

Open

                $relatedLink['RELATEDLIST_MASSACTIONS'][] = Vtiger_Link_Model::getInstanceFromValues([
                    'linktype' => 'RELATEDLIST_MASSACTIONS',
                    'linklabel' => 'LBL_MASS_DOWNLOAD',
                    'linkurl' => "javascript:Vtiger_RelatedList_Js.triggerMassDownload('index.php?module={$parentRecordModel->getModuleName()}&action=RelationAjax&mode=massDownload&src_record={$parentRecordModel->getId()}&relatedModule=Documents&mode=multiple','sendByForm')",
                    'linkclass' => '',

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'Vtiger_Link_Model' in method 'getLinks'.

Open

                Vtiger_Link_Model::getInstanceFromValues([
                    'linktype' => 'RELATEDLIST_VIEWS',
                    'linklabel' => 'LBL_RECORDS_PREVIEW_LIST',
                    'view' => 'ListPreview',
                    'linkicon' => 'fas fa-desktop',

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'Vtiger_Link_Model' in method 'getAddRelationLinks'.

Open

            $addLinkModel['RELATEDLIST_BASIC'][] = Vtiger_Link_Model::getInstanceFromValues([
                'linktype' => 'LISTVIEWBASIC',
                'linklabel' => App\Language::translate('LBL_MASS_ADD', 'Documents'),
                'linkdata' => [
                    'url' => 'index.php?module=Documents&view=MassAddDocuments&sourceView=Detail',

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'Vtiger_Loader' in method 'getInstance'.

Open

        $className = Vtiger_Loader::getComponentClassName('Model', 'RelationListView', $parentModuleModel->getName());

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class '\App\Log' in method 'loadOrderBy'.

Open

                    \App\Log::warning("[RelationListView] Incorrect value of sorting: '$fieldName'");

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'Vtiger_Module_Model' in method 'getHeaders'.

Open

                $fieldModel = Vtiger_Field_Model::getInstance($fieldName, Vtiger_Module_Model::getInstance($fieldInfo['module_name']));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class '\App\Log' in method 'getHeaders'.

Open

                    \App\Log::warning("The field does not exist: '$fieldName' | Module: " . $this->getRelationModel()->getRelationModuleModel()->getName(), __METHOD__);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class '\App\Log' in method 'getHeaders'.

Open

                \App\Log::warning("The field does not exist: '$fieldName' | Module: " . $this->getRelationModel()->getRelationModuleModel()->getName(), __METHOD__);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class '\Vtiger_Module_Model' in method 'loadSearchLockedFields'.

Open

                    $moduleModel = \Vtiger_Module_Model::getInstance($moduleName);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid assigning values to variables in if clauses and the like (line '220', column '7').

Open

    public function loadCondition()
    {
        $relatedModuleName = $this->getRelatedModuleModel()->getName();
        $queryGenerator = $this->getRelationModel()->getQueryGenerator();
        if ($entityState = $this->get('entityState')) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($foo = 'bar') { // possible typo
            // ...
        }
        if ($baz = 0) { // always false
            // ...
        }
    }
}

The method loadOrderBy uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

                } else {
                    \App\Log::warning("[RelationListView] Incorrect value of sorting: '$fieldName'");
                }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid using static access to class 'App\Fields\Tree' in method 'getTreeEntries'.

Open

                $treeDetail = App\Fields\Tree::getValueByTreeId($template, $parent);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class '\App\Fields\Owner' in method 'getTreeEntries'.

Open

                $tree['rel_created_user'] = \App\Fields\Owner::getLabel($row['rel_created_user']);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'Vtiger_Link_Model' in method 'getLinks'.

Open

                Vtiger_Link_Model::getInstanceFromValues([
                    'linktype' => 'RELATEDLIST_VIEWS',
                    'linklabel' => 'LBL_RECORDS_LIST',
                    'view' => 'List',
                    'linkicon' => 'far fa-list-alt',

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'Vtiger_Link_Model' in method 'getLinks'.

Open

                $relatedLink['RELATEDLIST_MASSACTIONS'][] = Vtiger_Link_Model::getInstanceFromValues([
                    'linktype' => 'RELATEDLIST_MASSACTIONS',
                    'linklabel' => 'LBL_QUICK_EXPORT',
                    'linkurl' => "javascript:Vtiger_RelatedList_Js.triggerMassAction('index.php?module={$parentRecordModel->getModuleName()}&action=RelationAjax&mode=exportToExcel&src_record={$parentRecordModel->getId()}&relatedModule={$relatedModuleModel->getName()}&relationId={$this->getRelationModel()->getId()}&isSortActive=true','sendByForm')",
                    'linkclass' => '',

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'Vtiger_Loader' in method 'getLinks'.

Open

                $handlerClass = Vtiger_Loader::getComponentClassName('Model', 'PDF', $relatedModuleModel->getName());

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'Vtiger_Link_Model' in method 'getSelectRelationLinks'.

Open

            Vtiger_Link_Model::getInstanceFromValues([
                'linktype' => 'LISTVIEWBASIC',
                'linklabel' => \App\Language::translate('LBL_SELECT_RELATION', $relatedModel->getName()),
                'linkurl' => '',
                'linkicon' => 'fas fa-level-up-alt',

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'Vtiger_Link_Model' in method 'getAddRelationLinks'.

Open

                Vtiger_Link_Model::getInstanceFromValues([
                    'linktype' => 'LISTVIEWBASIC',
                    'linklabel' => App\Language::translate('LBL_ADD_RELATION', $relatedModel->getName()),
                    'linkurl' => $this->getCreateViewUrl(),
                    'linkqcs' => $relatedModel->isQuickCreateSupported(),

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class '\App\Json' in method 'getWidgetsList'.

Open

            $row['data'] = \App\Json::decode($row['data']);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class '\App\Cache' in method 'getWidgetsList'.

Open

            return \App\Cache::get('RelatedModuleWidgets', $relationId);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Define a constant instead of duplicating this literal "Documents" 3 times.

Open

            if ('Documents' === $relatedModuleModel->getName()) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Define a constant instead of duplicating this literal "linkurl" 4 times.

Open

                    'linkurl' => "javascript:Vtiger_RelatedList_Js.triggerMassDownload('index.php?module={$parentRecordModel->getModuleName()}&action=RelationAjax&mode=massDownload&src_record={$parentRecordModel->getId()}&relatedModule=Documents&mode=multiple','sendByForm')",

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Define a constant instead of duplicating this literal "RELATEDLIST_MASSACTIONS" 6 times.

Open

                $relatedLink['RELATEDLIST_MASSACTIONS'][] = Vtiger_Link_Model::getInstanceFromValues([

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Define a constant instead of duplicating this literal "RELATEDLIST_VIEWS" 3 times.

Open

            'RELATEDLIST_VIEWS' => [

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Define a constant instead of duplicating this literal "RELATEDLIST_BASIC" 3 times.

Open

                    $relatedLink['RELATEDLIST_BASIC'][] = Vtiger_Link_Model::getInstanceFromValues([

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Define a constant instead of duplicating this literal "linkdata" 3 times.

Open

                    'linkdata' => ['url' => $smsNotifierModel->getMassSMSUrlForModule($relatedModuleModel->getName()), 'type' => 'modal'],

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Define a constant instead of duplicating this literal "LISTVIEWBASIC" 5 times.

Open

            $relatedLink['LISTVIEWBASIC'] = $selectLinks;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Define a constant instead of duplicating this literal "linkicon" 9 times.

Open

                    'linkicon' => 'far fa-list-alt',

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Define a constant instead of duplicating this literal "viewId" 5 times.

Open

            $instance->set('viewId', $cvId);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Define a constant instead of duplicating this literal "Query" 3 times.

Open

        if ($this->has('Query')) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Define a constant instead of duplicating this literal "linkclass" 5 times.

Open

                    'linkclass' => '',

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Define a constant instead of duplicating this literal "RelatedModuleWidgets" 3 times.

Open

        if (\App\Cache::has('RelatedModuleWidgets', $relationId)) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Rename "$parentRecordModel" which has the same name as the field declared at line 27.

Open

        $parentRecordModel = $this->getParentRecordModel();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
  public $myField;

  public function doSomething() {
    $myField = 0;
    ...
  }
}

Define a constant instead of duplicating this literal "linktype" 9 times.

Open

                    'linktype' => 'RELATEDLIST_VIEWS',

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Define a constant instead of duplicating this literal "search_params" 4 times.

Open

        if ($searchParams = $this->getArray('search_params')) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Define a constant instead of duplicating this literal "linklabel" 8 times.

Open

                    'linklabel' => 'LBL_RECORDS_LIST',

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Rename "$relatedModuleModel" which has the same name as the field declared at line 34.

Open

        $relatedModuleModel = $relationModelInstance->getRelationModuleModel();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
  public $myField;

  public function doSomething() {
    $myField = 0;
    ...
  }
}

Avoid unused parameters such as '$recordModel'.

Open

    public function getEntryExtend(Vtiger_Record_Model $recordModel)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar($howdy)
    {
        // $howdy is not used
    }
}

Returning type false but getInstance() is declared to return \Vtiger_RelationListView_Model

Open

            return false;

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \App\QueryGenerator (Did you mean class \Tests\App\QueryGenerator)

Open

        $queryGenerator = new \App\QueryGenerator($relationModelInstance->getRelationModuleModel()->getName());

• Exclude checks
  • Disable engine
  • Disable check

Call to method getModule from undeclared class \App\QueryGenerator (Did you mean class \Tests\App\QueryGenerator)

Open

            $relationModuleName = $queryGenerator->getModule();

• Exclude checks
  • Disable engine
  • Disable check

Call to method setCustomColumn from undeclared class \App\QueryGenerator (Did you mean class \Tests\App\QueryGenerator)

Open

                    $queryGenerator->setCustomColumn($columnName);

• Exclude checks
  • Disable engine
  • Disable check

Call to method addCondition from undeclared class \App\QueryGenerator (Did you mean class \Tests\App\QueryGenerator)

Open

            $queryGenerator->addCondition('id', $this->get('src_record'), 'n');

• Exclude checks
  • Disable engine
  • Disable check

Argument 3 (moduleName) is false but \Vtiger_Loader::getComponentClassName() takes string defined at /code/include/Loader.php:149

Open

        $className = Vtiger_Loader::getComponentClassName('Model', 'RelationListView', $parentModuleModel->getName());

• Exclude checks
  • Disable engine
  • Disable check

Call to method parseAdvFilter from undeclared class \App\QueryGenerator (Did you mean class \Tests\App\QueryGenerator)

Open

            $queryGenerator->parseAdvFilter($searchParams);

• Exclude checks
  • Disable engine
  • Disable check

Call to method addCondition from undeclared class \App\QueryGenerator (Did you mean class \Tests\App\QueryGenerator)

Open

            $queryGenerator->addCondition($this->get('search_key'), $this->get('search_value'), $this->get('operator'));

• Exclude checks
  • Disable engine
  • Disable check

Return type of getRelationQuery() is undeclared type \App\QueryGenerator (Did you mean class \Tests\App\QueryGenerator)

Open

    public function getRelationQuery($returnQueryGenerator = false)

• Exclude checks
  • Disable engine
  • Disable check

Call to method setStateCondition from undeclared class \App\QueryGenerator (Did you mean class \Tests\App\QueryGenerator)

Open

            $queryGenerator->setStateCondition($entityState);

• Exclude checks
  • Disable engine
  • Disable check

Call to method limit from undeclared class \App\QueryGenerator (Did you mean class \Tests\App\QueryGenerator)

Open

        $query->limit($pageLimit + 1)->offset($pagingModel->getStartIndex());

• Exclude checks
  • Disable engine
  • Disable check

Call to method warning from undeclared class \App\Log

Open

                    \App\Log::warning("The field does not exist: '$fieldName' | Module: " . $this->getRelationModel()->getRelationModuleModel()->getName(), __METHOD__);

• Exclude checks
  • Disable engine
  • Disable check

Call to method warning from undeclared class \App\Log

Open

                \App\Log::warning("The field does not exist: '$fieldName' | Module: " . $this->getRelationModel()->getRelationModuleModel()->getName(), __METHOD__);

• Exclude checks
  • Disable engine
  • Disable check

Call to method createQuery from undeclared class \App\QueryGenerator (Did you mean class \Tests\App\QueryGenerator)

Open

            $query = $queryGenerator->createQuery();

• Exclude checks
  • Disable engine
  • Disable check

Return type of getQueryGenerator() is undeclared type \App\QueryGenerator (Did you mean class \Tests\App\QueryGenerator)

Open

    public function getQueryGenerator()

• Exclude checks
  • Disable engine
  • Disable check

Call to method count from undeclared class \App\QueryGenerator (Did you mean class \Tests\App\QueryGenerator)

Open

        return $this->getRelationQuery()->count();

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (templateId) is \type but \App\Fields\Tree::getValueByTreeId() takes int defined at /code/app/Fields/Tree.php:47

Open

                $treeDetail = App\Fields\Tree::getValueByTreeId($template, $parent);

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property addSearchParamsToCreateView from undeclared class \Config\Relation

Open

        if (!empty(Config\Relation::$addSearchParamsToCreateView) && ($searchParams = $this->getArray('search_params')) && isset($searchParams['and']) && \is_array($searchParams['and'])) {

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \Vtiger_Module_Model::getMassSMSUrlForModule

Open

                    'linkdata' => ['url' => $smsNotifierModel->getMassSMSUrlForModule($relatedModuleModel->getName()), 'type' => 'modal'],

• Exclude checks
  • Disable engine
  • Disable check

Call to method getRelatedFields from undeclared class \App\QueryGenerator (Did you mean class \Tests\App\QueryGenerator)

Open

        foreach ($this->getQueryGenerator()->getRelatedFields() as $fieldInfo) {

• Exclude checks
  • Disable engine
  • Disable check

Argument 2 (moduleName) is false but \App\Language::translate() takes string defined at /code/app/Language.php:161

Open

                'name' => $parentName . App\Language::translate($row['name'], $relModuleName),

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db\Query::select

Open

        return (new App\Db\Query())->select(['relcrmid'])->from('u_#__favorites')

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                \App\Log::warning("The field does not exist: '$fieldName' | Module: " . $this->getRelationModel()->getRelationModuleModel()->getName(), __METHOD__);

• Exclude checks
  • Disable engine
  • Disable check

Call to method warning from undeclared class \App\Log

Open

                    \App\Log::warning("The field does not exist: '$fieldName' | Module: " . $this->getRelationModel()->getRelationModuleModel()->getName(), __METHOD__);

• Exclude checks
  • Disable engine
  • Disable check

Argument 2 (key) is int but \App\Cache::has() takes string defined at /code/app/Cache.php:74

Open

        if (\App\Cache::has('RelatedModuleWidgets', $relationId)) {

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared function \array_key_first()

Open

            $cvId = array_key_first($this->getRelationModel()->getCustomViewList());

• Exclude checks
  • Disable engine
  • Disable check

Call to method all from undeclared class \App\QueryGenerator (Did you mean class \Tests\App\QueryGenerator)

Open

        return $this->getRecordsFromArray($this->getRelationQuery()->all());

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                    \App\Log::warning("The field does not exist: '$fieldName' | Module: " . $this->getRelationModel()->getRelationModuleModel()->getName(), __METHOD__);

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db\Query::from

Open

        return (new \App\Db\Query())->from('vtiger_trees_templates_data ttd')->innerJoin('u_#__crmentity_rel_tree rel', 'rel.tree = ttd.tree')

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \Vtiger_Module_Model::isSMSActiveForModule

Open

            if ($relatedModuleModel->isPermitted('MassSendSMS') && ($smsNotifierModel = \Vtiger_Module_Model::getInstance('SMSNotifier'))->isSMSActiveForModule($relatedModuleModel->getName())) {

• Exclude checks
  • Disable engine
  • Disable check

Argument 2 (moduleName) is false but \App\Language::translate() takes string defined at /code/app/Language.php:161

Open

                    'linklabel' => App\Language::translate('LBL_ADD_RELATION', $relatedModel->getName()),

• Exclude checks
  • Disable engine
  • Disable check

Call to method getCurrentUserId from undeclared class \App\User (Did you mean class \Tests\App\User)

Open

                'userid' => App\User::getCurrentUserId(),

• Exclude checks
  • Disable engine
  • Disable check

Call to method initForCustomViewById from undeclared class \App\QueryGenerator (Did you mean class \Tests\App\QueryGenerator)

Open

            $this->getRelationModel()->getQueryGenerator()->initForCustomViewById($cvId);

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (moduleName) is false but \App\CustomView::getInstance() takes string defined at /code/app/CustomView.php:159

Open

            $customView = App\CustomView::getInstance($moduleModel->getName());

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                    \App\Log::warning("The field does not exist: '$fieldName' | Module: " . $this->getRelationModel()->getRelationModuleModel()->getName(), __METHOD__);

• Exclude checks
  • Disable engine
  • Disable check

Argument 2 (key) is int but \App\Cache::get() takes string defined at /code/app/Cache.php:61

Open

            return \App\Cache::get('RelatedModuleWidgets', $relationId);

• Exclude checks
  • Disable engine
  • Disable check

Call to method warning from undeclared class \App\Log

Open

                    \App\Log::warning("[RelationListView] Incorrect value of sorting: '$fieldName'");

• Exclude checks
  • Disable engine
  • Disable check

Argument 3 (moduleName) is false but \Vtiger_Loader::getComponentClassName() takes string defined at /code/include/Loader.php:149

Open

                $handlerClass = Vtiger_Loader::getComponentClassName('Model', 'PDF', $relatedModuleModel->getName());

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                    'linkurl' => "javascript:Vtiger_RelatedList_Js.triggerMassAction('index.php?module={$parentRecordModel->getModuleName()}&action=RelationAjax&mode=exportToExcel&src_record={$parentRecordModel->getId()}&relatedModule={$relatedModuleModel->getName()}&relationId={$this->getRelationModel()->getId()}&isSortActive=true','sendByForm')",

• Exclude checks
  • Disable engine
  • Disable check

Argument 2 (key) is int but \App\Cache::save() takes string defined at /code/app/Cache.php:89

Open

        App\Cache::save('RelatedModuleWidgets', $relationId, $widgets);

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (moduleName) is false but \App\EventHandler::setModuleName() takes string defined at /code/app/EventHandler.php:366

Open

        $eventHandler->setModuleName($relatedModuleModel->getName());

• Exclude checks
  • Disable engine
  • Disable check

Argument 2 (moduleName) is false but \App\Language::translate() takes string defined at /code/app/Language.php:161

Open

                'linklabel' => \App\Language::translate('LBL_SELECT_RELATION', $relatedModel->getName()),

• Exclude checks
  • Disable engine
  • Disable check

Argument 2 (moduleName) is false but \App\Language::translate() takes string defined at /code/app/Language.php:161

Open

                $parentName = '(' . App\Language::translate($treeDetail['name'], $relModuleName) . ') ';

• Exclude checks
  • Disable engine
  • Disable check

Returning type \Vtiger_TreeView_Model but getTreeViewModel() is declared to return \Vtiger_TreeCategoryModal_Model

Open

        return Vtiger_TreeCategoryModal_Model::getInstance($this->getRelatedModuleModel());

• Exclude checks
  • Disable engine
  • Disable check

Call to method all from undeclared class \App\QueryGenerator (Did you mean class \Tests\App\QueryGenerator)

Open

        $rows = $query->all();

• Exclude checks
  • Disable engine
  • Disable check

Call to method getLabel from undeclared class \App\Fields\Owner

Open

                $tree['rel_created_user'] = \App\Fields\Owner::getLabel($row['rel_created_user']);

• Exclude checks
  • Disable engine
  • Disable check

Argument 3 (moduleName) is false but \Vtiger_Loader::getComponentClassName() takes string defined at /code/include/Loader.php:149

Open

                $widgetName = Vtiger_Loader::getComponentClassName('Widget', $widget['type'], $moduleModel->getName());

• Exclude checks
  • Disable engine
  • Disable check

Call to method setOrder from undeclared class \App\QueryGenerator (Did you mean class \Tests\App\QueryGenerator)

Open

                    $this->getRelationModel()->getQueryGenerator()->setOrder($fieldName, $sortFlag);

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db\Query::from

Open

        $query = (new App\Db\Query())->from('a_#__relatedlists_widgets')->where(['relation_id' => $relationId]);

• Exclude checks
  • Disable engine
  • Disable check

Identical blocks of code found in 2 locations. Consider refactoring.

Open

            foreach ($relatedFields as $relatedModuleName => $fields) {
                foreach ($fields as $sourceField => $field) {
                    $recordData = [
                        'id' => $row[$sourceField . $relatedModuleName . 'id'] ?? 0,
                    ];

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

        if (!$request->isEmpty('lockedEmptyFields')) {
            foreach ($request->getArray('lockedEmptyFields') as $value) {
                if (strpos($value, ':')) {
                    [$fieldName, $moduleName] = explode(':', $value);
                    $moduleModel = \Vtiger_Module_Model::getInstance($moduleName);

• Read up
  Read up

Avoid excessively long variable names like $relationModelInstance. Keep variable name length under 20.

Open

        $relationModelInstance = $this->getRelationModel();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}

Avoid excessively long variable names like $relationModelInstance. Keep variable name length under 20.

Open

        $relationModelInstance = $this->getRelationModel();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}

Avoid excessively long variable names like $relationModelInstance. Keep variable name length under 20.

Open

        $relationModelInstance = $this->getRelationModel();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}

Avoid excessively long variable names like $relationModelInstance. Keep variable name length under 20.

Open

            $relationModelInstance = Vtiger_Relation_Model::getInstanceById($relationId);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}

Avoid excessively long variable names like $relationModelInstance. Keep variable name length under 20.

Open

        $relationModelInstance = $this->getRelationModel();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}

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

Open

class Vtiger_RelationListView_Model extends \App\Base

• Exclude checks
  • Disable engine
  • Disable check

The class Vtiger_RelationListView_Model is not named in CamelCase.

Open

class Vtiger_RelationListView_Model extends \App\Base
{
    /**
     * Relation model instance.
     *

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class class_name {
}

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

Open

     * Set relation model instance.

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        return $this;

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    public function isQuickSearchEnabled(): bool

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @param string              $relationModuleName

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @return self

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $instance->setParentRecordModel($parentRecordModel);

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $relationModelInstance->set('parentRecord', $parentRecordModel);

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @var Vtiger_Relation_Model

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * Function to identify if the module supports quick search or not.

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    public static function getInstance(Vtiger_Record_Model $parentRecordModel, string $relationModuleName, $relationId = false, $cvId = 0)

• Exclude checks
  • Disable engine
  • Disable check

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

Open

            $instance->set('viewId', $cvId);

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @var Vtiger_Module_Model

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @return Vtiger_Relation_Model

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @return \App\QueryGenerator

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * Get relation list view model instance.

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 138 characters

Open

    public static function getInstance(Vtiger_Record_Model $parentRecordModel, string $relationModuleName, $relationId = false, $cvId = 0)

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $instance->setRelatedModuleModel($relationModelInstance->getRelationModuleModel());

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    protected $mandatoryColumns = [];

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @param Vtiger_Record_Model $parentRecord

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $relationModelInstance->set('query_generator', $queryGenerator);

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $instance->setRelationModel($relationModelInstance);

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * Function to get Relation query.

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    protected $relationModel;

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * Mandatory columns.

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @return $this

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        return $this;

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * Get related module model instance.

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * Relation model instance.

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @param Vtiger_Module_Model $relatedModuleModel

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        } else {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        if (is_numeric($cvId)) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        return $instance;

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @var array

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        return $this->has('quickSearchEnabled') ? $this->get('quickSearchEnabled') : true;

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    public function getRelationModel()

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        return $this->getRelationModel()->getQueryGenerator();

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $instance = new $className();

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 160 characters

Open

            $relationModelInstance = Vtiger_Relation_Model::getInstance($parentModuleModel, Vtiger_Module_Model::getInstance($relationModuleName), $relationId);

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    public function getRelationQuery($returnQueryGenerator = false)

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $this->loadCondition();

• Exclude checks
  • Disable engine
  • Disable check

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

Open

            $relationModuleName = $queryGenerator->getModule();

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        if ($entityState = $this->get('entityState')) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        } else {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                    }

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        if ($this->has('viewId')) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * Function to get the related list view entries.

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $listViewRecordModels = $relatedFields = [];

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $recordId = $this->getParentRecordModel()->getId();

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                    $recordData = [

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                    $extRecordModel[$sourceField][$relatedModuleName] = Vtiger_Module_Model::getInstance($relatedModuleName)->getRecordFromArray($recordData);

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $this->parentRecordModel = $parentRecord;

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

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

Open

            $relationModelInstance = Vtiger_Relation_Model::getInstance($parentModuleModel, Vtiger_Module_Model::getInstance($relationModuleName), $relationId);

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        if (!$relationModelInstance) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                }

• Exclude checks
  • Disable engine
  • Disable check

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

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $pageLimit = $pagingModel->getPageLimit();

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @return \Vtiger_Record_Model[]

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @param array $rows

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    public function getRecordsFromArray(array $rows)

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     *

• Exclude checks
  • Disable engine
  • Disable check