YetiForceCompany/YetiForceCRM

View on GitHub
tests/App/Field.php

Summary

Maintainability
A
0 mins
Test Coverage
A
100%

Missing class import via use statement (line '27', column '21').
Open

        $fieldRoId = (new \App\Db\Query())->select(['fieldid'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'smcreatorid'])->scalar();
Severity: Minor
Found in tests/App/Field.php by phpmd

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 '26', column '19').
Open

        $fieldId = (new \App\Db\Query())->select(['fieldid'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'email'])->scalar();
Severity: Minor
Found in tests/App/Field.php by phpmd

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 '63', column '15').
Open

        return (new \App\Db\Query())->select(['tabid', 'related_tabid', 'relation_id'])->from('vtiger_relatedlists')->all();
Severity: Minor
Found in tests/App/Field.php by phpmd

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 '159', column '19').
Open

        $fieldId = (new \App\Db\Query())->select(['fieldid'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'email'])->scalar();
Severity: Minor
Found in tests/App/Field.php by phpmd

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

        $fieldColumn = (new \App\Db\Query())->select(['columnname'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'vat_id'])->scalar();
Severity: Minor
Found in tests/App/Field.php by phpmd

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 '45', column '25').
Open

        $fieldRoColumn = (new \App\Db\Query())->select(['columnname'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'smcreatorid'])->scalar();
Severity: Minor
Found in tests/App/Field.php by phpmd

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

Avoid using static access to class '\App\Field' in method 'testGetColumnPermission'.
Open

        $this->assertTrue(\App\Field::getColumnPermission($moduleId, $fieldColumn, true), 'Expected read perms(ids)');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetRelatedFieldForModulePair'.
Open

        $result0 = \App\Field::getRelatedFieldForModule(\App\Module::getModuleName($moduleId), \App\Module::getModuleName($forModuleId));
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Module' in method 'testGetRelatedFieldForModulePair'.
Open

        $result2 = \App\Field::getRelatedFieldForModule(\App\Module::getModuleName($moduleId), false);
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetColumnPermission'.
Open

        $this->assertFalse(\App\Field::getColumnPermission($moduleId, $fieldRoColumn, false), 'Expected no write perms(ids)');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetColumnPermission'.
Open

        $this->assertFalse(\App\Field::getColumnPermission($moduleId, $fieldRoColumn, false), 'Expected no write perms(ids, cached)');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Module' in method 'testGetRelatedFieldForModulePair'.
Open

        $result1 = \App\Field::getRelatedFieldForModule(false, \App\Module::getModuleName($forModuleId));
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetRelatedFieldForModulePair'.
Open

        $result2 = \App\Field::getRelatedFieldForModule(\App\Module::getModuleName($moduleId), false);
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetColumnPermission'.
Open

        $this->assertTrue(\App\Field::getColumnPermission($moduleId, $fieldColumn, false), 'Expected write perms(ids, cached)');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Module' in method 'testGetRelatedFieldForModulePair'.
Open

        $result0 = \App\Field::getRelatedFieldForModule(\App\Module::getModuleName($moduleId), \App\Module::getModuleName($forModuleId));
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetFieldPermission'.
Open

        $this->assertTrue(\App\Field::getFieldPermission('Leads', 'email', true), 'Expected read perms(strings)');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetColumnPermission'.
Open

        $this->assertTrue(\App\Field::getColumnPermission('Leads', $fieldColumn, true), 'Expected read perms(string)');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetFieldPermission'.
Open

        $this->assertTrue(\App\Field::getFieldPermission($moduleId, $fieldId, false), 'Expected write perms(ids)');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetFieldPermission'.
Open

        $this->assertFalse(\App\Field::getFieldPermission($moduleId, 'NxField', true), 'Expected no read perms(field not exists)');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetFieldPermission'.
Open

        $this->assertFalse(\App\Field::getFieldPermission($moduleId, 'NxField', false), 'Expected no write perms(field not exists)');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetColumnPermission'.
Open

        $this->assertFalse(\App\Field::getColumnPermission($moduleId, 'NxColumn', false), 'Expected write perms(field not exists)');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Module' in method 'testGetRelatedFieldForModulePair'.
