DeleteObjectJob - Code Metrics - symbiote/silverstripe-queuedjobs - Measure and Improve Code Quality continuously with Scrutinizer

DeleteObjectJob A
last analyzed 2019-12-18 20:39 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

Metric	Value
wmc	7
lcom	1
cbo	2
dl	0
loc	53
rs	10
c	0
b	0
f	0

5 Methods

Rating	Name	Size	Complexity
A	__construct()	9	2
A	getObject()	4	1
A	getJobType()	4	1
A	getTitle()	9	2
A	process()	7	1

<?php

namespace Symbiote\QueuedJobs\Jobs;

use SilverStripe\ORM\DataObject;
use Symbiote\QueuedJobs\Services\AbstractQueuedJob;
use Symbiote\QueuedJobs\Services\QueuedJob;

/**
 * A job used to delete a data object. Typically used for deletes that need to happen on
 * a schedule, or where the delete may have some onflow affect that takes a while to
 * finish the deletion.
 *
 * @author [email protected]
 * @license BSD License http://silverstripe.org/bsd-license/
 * @skipUpgrade
 */
class DeleteObjectJob extends AbstractQueuedJob
{
    /**
     * @param DataObject $node
     */
    public function __construct($node = null)
    {
        if ($node) {
            $this->TargetClass = get_class($node);
<?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->TargetID = $node->ID;
<?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->currentStep = 0;
            $this->totalSteps = 1;
        }
    }

    /**
     * @param  string $name
     * @return DataObject
     */
    protected function getObject($name = 'Object')
    {
        return DataObject::get_by_id($this->TargetClass, $this->TargetID);
<?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 string
     */
    public function getJobType()
    {
        return QueuedJob::IMMEDIATE;
    }

    /**
     * @return string
     */
    public function getTitle()
    {
        $obj = $this->getObject();
        if ($obj) {
            return _t(__CLASS__ . '.DELETE_OBJ2', 'Delete {title}', array('title' => $obj->Title));
        } else {
            return _t(__CLASS__ . '.DELETE_JOB', 'Delete node');
        }
    }

    public function process()
    {
        $obj = $this->getObject();
        $obj->delete();
        $this->currentStep = 1;
        $this->isComplete = true;
    }
}


1			<?php
2
3			namespace Symbiote\QueuedJobs\Jobs;
4
5			use SilverStripe\ORM\DataObject;
6			use Symbiote\QueuedJobs\Services\AbstractQueuedJob;
7			use Symbiote\QueuedJobs\Services\QueuedJob;
8
9			/**
10			* A job used to delete a data object. Typically used for deletes that need to happen on
11			* a schedule, or where the delete may have some onflow affect that takes a while to
12			* finish the deletion.
13			*
14			* @author [email protected]
15			* @license BSD License http://silverstripe.org/bsd-license/
16			* @skipUpgrade
17			*/
18			class DeleteObjectJob extends AbstractQueuedJob
19			{
20			/**
21			* @param DataObject $node
22			*/
23			public function __construct($node = null)
24			{
25			if ($node) {
26			$this->TargetClass = get_class($node);
			0 ignored issues – show Documentation introduced 2017-10-04 23:47 UTC by Report Bug Copy Issue Report The property `TargetClass` does not exist on `object<Symbiote\QueuedJobs\Jobs\DeleteObjectJob>`. 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...
27			$this->TargetID = $node->ID;
			0 ignored issues – show Documentation introduced 2017-10-04 23:47 UTC by Report Bug Copy Issue Report The property `TargetID` does not exist on `object<Symbiote\QueuedJobs\Jobs\DeleteObjectJob>`. 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...
28			$this->currentStep = 0;
29			$this->totalSteps = 1;
30			}
31			}
32
33			/**
34			* @param string $name
35			* @return DataObject
36			*/
37			protected function getObject($name = 'Object')
38			{
39			return DataObject::get_by_id($this->TargetClass, $this->TargetID);
			0 ignored issues – show Documentation introduced 2017-10-04 23:47 UTC by Report Bug Copy Issue Report The property `TargetClass` does not exist on `object<Symbiote\QueuedJobs\Jobs\DeleteObjectJob>`. Since you implemented `__get`, maybe consider adding a @property annotation. Since your code implements the magic getter `_get`, this function will be called for any read 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."); } } } If the property has read access only, you can use the @property-read 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 2017-10-04 23:47 UTC by Report Bug Copy Issue Report The property `TargetID` does not exist on `object<Symbiote\QueuedJobs\Jobs\DeleteObjectJob>`. Since you implemented `__get`, maybe consider adding a @property annotation. Since your code implements the magic getter `_get`, this function will be called for any read 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."); } } } If the property has read access only, you can use the @property-read 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...
40			}
41
42			/**
43			* @return string
44			*/
45			public function getJobType()
46			{
47			return QueuedJob::IMMEDIATE;
48			}
49
50			/**
51			* @return string
52			*/
53			public function getTitle()
54			{
55			$obj = $this->getObject();
56			if ($obj) {
57			return _t(__CLASS__ . '.DELETE_OBJ2', 'Delete {title}', array('title' => $obj->Title));
58			} else {
59			return _t(__CLASS__ . '.DELETE_JOB', 'Delete node');
60			}
61			}
62
63			public function process()
64			{
65			$obj = $this->getObject();
66			$obj->delete();
67			$this->currentStep = 1;
68			$this->isComplete = true;
69			}
70			}
71

symbiote / silverstripe-queuedjobs

GitHub Access Token became invalid

DeleteObjectJob A last analyzed 2019-12-18 20:39 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

5 Methods

Duplication Side-by-Side

Filter issues like

DeleteObjectJob A
last analyzed 2019-12-18 20:39 UTC