Acquia\LiftClient\Manager\ManagerBase

Complexity

Total Complexity

8

Size/Duplication

Total Lines	64
Duplicated Lines	12.5 %

Coupling/Cohesion

Components	2
Dependencies	2

Test Coverage

Coverage

94.74%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getResponseJson()	0	7	1
B	getQueryString()	8	17	6

<?php

namespace Acquia\LiftClient\Manager;

use GuzzleHttp\ClientInterface;
use Psr\Http\Message\RequestInterface;

abstract class ManagerBase
{
    /**
     * @var array A list of query parameters that the URL could possibly have
     */
    protected $queryParameters = [];

    /**
     * @var \GuzzleHttp\ClientInterface The request client
     */
    protected $client;

    /**
     * @param \GuzzleHttp\ClientInterface $client The request client
     */
    public function __construct(ClientInterface $client)
    {
        $this->client = $client;
    }

    /**
     * Make the given Request and return as JSON Decoded PHP object.
     *
     * @param \Psr\Http\Message\RequestInterface $request
     *
     * @return mixed the value encoded in <i>json</i> in appropriate
     *               PHP type. Values true, false and
     *               null (case-insensitive) are returned as <b>TRUE</b>, <b>FALSE</b>
     *               and <b>NULL</b> respectively. <b>NULL</b> is returned if the
     *               <i>json</i> cannot be decoded or if the encoded
     *               data is deeper than the recursion limit
     */
    protected function getResponseJson(RequestInterface $request)
    {
        $response = $this->client->send($request);
        $body = (string) $response->getBody();

        return json_decode($body, true);
    }

    /**
     * Get query string of using the options.
     *
     * @param $options The options
     * @return string  The query string
     */
    protected function getQueryString($options) {
      $queries = [];
      foreach ($this->queryParameters as $queryName => $queryDefaultValue) {
        // Use user value if possible.
        if (isset($options[$queryName])) {

This code seems to be duplicated across your project.

          $queries[] = $queryName . '=' . $options[$queryName];
          continue;
        }
        // Use default value if possible.
        if (!isset($options[$queryName]) && is_string($queryDefaultValue)) {

This code seems to be duplicated across your project.

          $queries[] = $queryName . '=' . $queryDefaultValue;
          continue;
        }
      }
      $queryString = implode('&', $queries);
      return empty($queryString) ? '' : '?' . $queryString;
    }
}