Symbiote\AdvancedWorkflow\Tasks\WorkflowReminderTask

Complexity

Total Complexity

8

Size/Duplication

Total Lines	57
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	6

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	run()	0	49	8

<?php

namespace Symbiote\AdvancedWorkflow\Tasks;

use SilverStripe\Dev\BuildTask;
use SilverStripe\CMS\Model\SiteTree;
use SilverStripe\Control\Email\Email;
use SilverStripe\ORM\FieldType\DBDatetime;
use Symbiote\AdvancedWorkflow\DataObjects\WorkflowDefinition;
use Symbiote\AdvancedWorkflow\DataObjects\WorkflowInstance;

/**
 * A task that sends a reminder email to users assigned to a workflow that has
 * not been actioned for n days.
 *
 * @package advancedworkflow
 */
class WorkflowReminderTask extends BuildTask
{
    protected $title       = 'Workflow Reminder Task';
    protected $description = 'Sends out workflow reminder emails to stale workflow instances';

    private static $segment = 'WorkflowReminderTask';

Consider using a different property name as you override a private property of the parent class.

    public function run($request)
    {
        $sent = 0;
        if (WorkflowInstance::get()->count()) { // Don't attempt the filter if no instances -- prevents a crash
            $active = WorkflowInstance::get()
                ->innerJoin('WorkflowDefinition', '"DefinitionID" = "WorkflowDefinition"."ID"')
                ->filter(array(
                    'WorkflowStatus' => array('Active', 'Paused')
                ))->where('RemindDays > 0');

            if ($active->exists()) {
                foreach ($active as $instance) {
                    $edited = strtotime($instance->LastEdited);
                    $days   = $instance->Definition()->RemindDays;

                    if ($edited + $days * 3600 * 24 > DBDatetime::now()->getTimestamp()) {
                        continue;
                    }

                    $email   = new Email();
                    $bcc     = '';

$bcc is not used, you could remove the assignment.

                    $members = $instance->getAssignedMembers();
                    $target  = $instance->getTarget();

                    if (!$members || !count($members)) {
                        continue;
                    }

                    $email->setSubject("Workflow Reminder: $instance->Title");
                    $email->setBCC($members->column('Email'));
                    $email->setHTMLTemplate('email\\WorkflowReminderEmail');
                    $email->setData(array(
                        'Instance' => $instance,
                        'Link' => $target instanceof SiteTree ? "admin/show/$target->ID" : '',
                        'Diff' => $instance->getTargetDiff()
                    ));

                    $email->send();


                    $sent++;

                    $instance->LastEdited = DBDatetime::now()->getTimestamp();
                    $instance->write();
                }
            }
        }
        echo "Sent $sent workflow reminder emails.\n";
    }
}