GoogleCloudClientStorage::write() - Code Metrics - Inspection of "Adds adapter for Google Cloud Client Library" - KnpLabs/Gaufrette - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#427)

unknown

created 2016-09-10 10:48 UTC

GoogleCloudClientStorage::write() B

↳ Parent: GoogleCloudClientStorage

Complexity

Conditions	2
Paths	2

Size

Total Lines	26
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	1

Metric	Value
c	2
b	0
f	1
dl	0
loc	26
rs	8.8571
cc	2
eloc	15
nc	2
nop	2

<?php

namespace Gaufrette\Adapter;

use Gaufrette\Adapter;
use Gaufrette\Adapter\MetadataSupporter;
use Gaufrette\Adapter\ResourcesSupporter;
use Gaufrette\Adapter\ListKeysAware;

/**
 * Google Cloud Storage adapter using the Google Cloud Client Library for PHP
 * http://googlecloudplatform.github.io/google-cloud-php/
 *
 * @package Gaufrette
 * @author  Lech Buszczynski <[email protected]>
 */
class GoogleCloudClientStorage implements Adapter, MetadataSupporter, ResourcesSupporter, ListKeysAware {

    protected $storageClient;
    protected $bucket;
    protected $bucketValidated;
    protected $options      = array();
    protected $metadata     = array();
    protected $resources    = array();