Open

        $result0 = \App\Field::getRelatedFieldForModule(\App\Module::getModuleName($moduleId), \App\Module::getModuleName($forModuleId));
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetFieldPermission'.
Open

        $this->assertTrue(\App\Field::getFieldPermission('Leads', 'email', false), 'Expected write perms(strings,cached)');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetFieldPermission'.
Open

        $this->assertTrue(\App\Field::getFieldPermission($moduleId, $fieldId, true), 'Expected read perms(ids)');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetFieldPermission'.
Open

        $this->assertFalse(\App\Field::getFieldPermission($moduleId, $fieldRoId, false), 'Expected no write perms(ids)');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetRelatedFieldForModulePair'.
Open

        $result1 = \App\Field::getRelatedFieldForModule(false, \App\Module::getModuleName($forModuleId));
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Module' in method 'testGetFieldPermission'.
Open

        $moduleId = \App\Module::getModuleId('Leads');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetFieldPermission'.
Open

        $this->assertTrue(\App\Field::getFieldPermission('Leads', 'email', false), 'Expected write perms(strings)');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Module' in method 'testGetColumnPermission'.
Open

        $moduleId = \App\Module::getModuleId('Leads');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetRelatedFieldForSpecificModulePairs'.
Open

        $result = \App\Field::getRelatedFieldForModule($moduleName, $forModuleName);
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetColumnPermission'.
Open

        $this->assertTrue(\App\Field::getColumnPermission($moduleId, $fieldColumn, false), 'Expected write perms(ids)');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetColumnPermission'.
Open

        $this->assertFalse(\App\Field::getColumnPermission($moduleId, 'NxColumn', true), 'Expected no read perms(field not exists)');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetRelatedFieldForModuleAll'.
Open

        $this->assertNotEmpty(\App\Field::getRelatedFieldForModule(), 'All relations list should be not empty');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetFieldsFromRelation'.
Open

        $result = \App\Field::getFieldsFromRelation($relationId);
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetFieldInfo'.
Open

        $fieldInfo = \App\Field::getFieldInfo($fieldId);
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetFieldInfo'.
Open

        $fieldInfoByName = \App\Field::getFieldInfo('email', $moduleId);
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetFieldsFromEmptyRelation'.
Open

        $result = \App\Field::getFieldsFromRelation('');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetFieldInfo'.
Open

        $this->assertSame($fieldInfo, \App\Field::getFieldInfo($fieldId), 'Field info from cache should be same as reference');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Field' in method 'testGetFieldsFromRelation'.
Open

        $this->assertSame($result, \App\Field::getFieldsFromRelation($relationId), 'Relation fields from cache should be equal as reference for relation: ' . $relationId);
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Module' in method 'testGetRelatedFieldForSpecificModulePairs'.
Open

        $this->assertSame(\App\Module::getModuleId($moduleName), $result['tabid'], 'Expected tabid differs from reference');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Avoid using static access to class '\App\Module' in method 'testGetFieldInfo'.
Open

        $moduleId = \App\Module::getModuleId('Leads');
Severity: Minor
Found in tests/App/Field.php by phpmd

StaticAccess

Since: 1.4.0

Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

Example

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

Source https://phpmd.org/rules/cleancode.html#staticaccess

Define a constant instead of duplicating this literal "Leads" 7 times.
Open

        $moduleId = \App\Module::getModuleId('Leads');
Severity: Critical
Found in tests/App/Field.php by sonar-php

Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.

On the other hand, constants can be referenced from many places, but only need to be updated in a single place.

Noncompliant Code Example

With the default threshold of 3:

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

Compliant Solution

ACTION_1 = 'action1';

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

Exceptions

To prevent generating some false-positives, literals having less than 5 characters are excluded.

Define a constant instead of duplicating this literal "Relation list should be array type" 4 times.
Open

        $this->assertIsArray($result0, 'Relation list should be array type');
Severity: Critical
Found in tests/App/Field.php by sonar-php

Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.

On the other hand, constants can be referenced from many places, but only need to be updated in a single place.

Noncompliant Code Example

With the default threshold of 3:

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

Compliant Solution

ACTION_1 = 'action1';

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

Exceptions

To prevent generating some false-positives, literals having less than 5 characters are excluded.

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

        $fieldId = (new \App\Db\Query())->select(['fieldid'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'email'])->scalar();
Severity: Critical
Found in tests/App/Field.php by sonar-php

Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.

