LDAPMemberSyncJob

Complexity

Total Complexity

9

Size/Duplication

Total Lines	57
Duplicated Lines	26.32 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getJobType()	0	4	1
A	getTitle()	0	4	1
A	getSignature()	0	4	1
A	validateRegenerateTime()	9	9	3
A	process()	6	15	2

<?php
/**
 * Class LDAPMemberSyncJob
 *
 * A {@link QueuedJob} job to sync all users to the site using LDAP.
 * This doesn't do the actual sync work, but rather just triggers {@link LDAPMemberSyncTask}
 */
class LDAPMemberSyncJob extends AbstractQueuedJob

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    /**
     * If you specify this value in seconds, it tells the completed job to queue another of itself
     * x seconds ahead of time.
     *
     * @var mixed
     * @config
     */
    private static $regenerate_time = null;

The property $regenerate_time is not used and could be removed.

    public function __construct()
    {
        // noop, but needed for QueuedJobsAdmin::createjob() to work
    }

    public function getJobType()
    {
        return QueuedJob::QUEUED;
    }

    public function getTitle()
    {
        return _t('LDAPMemberSyncJob.SYNCTITLE', 'Sync all users from Active Directory');
    }

    public function getSignature()
    {
        return md5(get_class($this));
    }

    public function validateRegenerateTime()

This method seems to be duplicated in your project.

    {
        $regenerateTime = Config::inst()->get('LDAPMemberSyncJob', 'regenerate_time');

        // don't allow this job to run less than every 15 minutes, as it could take a while.
        if ($regenerateTime !== null && $regenerateTime < 900) {
            throw new Exception('LDAPMemberSyncJob::regenerate_time must be 15 minutes or greater');
        }
    }

    public function process()
    {
        $regenerateTime = Config::inst()->get('LDAPMemberSyncJob', 'regenerate_time');
        if ($regenerateTime) {

This code seems to be duplicated across your project.

            $this->validateRegenerateTime();

            $nextJob = Injector::inst()->create('LDAPMemberSyncJob');
            singleton('QueuedJobService')->queueJob($nextJob, date('Y-m-d H:i:s', time() + $regenerateTime));
        }

        $task = Injector::inst()->create('LDAPMemberSyncTask');
        $task->run(null);

        $this->isComplete = true;
    }
}