Function davSync has a Cognitive Complexity of 24 (exceeds 5 allowed). Consider refactoring.

Open

    public function davSync(): int
    {
        $create = $updates = 0;
        foreach ($this->davUsers as $userId => $user) {
            $this->calendarId = $user->get('calendarsid');

• Read up
  Read up

File CalDAV.php has 267 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php

/**
 * Api CalDAV Model Class.
 *

Function recordSync has a Cognitive Complexity of 17 (exceeds 5 allowed). Consider refactoring.

Open

    public function recordSync(): int
    {
        \App\Log::trace('Start', __METHOD__);
        $query = (new \App\Db\Query())->select([
            'dav_calendarobjects.*',

• Read up
  Read up

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

Open

    public function recordSync(): int
    {
        \App\Log::trace('Start', __METHOD__);
        $query = (new \App\Db\Query())->select([
            'dav_calendarobjects.*',

Method davSync has 33 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function davSync(): int
    {
        $create = $updates = 0;
        foreach ($this->davUsers as $userId => $user) {
            $this->calendarId = $user->get('calendarsid');

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

Open

    public function recordCreate($cal)
    {
        \App\Log::trace(__METHOD__ . ' | Start Cal ID' . $cal['id']);
        $calendar = \App\Integrations\Dav\Calendar::loadFromContent($cal['calendardata']);
        foreach ($calendar->getRecordInstance() as $recordModel) {

• Read up
  Read up

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

Open

    public function recordUpdate(Vtiger_Record_Model $record, $cal)
    {
        \App\Log::trace(__METHOD__ . ' | Start Cal ID:' . $cal['crmid']);
        $calendar = \App\Integrations\Dav\Calendar::loadFromContent($cal['calendardata'], $record, $cal['uid']);
        foreach ($calendar->getRecordInstance() as $recordModel) {

• Read up
  Read up

Method recordCreate has 27 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function recordCreate($cal)
    {
        \App\Log::trace(__METHOD__ . ' | Start Cal ID' . $cal['id']);
        $calendar = \App\Integrations\Dav\Calendar::loadFromContent($cal['calendardata']);
        foreach ($calendar->getRecordInstance() as $recordModel) {

Method davCreate has 27 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function davCreate()
    {
        \App\Log::trace(__METHOD__ . ' | Start CRM ID:' . $this->record['id']);
        $instance = \App\Integrations\Dav\Calendar::createEmptyInstance();
        $instance->loadFromArray($this->record);

Method recordUpdate has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function recordUpdate(Vtiger_Record_Model $record, $cal)
    {
        \App\Log::trace(__METHOD__ . ' | Start Cal ID:' . $cal['crmid']);
        $calendar = \App\Integrations\Dav\Calendar::loadFromContent($cal['calendardata'], $record, $cal['uid']);
        foreach ($calendar->getRecordInstance() as $recordModel) {

Function toDelete has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    protected function toDelete($cal)
    {
        if ('' === $cal['assigned_user_id'] || 0 !== (int) $cal['deleted']) {
            return true;
        }

• Read up
  Read up

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

Open

    public function davSync(): int
    {
        $create = $updates = 0;
        foreach ($this->davUsers as $userId => $user) {
            $this->calendarId = $user->get('calendarsid');

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

Open

    protected function toDelete($cal)
    {
        if ('' === $cal['assigned_user_id'] || 0 !== (int) $cal['deleted']) {
            return true;
        }

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

Open

    public function davSync(): int

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

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

Open

    public function recordSync(): int

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

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

Open

        return (new \App\Db\Query())->from('dav_calendarobjects')->where(['calendarid' => $this->calendarId, 'crmid' => $this->record['id']])->one();

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

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

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

Open

        $query = (new \App\Db\Query())->select([

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

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

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

Open

        $queryGenerator = new \App\QueryGenerator('Calendar');

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

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

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

Open

        \App\Log::trace(__METHOD__ . ' | Start CRM ID:' . $this->record['id']);

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

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

Avoid using static access to class '\App\Integrations\Dav\Calendar' in method 'davCreate'.

Open

        \App\Integrations\Dav\Calendar::addChange($this->calendarId, $calUri, 1);

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

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

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

Open

        \App\Log::trace(__METHOD__ . ' | End');

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

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

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

Open

                    } else {
                        ++$skipped;
                    }

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

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

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

Open

                        \App\Log::info(__METHOD__ . ' | End exclusion');

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

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

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

Open

        \App\Log::trace(__METHOD__ . ' | End');

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

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

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

Open

                } else {
                    ++$skipped;
                }

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

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

Avoid using static access to class '\App\Fields\SharedOwner' in method 'toDelete'.

Open

                $isPermitted = (int) $cal['assigned_user_id'] === $userId || \in_array($userId, \App\Fields\SharedOwner::getById($cal['id']));

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

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

Avoid using static access to class '\App\Integrations\Dav\Calendar' in method 'recordUpdate'.

Open

        $calendar = \App\Integrations\Dav\Calendar::loadFromContent($cal['calendardata'], $record, $cal['uid']);

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

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

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

Open

        \App\Log::trace(__METHOD__ . ' | Start Cal ID:' . $cal['crmid']);

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

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

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

Open

        \App\Log::trace(__METHOD__ . ' | End');

• 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\SharedOwner' in method 'toDelete'.

Open

                $shownerIds = \App\Fields\SharedOwner::getById($cal['id']);

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

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

Avoid using static access to class '\App\Config' in method 'davSync'.

Open

                $exclusion = \App\Config::component('Dav', 'CALDAV_EXCLUSION_TO_DAV');

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

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

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

Open

        \App\Log::trace(__METHOD__ . ' | Start');

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

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

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

Open

                    App\User::setCurrentUserId($orgUserId);

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

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

Avoid using static access to class '\App\Integrations\Dav\Calendar' in method 'recordCreate'.

Open

        $calendar = \App\Integrations\Dav\Calendar::loadFromContent($cal['calendardata']);

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

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

Avoid using static access to class '\App\Integrations\Dav\Calendar' in method 'davUpdate'.

Open

        $instance = \App\Integrations\Dav\Calendar::loadFromDav($dav['calendardata']);

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

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

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

Open

        \App\Log::trace(__METHOD__ . ' | End');

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

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

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

Open

        \App\Log::trace(__METHOD__ . ' | Start Cal ID' . $cal['id']);

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

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

Avoid using static access to class '\App\Config' in method 'recordUpdate'.

Open

            $exclusion = \App\Config::component('Dav', 'CALDAV_EXCLUSION_FROM_DAV');

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

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

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

Open

                        \App\Log::info(__METHOD__ . ' | End exclusion');

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

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

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

Open

                    $orgUserId = App\User::getCurrentUserId();

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

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

Avoid using static access to class '\App\Integrations\Dav\Calendar' in method 'davCreate'.

Open

        $instance = \App\Integrations\Dav\Calendar::createEmptyInstance();

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

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

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

Open

            } else {
                if (strtotime($row['modifiedtime']) < $row['lastmodified']) { // Updating
                    if ($this->recordUpdate(Vtiger_Record_Model::getInstanceById($row['crmid'], $row['setype']), $row)) {
                        ++$updates;
                    } else {

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

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

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

Open

        \App\Log::trace("Calendar end - DAV >> CRM | create: $create | deletes: $deletes | updates: $updates | skipped: $skipped", __METHOD__);

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

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

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

Open

                    App\User::setCurrentUserId($userId);

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

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

Avoid using static access to class 'Vtiger_Record_Model' in method 'recordSync'.

Open

                    if ($this->recordUpdate(Vtiger_Record_Model::getInstanceById($row['crmid'], $row['setype']), $row)) {

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

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

Avoid using static access to class '\App\Config' in method 'recordCreate'.

Open

            $exclusion = \App\Config::component('Dav', 'CALDAV_EXCLUSION_FROM_DAV');

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

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

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

Open

        \App\Log::trace(__METHOD__ . ' | End');

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

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

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

Open

        \App\Log::trace("Calendar end - CRM >> DAV | create: $create | updates: $updates", __METHOD__);

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

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

Avoid using static access to class '\App\Integrations\Dav\Calendar' in method 'davUpdate'.

Open

        \App\Integrations\Dav\Calendar::addChange($this->calendarId, $dav['uri'], 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\Integrations\Dav\Calendar' in method 'recordSync'.

Open

                \App\Integrations\Dav\Calendar::delete($row);

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

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

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

Open

        \App\Log::trace(__METHOD__ . ' | Start');

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

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

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

Open

        \App\Log::trace(__METHOD__ . ' | End');

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

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

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

Open

        \App\Log::trace('Start', __METHOD__);

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

Open

        $queryGenerator->setCustomColumn(['vtiger_crmentity.deleted']);

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

Open

                    } elseif (strtotime($this->record['modifiedtime']) > $event['lastmodified']) { // Updating

• 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 "crmid" 11 times.

Open

            'crmid' => $this->record['id'],

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

Open

            'calendarid' => $this->calendarId,

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

Open

            ->leftJoin('vtiger_crmentity', 'vtiger_crmentity.crmid = dav_calendarobjects.crmid')

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

Open

        \App\Db::getInstance()->createCommand()->insert('dav_calendarobjects', [

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

Open

            'calendardata' => $calendarData,

• 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 "assigned_user_id" 8 times.

Open

        $queryGenerator->setFields(['id', 'subject', 'subject', 'activitytype', 'date_start', 'due_date', 'time_start', 'time_end', 'activitystatus', 'taskpriority', 'location', 'visibility', 'assigned_user_id', 'allday', 'state', 'createdtime', 'modifiedtime', 'description']);

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

Open

        \App\Log::trace(__METHOD__ . ' | End');

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

Open

        $queryGenerator->setFields(['id', 'subject', 'subject', 'activitytype', 'date_start', 'due_date', 'time_start', 'time_end', 'activitystatus', 'taskpriority', 'location', 'visibility', 'assigned_user_id', 'allday', 'state', 'createdtime', 'modifiedtime', 'description']);

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

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

Open

                    App\User::setCurrentUserId($orgUserId);

• Exclude checks
  • Disable engine
  • Disable check

Call to method serialize from undeclared class \Sabre\VObject\Component\VCalendar

Open

        $calendarData = $calendar->serialize();

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $query = $queryGenerator->createQuery();

• Exclude checks
  • Disable engine
  • Disable check

Call to method trace from undeclared class \App\Log

Open

        \App\Log::trace(__METHOD__ . ' | Start CRM ID:' . $this->record['id']);

• Exclude checks
  • Disable engine
  • Disable check

Call to method info from undeclared class \App\Log

Open

                        \App\Log::info(__METHOD__ . ' | End exclusion');

• Exclude checks
  • Disable engine
  • Disable check

Assigning false to property but \API_CalDAV_Model->calendarId is int

Open

    public $calendarId = false;

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $queryGenerator->setFields(['id', 'subject', 'subject', 'activitytype', 'date_start', 'due_date', 'time_start', 'time_end', 'activitystatus', 'taskpriority', 'location', 'visibility', 'assigned_user_id', 'allday', 'state', 'createdtime', 'modifiedtime', 'description']);

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property name from undeclared class \Sabre\VObject\Component

Open

        if ('VEVENT' === (string) $component->name) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method trace from undeclared class \App\Log

Open

        \App\Log::trace(__METHOD__ . ' | End');

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property name from undeclared class \Sabre\VObject\Component

Open

        if ('VEVENT' === (string) $component->name) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method trace from undeclared class \App\Log

Open

        \App\Log::trace(__METHOD__ . ' | End');

• Exclude checks
  • Disable engine
  • Disable check

Call to method trace from undeclared class \App\Log

Open

        \App\Log::trace(__METHOD__ . ' | End');

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                    App\User::setCurrentUserId($userId);

• Exclude checks
  • Disable engine
  • Disable check

Call to method trace from undeclared class \App\Log

Open

        \App\Log::trace(__METHOD__ . ' | Start Cal ID:' . $cal['crmid']);

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::createCommand

Open

        \App\Db::getInstance()->createCommand()->update('dav_calendarobjects', [

• Exclude checks
  • Disable engine
  • Disable check

Call to method trace from undeclared class \App\Log

Open

        \App\Log::trace(__METHOD__ . ' | End');

• Exclude checks
  • Disable engine
  • Disable check

Call to method info from undeclared class \App\Log

Open

                        \App\Log::info(__METHOD__ . ' | End exclusion');

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property permissions from undeclared class \App\QueryGenerator

Open

        $queryGenerator->permissions = false;

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property UID from undeclared class \Sabre\VObject\Component

Open

        $uid = (string) $component->UID;

• Exclude checks
  • Disable engine
  • Disable check

Call to method trace from undeclared class \App\Log

Open

        \App\Log::trace('Start', __METHOD__);

• Exclude checks
  • Disable engine
  • Disable check

Call to method trace from undeclared class \App\Log

Open

        \App\Log::trace(__METHOD__ . ' | Start Cal ID' . $cal['id']);

• Exclude checks
  • Disable engine
  • Disable check

Call to method trace from undeclared class \App\Log

Open

        \App\Log::trace(__METHOD__ . ' | End');

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $queryGenerator = new \App\QueryGenerator('Calendar');

• 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 undeclared method \App\Db::createCommand

Open

        App\Db::getInstance()->createCommand()->update('vtiger_activity', [

• Exclude checks
  • Disable engine
  • Disable check

Call to method trace from undeclared class \App\Log

Open

        \App\Log::trace(__METHOD__ . ' | Start');

• 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 undeclared method \App\Db\Query::from

Open

        return (new \App\Db\Query())->from('dav_calendarobjects')->where(['calendarid' => $this->calendarId, 'crmid' => $this->record['id']])->one();

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $queryGenerator->setCustomColumn(['vtiger_crmentity.deleted']);

• Exclude checks
  • Disable engine
  • Disable check

Call to method trace from undeclared class \App\Log

Open

        \App\Log::trace("Calendar end - CRM >> DAV | create: $create | updates: $updates", __METHOD__);

• Exclude checks
  • Disable engine
  • Disable check

Call to method serialize from undeclared class \Sabre\VObject\Component\VCalendar

Open

        $calendarData = $calendar->serialize();

• Exclude checks
  • Disable engine
  • Disable check

Call to method trace from undeclared class \App\Log

Open

        \App\Log::trace(__METHOD__ . ' | End');

• Exclude checks
  • Disable engine
  • Disable check

Call to method trace from undeclared class \App\Log

Open

        \App\Log::trace("Calendar end - DAV >> CRM | create: $create | deletes: $deletes | updates: $updates | skipped: $skipped", __METHOD__);

• Exclude checks
  • Disable engine
  • Disable check

Assigning false to property but \API_CalDAV_Model->record is array

Open

    public $record = false;

• Exclude checks
  • Disable engine
  • Disable check

Call to method trace from undeclared class \App\Log

Open

        \App\Log::trace(__METHOD__ . ' | Start');

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $query = (new \App\Db\Query())->select([

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                    $orgUserId = App\User::getCurrentUserId();

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::createCommand

Open

        \App\Db::getInstance()->createCommand()->insert('dav_calendarobjects', [

• Exclude checks
  • Disable engine
  • Disable check

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

Open

            case 'PLL_OWNER_PERSON_GROUP':
                $shownerIds = \App\Fields\SharedOwner::getById($cal['id']);
                $isPermitted = (int) $cal['assigned_user_id'] === $userId || \in_array($cal['assigned_user_id'], $this->user->get('groups')) || \in_array($userId, $shownerIds) || \count(array_intersect($shownerIds, $this->user->get('groups'))) > 0;
                break;

• Read up
  Read up

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

Open

class API_CalDAV_Model

• Exclude checks
  • Disable engine
  • Disable check

The class API_CalDAV_Model is not named in CamelCase.

Open

class API_CalDAV_Model
{
    /**
     * Prod id.
     *

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

class class_name {
}

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

Open

        $i = 0;

• 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 $i. Configured minimum length is 3.

Open

        $i = 0;

• 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 for alignment; tabs are not allowed

Open

            ->where(['calendarid' => $this->calendarId]);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                } else {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        \App\Log::trace("Calendar end - DAV >> CRM | create: $create | deletes: $deletes | updates: $updates | skipped: $skipped", __METHOD__);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * Record create.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $dbCommand->update('dav_calendarobjects', [

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $dbCommand->update('vtiger_crmentity', [

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $calendar->recordSaveAttendee($recordModel);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        return true;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $skipped = $create = $deletes = $updates = 0;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $dataReader->close();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                        \App\Log::info(__METHOD__ . ' | End exclusion');

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $recordModel->save();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            )->execute();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * Record update.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            ->leftJoin('vtiger_crmentity', 'vtiger_crmentity.crmid = dav_calendarobjects.crmid')

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                \App\Integrations\Dav\Calendar::delete($row);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            } else {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $dataReader = $query->createCommand()->query();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            if (!$row['crmid']) { //Creating

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    public function recordCreate($cal)

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            ], ['crmid' => $recordModel->getId()]

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            )->execute();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        \App\Log::trace(__METHOD__ . ' | Start Cal ID:' . $cal['crmid']);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @return bool

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $dbCommand->update('vtiger_crmentity', [

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                'crmid' => $recordModel->getId(),

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    ++$create;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        return $create + $updates + $deletes;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @param array               $cal

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                foreach ($exclusion as $key => $value) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $calendar = \App\Integrations\Dav\Calendar::loadFromContent($cal['calendardata']);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            'vtiger_crmentity.modifiedtime', 'vtiger_crmentity.setype',    'assigned_user_id' => 'vtiger_crmentity.smownerid',

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            ->leftJoin('vtiger_activity', 'vtiger_crmentity.crmid = vtiger_activity.activityid')

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                        \App\Log::info(__METHOD__ . ' | End exclusion');

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            )->execute();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        ])->from('dav_calendarobjects')

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    if ($this->recordUpdate(Vtiger_Record_Model::getInstanceById($row['crmid'], $row['setype']), $row)) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @param array $cal

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            if (\is_array($exclusion)) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                'modifiedtime' => date('Y-m-d H:i:s', $cal['lastmodified']),

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            } elseif ($this->toDelete(array_merge($row, ['id' => $row['crmid']]))) { // Deleting

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                ++$deletes;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    } else {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                        return false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $recordModel->save();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @return bool

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        foreach ($calendar->getRecordInstance() as $recordModel) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            ], ['id' => $cal['id']]

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        \App\Log::trace(__METHOD__ . ' | End');

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $calendar = \App\Integrations\Dav\Calendar::loadFromContent($cal['calendardata'], $record, $cal['uid']);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

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

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $dbCommand->update('dav_calendarobjects', [

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            ], ['id' => $cal['id']]

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                if ($this->recordCreate($row)) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                if (strtotime($row['modifiedtime']) < $row['lastmodified']) { // Updating

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

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

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $exclusion = \App\Config::component('Dav', 'CALDAV_EXCLUSION_FROM_DAV');

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                foreach ($exclusion as $key => $value) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

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

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                'crmid' => $recordModel->getId(),

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @param Vtiger_Record_Model $record

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        foreach ($calendar->getRecordInstance() as $recordModel) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $exclusion = \App\Config::component('Dav', 'CALDAV_EXCLUSION_FROM_DAV');

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                        return false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        \App\Log::trace(__METHOD__ . ' | Start Cal ID' . $cal['id']);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    if ($recordModel->get($key) == $value) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            'vtiger_crmentity.crmid', 'vtiger_crmentity.deleted', 'vtiger_activity.visibility'

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                        ++$updates;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    if ($recordModel->get($key) == $value) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                        ++$skipped;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            if (false !== $exclusion) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        while ($row = $dataReader->read()) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    ++$skipped;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    public function recordUpdate(Vtiger_Record_Model $record, $cal)

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    protected function recordMarkComplete()

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    protected function toDelete($cal)

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        if (!$isPermitted && 'Public' !== $cal['visibility']) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                'modifiedtime' => date('Y-m-d H:i:s', $cal['lastmodified']),

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            ], ['crmid' => $recordModel->getId()]

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    public function getDavDetail()

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            )->execute();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        return true;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @return array|bool

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            default:

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * Record mark complete.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $userId = (int) $this->user->getId();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            case 'PLL_OWNER_PERSON_GROUP':

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        App\Db::getInstance()->createCommand()->update('vtiger_activity', [

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                $isPermitted = (int) $cal['assigned_user_id'] === $userId || \in_array($userId, \App\Fields\SharedOwner::getById($cal['id']));

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * To delete.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @param array $cal

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                break;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            return true;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        if ('' === $cal['assigned_user_id'] || 0 !== (int) $cal['deleted']) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            return true;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            case 'PLL_OWNER':

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                $isPermitted = (int) $cal['assigned_user_id'] === $userId;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        \App\Log::trace(__METHOD__ . ' | End');

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        return (new \App\Db\Query())->from('dav_calendarobjects')->where(['calendarid' => $this->calendarId, 'crmid' => $this->record['id']])->one();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        ], ['activityid' => $this->record['id']]

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        switch ($this->user->get('sync_caldav')) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                $shownerIds = \App\Fields\SharedOwner::getById($cal['id']);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * Get dav detail.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @return bool

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                break;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                $isPermitted = (int) $cal['assigned_user_id'] === $userId || \in_array($cal['assigned_user_id'], $this->user->get('groups')) || \in_array($userId, $shownerIds) || \count(array_intersect($shownerIds, $this->user->get('groups'))) > 0;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        return false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $calendar->recordSaveAttendee($recordModel);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                break;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            'dav_status' => 0,

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        )->execute();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            case 'PLL_OWNER_PERSON':

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * Calendar name.

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @var string

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @var string

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    const PRODID = 'YetiForce';

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * Prod 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

    const CALENDAR_NAME = 'YFCalendar';

• 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 $record = false;

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $query = $queryGenerator->createQuery();

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $i = 0;

• Exclude checks
  • Disable engine
  • Disable check

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

Open

     * @return int

• Exclude checks
  • Disable engine
  • Disable check

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

Open

            'calendarid' => $this->calendarId,

• Exclude checks
  • Disable engine
  • Disable check

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

Open

            'crmid' => $this->record['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

    public $user = false;

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    public function davSync(): int

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $create = $updates = 0;

• Exclude checks
  • Disable engine
  • Disable check

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

Open

            $this->calendarId = $user->get('calendarsid');

• Exclude checks
  • Disable engine
  • Disable check

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

Open

                        // Creating