On the other hand, constants can be referenced from many places, but only need to be updated in a single place.

Noncompliant Code Example

With the default threshold of 3:

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

Compliant Solution

ACTION_1 = 'action1';

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

Exceptions

To prevent generating some false-positives, literals having less than 5 characters are excluded.

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

        $fieldId = (new \App\Db\Query())->select(['fieldid'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'email'])->scalar();
Severity: Critical
Found in tests/App/Field.php by sonar-php

Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.

On the other hand, constants can be referenced from many places, but only need to be updated in a single place.

Noncompliant Code Example

With the default threshold of 3:

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

Compliant Solution

ACTION_1 = 'action1';

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

Exceptions

To prevent generating some false-positives, literals having less than 5 characters are excluded.

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

        $fieldId = (new \App\Db\Query())->select(['fieldid'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'email'])->scalar();
Severity: Critical
Found in tests/App/Field.php by sonar-php

Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.

On the other hand, constants can be referenced from many places, but only need to be updated in a single place.

Noncompliant Code Example

With the default threshold of 3:

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

Compliant Solution

ACTION_1 = 'action1';

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

Exceptions

To prevent generating some false-positives, literals having less than 5 characters are excluded.

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

        $fieldId = (new \App\Db\Query())->select(['fieldid'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'email'])->scalar();
Severity: Critical
Found in tests/App/Field.php by sonar-php

Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.

On the other hand, constants can be referenced from many places, but only need to be updated in a single place.

Noncompliant Code Example

With the default threshold of 3:

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

Compliant Solution

ACTION_1 = 'action1';

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

Exceptions

To prevent generating some false-positives, literals having less than 5 characters are excluded.

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

        $fieldId = (new \App\Db\Query())->select(['fieldid'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'email'])->scalar();
Severity: Critical
Found in tests/App/Field.php by sonar-php

Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.

On the other hand, constants can be referenced from many places, but only need to be updated in a single place.

Noncompliant Code Example

With the default threshold of 3:

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

Compliant Solution

ACTION_1 = 'action1';

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

Exceptions

To prevent generating some false-positives, literals having less than 5 characters are excluded.

Avoid unused parameters such as '$relationId'.
Open

    public function testGetRelatedFieldForModulePair($forModuleId, $moduleId, $relationId)
Severity: Minor
Found in tests/App/Field.php by phpmd

UnusedFormalParameter

Since: 0.2

Avoid passing parameters to methods or constructors and then not using those parameters.

Example

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

Source https://phpmd.org/rules/unusedcode.html#unusedformalparameter

Avoid unused parameters such as '$moduleId'.
Open

    public function testGetFieldsFromRelation($forModuleId, $moduleId, $relationId)
Severity: Minor
Found in tests/App/Field.php by phpmd

UnusedFormalParameter

Since: 0.2

Avoid passing parameters to methods or constructors and then not using those parameters.

Example

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

Source https://phpmd.org/rules/unusedcode.html#unusedformalparameter

Avoid unused parameters such as '$forModuleId'.
Open

    public function testGetFieldsFromRelation($forModuleId, $moduleId, $relationId)
Severity: Minor
Found in tests/App/Field.php by phpmd

UnusedFormalParameter

Since: 0.2

Avoid passing parameters to methods or constructors and then not using those parameters.

Example

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

Source https://phpmd.org/rules/unusedcode.html#unusedformalparameter

Call to undeclared method \Tests\App\Field::assertFalse
Open

        $this->assertFalse(\App\Field::getFieldPermission($moduleId, 'NxField', false), 'Expected no write perms(field not exists)');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertFalse
Open

        $this->assertFalse(\App\Field::getColumnPermission($moduleId, $fieldRoColumn, false), 'Expected no write perms(ids)');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertNotEmpty
Open

        $this->assertNotEmpty(\App\Field::getRelatedFieldForModule(), 'All relations list should be not empty');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertSame
Open

        $this->assertSame($fieldInfo, \App\Field::getFieldInfo($fieldId), 'Field info from cache should be same as reference');
Severity: Critical
Found in tests/App/Field.php by phan

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

        $fieldId = (new \App\Db\Query())->select(['fieldid'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'email'])->scalar();
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertFalse
Open

        $this->assertFalse(\App\Field::getFieldPermission($moduleId, $fieldRoId, false), 'Expected no write perms(ids)');
