TaskManagerCommand::actionRun() - Code Metrics - Inspection of "Added the ability to run and watch for tasks via t..." - boboldehampsink/taskmanager - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( b0afbf...396dfb )

by Bob Olde

created 2016-04-01 06:22 UTC

TaskManagerCommand::actionRun() A

↳ Parent: TaskManagerCommand

Complexity

Conditions	3
Paths	3

Size

Total Lines	23
Code Lines	11

Duplication

Lines	15
Ratio	65.22 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
c	1
b	0
f	1
dl	15
loc	23
rs	9.0856
cc	3
eloc	11
nc	3
nop	0

<?php

namespace Craft;

/**
 * Task Manager.
 *
 * @author    Bob Olde Hampsink <[email protected]>
 * @copyright Copyright (c) 2015, Bob Olde Hampsink
 * @license   MIT
 *
 * @link      http://github.com/boboldehampsink
 */
class TaskManagerCommand extends BaseCommand
{
    /**
     * Runs pending tasks.
     *
     * @return int
     */
    public function actionRun()
    {
        Craft::log(Craft::t('Running new tasks.'));

        // Make sure tasks aren't already running
        if (!craft()->tasks->isTaskRunning()) {


            // Is there a pending task?
            if (craft()->tasks->getNextPendingTask()) {

                // Start running tasks
                craft()->tasks->runPendingTasks();

                return 1;
            } else {
                Craft::log(Craft::t('No pending tasks found.'));
            }
        } else {
            Craft::log(Craft::t('Tasks are already running.'));
        }

        return 0;
    }

    /**
     * Watch for tasks and run them.
     */
    public function actionWatch()
    {
        Craft::log(Craft::t('Watching for new tasks.'));

        // Keep on checking for pending tasks
        while (true) {

            // Make sure tasks aren't already running
            if (!craft()->tasks->isTaskRunning()) {


                // Is there a pending task?
                if (craft()->tasks->getNextPendingTask()) {

                    // Start running tasks
                    craft()->tasks->runPendingTasks();
                } else {
                    Craft::log(Craft::t('No pending tasks found.'));
                }
            } else {
                Craft::log(Craft::t('Tasks are already running, skipping iteration.'));
            }

            // Sleep a little
            sleep(10);
        }
    }
}


1		<?php
2
3		namespace Craft;
4
5		/**
6		* Task Manager.
7		*
8		* @author Bob Olde Hampsink <[email protected]>
9		* @copyright Copyright (c) 2015, Bob Olde Hampsink
10		* @license MIT
11		*
12		* @link http://github.com/boboldehampsink
13		*/
14		class TaskManagerCommand extends BaseCommand
15		{
16		/**
17		* Runs pending tasks.
18		*
19		* @return int
20		*/
21		public function actionRun()
22		{
23		Craft::log(Craft::t('Running new tasks.'));
24
25		// Make sure tasks aren't already running
26	View Code Duplication	if (!craft()->tasks->isTaskRunning()) {
		1 ignored issue – show Duplication introduced 2016-04-01 06:24 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
27
28		// Is there a pending task?
29		if (craft()->tasks->getNextPendingTask()) {
30
31		// Start running tasks
32		craft()->tasks->runPendingTasks();
33
34		return 1;
35		} else {
36		Craft::log(Craft::t('No pending tasks found.'));
37		}
38		} else {
39		Craft::log(Craft::t('Tasks are already running.'));
40		}
41
42		return 0;
43		}
44
45		/**
46		* Watch for tasks and run them.
47		*/
48		public function actionWatch()
49		{
50		Craft::log(Craft::t('Watching for new tasks.'));
51
52		// Keep on checking for pending tasks
53		while (true) {
54
55		// Make sure tasks aren't already running
56	View Code Duplication	if (!craft()->tasks->isTaskRunning()) {
		1 ignored issue – show Duplication introduced 2016-04-01 06:24 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
57
58		// Is there a pending task?
59		if (craft()->tasks->getNextPendingTask()) {
60
61		// Start running tasks
62		craft()->tasks->runPendingTasks();
63		} else {
64		Craft::log(Craft::t('No pending tasks found.'));
65		}
66		} else {
67		Craft::log(Craft::t('Tasks are already running, skipping iteration.'));
68		}
69
70		// Sleep a little
71		sleep(10);
72		}
73		}
74		}
75

boboldehampsink / taskmanager

Push — develop ( b0afbf...396dfb )

TaskManagerCommand::actionRun() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like