Function parsePhone has a Cognitive Complexity of 32 (exceeds 5 allowed). Consider refactoring.

Open

    private function parsePhone(): void
    {
        $country = null;
        if ($userId = $this->user ? $this->user->getId() : null) {
            $country = \App\Fields\Country::getCountryCode(\App\User::getUserModel($userId)->getDetail('sync_carddav_default_country'));

• Read up
  Read up

The class Card has an overall complexity of 75 which is very high. The configured complexity threshold is 50.

Open

class Card
{
    /** @var array Address mapping for modules. */
    const ADDRESS_MAPPING = [
        'Contacts' => [

• Exclude checks
  • Disable engine
  • Disable check

Function setValuesForRecord has a Cognitive Complexity of 23 (exceeds 5 allowed). Consider refactoring.

Open

    public function setValuesForRecord(\Vtiger_Record_Model $record): void
    {
        $this->record = $record;
        if (isset($this->card->N)) {
            $head = $this->card->N->getParts();

• Read up
  Read up

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

Open

    private function parseMail(): void
    {
        $moduleName = $this->record->getModuleName();
        $usedTypes = [];
        foreach (self::MAIL_FIELDS[$moduleName] as $key => $type) {

• Read up
  Read up

File Card.php has 274 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php
/**
 * CardDav address books file.
 *
 * @see https://en.wikipedia.org/wiki/VCard#Properties

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

Open

    public function setRecordAddress(string $moduleName, \Vtiger_Record_Model $record): void
    {
        foreach ($this->card->ADR as $property) {
            if ($typeOfAddress = $this->getTypeOfAddress($property)) {
                $address = $this->convertAddress($property->getParts());

• Read up
  Read up

Method setValuesForRecord has 42 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function setValuesForRecord(\Vtiger_Record_Model $record): void
    {
        $this->record = $record;
        if (isset($this->card->N)) {
            $head = $this->card->N->getParts();

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

Open

    private function parsePhone(): void
    {
        $country = null;
        if ($userId = $this->user ? $this->user->getId() : null) {
            $country = \App\Fields\Country::getCountryCode(\App\User::getUserModel($userId)->getDetail('sync_carddav_default_country'));

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

Open

    public function setValuesForCreateRecord(\Vtiger_Record_Model $record): void
    {
        if ('Contacts' === $record->getModuleName() && isset($this->card->ORG)) {
            $lead = \Vtiger_Record_Model::getCleanInstance('Leads');
            $lead->set('assigned_user_id', $this->user->get('id'));

• Read up
  Read up

The method setValuesForRecord() has an NPath complexity of 6336. The configured NPath complexity threshold is 200.

Open

    public function setValuesForRecord(\Vtiger_Record_Model $record): void
    {
        $this->record = $record;
        if (isset($this->card->N)) {
            $head = $this->card->N->getParts();

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

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

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

Open

    public function setValuesForRecord(\Vtiger_Record_Model $record): void
    {
        $this->record = $record;
        if (isset($this->card->N)) {
            $head = $this->card->N->getParts();

• 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 parsePhone() has a Cyclomatic Complexity of 14. The configured cyclomatic complexity threshold is 10.

Open

    private function parsePhone(): void
    {
        $country = null;
        if ($userId = $this->user ? $this->user->getId() : null) {
            $country = \App\Fields\Country::getCountryCode(\App\User::getUserModel($userId)->getDetail('sync_carddav_default_country'));

• 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 31 to the 15 allowed.

Open

    public function setValuesForRecord(\Vtiger_Record_Model $record): void

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

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

Open

    private function parseMail(): void

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

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

Open

    private function parsePhone(): void

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

Missing class import via use statement (line '120', column '22').

Open

        $dataReader = (new \App\Db\Query())->select(['addressbookid'])->from('dav_cards')->where(['crmid' => $id])->createCommand()->query();

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

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

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

Open

        return (new \App\Db\Query())->from('dav_addressbooks')->where(['id' => $id])->one();

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

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

Avoid using static access to class '\App\Purifier' in method 'setValuesForRecord'.

Open

            $record->set('description', $fieldModel->getDBValue(\App\Purifier::purify((string) $this->card->NOTE)));

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

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

Avoid using static access to class '\App\Purifier' in method 'parsePhone'.

Open

                        $orgPhone = \App\Purifier::purify($t->getValue());

• 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 '198', column '28').

Open

    public function setValuesForRecord(\Vtiger_Record_Model $record): void
    {
        $this->record = $record;
        if (isset($this->card->N)) {
            $head = $this->card->N->getParts();

• 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\Purifier' in method 'convertAddress'.

Open

            'city' => \App\Purifier::purify($addressFromVCard[3]),

• 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\Country' in method 'convertAddress'.

Open

            'country' => \App\Fields\Country::findCountryName(\App\Purifier::purify($addressFromVCard[6])),

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

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

Avoid using static access to class '\App\Purifier' in method 'convertAddress'.

Open

            'postCode' => \App\Purifier::purify($addressFromVCard[5]),

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

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

Avoid using static access to class '\App\Purifier' in method 'convertAddress'.

Open

            'postOfficeBox' => \App\Purifier::purify($addressFromVCard[0]),

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

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

Avoid using static access to class '\App\Purifier' in method 'setValuesForRecord'.

Open

                $record->set('firstname', $fieldModel->getDBValue(\App\Purifier::purify($head[1])));

• 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 '351', column '8').

Open

    public function setRecordAddress(string $moduleName, \Vtiger_Record_Model $record): void
    {
        foreach ($this->card->ADR as $property) {
            if ($typeOfAddress = $this->getTypeOfAddress($property)) {
                $address = $this->convertAddress($property->getParts());

• 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\Purifier' in method 'setValuesForRecord'.

Open

                $record->set('name', $fieldModel->getDBValue(\App\Purifier::purify($head[1])));

• 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 '36').

Open

    public function setValuesForRecord(\Vtiger_Record_Model $record): void
    {
        $this->record = $record;
        if (isset($this->card->N)) {
            $head = $this->card->N->getParts();

• 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 assigning values to variables in if clauses and the like (line '358', column '10').

Open

    public function setRecordAddress(string $moduleName, \Vtiger_Record_Model $record): void
    {
        foreach ($this->card->ADR as $property) {
            if ($typeOfAddress = $this->getTypeOfAddress($property)) {
                $address = $this->convertAddress($property->getParts());

• 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\Purifier' in method 'setValuesForRecord'.

Open

                $record->set('lastname', $fieldModel->getDBValue(\App\Purifier::purify($head[0])));

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

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

Avoid using static access to class '\App\Purifier' in method 'setValuesForRecord'.

Open

                $record->set('jobtitle', $fieldModel->getDBValue(\App\Purifier::purify((string) $this->card->TITLE)));

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

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

Avoid using static access to class '\App\Purifier' in method 'convertAddress'.

Open

            'street' => \App\Purifier::purify(trim($addressFromVCard[2])),

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

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

Avoid using static access to class '\App\Purifier' in method 'setValuesForRecord'.

Open

                $record->set('last_name', $fieldModel->getDBValue(\App\Purifier::purify($head[0])));

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

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

Avoid using static access to class '\App\Purifier' in method 'convertAddress'.

Open

            'localNumber' => \App\Purifier::purify($addressFromVCard[1]),

• 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 '201', column '28').

Open

    public function setValuesForRecord(\Vtiger_Record_Model $record): void
    {
        $this->record = $record;
        if (isset($this->card->N)) {
            $head = $this->card->N->getParts();

• 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 assigning values to variables in if clauses and the like (line '194', column '39').

Open

    public function setValuesForRecord(\Vtiger_Record_Model $record): void
    {
        $this->record = $record;
        if (isset($this->card->N)) {
            $head = $this->card->N->getParts();

• 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 assigning values to variables in if clauses and the like (line '231', column '8').

Open

    public function setValuesForCreateRecord(\Vtiger_Record_Model $record): void
    {
        if ('Contacts' === $record->getModuleName() && isset($this->card->ORG)) {
            $lead = \Vtiger_Record_Model::getCleanInstance('Leads');
            $lead->set('assigned_user_id', $this->user->get('id'));

• 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 assigning values to variables in if clauses and the like (line '258', column '36').

Open

    private function parsePhone(): void
    {
        $country = null;
        if ($userId = $this->user ? $this->user->getId() : null) {
            $country = \App\Fields\Country::getCountryCode(\App\User::getUserModel($userId)->getDetail('sync_carddav_default_country'));

• 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\Fields\Country' in method 'parsePhone'.

Open

            $country = \App\Fields\Country::getCountryCode(\App\User::getUserModel($userId)->getDetail('sync_carddav_default_country'));

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

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

Avoid using static access to class '\App\Purifier' in method 'parseMail'.

Open

                        $this->record->set($key, $fieldModel->getDBValue(\App\Purifier::purify($e->getValue())));

• 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 '185', column '28').

Open

    public function setValuesForRecord(\Vtiger_Record_Model $record): void
    {
        $this->record = $record;
        if (isset($this->card->N)) {
            $head = $this->card->N->getParts();

• 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 assigning values to variables in if clauses and the like (line '188', column '38').

Open

    public function setValuesForRecord(\Vtiger_Record_Model $record): void
    {
        $this->record = $record;
        if (isset($this->card->N)) {
            $head = $this->card->N->getParts();

• 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 assigning values to variables in if clauses and the like (line '252', column '7').

Open

    private function parsePhone(): void
    {
        $country = null;
        if ($userId = $this->user ? $this->user->getId() : null) {
            $country = \App\Fields\Country::getCountryCode(\App\User::getUserModel($userId)->getDetail('sync_carddav_default_country'));

• 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\Purifier' in method 'convertAddress'.

Open

            'country' => \App\Fields\Country::findCountryName(\App\Purifier::purify($addressFromVCard[6])),

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

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

Avoid using static access to class '\Sabre\VObject\Reader' in method 'loadFromContent'.

Open

        $instance->card = \Sabre\VObject\Reader::read($content);

• 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 '297', column '38').

Open

    private function parseMail(): void
    {
        $moduleName = $this->record->getModuleName();
        $usedTypes = [];
        foreach (self::MAIL_FIELDS[$moduleName] as $key => $type) {

• 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\Purifier' in method 'convertAddress'.

Open

            'state' => \App\Purifier::purify($addressFromVCard[4]),

• 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 '182', column '28').

Open

    public function setValuesForRecord(\Vtiger_Record_Model $record): void
    {
        $this->record = $record;
        if (isset($this->card->N)) {
            $head = $this->card->N->getParts();

• 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_Record_Model' in method 'setValuesForCreateRecord'.

Open

            $lead = \Vtiger_Record_Model::getCleanInstance('Leads');

• 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 "synctoken" 3 times.

Open

        $syncToken = (int) static::getAddressBook($addressBookId)['synctoken'];

• 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 "Contacts" 5 times.

Open

        'Contacts' => [

• 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 "country" 6 times.

Open

                'addresslevel1a' => ['country'],

• 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 "state" 6 times.

Open

                'addresslevel2a' => ['state'],

• 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 "postCode" 5 times.

Open

                'addresslevel7a' => ['postCode'],

• 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 "localNumber" 5 times.

Open

                'localnumbera' => ['localNumber'],

• 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 "OSSEmployees" 4 times.

Open

        'OSSEmployees' => [

• 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 "street" 6 times.

Open

                'addresslevel8a' => ['street'],

• 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);
}

Reference to instance property GENDER from undeclared class \Sabre\VObject\Component\VCard

Open

            if (isset($this->card->GENDER) && ($fieldModel = $record->getField('salutationtype'))) {

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (value) is string but \Vtiger_Field_Model::getDBValue() takes \type defined at /code/modules/Vtiger/models/Field.php:1357

Open

                $record->set('salutationtype', $fieldModel->getDBValue($this->getCardGender((string) $this->card->GENDER)));

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property ADR from undeclared class \Sabre\VObject\Component\VCard

Open

        if (isset($this->card->ADR)) {

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (value) is string but \Vtiger_Field_Model::getDBValue() takes \type defined at /code/modules/Vtiger/models/Field.php:1357

Open

                $record->set('firstname', $fieldModel->getDBValue(\App\Purifier::purify($head[1])));

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property GENDER from undeclared class \Sabre\VObject\Component\VCard

Open

                $record->set('salutationtype', $fieldModel->getDBValue($this->getCardGender((string) $this->card->GENDER)));

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property NOTE from undeclared class \Sabre\VObject\Component\VCard

Open

        if (isset($this->card->NOTE) && ($fieldModel = $record->getField('description'))) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property NOTE from undeclared class \Sabre\VObject\Component\VCard

Open

            $record->set('description', $fieldModel->getDBValue(\App\Purifier::purify((string) $this->card->NOTE)));

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        return (new \App\Db\Query())->from('dav_addressbooks')->where(['id' => $id])->one();

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property BDAY from undeclared class \Sabre\VObject\Component\VCard

Open

            if (isset($this->card->BDAY) && $record->getField('birth_date')) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property ADR from undeclared class \Sabre\VObject\Component\VCard

Open

        foreach ($this->card->ADR as $property) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property FN from undeclared class \Sabre\VObject\Component\VCard

Open

            $head = $this->card->FN->getParts();

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property TEL from undeclared class \Sabre\VObject\Component\VCard

Open

            if (isset($this->card->TEL) && ($fieldModel = $this->record->getField($key))) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $dataReader = (new \App\Db\Query())->select(['addressbookid'])->from('dav_cards')->where(['crmid' => $id])->createCommand()->query();

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property N from undeclared class \Sabre\VObject\Component\VCard

Open

            $head = $this->card->N->getParts();

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property FN from undeclared class \Sabre\VObject\Component\VCard

Open

        } elseif (isset($this->card->FN)) {

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (value) is string but \Vtiger_Field_Model::getDBValue() takes \type defined at /code/modules/Vtiger/models/Field.php:1357

Open

                $record->set('lastname', $fieldModel->getDBValue(\App\Purifier::purify($head[0])));

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \App\Integrations\Dav\Card->records (Did you mean expr->record)

Open

            $instance->records[$uid] = $recordModel;

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property TITLE from undeclared class \Sabre\VObject\Component\VCard

Open

                $record->set('jobtitle', $fieldModel->getDBValue(\App\Purifier::purify((string) $this->card->TITLE)));

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property BDAY from undeclared class \Sabre\VObject\Component\VCard

Open

                $record->set('birth_date', date('Y-m-d', strtotime($this->card->BDAY)));

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (value) is string but \Vtiger_Field_Model::getDBValue() takes \type defined at /code/modules/Vtiger/models/Field.php:1357

Open

                $lead->set('company', $fieldModel->getDBValue((string) $this->card->ORG));

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property parameters from undeclared class \Sabre\VObject\Property

Open

        foreach ($property->parameters as $parameter) {

• Exclude checks
  • Disable engine
  • Disable check

Property \App\Integrations\Dav\Card->card has undeclared type \Sabre\VObject\Component\VCard

Open

    private $card;

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::createCommand

Open

        $dbCommand = \App\Db::getInstance()->createCommand();

• Exclude checks
  • Disable engine
  • Disable check

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

Open

            $country = \App\Fields\Country::getCountryCode(\App\User::getUserModel($userId)->getDetail('sync_carddav_default_country'));

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (value) is string but \Vtiger_Field_Model::getDBValue() takes \type defined at /code/modules/Vtiger/models/Field.php:1357

Open

                        $this->record->set($key, $fieldModel->getDBValue($orgPhone));

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (value) is string but \Vtiger_Field_Model::getDBValue() takes \type defined at /code/modules/Vtiger/models/Field.php:1357

Open

                        $this->record->set($key, $fieldModel->getDBValue(\App\Purifier::purify($e->getValue())));

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (value) is string but \Vtiger_Field_Model::getDBValue() takes \type defined at /code/modules/Vtiger/models/Field.php:1357

Open

                $record->set('last_name', $fieldModel->getDBValue(\App\Purifier::purify($head[0])));

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property N from undeclared class \Sabre\VObject\Component\VCard

Open

        if (isset($this->card->N)) {

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (value) is string but \Vtiger_Field_Model::getDBValue() takes \type defined at /code/modules/Vtiger/models/Field.php:1357

Open

                $record->set('jobtitle', $fieldModel->getDBValue(\App\Purifier::purify((string) $this->card->TITLE)));

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property ORG from undeclared class \Sabre\VObject\Component\VCard

Open

        if ('Contacts' === $record->getModuleName() && isset($this->card->ORG)) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property EMAIL from undeclared class \Sabre\VObject\Component\VCard

Open

            if (isset($this->card->EMAIL) && ($fieldModel = $this->record->getField($key))) {

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (value) is string but \Vtiger_Field_Model::getDBValue() takes \type defined at /code/modules/Vtiger/models/Field.php:1357

Open

                $record->set('name', $fieldModel->getDBValue(\App\Purifier::purify($head[1])));

• Exclude checks
  • Disable engine
  • Disable check

Doc-block of $property in getTypeOfAddress contains phpdoc param type mixed which is incompatible with the param type \Sabre\VObject\Property declared in the signature

Open

     * @param mixed $property

• Exclude checks
  • Disable engine
  • Disable check

Return type of getVCard() is undeclared type \Sabre\VObject\Component\VCard

Open

    public function getVCard(): \Sabre\VObject\Component\VCard

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \Vtiger_Base_UIType::getPhoneDetails

Open

                            $details = $fieldModel->getUITypeModel()->getPhoneDetails($orgPhone, $country);

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property TITLE from undeclared class \Sabre\VObject\Component\VCard

Open

            if (isset($this->card->TITLE) && ($fieldModel = $record->getField('jobtitle'))) {

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (value) is string but \Vtiger_Field_Model::getDBValue() takes \type defined at /code/modules/Vtiger/models/Field.php:1357

Open

                        $record->set($fieldName, $fieldModel->getDBValue(implode(' ', $fieldsForJoin)));

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property BDAY from undeclared class \Sabre\VObject\Component\VCard

Open

                $record->set('birthday', date('Y-m-d', strtotime($this->card->BDAY)));

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (value) is string but \Vtiger_Field_Model::getDBValue() takes \type defined at /code/modules/Vtiger/models/Field.php:1357

Open

            $record->set('description', $fieldModel->getDBValue(\App\Purifier::purify((string) $this->card->NOTE)));

• Exclude checks
  • Disable engine
  • Disable check

Parameter $property has undeclared type \Sabre\VObject\Property

Open

    private function getTypeOfAddress(\Sabre\VObject\Property $property): ?string

• Exclude checks
  • Disable engine
  • Disable check

Call to method read from undeclared class \Sabre\VObject\Reader

Open

        $instance->card = \Sabre\VObject\Reader::read($content);

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::createCommand

Open

        $dbCommand = \App\Db::getInstance()->createCommand();

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property ORG from undeclared class \Sabre\VObject\Component\VCard

Open

                $lead->set('company', $fieldModel->getDBValue((string) $this->card->ORG));

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property EMAIL from undeclared class \Sabre\VObject\Component\VCard

Open

                foreach ($this->card->EMAIL as $e) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property BDAY from undeclared class \Sabre\VObject\Component\VCard

Open

            if (isset($this->card->BDAY) && 8 === \strlen($this->card->BDAY) && $record->getField('birthday')) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property TEL from undeclared class \Sabre\VObject\Component\VCard

Open

                foreach ($this->card->TEL as $t) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    public static function deleteByCrmId(int $id): void
    {
        $dbCommand = \App\Db::getInstance()->createCommand();
        $dataReader = (new \App\Db\Query())->select(['addressbookid'])->from('dav_cards')->where(['crmid' => $id])->createCommand()->query();
        $dbCommand->delete('dav_cards', ['crmid' => $id])->execute();

• Read up
  Read up

Avoid variables with short names like $id. Configured minimum length is 3.

Open

    public static function deleteByCrmId(int $id): void

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

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

Avoid variables with short names like $id. Configured minimum length is 3.

Open

    public static function getAddressBook(int $id)

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

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

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

Open

    /** @var array Address mapping for modules. */

• Exclude checks
  • Disable engine
  • Disable check

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

Open

            'HOME' => [

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                'ship_code' => ['postCode'],

• 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

                'addresslevel2a' => ['state'],

• 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

                'addresslevel1b' => ['country'],

• 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

            'WORK' => [

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                'code' => ['postCode'],

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                'city' => ['city'],

• Exclude checks
  • Disable engine
  • Disable check

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

Open

            'WORK' => [

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                'localnumbera' => ['localNumber'],

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                'localnumberb' => ['localNumber'],

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                'addresslevel2b' => ['state'],

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                'addresslevel8b' => ['street'],

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        'Contacts' => [

• 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

                'state' => ['state'],

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                'street' => ['street', 'localNumber'],

• 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

            'HOME' => [

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                'addresslevel8a' => ['street'],

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                'ship_state' => ['state'],

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                'ship_city' => ['city'],

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                'addresslevel7a' => ['postCode'],

• 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

                'ship_street' => ['street', 'localNumber'],

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    const ADDRESS_MAPPING = [

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                'addresslevel1a' => ['country'],

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                'addresslevel7b' => ['postCode'],

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                'addresslevel5b' => ['city'],

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        'OSSEmployees' => [

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                'ship_country' => ['country'],

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                'addresslevel5a' => ['city'],

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                'country' => ['country'],

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        'Contacts' => ['email' => 'WORK', 'secondary_email' => 'HOME'],

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    const PHONE_FIELDS = [

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    /** @var \Users_Record_Model User record model. */

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    public $user;

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @param string|null               $uid

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $instance->card = \Sabre\VObject\Reader::read($content);

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * Get card instance.

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @return \Sabre\VObject\Component\VCard

• 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

        $dataReader = (new \App\Db\Query())->select(['addressbookid'])->from('dav_cards')->where(['crmid' => $id])->createCommand()->query();

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $dbCommand->delete('dav_cards', ['crmid' => $id])->execute();

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $dataReader->close();

• Exclude checks
  • Disable engine
  • Disable check

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

Open

            'operation' => $operation,

• 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

        $this->record = $record;

• Exclude checks
  • Disable engine
  • Disable check

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

Open

            $head = $this->card->FN->getParts();

• 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

        $moduleName = $record->getModuleName();

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                $record->set('lastname', $fieldModel->getDBValue(\App\Purifier::purify($head[0])));

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                $record->set('jobtitle', $fieldModel->getDBValue(\App\Purifier::purify((string) $this->card->TITLE)));

• 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->parsePhone();

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    private $card;

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * Load from content.

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    public static function loadFromContent(string $content, ?\Vtiger_Record_Model $recordModel = null, ?string $uid = null): self

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        if ($recordModel && $uid) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        return $this->card;

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @return void

• 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

            $head = $this->card->N->getParts();

• Exclude checks
  • Disable engine
  • Disable check

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

Open

            if (isset($this->card->GENDER) && ($fieldModel = $record->getField('salutationtype'))) {

• 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 (isset($this->card->ADR)) {

• 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 self

• 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

            ->execute();

• 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 (isset($head[0]) && ($fieldModel = $record->getField('last_name'))) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @param string                    $content

• 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 ('Contacts' === $moduleName) {

• 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

     * @return void

• 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->parseMail();

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        'OSSEmployees' => ['business_mail' => 'WORK', 'private_mail' => 'HOME'],

• 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

            'synctoken' => $syncToken,

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                $record->set('firstname', $fieldModel->getDBValue(\App\Purifier::purify($head[1])));

• 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

        ], ['id' => $addressBookId])

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @return array

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        if (isset($this->card->N)) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                $record->set('salutationtype', $fieldModel->getDBValue($this->getCardGender((string) $this->card->GENDER)));

• 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

        $dbCommand = \App\Db::getInstance()->createCommand();

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $syncToken = (int) static::getAddressBook($addressBookId)['synctoken'];

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $dbCommand->update('dav_addressbooks', [

• 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

                $record->set('name', $fieldModel->getDBValue(\App\Purifier::purify($head[1])));

• 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->setRecordAddress($moduleName, $record);

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @param int $id

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $dbCommand = \App\Db::getInstance()->createCommand();

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        while ($addressBookId = $dataReader->readColumn(0)) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

            static::addChange($addressBookId, $id . '.vcf', 3);

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @param int    $operation

• 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

            'synctoken' => $syncToken + 1,

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @param int $id

• Exclude checks
  • Disable engine
  • Disable check

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

Open

            if (isset($head[1]) && ($fieldModel = $record->getField('firstname'))) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                $record->set('birth_date', date('Y-m-d', strtotime($this->card->BDAY)));

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        'OSSEmployees' => ['business_phone' => 'WORK', 'private_phone' => 'CELL', 'secondary_phone' => 'OTHER'],

• Exclude checks
  • Disable engine
  • Disable check

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

Open

            $instance->records[$uid] = $recordModel;

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 141 characters

Open

        $dataReader = (new \App\Db\Query())->select(['addressbookid'])->from('dav_cards')->where(['crmid' => $id])->createCommand()->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

     *

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * Get address books.

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        return (new \App\Db\Query())->from('dav_addressbooks')->where(['id' => $id])->one();

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 124 characters

Open

                $record->set('salutationtype', $fieldModel->getDBValue($this->getCardGender((string) $this->card->GENDER)));

• 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 (isset($head[1]) && ($fieldModel = $record->getField('name'))) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        'Contacts' => ['phone' => 'WORK', 'mobile' => 'CELL'],

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    /** @var \Sabre\VObject\Component\VCard Card object. */

• 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

     * Delete card by crm id.

• 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

            'uri' => $uri,

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @param \Vtiger_Record_Model $record

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    public function setValuesForRecord(\Vtiger_Record_Model $record): void

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        if (isset($this->card->NOTE) && ($fieldModel = $record->getField('description'))) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    /** @var array Mail fields. */

• 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 string $uri

• Exclude checks
  • Disable engine
  • Disable check

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

Open

            'addressbookid' => $addressBookId,

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    ];

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 129 characters

Open

    public static function loadFromContent(string $content, ?\Vtiger_Record_Model $recordModel = null, ?string $uid = null): self

• 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 static function deleteByCrmId(int $id): void

• 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

                $record->set('birthday', date('Y-m-d', strtotime($this->card->BDAY)));

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                $record->set('last_name', $fieldModel->getDBValue(\App\Purifier::purify($head[0])));

• Exclude checks
  • Disable engine
  • Disable check

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

Open

            if (isset($this->card->BDAY) && $record->getField('birth_date')) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    /** @var array Phone fields. */

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    private $record;

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @param \Vtiger_Record_Model|null $recordModel

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $instance = new self();

• 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 static function addChange(int $addressBookId, string $uri, int $operation): void

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        ])->execute();

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * Set values for record.

• 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 (isset($this->card->BDAY) && 8 === \strlen($this->card->BDAY) && $record->getField('birthday')) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        } elseif ('OSSEmployees' === $moduleName) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    /** @var \Vtiger_Record_Model Record model. */

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @return void

• 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 int    $addressBookId

• 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 (isset($this->card->TITLE) && ($fieldModel = $record->getField('jobtitle'))) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

            $record->set('description', $fieldModel->getDBValue(\App\Purifier::purify((string) $this->card->NOTE)));

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                $record->set($fieldModel->getFieldName(), $lead->getId());

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $record->set('assigned_user_id', $this->user->get('id'));

• 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

                        $types[] = 'WORK';

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                        $usedTypes = array_merge($usedTypes, $types);

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                    if (\in_array($type, $types)) {

• 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 string $gender

• Exclude checks
  • Disable engine
  • Disable check

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

Open

            case 'M':

• Exclude checks
  • Disable engine
  • Disable check