flipbox\organizations\actions\organizations\UpdateOrganization

Complexity

Total Complexity

5

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	run()	0	4	1
A	find()	0	8	3
A	populate()	0	7	1

<?php

/**
 * @copyright  Copyright (c) Flipbox Digital Limited
 * @license    https://flipboxfactory.com/software/organization/license
 * @link       https://www.flipboxfactory.com/software/organization/
 */

namespace flipbox\organizations\actions\organizations;

use flipbox\craft\ember\actions\elements\UpdateElement;
use flipbox\organizations\elements\Organization;
use yii\base\BaseObject;

/**
 * @author Flipbox Factory <hello@flipboxfactory.com>
 * @since 1.0.0
 */
class UpdateOrganization extends UpdateElement
{
    use PopulateOrganizationTrait;

    /**
     * @inheritdoc
     */
    public function run($organization)
    {
        return parent::run($organization);
    }

    /**
     * @inheritdoc
     * @return Organization
     */
    public function find($identifier)
    {
        $site = $this->resolveSiteFromRequest();
        return Organization::findOne([

The expression \flipbox\organizations\elements\Organization::findOne(array(is_numeric($identifier) ? 'id' : 'slug' => $identifier, 'siteId' => $site ? $site->id : null)); of type craft\base\Element|null|craft\base\Element[] adds the type craft\base\Element[] to the return on line 38 which is incompatible with the return type documented by flipbox\organizations\actions\organizations\UpdateOrganization::find of type flipbox\organizations\elements\Organization|null.

            is_numeric($identifier) ? 'id' : 'slug' => $identifier,
            'siteId' => $site ? $site->id : null
        ]);
    }

    /**
     * @inheritdoc
     * @param Organization $object
     * @return Organization
     * @throws \flipbox\craft\ember\exceptions\RecordNotFoundException
     */
    protected function populate(BaseObject $object): BaseObject
    {
        parent::populate($object);
        $this->populateFromRequest($object);

        return $object;
    }
}