Severity: Critical
Found in tests/App/Field.php by phan

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

        return (new \App\Db\Query())->select(['tabid', 'related_tabid', 'relation_id'])->from('vtiger_relatedlists')->all();
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertNotEmpty
Open

        $this->assertNotEmpty($fieldInfo, 'Field info should be not empty');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertNotEmpty
Open

        $this->assertNotEmpty($result, 'Relation data should be not empty');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertIsArray
Open

        $this->assertIsArray($fieldInfo, 'Expected field info array');
Severity: Critical
Found in tests/App/Field.php by phan

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

        $fieldId = (new \App\Db\Query())->select(['fieldid'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'email'])->scalar();
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertTrue
Open

        $this->assertTrue(\App\Field::getFieldPermission('Leads', 'email', true), 'Expected read perms(strings)');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertTrue
Open

        $this->assertTrue(\App\Field::getFieldPermission('Leads', 'email', false), 'Expected write perms(strings,cached)');
Severity: Critical
Found in tests/App/Field.php by phan

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

        $fieldColumn = (new \App\Db\Query())->select(['columnname'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'vat_id'])->scalar();
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertTrue
Open

        $this->assertTrue(\App\Field::getColumnPermission('Leads', $fieldColumn, true), 'Expected read perms(string)');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertIsArray
Open

        $this->assertIsArray($result2, 'Relation list should be array type');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertFalse
Open

        $this->assertFalse(\App\Field::getColumnPermission($moduleId, 'NxColumn', false), 'Expected write perms(field not exists)');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertIsArray
Open

        $this->assertIsArray($result, 'Expected result type array for empty relation');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertSame
Open

        $this->assertSame(\App\Module::getModuleId($moduleName), $result['tabid'], 'Expected tabid differs from reference');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertTrue
Open

        $this->assertTrue(\App\Field::getColumnPermission($moduleId, $fieldColumn, false), 'Expected write perms(ids)');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertSame
Open

        $this->assertSame($moduleId, $fieldInfoByName['tabid'], 'Expected moduleid in fieldinfo same as reference');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertTrue
Open

        $this->assertTrue(\App\Field::getColumnPermission($moduleId, $fieldColumn, false), 'Expected write perms(ids, cached)');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertIsArray
Open

        $this->assertIsArray($result1, 'Relation list should be array type');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertEmpty
Open

        $this->assertEmpty($result, 'Fields array from empty relation should be empty');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertSame
Open

        $this->assertSame($fieldId, $fieldInfo['fieldid'], 'Expected fieldid in fieldinfo same as reference');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertIsArray
Open

        $this->assertIsArray($fieldInfoByName, 'Expected field info array');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertTrue
Open

        $this->assertTrue(\App\Field::getFieldPermission('Leads', 'email', false), 'Expected write perms(strings)');
Severity: Critical
Found in tests/App/Field.php by phan

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

        $fieldRoColumn = (new \App\Db\Query())->select(['columnname'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'smcreatorid'])->scalar();
Severity: Critical
Found in tests/App/Field.php by phan

Expected @param annotation for moduleId to be before the @param annotation for relationId
Open

     * @param int   $relationId
Severity: Info
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertSame
Open

        $this->assertSame($result, \App\Field::getFieldsFromRelation($relationId), 'Relation fields from cache should be equal as reference for relation: ' . $relationId);
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertTrue
Open

        $this->assertTrue(\App\Field::getFieldPermission($moduleId, $fieldId, false), 'Expected write perms(ids)');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertFalse
Open

        $this->assertFalse(\App\Field::getFieldPermission($moduleId, 'NxField', true), 'Expected no read perms(field not exists)');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertIsArray
Open

        $this->assertIsArray($result0, 'Relation list should be array type');
Severity: Critical
Found in tests/App/Field.php by phan

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

        $fieldRoId = (new \App\Db\Query())->select(['fieldid'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'smcreatorid'])->scalar();
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertTrue
Open

        $this->assertTrue(\App\Field::getFieldPermission($moduleId, $fieldId, true), 'Expected read perms(ids)');
Severity: Critical
Found in tests/App/Field.php by phan

