LabelFileFactory::read() - Code Metrics - Inspection of "coding" - loevgaard/pakkelabels-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( a3a68a...0a1479 )

by Joachim

created 2017-11-06 13:54 UTC

LabelFileFactory::read() B

↳ Parent: LabelFileFactory

Complexity

Conditions	4
Paths	3

Size

Total Lines	24
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	24
rs	8.6845
c	0
b	0
f	0
cc	4
eloc	13
nc	3
nop	2

<?php

namespace Loevgaard\PakkelabelsBundle\File;

use Loevgaard\Pakkelabels\Client;
use Loevgaard\PakkelabelsBundle\Entity\Label;

class LabelFileFactory
{
    /**
     * @var string
     */
    protected $labelDir;

    /**
     * @var Client
     */
    protected $pakkelabelsClient;

    public function __construct(Client $client, string $labelDir)
    {
        $this->pakkelabelsClient = $client;
        $this->labelDir = $labelDir;
    }

    /**
     * @param Label $label
     *
     * @return LabelFile
     */
    public function create(Label $label) : LabelFile
    {

        $file = new LabelFile($this->labelDir.'/'.$label->getId().'.png', 'w+');

        return $file;
    }

    /**
     * @todo This is a bit wrong. This should probably be a service of some sort instead of being called a 'factory'
     *
     * @param Label $label
     * @param bool $verifyExistence
     * @return LabelFile
     */
    public function read(Label $label, bool $verifyExistence = false) : LabelFile
    {
        $labelFilePath = $this->labelDir.'/'.$label->getId().'.png';

        if($verifyExistence && !file_exists($labelFilePath)) {
            $file = new LabelFile($labelFilePath, 'w+');

            $labelRes = $this->pakkelabelsClient->doRequest('get', '/shipments/'.$label->getExternalId().'/labels', [
                'query' => [
                    'label_format' => 'png',
                ],
            ]);

            if (isset($labelRes['error'])) {
                throw new \RuntimeException('The labels for label id '.$label->getId().' could not be downloaded from Pakkelabels. Error was: '.$labelRes['error']);
            }

            $file->fwrite(base64_decode($labelRes['base64']));
        } else {
            $file = new LabelFile($labelFilePath, 'r');
        }

        return $file;
    }
}


1			<?php
2
3			namespace Loevgaard\PakkelabelsBundle\File;
4
5			use Loevgaard\Pakkelabels\Client;
6			use Loevgaard\PakkelabelsBundle\Entity\Label;
7
8			class LabelFileFactory
9			{
10			/**
11			* @var string
12			*/
13			protected $labelDir;
14
15			/**
16			* @var Client
17			*/
18			protected $pakkelabelsClient;
19
20			public function __construct(Client $client, string $labelDir)
21			{
22			$this->pakkelabelsClient = $client;
23			$this->labelDir = $labelDir;
24			}
25
26			/**
27			* @param Label $label
28			*
29			* @return LabelFile
30			*/
31			public function create(Label $label) : LabelFile
32			{
33
34			$file = new LabelFile($this->labelDir.'/'.$label->getId().'.png', 'w+');
35
36			return $file;
37			}
38
39			/**
40			* @todo This is a bit wrong. This should probably be a service of some sort instead of being called a 'factory'
41			*
42			* @param Label $label
43			* @param bool $verifyExistence
44			* @return LabelFile
45			*/
46			public function read(Label $label, bool $verifyExistence = false) : LabelFile
47			{
48			$labelFilePath = $this->labelDir.'/'.$label->getId().'.png';
49
50			if($verifyExistence && !file_exists($labelFilePath)) {
51			$file = new LabelFile($labelFilePath, 'w+');
52
53			$labelRes = $this->pakkelabelsClient->doRequest('get', '/shipments/'.$label->getExternalId().'/labels', [
54			'query' => [
55			'label_format' => 'png',
56			],
57			]);
58
59			if (isset($labelRes['error'])) {
60			throw new \RuntimeException('The labels for label id '.$label->getId().' could not be downloaded from Pakkelabels. Error was: '.$labelRes['error']);
61			}
62
63			$file->fwrite(base64_decode($labelRes['base64']));
64			} else {
65			$file = new LabelFile($labelFilePath, 'r');
66			}
67
68			return $file;
69			}
70			}
71

loevgaard / pakkelabels-bundle

Push — master ( a3a68a...0a1479 )

LabelFileFactory::read() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like