netgen / HtmlPdfApi

HttpClient/Guzzle.php

<?php

namespace Netgen\HtmlPdfApi\HttpClient;

use Netgen\HtmlPdfApi\HttpClientInterface;
use Guzzle\Service\Client;
use Guzzle\Common\Collection;
use Guzzle\Service\Description\ServiceDescription;
use Guzzle\Http\Exception\ClientErrorResponseException;
use Symfony\Component\Finder\Finder;

class Guzzle extends Client implements HttpClientInterface {

    /**
     * Constructor
     *
     * @param string $host          Base url of the api
     * @param string $token         Security token
     * @param string $json_location Full path to service.json file
     */
    public function __construct($host, $token, $json_location)
    {
        //fallback value
        $default = array(
            //'hostname' => '{host}',

67% of this comment could be valid code. Did you maybe forget this after debugging?

            //'host' => 'https://htmlpdfapi.com//api//v1//'
        );
        $config = array(
            'host' => $host,
            'token' => $token
        );
        $required = array('host', 'token');
        $config = Collection::fromConfig($config, $default, $required);

        parent::__construct($config['host']);

        $this->setDescription(ServiceDescription::factory($json_location));

        $header = array( 'Authentication' => 'Token' . " " . $config->get('token') );
        $this->setDefaultOption("headers", $header);
    }

    /**
     * Sends the request via Guzzle Client
     *
     * @param string $url       Relative url for the request
     * @param array $params     Parameters for the request
     * @param string $method    Method for the request
     * @return string           Response body
     * @throws \Exception
     */
    public function sendRequest($url, $params, $method)
    {
        try{
            if (!empty($params['html'])){
                $commandName = 'GenerateFromHTML';
            }
            else if (!empty($params['url'])){
                $commandName = 'GenerateFromURL';
            }
            else if (!empty($params['file'])&& $url=='pdf'){
                $commandName = 'GenerateFromFile';
            }
            else {
                $operations = $this->getDescription()->getOperations();
                foreach ($operations as $operation)
                {
                    if ($operation->getUri()==$url && $operation->getHttpMethod() == $method)
                    {
                        $commandName = $operation->getName();
                    }
                }
            }
            $command = $this->getCommand($commandName, $params);

The variable $commandName does not seem to be defined for all execution paths leading up to this point.

            $ret = $this->execute($command);

            if ( $commandName == 'GetAssetList' || $commandName == 'UploadAsset' )
            {
                return json_encode($ret);
            }
            return $ret->getBody(true);

        }catch(ClientErrorResponseException $exception){
            throw new \Exception($exception->getResponse()->getBody(true), $exception->getResponse()->getStatusCode());
        }
    }
}