Argument 1 (relationId) is '' but \App\Field::getFieldsFromRelation() takes int defined at /code/app/Field.php:297
Open

        $result = \App\Field::getFieldsFromRelation('');
Severity: Minor
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertSame
Open

        $this->assertSame($fieldId, $fieldInfoByName['fieldid'], 'Expected fieldid in fieldinfo same as reference');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertFalse
Open

        $this->assertFalse(\App\Field::getColumnPermission($moduleId, $fieldRoColumn, false), 'Expected no write perms(ids, cached)');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertIsArray
Open

        $this->assertIsArray($result, 'Expected result type array for relation: ' . $relationId);
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertTrue
Open

        $this->assertTrue(\App\Field::getColumnPermission($moduleId, $fieldColumn, true), 'Expected read perms(ids)');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertFalse
Open

        $this->assertFalse(\App\Field::getColumnPermission($moduleId, 'NxColumn', true), 'Expected no read perms(field not exists)');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertIsArray
Open

        $this->assertIsArray($result, 'Relation list should be array type');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertSame
Open

        $this->assertSame($moduleId, $fieldInfo['tabid'], 'Expected moduleid in fieldinfo same as reference');
Severity: Critical
Found in tests/App/Field.php by phan

Call to undeclared method \Tests\App\Field::assertNotEmpty
Open

        $this->assertNotEmpty($fieldInfoByName, 'Field info should be not empty');
Severity: Critical
Found in tests/App/Field.php by phan

