BuildAddonJob

Complexity

Total Complexity

10

Size/Duplication

Total Lines	78
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	2
A	setUp()	0	8	2
A	perform()	0	14	2
A	getTitle()	0	4	1
A	getSignature()	0	3	1
A	getJobData()	0	11	1
A	process()	0	5	1

<?php
/**
 * A background job which builds a single add-on.
 */
class BuildAddonJob extends AbstractQueuedJob implements QueuedJob
{
    private $packageID;

    /**
     * @var array params
     * @throws Exception
     */
    public function __construct($params = array()) {
        if (!empty($params['package'])) {
            $this->setObject(Addon::get()->byID($params['package']));

It seems like \Addon::get()->byID($params['package']) can be null; however, setObject() does not accept null, maybe add an additional type check?

        }
    }

    public function setUp()
    {
        global $databaseConfig;

        if (!DB::isActive()) {

The method DB::isActive() has been deprecated with message: since version 4.0 Use DB::is_active instead

            DB::connect($databaseConfig);
        }
    }

    public function perform()
    {
        $package = $this->getObject();
        if (!$package->ID) throw new Exception('Package not specified');

        $builder = Injector::inst()->get('AddonBuilder');

        $builder->build($package);

        $package->BuildQueued = false;
        $package->write();

        $this->isComplete = true;
    }

    /**
     * @return string
     */
    public function getTitle()
    {
        return 'Build Extended Addon Information';
    }
    /**
     * Return a signature for this queued job
     *
     * @return string
     */
    public function getSignature() {
        return md5(get_class($this) . serialize($this->jobData) . $this->packageID);
    }

    /**
     * @return stdClass
     */
    public function getJobData() {
        $data = new stdClass();
        $data->totalSteps = $this->totalSteps;
        $data->currentStep = $this->currentStep;
        $data->isComplete = $this->isComplete;
        $data->jobData = $this->jobData;
        $data->messages = $this->messages;
        $data->packageID = $this->packageID;

        return $data;
    }

    /**
     * Do some processing yourself!
     */
    public function process()
    {
        $this->setUp();
        $this->perform();
    }
}