CloudImageCached - Code Metrics - markguinn/silverstripe-cloudassets - Measure and Improve Code Quality continuously with Scrutinizer

CloudImageCached A
last analyzed 2018-09-25 15:47 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	150
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

Metric	Value
dl	0
loc	150
c	0
b	0
f	0
wmc	20
lcom	1
cbo	2
rs	10

11 Methods

Rating	Name	Size	Complexity
A	__construct()	11	3
A	exists()	4	1
A	getRelativePath()	4	1
A	requireTable()	4	1
A	write()	8	2
A	delete()	20	4
A	setStoreRecord()	8	1
A	getStoreRecord()	4	1
A	setCloudMetaJson()	8	2
A	setCloudStatus()	8	2
A	setCloudSize()	8	2

<?php
/**
 * Wraps Image_Cached. This one we have to be a little more
 * careful with because we don't keep a database record.
 *
 * NOTE: An Image_Cached can never actually be converted to
 * one of these because it's not in the db. It must be created
 * as this class (see CloudImage::getFormattedImage).
 *
 * @author Mark Guinn <[email protected]>
 * @date 01.13.2014
 * @package cloudassets
 * @subpackage wrappers
 */
class CloudImageCached extends CloudImage
{
    /** @var CloudImageCachedStore */
    protected $storeRecord;

    /**
     * Create a new cached image.
     * @param string $filename The filename of the image.
     * @param boolean $isSingleton This this to true if this is a singleton() object, a stub for calling methods.
     *                             Singletons don't have their defaults set.
     */
    public function __construct($filename = null, $isSingleton = false)
    {
        parent::__construct(array(), $isSingleton);
        $this->ID = -1;
        $this->Filename = $filename;

        // this covers the case where the image already exists in the cloud from a previous call
        if (file_exists($this->getFullPath()) && $this->containsPlaceholder()) {
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
            $this->CloudStatus = 'Live';
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
        }
    }

    /**
     * @return bool
     */
    public function exists()
    {
        return file_exists($this->getFullPath());
    }

    /**
     * @return String
     */
    public function getRelativePath()
    {
        return $this->getField('Filename');
    }


    /**
     * Prevent creating new tables for the cached record
     *
     * @return false
     */
    public function requireTable()
    {
        return false;
    }


    /**
     * Prevent writing the cached image to the database, but write the store record instead
     */
    public function write($showDebug = false, $forceInsert = false, $forceWrite = false, $writeComponents = false)
    {
        //throw new Exception("{$this->ClassName} can not be written back to the database.");
        // NOTE: we need to fail silently on writes because writing is part of the cloud upload process
        if ($this->storeRecord) {
            $this->storeRecord->write($showDebug, $forceInsert, $forceWrite, $writeComponents);
        }
    }


    /**
     * Simulates a delete
     */
    public function delete()
    {
        $this->brokenOnDelete = true;
        $this->onBeforeDelete();
        if ($this->brokenOnDelete) {
            user_error("$this->class has a broken onBeforeDelete() function."
            . " Make sure that you call parent::onBeforeDelete().", E_USER_ERROR);
        }

        $path = $this->getFullPath();
        if (file_exists($path)) {
            unlink($path);
        }
        if ($this->storeRecord) {
            $this->storeRecord->delete();
        }

        $this->flushCache();
        $this->onAfterDelete();
    }


    /**
     * @param CloudImageCachedStore $store
     * @return $this
     */
    public function setStoreRecord(CloudImageCachedStore $store)
    {
        $this->storeRecord    = $store;
        $this->CloudStatus   = $store->CloudStatus;
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
        $this->CloudSize     = $store->CloudSize;
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
        $this->CloudMetaJson = $store->CloudMetaJson;
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
        return $this;
    }


    /**
     * @return CloudImageCachedStore
     */
    public function getStoreRecord()
    {
        return $this->storeRecord;
    }


    /**
     * @param $val
     */
    public function setCloudMetaJson($val)
    {
        $this->setField('CloudMetaJson', $val);
        if ($this->storeRecord) {
            $this->storeRecord->CloudMetaJson = $val;
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
            //$this->storeRecord->write();
        }
    }


