Mrcnpdlk\Api\Unoconv\Api::wsGetPdf()

Complexity

Conditions	7
Paths	11

Size

Total Lines	47
Code Lines	31

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	20
CRAP Score	9.5839

Importance

Changes

0

<?php
    public function wsGetPdf(string $sourceFile, ?string $destination): SplFileObject
    {
        $sourceFile = realpath($sourceFile);

        if (!is_file($sourceFile)) {
            throw new InvalidFileArgumentException(sprintf('Input file "%s" not exists', $sourceFile));
        }
        if (!is_readable($sourceFile)) {
            throw new InvalidFileArgumentException(sprintf('Input file "%s" is not readable', $sourceFile));
        }

        $fromPathInfo = pathinfo($sourceFile);

        if (null === $destination) {
            $destination = sprintf('%s%s%s.pdf',
                $fromPathInfo['dirname'],
                DIRECTORY_SEPARATOR,
                $fromPathInfo['filename']
            );
        } elseif (is_dir($destination)) {
            $destination = sprintf('%s%s%s.pdf',
                $destination,
                DIRECTORY_SEPARATOR,
                $fromPathInfo['filename']
            );
        }

        $this->logger->debug(sprintf('Creating "%s" from "%s"', $destination, $sourceFile));

        $ch = curl_init();
        curl_setopt($ch, \CURLOPT_URL, sprintf('%s/unoconv/pdf', $this->ws));

It seems like $ch can also be of type false; however, parameter $ch of curl_setopt() does only seem to accept resource, maybe add an additional type check?

        curl_setopt($ch, \CURLOPT_RETURNTRANSFER, 1);
        curl_setopt($ch, \CURLOPT_POST, true);
        curl_setopt($ch, \CURLOPT_POSTFIELDS, ['file' => new \CURLFile($sourceFile)]);
        $output = curl_exec($ch);

It seems like $ch can also be of type false; however, parameter $ch of curl_exec() does only seem to accept resource, maybe add an additional type check?

        if (false === $output) {
            throw new UnoconvException('Curl error: ' . curl_error($ch));

It seems like $ch can also be of type false; however, parameter $ch of curl_error() does only seem to accept resource, maybe add an additional type check?

        }
        $ret = json_decode($output);

It seems like $output can also be of type true; however, parameter $json of json_decode() does only seem to accept string, maybe add an additional type check?

        if (\JSON_ERROR_NONE === json_last_error()) {
            throw new UnoconvException('WebService Error: ' . $ret->message);
        }

        curl_close($ch);

It seems like $ch can also be of type false; however, parameter $ch of curl_close() does only seem to accept resource, maybe add an additional type check?

        file_put_contents($destination, $output);

        return new SplFileObject($destination);
    }