flipbox\organizations\events\handlers\ProjectConfigHandler

Complexity

Total Complexity

11

Size/Duplication

Total Lines	204
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	5

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	handleChangedOrganizationType()	0	51	2
A	handleDeletedOrganizationType()	0	31	2
A	handleChangedUserType()	0	51	2
A	handleDeletedUserType()	0	31	2
A	rebuild()	0	14	3

<?php

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

namespace flipbox\organizations\events\handlers;

use Craft;
use craft\events\ConfigEvent;
use flipbox\organizations\events\ManageOrganizationTypeProjectConfig;
use flipbox\organizations\events\ManageUserTypeProjectConfig;
use flipbox\organizations\records\OrganizationType;
use flipbox\organizations\records\UserType;
use yii\base\Event;

/**
 * @author Flipbox Factory <hello@flipboxfactory.com>
 * @since 3.0.0
 */
class ProjectConfigHandler
{
    /**
     * @param ConfigEvent $event
     */
    public static function handleChangedOrganizationType(ConfigEvent $event)
    {
        Event::off(
            OrganizationType::class,
            OrganizationType::EVENT_AFTER_INSERT,
            [
                ManageOrganizationTypeProjectConfig::class,
                'save'
            ]
        );

        Event::off(
            OrganizationType::class,
            OrganizationType::EVENT_AFTER_UPDATE,
            [
                ManageOrganizationTypeProjectConfig::class,
                'save'
            ]
        );

        // Get the UID that was matched in the config path
        $uid = $event->tokenMatches[0];

        if (null === ($provider = OrganizationType::findOne([
                'uid' => $uid
            ]))) {
            $provider = new OrganizationType();
        }

        Craft::configure($provider, $event->newValue);

It seems like $provider defined by \flipbox\organizations\records\OrganizationType::findOne(array('uid' => $uid)) on line 51 can also be of type array; however, yii\BaseYii::configure() does only seem to accept object, maybe add an additional type check?

        $provider->save();

        Event::on(
            OrganizationType::class,
            OrganizationType::EVENT_AFTER_INSERT,
            [
                ManageOrganizationTypeProjectConfig::class,
                'save'
            ]
        );

        Event::on(
            OrganizationType::class,
            OrganizationType::EVENT_AFTER_UPDATE,
            [
                ManageOrganizationTypeProjectConfig::class,
                'save'
            ]
        );
    }

    /**
     * @param ConfigEvent $event
     * @throws \Throwable
     * @throws \yii\db\StaleObjectException
     */
    public static function handleDeletedOrganizationType(ConfigEvent $event)
    {
        Event::off(
            OrganizationType::class,
            OrganizationType::EVENT_AFTER_DELETE,
            [
                ManageOrganizationTypeProjectConfig::class,
                'delete'
            ]
        );

        // Get the UID that was matched in the config path
        $uid = $event->tokenMatches[0];

        if (null === $provider = OrganizationType::findOne([
                'uid' => $uid
            ])) {
            return;
        }

        $provider->delete();

        Event::on(
            OrganizationType::class,
            OrganizationType::EVENT_AFTER_DELETE,
            [
                ManageOrganizationTypeProjectConfig::class,
                'delete'
            ]
        );
    }

    /**
     * @param ConfigEvent $event
     */
    public static function handleChangedUserType(ConfigEvent $event)
    {
        Event::off(
            UserType::class,
            UserType::EVENT_AFTER_INSERT,
            [
                ManageUserTypeProjectConfig::class,
                'save'
            ]
        );

        Event::off(
            UserType::class,
            UserType::EVENT_AFTER_UPDATE,
            [
                ManageUserTypeProjectConfig::class,
                'save'
            ]
        );

        // Get the UID that was matched in the config path
        $uid = $event->tokenMatches[0];

        if (null === ($token = UserType::findOne([
                'uid' => $uid
            ]))) {
            $token = new UserType();
        }

        Craft::configure($token, $event->newValue);

It seems like $token defined by \flipbox\organizations\records\UserType::findOne(array('uid' => $uid)) on line 143 can also be of type array; however, yii\BaseYii::configure() does only seem to accept object, maybe add an additional type check?

        $token->save();

        Event::on(
            UserType::class,
            UserType::EVENT_AFTER_INSERT,
            [
                ManageUserTypeProjectConfig::class,
                'save'
            ]
        );

        Event::on(
            UserType::class,
            UserType::EVENT_AFTER_UPDATE,
            [
                ManageUserTypeProjectConfig::class,
                'save'
            ]
        );
    }

    /**
     * @param ConfigEvent $event
     * @throws \Throwable
     * @throws \yii\db\StaleObjectException
     */
    public static function handleDeletedUserType(ConfigEvent $event)
    {
        Event::off(
            UserType::class,
            UserType::EVENT_AFTER_DELETE,
            [
                ManageUserTypeProjectConfig::class,
                'delete'
            ]
        );

        // Get the UID that was matched in the config path
        $uid = $event->tokenMatches[0];

        if (null === $token = UserType::findOne([
                'uid' => $uid
            ])) {
            return;
        }

        $token->delete();

        Event::on(
            UserType::class,
            UserType::EVENT_AFTER_DELETE,
            [
                ManageUserTypeProjectConfig::class,
                'delete'
            ]
        );
    }

    /**
     * @return array
     */
    public static function rebuild(): array
    {
        $return = [];

        foreach (OrganizationType::findAll([]) as $record) {
            $return['plugins']['organizations']['organizationTypes'][$record->uid] = $record->toProjectConfig();
        }

        foreach (UserType::findAll([]) as $token) {
            $return['plugins']['organizations']['userTypes'][$record->uid] = $record->toProjectConfig();

The variable $record seems to be defined by a foreach iteration on line 216. Are you sure the iterator is never empty, otherwise this variable is not defined?

        }

        return $return;
    }
}