    /**
     * @param $val
     */
    public function setCloudStatus($val)
    {
        $this->setField('CloudStatus', $val);
        if ($this->storeRecord) {
            $this->storeRecord->CloudStatus = $val;
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
            //$this->storeRecord->write();
        }
    }


    /**
     * @param $val
     */
    public function setCloudSize($val)
    {
        $this->setField('CloudSize', $val);
        if ($this->storeRecord) {
            $this->storeRecord->CloudSize = $val;
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
            //$this->storeRecord->write();
        }
    }
}


1			<?php
2			/**
3			* Wraps Image_Cached. This one we have to be a little more
4			* careful with because we don't keep a database record.
5			*
6			* NOTE: An Image_Cached can never actually be converted to
7			* one of these because it's not in the db. It must be created
8			* as this class (see CloudImage::getFormattedImage).
9			*
10			* @author Mark Guinn <[email protected]>
11			* @date 01.13.2014
12			* @package cloudassets
13			* @subpackage wrappers
14			*/
15			class CloudImageCached extends CloudImage
16			{
17			/** @var CloudImageCachedStore */
18			protected $storeRecord;
19
20			/**
21			* Create a new cached image.
22			* @param string $filename The filename of the image.
23			* @param boolean $isSingleton This this to true if this is a singleton() object, a stub for calling methods.
24			* Singletons don't have their defaults set.
25			*/
26			public function __construct($filename = null, $isSingleton = false)
27			{
28			parent::__construct(array(), $isSingleton);
29			$this->ID = -1;
30			$this->Filename = $filename;
31
32			// this covers the case where the image already exists in the cloud from a previous call
33			if (file_exists($this->getFullPath()) && $this->containsPlaceholder()) {
			0 ignored issues – show Documentation Bug introduced 2015-12-20 20:45 UTC by Report Bug Copy Issue Report The method `containsPlaceholder` does not exist on `object<CloudImageCached>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
34			$this->CloudStatus = 'Live';
			0 ignored issues – show Documentation introduced 2015-12-20 20:45 UTC by Report Bug Copy Issue Report The property `CloudStatus` does not exist on `object<CloudImageCached>`. Since you implemented `__set`, maybe consider adding a @property annotation. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
35			}
36			}
37
38			/**
39			* @return bool
40			*/
41			public function exists()
42			{
43			return file_exists($this->getFullPath());
44			}
45
46			/**
47			* @return String
48			*/
49			public function getRelativePath()
50			{
51			return $this->getField('Filename');
52			}
53
54
55			/**
56			* Prevent creating new tables for the cached record
57			*
58			* @return false
59			*/
60			public function requireTable()
61			{
62			return false;
63			}
64
65
66			/**
67			* Prevent writing the cached image to the database, but write the store record instead
68			*/
69			public function write($showDebug = false, $forceInsert = false, $forceWrite = false, $writeComponents = false)
70			{
71			//throw new Exception("{$this->ClassName} can not be written back to the database.");
72			// NOTE: we need to fail silently on writes because writing is part of the cloud upload process
73			if ($this->storeRecord) {
74			$this->storeRecord->write($showDebug, $forceInsert, $forceWrite, $writeComponents);
75			}
76			}
77
78
79			/**
80			* Simulates a delete
81			*/
82			public function delete()
83			{
84			$this->brokenOnDelete = true;
85			$this->onBeforeDelete();
86			if ($this->brokenOnDelete) {
87			user_error("$this->class has a broken onBeforeDelete() function."
88			. " Make sure that you call parent::onBeforeDelete().", E_USER_ERROR);
89			}
90
91			$path = $this->getFullPath();
92			if (file_exists($path)) {
93			unlink($path);
94			}
95			if ($this->storeRecord) {
96			$this->storeRecord->delete();
97			}
98
99			$this->flushCache();
100			$this->onAfterDelete();
101			}
102
103
104			/**
105			* @param CloudImageCachedStore $store
106			* @return $this
107			*/
108			public function setStoreRecord(CloudImageCachedStore $store)
109			{
110			$this->storeRecord = $store;
111			$this->CloudStatus = $store->CloudStatus;
			0 ignored issues – show Documentation introduced 2015-12-17 04:10 UTC by Report Bug Copy Issue Report The property `CloudStatus` does not exist on `object<CloudImageCached>`. Since you implemented `__set`, maybe consider adding a @property annotation. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text / class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history... Documentation introduced 2015-12-17 04:10 UTC by Report Bug Copy Issue Report The property `CloudStatus` does not exist on `object<CloudImageCachedStore>`. Since you implemented `__set`, maybe consider adding a @property annotation. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /* * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
112			$this->CloudSize = $store->CloudSize;
			0 ignored issues – show Documentation introduced 2015-12-17 04:10 UTC by Report Bug Copy Issue Report The property `CloudSize` does not exist on `object<CloudImageCached>`. Since you implemented `__set`, maybe consider adding a @property annotation. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text / class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history... Documentation introduced 2015-12-17 04:10 UTC by Report Bug Copy Issue Report The property `CloudSize` does not exist on `object<CloudImageCachedStore>`. Since you implemented `__set`, maybe consider adding a @property annotation. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /* * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
113			$this->CloudMetaJson = $store->CloudMetaJson;
			0 ignored issues – show Documentation introduced 2015-12-17 04:10 UTC by Report Bug Copy Issue Report The property `CloudMetaJson` does not exist on `object<CloudImageCached>`. Since you implemented `__set`, maybe consider adding a @property annotation. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text / class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history... Documentation introduced 2015-12-17 04:10 UTC by Report Bug Copy Issue Report The property `CloudMetaJson` does not exist on `object<CloudImageCachedStore>`. Since you implemented `__set`, maybe consider adding a @property annotation. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /* * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
114			return $this;
115			}
116
117
118			/**
119			* @return CloudImageCachedStore
120			*/
121			public function getStoreRecord()
122			{
123			return $this->storeRecord;
124			}
125
126
127			/**
128			* @param $val
129			*/
130			public function setCloudMetaJson($val)
131			{
132			$this->setField('CloudMetaJson', $val);
133			if ($this->storeRecord) {
134			$this->storeRecord->CloudMetaJson = $val;
			0 ignored issues – show Documentation introduced 2015-12-17 04:10 UTC by Report Bug Copy Issue Report The property `CloudMetaJson` does not exist on `object<CloudImageCachedStore>`. Since you implemented `__set`, maybe consider adding a @property annotation. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
135			//$this->storeRecord->write();
136			}
137			}
138
139
140			/**
141			* @param $val
142			*/
143			public function setCloudStatus($val)
144			{
145			$this->setField('CloudStatus', $val);
146			if ($this->storeRecord) {
147			$this->storeRecord->CloudStatus = $val;
			0 ignored issues – show Documentation introduced 2015-12-17 04:10 UTC by Report Bug Copy Issue Report The property `CloudStatus` does not exist on `object<CloudImageCachedStore>`. Since you implemented `__set`, maybe consider adding a @property annotation. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
148			//$this->storeRecord->write();
149			}
150			}
151
152
153			/**
154			* @param $val
155			*/
156			public function setCloudSize($val)
157			{
158			$this->setField('CloudSize', $val);
159			if ($this->storeRecord) {
160			$this->storeRecord->CloudSize = $val;
			0 ignored issues – show Documentation introduced 2015-12-17 04:10 UTC by Report Bug Copy Issue Report The property `CloudSize` does not exist on `object<CloudImageCachedStore>`. Since you implemented `__set`, maybe consider adding a @property annotation. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
161			//$this->storeRecord->write();
162			}
163			}
164			}
165

markguinn / silverstripe-cloudassets

CloudImageCached A last analyzed 2018-09-25 15:47 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

11 Methods

Duplication Side-by-Side

Filter issues like

CloudImageCached A
last analyzed 2018-09-25 15:47 UTC