DownloadOperation::execute() - Code Metrics - Inspection of "Renaming" - storeman-io/storeman - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 2cde75...6c52d6 )

by Arne

created 2018-06-08 21:56 UTC

DownloadOperation::execute() A

↳ Parent: DownloadOperation

Complexity

Conditions	2
Paths	2

Size

Total Lines	17
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	17
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	9
nc	2
nop	2

<?php

namespace Storeman\Operation;

use Storeman\StorageAdapter\StorageAdapterInterface;

class DownloadOperation implements OperationInterface
{
    /**
     * @var string
     */
    protected $relativePath;

    /**
     * @var string
     */
    protected $blobId;

    /**
     * @var array
     */
    protected $streamFilterConfigMap;

    public function __construct(string $relativePath, string $blobId, array $streamFilterConfigMap = [])
    {
        $this->relativePath = $relativePath;
        $this->blobId = $blobId;
        $this->streamFilterConfigMap = $streamFilterConfigMap;
    }

    public function execute(string $localBasePath, StorageAdapterInterface $storageAdapter): bool
    {
        $localStream = fopen($localBasePath . $this->relativePath, 'wb');
        $remoteStream = $storageAdapter->getReadStream($this->blobId);

        foreach ($this->streamFilterConfigMap as $filterName => $filterParams)
        {
            stream_filter_append($remoteStream, $filterName, STREAM_FILTER_READ, $filterParams);
        }

        $bytesCopied = stream_copy_to_stream($remoteStream, $localStream);

        fclose($remoteStream);
        fclose($localStream);

        return $bytesCopied !== false;
    }

    /**
     * @codeCoverageIgnore
     */
    public function __toString(): string
    {
        $filterNames = implode(',', array_keys($this->streamFilterConfigMap)) ?: '-';

        return sprintf('Download %s (blobId %s, filters: %s)', $this->relativePath, $this->blobId, $filterNames);
    }
}


1			<?php
2
3			namespace Storeman\Operation;
4
5			use Storeman\StorageAdapter\StorageAdapterInterface;
6
7			class DownloadOperation implements OperationInterface
8			{
9			/**
10			* @var string
11			*/
12			protected $relativePath;
13
14			/**
15			* @var string
16			*/
17			protected $blobId;
18
19			/**
20			* @var array
21			*/
22			protected $streamFilterConfigMap;
23
24			public function __construct(string $relativePath, string $blobId, array $streamFilterConfigMap = [])
25			{
26			$this->relativePath = $relativePath;
27			$this->blobId = $blobId;
28			$this->streamFilterConfigMap = $streamFilterConfigMap;
29			}
30
31			public function execute(string $localBasePath, StorageAdapterInterface $storageAdapter): bool
32			{
33			$localStream = fopen($localBasePath . $this->relativePath, 'wb');
34			$remoteStream = $storageAdapter->getReadStream($this->blobId);
35
36			foreach ($this->streamFilterConfigMap as $filterName => $filterParams)
37			{
38			stream_filter_append($remoteStream, $filterName, STREAM_FILTER_READ, $filterParams);
39			}
40
41			$bytesCopied = stream_copy_to_stream($remoteStream, $localStream);
42
43			fclose($remoteStream);
44			fclose($localStream);
45
46			return $bytesCopied !== false;
47			}
48
49			/**
50			* @codeCoverageIgnore
51			*/
52			public function __toString(): string
53			{
54			$filterNames = implode(',', array_keys($this->streamFilterConfigMap)) ?: '-';
55
56			return sprintf('Download %s (blobId %s, filters: %s)', $this->relativePath, $this->blobId, $filterNames);
57			}
58			}
59

storeman-io / storeman

Push — master ( 2cde75...6c52d6 )

DownloadOperation::execute() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like