Line exceeds 120 characters; contains 149 characters
Open

        $fieldId = (new \App\Db\Query())->select(['fieldid'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'email'])->scalar();
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * Testing getFieldPermission function.
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertTrue(\App\Field::getFieldPermission($moduleId, $fieldId, true), 'Expected read perms(ids)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertTrue(\App\Field::getFieldPermission($moduleId, $fieldId, false), 'Expected write perms(ids)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertTrue(\App\Field::getFieldPermission('Leads', 'email', false), 'Expected write perms(strings)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertFalse(\App\Field::getFieldPermission($moduleId, 'NxField', true), 'Expected no read perms(field not exists)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    }
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $fieldId = (new \App\Db\Query())->select(['fieldid'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'email'])->scalar();
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    {
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertTrue(\App\Field::getFieldPermission('Leads', 'email', false), 'Expected write perms(strings,cached)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     *
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    }
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

Line exceeds 120 characters; contains 124 characters
Open

        $this->assertSame(\App\Module::getModuleId($moduleName), $result['tabid'], 'Expected tabid differs from reference');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * Testing getRelatedFieldForModule without params.
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * @param int   $relationId
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $result = \App\Field::getFieldsFromRelation($relationId);
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    {
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

Line exceeds 120 characters; contains 121 characters
Open

        $this->assertFalse(\App\Field::getFieldPermission($moduleId, $fieldRoId, false), 'Expected no write perms(ids)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

Line exceeds 120 characters; contains 133 characters
Open

        $this->assertFalse(\App\Field::getFieldPermission($moduleId, 'NxField', false), 'Expected no write perms(field not exists)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    /**
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $fieldColumn = (new \App\Db\Query())->select(['columnname'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'vat_id'])->scalar();
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    /**
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

Line exceeds 120 characters; contains 128 characters
Open

        $this->assertTrue(\App\Field::getColumnPermission($moduleId, $fieldColumn, false), 'Expected write perms(ids, cached)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertFalse(\App\Field::getFieldPermission($moduleId, $fieldRoId, false), 'Expected no write perms(ids)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     */
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * @param mixed $moduleId
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        ];
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     */
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertSame($result, \App\Field::getFieldsFromRelation($relationId), 'Relation fields from cache should be equal as reference for relation: ' . $relationId);
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

Line exceeds 120 characters; contains 171 characters
Open

        $this->assertSame($result, \App\Field::getFieldsFromRelation($relationId), 'Relation fields from cache should be equal as reference for relation: ' . $relationId);
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $moduleId = \App\Module::getModuleId('Leads');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     */
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

Line exceeds 120 characters; contains 123 characters
Open

        $this->assertTrue(\App\Field::getFieldPermission('Leads', 'email', false), 'Expected write perms(strings,cached)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    {
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    public function testGetRelatedFieldForModulePair($forModuleId, $moduleId, $relationId)
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertIsArray($result1, 'Relation list should be array type');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * Relations modules list provider.
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    /**
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * @param mixed $forModuleName
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $result = \App\Field::getRelatedFieldForModule($moduleName, $forModuleName);
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertIsArray($result, 'Relation list should be array type');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertNotEmpty($fieldInfoByName, 'Field info should be not empty');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

Line exceeds 120 characters; contains 131 characters
Open

        $this->assertFalse(\App\Field::getFieldPermission($moduleId, 'NxField', true), 'Expected no read perms(field not exists)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

            ['DataSetRegister', 'IncidentRegister'],
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertNotEmpty(\App\Field::getRelatedFieldForModule(), 'All relations list should be not empty');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * @param mixed $moduleId
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    /**
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $fieldId = (new \App\Db\Query())->select(['fieldid'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'email'])->scalar();
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

Line exceeds 120 characters; contains 127 characters
Open

        $this->assertSame($fieldInfo, \App\Field::getFieldInfo($fieldId), 'Field info from cache should be same as reference');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $fieldInfoByName = \App\Field::getFieldInfo('email', $moduleId);
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertSame($moduleId, $fieldInfoByName['tabid'], 'Expected moduleid in fieldinfo same as reference');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

Line exceeds 120 characters; contains 133 characters
Open

        $this->assertFalse(\App\Field::getColumnPermission($moduleId, 'NxColumn', true), 'Expected no read perms(field not exists)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    /**
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     */
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     */
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    }
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    }
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    public function testGetFieldPermission()
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $moduleId = \App\Module::getModuleId('Leads');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $fieldRoId = (new \App\Db\Query())->select(['fieldid'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'smcreatorid'])->scalar();
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertFalse(\App\Field::getFieldPermission($moduleId, 'NxField', false), 'Expected no write perms(field not exists)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * Testing getColumnPermission function.
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    public function testGetColumnPermission()
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

Line exceeds 120 characters; contains 157 characters
Open

        $fieldColumn = (new \App\Db\Query())->select(['columnname'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'vat_id'])->scalar();
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertTrue(\App\Field::getColumnPermission($moduleId, $fieldColumn, false), 'Expected write perms(ids, cached)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    {
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    }
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * @codeCoverageIgnore
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     *
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    /**
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * @param mixed $forModuleId
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $result = \App\Field::getFieldsFromRelation('');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertIsArray($fieldInfoByName, 'Expected field info array');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    /**
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        return [
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * @param mixed $moduleName
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    }
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     *
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    public function testGetFieldInfo()
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertIsArray($fieldInfo, 'Expected field info array');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertFalse(\App\Field::getColumnPermission($moduleId, $fieldRoColumn, false), 'Expected no write perms(ids)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * Testing getRelatedFieldForModule function with params from vtiger_relatedlist table.
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    public function testGetRelatedFieldForModuleAll()
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * @dataProvider relationsProvider
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

Line exceeds 120 characters; contains 157 characters
Open

        $fieldRoId = (new \App\Db\Query())->select(['fieldid'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'smcreatorid'])->scalar();
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertTrue(\App\Field::getFieldPermission('Leads', 'email', true), 'Expected read perms(strings)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertFalse(\App\Field::getColumnPermission($moduleId, $fieldRoColumn, false), 'Expected no write perms(ids, cached)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

Line exceeds 120 characters; contains 134 characters
Open

        $this->assertFalse(\App\Field::getColumnPermission($moduleId, $fieldRoColumn, false), 'Expected no write perms(ids, cached)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

Line exceeds 120 characters; contains 124 characters
Open

        return (new \App\Db\Query())->select(['tabid', 'related_tabid', 'relation_id'])->from('vtiger_relatedlists')->all();
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    }
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     */
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    {
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     *
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    public function testGetFieldsFromRelation($forModuleId, $moduleId, $relationId)
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertSame($moduleId, $fieldInfo['tabid'], 'Expected moduleid in fieldinfo same as reference');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $fieldRoColumn = (new \App\Db\Query())->select(['columnname'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'smcreatorid'])->scalar();
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

Line exceeds 120 characters; contains 126 characters
Open

        $this->assertFalse(\App\Field::getColumnPermission($moduleId, $fieldRoColumn, false), 'Expected no write perms(ids)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertFalse(\App\Field::getColumnPermission($moduleId, 'NxColumn', true), 'Expected no read perms(field not exists)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * @param mixed $forModuleId
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

Line exceeds 120 characters; contains 137 characters
Open

        $result0 = \App\Field::getRelatedFieldForModule(\App\Module::getModuleName($moduleId), \App\Module::getModuleName($forModuleId));
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertNotEmpty($result, 'Relation data should be not empty');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     *
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    {
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * Testing getFieldsFromRelation for empty relation id.
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertTrue(\App\Field::getColumnPermission($moduleId, $fieldColumn, true), 'Expected read perms(ids)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     *
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     *
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    {
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    }
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    /**
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     */
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    public function testGetFieldsFromEmptyRelation()
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    {
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * Relations modules ids provider.
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    /**
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertIsArray($result0, 'Relation list should be array type');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $result1 = \App\Field::getRelatedFieldForModule(false, \App\Module::getModuleName($forModuleId));
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertIsArray($result2, 'Relation list should be array type');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * @return array
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * @dataProvider relationSelectedModulesProvider
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    {
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertEmpty($result, 'Fields array from empty relation should be empty');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * Testing getFieldInfo.
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

Line exceeds 120 characters; contains 164 characters
Open

        $fieldRoColumn = (new \App\Db\Query())->select(['columnname'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'smcreatorid'])->scalar();
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertTrue(\App\Field::getColumnPermission('Leads', $fieldColumn, true), 'Expected read perms(string)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        return (new \App\Db\Query())->select(['tabid', 'related_tabid', 'relation_id'])->from('vtiger_relatedlists')->all();
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $moduleId = \App\Module::getModuleId('Leads');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     *
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertFalse(\App\Field::getColumnPermission($moduleId, 'NxColumn', false), 'Expected write perms(field not exists)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    }
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $result0 = \App\Field::getRelatedFieldForModule(\App\Module::getModuleName($moduleId), \App\Module::getModuleName($forModuleId));
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $result2 = \App\Field::getRelatedFieldForModule(\App\Module::getModuleName($moduleId), false);
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * @param mixed $relationId
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    public function relationSelectedModulesProvider()
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

            ['FInvoice', 'FCorectingInvoice'],
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

            ['DataSetRegister', 'AuditRegister'],
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertSame(\App\Module::getModuleId($moduleName), $result['tabid'], 'Expected tabid differs from reference');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * Testing getFieldsFromRelation.
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    /**
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertNotEmpty($fieldInfo, 'Field info should be not empty');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

Line exceeds 120 characters; contains 149 characters
Open

        $fieldId = (new \App\Db\Query())->select(['fieldid'])->from('vtiger_field')->where(['tabid' => $moduleId, 'fieldname' => 'email'])->scalar();
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertSame($fieldInfo, \App\Field::getFieldInfo($fieldId), 'Field info from cache should be same as reference');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $fieldInfo = \App\Field::getFieldInfo($fieldId);
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertSame($fieldId, $fieldInfoByName['fieldid'], 'Expected fieldid in fieldinfo same as reference');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    }
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     */
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertTrue(\App\Field::getColumnPermission($moduleId, $fieldColumn, false), 'Expected write perms(ids)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    public function relationsProvider()
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * @dataProvider relationsProvider
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     *
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

            ['LocationRegister', 'IncidentRegister'],
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

            ['LocationRegister', 'AuditRegister'],
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    public function testGetRelatedFieldForSpecificModulePairs($forModuleName, $moduleName)
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     */
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertSame($fieldId, $fieldInfo['fieldid'], 'Expected fieldid in fieldinfo same as reference');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

Line exceeds 120 characters; contains 132 characters
Open

        $this->assertFalse(\App\Field::getColumnPermission($moduleId, 'NxColumn', false), 'Expected write perms(field not exists)');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * @return array
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     * Testing getRelatedFieldForModule function with params from vtiger_relatedlist table.
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

     */
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

    {
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertIsArray($result, 'Expected result type array for relation: ' . $relationId);
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

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

        $this->assertIsArray($result, 'Expected result type array for empty relation');
Severity: Minor
Found in tests/App/Field.php by phpcodesniffer

There are no issues that match your filters.

Category
Status