BWC\Share\Net\HttpClient\HttpClientCallbackMock

Complexity

Total Complexity

15

Size/Duplication

Total Lines	139
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

14 Methods

Rating	Name	Duplication	Size	Complexity
A	getTimeoutMS()	0	3	1
A	setTimeoutMS()	0	3	1
A	setCaPath()	0	4	1
A	setCaFile()	0	4	1
A	getStatusCode()	0	3	1
A	getHeader()	0	3	1
A	setCredentials()	0	3	1
A	accept()	0	3	1
A	get()	0	4	1
A	post()	0	4	1
A	delete()	0	4	1
A	request()	0	7	2
A	looseSslCheck()	0	4	1
A	getErrorText()	0	5	1

<?php

namespace BWC\Share\Net\HttpClient;

use BWC\Share\Net\HttpStatusCode;


class HttpClientCallbackMock implements HttpClientInterface
{
    private $_timeout_ms = 10000;

    public $statusCode = HttpStatusCode::OK;
    public $header;

    public $callbackRequest;


    /**
     * @return int
     */
    function getTimeoutMS() {

It is generally recommended to explicitly declare the visibility for methods.

        return $this->_timeout_ms;
    }

    /**
     * @param int $milliseconds
     * @return void
     */
    function setTimeoutMS($milliseconds) {

It is generally recommended to explicitly declare the visibility for methods.

        $this->_timeout_ms = $milliseconds;
    }

    /**
     * @param $path
     * @return void
     */
    public function setCaPath($path)
    {

    }

    /**
     * @param $file
     * @return void
     */
    public function setCaFile($file)
    {

    }


    /**
     * @return int
     */
    function getStatusCode() {

It is generally recommended to explicitly declare the visibility for methods.

        return $this->statusCode;
    }

    /**
     * @return string
     */
    function getHeader() {

It is generally recommended to explicitly declare the visibility for methods.

        return $this->header;
    }


    /**
     * @param string $username
     * @param string $password
     * @return void
     */
    function setCredentials($username, $password) {

It is generally recommended to explicitly declare the visibility for methods.

        // TODO: Implement setCredentials() method.
    }


    /**
     * @param string $type   mime type for Accept http header, like text/xml or application/json
     * @return void
     */
    function accept($type) {

It is generally recommended to explicitly declare the visibility for methods.

        // TODO: Implement accept() method.
    }

    /**
     * @param string $url
     * @param array $queryData
     * @param array $arrHeaders
     * @return string
     */
    function get($url, array $queryData = array(), array $arrHeaders = null) {

It is generally recommended to explicitly declare the visibility for methods.

        $result = $this->request($url, 'GET', $queryData, null, null, $arrHeaders);
        return $result;
    }

    function post($url, array $queryData = array(), $postData, $contentType = null, array $arrHeaders = null) {

It is generally recommended to explicitly declare the visibility for methods.

        $result = $this->request($url, 'POST', $queryData, $postData, $contentType, $arrHeaders);

It seems like $postData defined by parameter $postData on line 96 can also be of type object; however, BWC\Share\Net\HttpClient\HttpClientCallbackMock::request() does only seem to accept array|string, maybe add an additional type check?

It seems like $contentType defined by parameter $contentType on line 96 can also be of type string; however, BWC\Share\Net\HttpClient\HttpClientCallbackMock::request() does only seem to accept null, maybe add an additional type check?

        return $result;
    }

    /**
     * @param string $url
     * @param array $queryData
     * @param array $arrHeaders
     * @return string
     */
    function delete($url, array $queryData = array(), array $arrHeaders = null) {

It is generally recommended to explicitly declare the visibility for methods.

        $result = $this->request($url, 'DELETE', $queryData, null, null, $arrHeaders);
        return $result;
    }

    /**
     * @param $url
     * @param $method
     * @param array $queryData
     * @param array|string $postData
     * @param null $contentType
     * @param array $arrHeaders
     * @return string
     */
    function request($url, $method, array $queryData, $postData, $contentType = null, array $arrHeaders = null) {

It is generally recommended to explicitly declare the visibility for methods.

        if ($this->callbackRequest) {
            return call_user_func($this->callbackRequest, $this, $url, $method, $queryData, $postData, $contentType, $arrHeaders);
        } else {
            return '';
        }
    }

    /**
     * @param bool $value
     * @return void
     */
    public function looseSslCheck($value)
    {
        // TODO: Implement looseSslCheck() method.
    }

    /**
     * @return string
     */
    public function getErrorText()
    {
        // TODO: Implement getErrorText() method.
        return '';
    }
}