Acquia\LiftClient\Manager\SlotManager

Complexity

Total Complexity

7

Size/Duplication

Total Lines	102
Duplicated Lines	20.59 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
B	query()	0	23	4
A	get()	9	9	1
A	add()	8	8	1
A	delete()	0	6	1

<?php

namespace Acquia\LiftClient\Manager;

use Acquia\LiftClient\Entity\Slot;
use GuzzleHttp\Psr7\Request;

class SlotManager extends ManagerBase
{
    /**
     * Get a list of slots.
     *
     * Example of how to structure the $options parameter:
     * <code>
     * $options = [
     *     'visible_on_page'  => 'http://localhost/blog/*',
     *     'status' => 'enabled',
     * ];
     * </code>
     *
     * @see http://docs.decision-api.acquia.com/#slots_get
     *
     * @param array $options
     *
     * @throws \GuzzleHttp\Exception\RequestException
     *
     * @return \Acquia\LiftClient\Entity\Slot[]
     */
    public function query($options = [])
    {
        $variables = $options + [
            'limit' => 1000,
            'start' => 0,
          ];

        $url = '/slots';
        $url .= isset($variables['visible_on_page']) ? "&visible_on_page={$variables['visible_on_page']}" : '';
        $url .= isset($variables['status']) ? "&status={$variables['status']}" : '';

        // Now make the request.
        $request = new Request('GET', $url);
        $data = $this->client->getResponseJson($request);

        // Get them as Slot objects
        $slots = [];
        foreach ($data as $dataItem) {
            $slots[] = new Slot($dataItem);
        }

        return $slots;
    }

    /**
     * Get a specific slot.
     *
     * Example of how to structure the $options parameter:
     *
     * @see http://docs.decision-api.acquia.com/#slots__slotId__get
     *
     * @param array $id
     *
     * @throws \GuzzleHttp\Exception\RequestException
     *
     * @return \Acquia\LiftClient\Entity\Slot
     */
    public function get($id) {

This method seems to be duplicated in your project.

        $url = "/slots/{$id}";

        // Now make the request.
        $request = new Request('GET', $url);
        $data = $this->client->getResponseJson($request);

        return new Slot($data);
    }

    /**
     * Add a slot.
     *
     * @param \Acquia\LiftClient\Entity\Slot $slot
     *
     * @throws \GuzzleHttp\Exception\RequestException
     *
     * @return \Acquia\LiftClient\Entity\Slot
     */
    public function add(Slot $slot) {

This method seems to be duplicated in your project.

        $body = $slot->json();
        $url = '/slots';
        $request = new Request('POST', $url, [], $body);
        $data = $this->client->getResponseJson($request);

        return new Slot($data);
    }

    /**
     * Deletes a slot by ID.
     *
     * @param string $id
     *
     * @throws \GuzzleHttp\Exception\RequestException
     *
     * @return bool
     */
    public function delete($id) {
        $url = "/slots/{$id}";
        $this->client->delete($url);

        return true;
    }
}