    /**
     * @param Google\Cloud\Storage\StorageClient    $service    Authenticated storage client class
/**
 * @param array $germany
 * @param array $island
 * @param array $italy
 */
function finale($germany, $island) {
    return "2:1";
}
     * @param string                                $bucketName Name of the bucket
     * @param array                                 $options    Options are: "directory" and "acl" (see https://cloud.google.com/storage/docs/access-control/lists)
     */
    public function __construct(\Google\Cloud\Storage\StorageClient $storageClient, $bucketName, $options = array())
    {
        $this->storageClient = $storageClient;
        $this->setBucket($bucketName);
        $this->options = array_replace_recursive(
            array(
                'directory' => '',
                'acl'       => array()
            ),
            $options
        );
    }
    
    /**
     * Get adapter options
     * 
     * @return  array
     */
    public function getOptions()
    {
        return $this->options;
    }

    /**
     * Set adapter options
     * 
     * @param   array   $options
     */
    public function setOptions($options)
    {
        $this->options = array_replace($this->options, $options);
    }
    
    protected function computePath($key)
    {
        if (strlen($this->options['directory']))
        {
            if (strcmp(substr($this->options['directory'], -1), '/') == 0)
            {
                return $this->options['directory'].$key;
            }
            return $this->options['directory'].'/'.$key;
        }
        return $key;
    }
    
    protected function isBucket()
    {
        if ($this->bucketValidated === true)
        {
            return true;
        } elseif (!$this->bucket->exists()) {
            throw new \RuntimeException(sprintf('Bucket %s does not exist.', $this->bucket->name()));
        }
        $this->bucketValidated = true;
        return true;
    }
    
    public function setBucket($name)
    {
        $this->bucketValidated = null;
        $this->bucket = $this->storageClient->bucket($name);       
        $this->isBucket();
    }
    
    public function getBucket()
    {
        return $this->bucket;
    }
    
    /**
     * {@inheritdoc}
     */
    public function read($key)

    {   
        $this->isBucket();     
        $object = $this->bucket->object($this->computePath($key));
        $info = $object->info();
        $this->setResources($key, $info);
        return $object->downloadAsString();
    }
    
    /**
     * {@inheritdoc}
     */
    public function write($key, $content)
    {
        $this->isBucket();
        
        $options = array(
            'resumable'     => true,
            'name'          => $this->computePath($key),
            'metadata'      => $this->getMetadata($key),
        );

        $this->bucket->upload(
            $content,
            $options
        );
        
        $this->updateKeyProperties($key,
            array(
                'acl'       => $this->options['acl'],
                'metadata'  => $this->getMetadata($key)
            )
        );
        
        $size = $this->getResourceByName($key, 'size');
        
        return $size === null ? false : $size;
class Author {
    private $name;

    public function __construct($name) {
        $this->name = $name;
    }

    public function getName() {
        return $this->name;
    }
}

abstract class Post {
    public function getAuthor() {
        return 'Johannes';
    }
}

class BlogPost extends Post {
    public function getAuthor() {
        return new Author('Johannes');
    }
}

class ForumPost extends Post { /* ... */ }

function my_function(Post $post) {
    echo strtoupper($post->getAuthor());
}
    }
    
    /**
     * {@inheritdoc}
     */
    public function exists($key)

    {
        $this->isBucket();
        $object = $this->bucket->object($this->computePath($key));
        if ($object->exists())
        {
            return true;
        }
        return false;
    }
    
    /**
     * {@inheritdoc}
     */
    public function isDirectory($key)
    {
        if ($this->exists($key . '/'))
        {
            return true;
        }
        return false;
    }
    
    /**
     * {@inheritdoc}
     */
    public function listKeys($prefix = null)
    {
        $this->isBucket();        
        $keys = array();        
        if ($prefix === null)
        {
            $prefix = $this->options['directory'];
        } else {
            $prefix = $this->computePath($prefix);
        }
        foreach ($this->bucket->objects(array('prefix' => $prefix)) as $e)
        {
            $keys[] = $e->name();
        }
        sort($keys);
        return $keys;
    }
    
    /**
     * {@inheritdoc}
     */
    public function keys()
    {
        return $this->listKeys();
    }

    /**
     * {@inheritdoc}
     */
    public function mtime($key)

    {
        $this->isBucket();
        $object = $object = $this->bucket->object($this->computePath($key));
        $info = $object->info();
        $this->setResources($key, $info);
        return strtotime($info['updated']);
    }
    
    /**
     * {@inheritdoc}
     */
    public function delete($key)

    {
        $this->isBucket();
        $object = $this->bucket->object($this->computePath($key));
        $object->delete();
        $this->setMetadata($key, null);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        return true;
    }
    
    /**
     * {@inheritdoc}
     */
    public function rename($sourceKey, $targetKey)
    {
        $this->isBucket();
        
        $metadata = $this->getMetadata($sourceKey);
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        
        $sourceKey = $this->computePath($sourceKey);
        $targetKey = $this->computePath($targetKey);
        
        $object = $this->bucket->object($sourceKey);
        
        $copiedObject = $object->copy($this->bucket,
            array(
                'name' => $targetKey
            )
        );
        
        if ($copiedObject->exists())
        {
            $this->updateKeyProperties($targetKey,
                array(
                    'acl'       => $this->options['acl'],
                    'metadata'  => $this->getMetadata($sourceKey)
                )
            );
            $this->setMetadata($targetKey, $this->getMetadata($sourceKey));
            $this->setMetadata($sourceKey, null);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $object->delete();
            return true;
        }
        return false;
    }
    
    /**
     * {@inheritdoc}
     */
    public function setMetadata($key, $metadata)
    {
        $this->metadata[$key] = $metadata;
    }

    /**
     * {@inheritdoc}
     */
    public function getMetadata($key)
    {    
        return isset($this->metadata[$key]) ? $this->metadata[$key] : array();
    }
    
    /**
     * {@inheritdoc}
     */
    public function setResources($key, $data)
    {
        $this->resources[$key] = $data;
    }
    
    /**
     * {@inheritdoc}
     */
    public function getResources($key)
    {
        return isset($this->resources[$key]) ? $this->resources[$key] : array();
    }
    
    /**
     * {@inheritdoc}
     */
    public function getResourceByName($key, $resourceName)
    {
        return isset($this->resources[$key][$resourceName]) ? $this->resources[$key][$resourceName] : null;
    }
    
    /**
     * Sets ACL and metadata information.
     * 
     * @param   string  $key
     * @param   array   $options    Can contain "acl" and/or "metadata" arrays.
     * @return  boolean
     */
    protected function updateKeyProperties($key, $options = array())
    {
        if ($this->exists($key) === false)
        {
            return false;
        }
        
        $object = $this->bucket->object($this->computePath($key));
        
        $properties = array_replace_recursive(
            array(
                'acl'       => array(),
                'metadata'  => array()
            ), $options
        );

        $acl = $object->acl();
        foreach ($properties['acl'] as $k => $v)
        {
            $acl->add($k, $v);
        }
        $object->update(array('metadata' => $properties['metadata']));

        $info = $object->info();

        $this->setResources($key, $info);
        return true;
    }
}

1		<?php
2
3		namespace Gaufrette\Adapter;
4
5		use Gaufrette\Adapter;
6		use Gaufrette\Adapter\MetadataSupporter;
7		use Gaufrette\Adapter\ResourcesSupporter;
8		use Gaufrette\Adapter\ListKeysAware;
9
10		/**
11		* Google Cloud Storage adapter using the Google Cloud Client Library for PHP
12		* http://googlecloudplatform.github.io/google-cloud-php/
13		*
14		* @package Gaufrette
15		* @author Lech Buszczynski <[email protected]>
16		*/
17		class GoogleCloudClientStorage implements Adapter, MetadataSupporter, ResourcesSupporter, ListKeysAware {
18
19		protected $storageClient;
20		protected $bucket;
21		protected $bucketValidated;
22		protected $options = array();
23		protected $metadata = array();
24		protected $resources = array();
25
26		/**
27		* @param Google\Cloud\Storage\StorageClient $service Authenticated storage client class
		0 ignored issues – show Bug introduced 2016-09-09 12:17 UTC by Report Bug Copy Issue Report There is no parameter named `$service`. Was it maybe removed? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. Consider the following example. The parameter `$italy` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $island * @param array $italy */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was removed, but the annotation was not. Loading history...
28		* @param string $bucketName Name of the bucket
29		* @param array $options Options are: "directory" and "acl" (see https://cloud.google.com/storage/docs/access-control/lists)
30		*/
31		public function __construct(\Google\Cloud\Storage\StorageClient $storageClient, $bucketName, $options = array())
32		{
33		$this->storageClient = $storageClient;
34		$this->setBucket($bucketName);
35		$this->options = array_replace_recursive(
36		array(
37		'directory' => '',
38		'acl' => array()
39		),
40		$options
41		);
42		}
43
44		/**
45		* Get adapter options
46		*
47		* @return array
48		*/
49		public function getOptions()
50		{
51		return $this->options;
52		}
53
54		/**
55		* Set adapter options
56		*
57		* @param array $options
58		*/
59		public function setOptions($options)
60		{
61		$this->options = array_replace($this->options, $options);
62		}
63
64		protected function computePath($key)
65		{
66		if (strlen($this->options['directory']))
67		{
68		if (strcmp(substr($this->options['directory'], -1), '/') == 0)
69		{
70		return $this->options['directory'].$key;
71		}
72		return $this->options['directory'].'/'.$key;
73		}
74		return $key;
75		}
76
77		protected function isBucket()
78		{
79		if ($this->bucketValidated === true)
80		{
81		return true;
82		} elseif (!$this->bucket->exists()) {
83		throw new \RuntimeException(sprintf('Bucket %s does not exist.', $this->bucket->name()));
84		}
85		$this->bucketValidated = true;
86		return true;
87		}
88
89		public function setBucket($name)
90		{
91		$this->bucketValidated = null;
92		$this->bucket = $this->storageClient->bucket($name);
93		$this->isBucket();
94		}
95
96		public function getBucket()
97		{
98		return $this->bucket;
99		}
100
101		/**
102		* {@inheritdoc}
103		*/
104	View Code Duplication	public function read($key)
		0 ignored issues – show Duplication introduced 2016-09-09 12:17 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
105		{
106		$this->isBucket();
107		$object = $this->bucket->object($this->computePath($key));
108		$info = $object->info();
109		$this->setResources($key, $info);
110		return $object->downloadAsString();
111		}
112
113		/**
114		* {@inheritdoc}
115		*/
116		public function write($key, $content)
117		{
118		$this->isBucket();
119
120		$options = array(
121		'resumable' => true,
122		'name' => $this->computePath($key),
123		'metadata' => $this->getMetadata($key),
124		);
125
126		$this->bucket->upload(
127		$content,
128		$options
129		);
130
131		$this->updateKeyProperties($key,
132		array(
133		'acl' => $this->options['acl'],
134		'metadata' => $this->getMetadata($key)
135		)
136		);
137
138		$size = $this->getResourceByName($key, 'size');
139
140		return $size === null ? false : $size;
		0 ignored issues – show Bug Best Practice introduced 2016-09-10 10:54 UTC by Report Bug Copy Issue Report The return type of `return $size === null ? false : $size;` (`array\|string`) is incompatible with the return type declared by the interface `Gaufrette\Adapter::write` of type `integer\|boolean`. If you return a value from a function or method, it should be a sub-type of the type that is given by the parent type f.e. an interface, or abstract method. This is more formally defined by the Lizkov substitution principle, and guarantees that classes that depend on the parent type can use any instance of a child type interchangably. This principle also belongs to the SOLID principles for object oriented design. Let’s take a look at an example: class Author { private $name; public function __construct($name) { $this->name = $name; } public function getName() { return $this->name; } } abstract class Post { public function getAuthor() { return 'Johannes'; } } class BlogPost extends Post { public function getAuthor() { return new Author('Johannes'); } } class ForumPost extends Post { /* ... */ } function my_function(Post $post) { echo strtoupper($post->getAuthor()); } Our function `my_function` expects a `Post` object, and outputs the author of the post. The base class `Post` returns a simple string and outputting a simple string will work just fine. However, the child class `BlogPost` which is a sub-type of `Post` instead decided to return an `object`, and is therefore violating the SOLID principles. If a `BlogPost` were passed to `my_function`, PHP would not complain, but ultimately fail when executing the `strtoupper` call in its body. Loading history...
141		}
142
143		/**
144		* {@inheritdoc}
145		*/
146	View Code Duplication	public function exists($key)
		0 ignored issues – show Duplication introduced 2016-09-10 10:58 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
147		{
148		$this->isBucket();
149		$object = $this->bucket->object($this->computePath($key));
150		if ($object->exists())
151		{
152		return true;
153		}
154		return false;
155		}
156
157		/**
158		* {@inheritdoc}
159		*/
160		public function isDirectory($key)
161		{
162		if ($this->exists($key . '/'))
163		{
164		return true;
165		}
166		return false;
167		}
168
169		/**
170		* {@inheritdoc}
171		*/
172		public function listKeys($prefix = null)
173		{
174		$this->isBucket();
175		$keys = array();
176		if ($prefix === null)
177		{
178		$prefix = $this->options['directory'];
179		} else {
180		$prefix = $this->computePath($prefix);
181		}
182		foreach ($this->bucket->objects(array('prefix' => $prefix)) as $e)
183		{
184		$keys[] = $e->name();
185		}
186		sort($keys);
187		return $keys;
188		}
189
190		/**
191		* {@inheritdoc}
192		*/
193		public function keys()
194		{
195		return $this->listKeys();
196		}
197
198		/**
199		* {@inheritdoc}
200		*/
201	View Code Duplication	public function mtime($key)
		0 ignored issues – show Duplication introduced 2016-09-09 12:17 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
202		{
203		$this->isBucket();
204		$object = $object = $this->bucket->object($this->computePath($key));
205		$info = $object->info();
206		$this->setResources($key, $info);
207		return strtotime($info['updated']);
208		}
209
210		/**
211		* {@inheritdoc}
212		*/
213	View Code Duplication	public function delete($key)
		0 ignored issues – show Duplication introduced 2016-09-10 10:58 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
214		{
215		$this->isBucket();
216		$object = $this->bucket->object($this->computePath($key));
217		$object->delete();
218		$this->setMetadata($key, null);
		0 ignored issues – show Documentation introduced 2016-09-10 10:58 UTC by Report Bug Copy Issue Report `null` is of type `null`, but the function expects a `array`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
219		return true;
220		}
221
222		/**
223		* {@inheritdoc}
224		*/
225		public function rename($sourceKey, $targetKey)
226		{
227		$this->isBucket();
228
229		$metadata = $this->getMetadata($sourceKey);
		0 ignored issues – show Unused Code introduced 2016-09-10 10:54 UTC by Report Bug Copy Issue Report `$metadata` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
230
231		$sourceKey = $this->computePath($sourceKey);
232		$targetKey = $this->computePath($targetKey);
233
234		$object = $this->bucket->object($sourceKey);
235
236		$copiedObject = $object->copy($this->bucket,
237		array(
238		'name' => $targetKey
239		)
240		);
241
242		if ($copiedObject->exists())
243		{
244		$this->updateKeyProperties($targetKey,
245		array(
246		'acl' => $this->options['acl'],
247		'metadata' => $this->getMetadata($sourceKey)
248		)
249		);
250		$this->setMetadata($targetKey, $this->getMetadata($sourceKey));
251		$this->setMetadata($sourceKey, null);
		0 ignored issues – show Documentation introduced 2016-09-10 10:54 UTC by Report Bug Copy Issue Report `null` is of type `null`, but the function expects a `array`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
252		$object->delete();
253		return true;
254		}
255		return false;
256		}
257
258		/**
259		* {@inheritdoc}
260		*/
261		public function setMetadata($key, $metadata)
262		{
263		$this->metadata[$key] = $metadata;
264		}
265
266		/**
267		* {@inheritdoc}
268		*/
269		public function getMetadata($key)
270		{
271		return isset($this->metadata[$key]) ? $this->metadata[$key] : array();
272		}
273
274		/**
275		* {@inheritdoc}
276		*/
277		public function setResources($key, $data)
278		{
279		$this->resources[$key] = $data;
280		}
281
282		/**
283		* {@inheritdoc}
284		*/
285		public function getResources($key)
286		{
287		return isset($this->resources[$key]) ? $this->resources[$key] : array();
288		}
289
290		/**
291		* {@inheritdoc}
292		*/
293		public function getResourceByName($key, $resourceName)
294		{
295		return isset($this->resources[$key][$resourceName]) ? $this->resources[$key][$resourceName] : null;
296		}
297
298		/**
299		* Sets ACL and metadata information.
300		*
301		* @param string $key
302		* @param array $options Can contain "acl" and/or "metadata" arrays.
303		* @return boolean
304		*/
305		protected function updateKeyProperties($key, $options = array())
306		{
307		if ($this->exists($key) === false)
308		{
309		return false;
310		}
311
312		$object = $this->bucket->object($this->computePath($key));
313
314		$properties = array_replace_recursive(
315		array(
316		'acl' => array(),
317		'metadata' => array()
318		), $options
319		);
320
321		$acl = $object->acl();
322		foreach ($properties['acl'] as $k => $v)
323		{
324		$acl->add($k, $v);
325		}
326		$object->update(array('metadata' => $properties['metadata']));
327
328		$info = $object->info();
329
330		$this->setResources($key, $info);
331		return true;
332		}
333		}

KnpLabs / Gaufrette

Pull Request — master (#427)

GoogleCloudClientStorage::write() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like