NerdsAndCompany\Schematic\Converters\Models\UserGroup

Complexity

Total Complexity

14

Size/Duplication

Total Lines	87
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
B	getRecordDefinition()	0	23	4
A	saveRecord()	0	10	3
A	deleteRecord()	0	4	1
A	getAllMappedPermissions()	0	9	2
A	getMappedPermissions()	0	12	4

<?php

namespace NerdsAndCompany\Schematic\Converters\Models;

use Craft;
use craft\base\Model;

/**
 * Schematic User Groups Service.
 *
 * Sync Craft Setups.
 *
 * @author    Nerds & Company
 * @copyright Copyright (c) 2015-2018, Nerds & Company
 * @license   MIT
 *
 * @see      http://www.nerds.company
 */
class UserGroup extends Base
{
    /**
     * {@inheritdoc}
     */
    public function getRecordDefinition(Model $record): array
    {
        $definition = parent::getRecordDefinition($record);
        $mappedPermissions = $this->getAllMappedPermissions();

        $groupPermissions = [];
        if ($record->id) {
            foreach (Craft::$app->userPermissions->getPermissionsByGroupId($record->id) as $permission) {
                if (array_key_exists($permission, $mappedPermissions)) {
                    $groupPermissions[] = $mappedPermissions[$permission];
                } else {
                    $groupPermissions[] = $permission;
                }
            }
        }

        $permissionDefinitions = $this->getSources(false, $groupPermissions, 'id', 'handle');

The method getSources does not exist on object<NerdsAndCompany\Schematic\Converters\Models\UserGroup>? Since you implemented __call, maybe consider adding a @method annotation.

        sort($permissionDefinitions);

        $definition['permissions'] = $permissionDefinitions;

        return $definition;
    }

    /**
     * {@inheritdoc}
     */
    public function saveRecord(Model $record, array $definition): bool
    {
        if (Craft::$app->userGroups->saveGroup($record) && array_key_exists('permissions', $definition)) {
            $permissions = $this->getSources(false, $definition['permissions'], 'handle', 'id');

The method getSources does not exist on object<NerdsAndCompany\Schematic\Converters\Models\UserGroup>? Since you implemented __call, maybe consider adding a @method annotation.

            return Craft::$app->userPermissions->saveGroupPermissions($record->id, $permissions);
        }

        return false;
    }

    /**
     * {@inheritdoc}
     */
    public function deleteRecord(Model $record): bool
    {
        return Craft::$app->userGroups->deleteGroupById($record->id);
    }

    /**
     * Get a mapping of all permissions from lowercase to camelcase
     * savePermissions only accepts camelcase.
     *
     * @return array
     */
    private function getAllMappedPermissions(): array
    {
        $mappedPermissions = [];
        foreach (Craft::$app->userPermissions->getAllPermissions() as $permissions) {
            $mappedPermissions = array_merge($mappedPermissions, $this->getMappedPermissions($permissions));
        }

        return $mappedPermissions;
    }

    /**
     * Recursive function to get mapped permissions.
     *
     * @param array $permissions
     *
     * @return array
     */
    private function getMappedPermissions(array $permissions): array
    {
        $mappedPermissions = [];
        foreach ($permissions as $permission => $options) {
            $mappedPermissions[strtolower($permission)] = $permission;
            if (is_array($options) && array_key_exists('nested', $options)) {
                $mappedPermissions = array_merge($mappedPermissions, $this->getMappedPermissions($options['nested']));
            }
        }

        return $mappedPermissions;
    }
}