Marek\Toggable\Service\WorkspaceUsers\WorkspaceUsersService

Complexity

Total Complexity

7

Size/Duplication

Total Lines	98
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	12

Test Coverage

Coverage

100%

Importance

Changes	4
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
B	inviteUsers()	0	32	3
A	updateWorkspaceUser()	0	17	1
A	deleteWorkspaceUser()	0	10	1
A	getWorkspaceUsers()	0	21	2

<?php

namespace Marek\Toggable\Service\WorkspaceUsers;

use Marek\Toggable\API\Http\Request\WorkspaceUsers\DeleteWorkspaceUser;
use Marek\Toggable\API\Http\Request\WorkspaceUsers\GetWorkspaceUsers;
use Marek\Toggable\API\Http\Request\WorkspaceUsers\InviteUserToWorkspace;
use Marek\Toggable\API\Http\Request\WorkspaceUsers\UpdateWorkspaceUser;
use Marek\Toggable\API\Http\Response\WorkspaceUsers\Invitation;
use Marek\Toggable\API\Http\Response\WorkspaceUsers\WorkspaceUser;
use Marek\Toggable\API\Http\Response\WorkspaceUsers\WorkspaceUsers;
use Marek\Toggable\API\Toggl\Values\Notification;
use Marek\Toggable\API\Toggl\Values\Workspace\User;
use Marek\Toggable\Service\AbstractService;

/**
 * Class WorkspaceUsersService
 * @package Marek\Toggable\Service\WorkspaceUsers
 */
class WorkspaceUsersService extends AbstractService implements \Marek\Toggable\API\Toggl\WorkspaceUsersServiceInterface
{
    /**
     * @inheritDoc
     */
    public function inviteUsers($workspaceId, array $emails)
    {
        $request = new InviteUserToWorkspace(
            array(
                'workspaceId' => $this->validate($workspaceId),
                'data' => $emails,
            )
        );

        $response = $this->delegate($request);

        $workspaceUsers = array();
        foreach($response->body['data'] as $workspaceUser) {
            $workspaceUsers[] = $this->hydrator->hydrate($workspaceUser, new User());
        }

        $notifications = array();
        foreach($response->body['notifications'] as $notification) {
            $notifications[] = new Notification(
                array(
                    'message' => $notification,
                )
            );
        }

        return new Invitation(
            array(
                'workspaceUsers' => $workspaceUsers,
                'notifications' => $notifications,
            )
        );
    }

    /**
     * @inheritDoc
     */
    public function updateWorkspaceUser($workspaceUserId, \Marek\Toggable\API\Toggl\Values\Workspace\User $user)
    {
        $request = new UpdateWorkspaceUser(
            array(
                'workspaceUserId' => $this->validate($workspaceUserId),
                'data' => $this->extractDataFromObject($user),
            )
        );

        $response = $this->delegate($request);

        return new WorkspaceUser(
            array(
                'workspaceUser' => $this->hydrateDataFromArrayToObject($response, new User())
            )
        );
    }

    /**
     * @inheritDoc
     */
    public function deleteWorkspaceUser($workspaceUserId)
    {
        $request = new DeleteWorkspaceUser(
            array(
                'workspaceUserId' => $this->validate($workspaceUserId),
            )
        );

        return $this->delegate($request);
    }

    /**
     * @inheritDoc
     */
    public function getWorkspaceUsers($workspaceId)

This method seems to be duplicated in your project.

    {
        $request = new GetWorkspaceUsers(
            array(
                'workspaceId' => $this->validate($workspaceId),
            )
        );

        $response = $this->delegate($request);

        $workspaceUsers = array();
        foreach($response->body as $workspaceUser) {
            $workspaceUsers[] = $this->hydrator->hydrate($workspaceUser, new User());
        }

        return new WorkspaceUsers(
            array(

The call to WorkspaceUsers::__construct() has too many arguments starting with array('workspaceUsers' => $workspaceUsers).

                'workspaceUsers' => $workspaceUsers,
            )
        );
    }
}