ThreePlCentral\Request

Complexity

Total Complexity

17

Size/Duplication

Total Lines	119
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes	1
Bugs	0	Features	0

15 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	fetch()	0	22	2
A	getBody()	0	18	2
A	setTemplate()	0	4	1
A	getId()	0	4	1
A	setId()	0	4	1
A	getCustomerId()	0	4	1
A	setCustomerId()	0	4	1
A	getFacilityId()	0	4	1
A	setFacilityId()	0	4	1
A	getLogin()	0	4	1
A	setLogin()	0	4	1
A	getPassword()	0	4	1
A	setPassword()	0	4	1
A	getUrl()	0	4	1

<?php

namespace ThreePlCentral;

use GuzzleHttp\Client;
use GuzzleHttp\Psr7;

class Request implements RequestInterface
{
    private $id;
    private $customerId;
    private $facilityId;
    private $login;
    private $password;
    private $method;
    private $url;
    private $template;

    public function __construct($method, $url)
    {
        $this->method = $method;
        $this->url = $url;
    }

    public function fetch(array $data)
    {
        $client = new Client();
        $request = new Psr7\Request(
            $this->method,
            'https://secure-wms.com/webserviceexternal/contracts.asmx',
            [
                'SOAPAction' => $this->url,
                'Content-Type' => 'text/xml; charset=utf-8'
            ],
            $this->getBody($data)
        );

        try {
            $reponse = $client->send($request);
        } catch (\Exception $error) {
            $response = new Response($error->getResponse());

It seems like you code against a specific sub-type and not the parent class Exception as the method getResponse() does only exist in the following sub-classes of Exception: GuzzleHttp\Exception\BadResponseException, GuzzleHttp\Exception\ClientException, GuzzleHttp\Exception\ConnectException, GuzzleHttp\Exception\RequestException, GuzzleHttp\Exception\ServerException, GuzzleHttp\Exception\TooManyRedirectsException. Maybe you want to instanceof check for one of these explicitly?

            throw new Exception($response->json()['error'], $error->getCode(), $error);
        }
        
        return new Response($reponse);
    }

    private function getBody(array $data)
    {
        $content = file_get_contents($this->template);

        $data = array_merge($data, [
          'ThreePLID' => $this->getId(),
          'Login' => $this->getLogin(),
          'Password' => $this->getPassword(),
          'CustomerID' => $this->getCustomerId(),
          'FacilityID' => $this->getFacilityId()
        ]);

        foreach ($data as $prop => $value) {
            $content = str_replace("{{$prop}}", $value, $content);
        }

        return $content;
    }

    public function setTemplate($template)
    {
        $this->template = $template;
    }

    public function getId()
    {
        return $this->id;
    }

    public function setId($id)
    {
        $this->id = $id;
    }

    public function getCustomerId()
    {
        return $this->customerId;
    }

    public function setCustomerId($customerId)
    {
        $this->customerId = $customerId;
    }

    public function getFacilityId()
    {
        return $this->facilityId;
    }

    public function setFacilityId($facilityId)
    {
        $this->facilityId = $facilityId;
    }

    public function getLogin()
    {
        return $this->login;
    }

    public function setLogin($login)
    {
        $this->login = $login;
    }

    public function getPassword()
    {
        return $this->password;
    }

    public function setPassword($password)
    {
        $this->password = $password;
    }

    public function getUrl()
    {
        return $this->url;
    }
}