CloseNonClosedTimeTrackerTasks::run() - Code Metrics - Inspection of "refactor time tracker open action" - Porto-SAP/Hello-API - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 0cd623...76b2d2 )

by Mahmoud

created 2016-08-18 12:45 UTC

CloseNonClosedTimeTrackerTasks::run() A

↳ Parent: CloseNonClosedTimeTrackerTasks

Complexity

Conditions	3
Paths	2

Size

Total Lines	9
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	9
rs	9.6666
cc	3
eloc	4
nc	2
nop	1

<?php

namespace App\Containers\Tracker\Tasks;

use App\Containers\Tracker\Data\Repositories\TimeTrackerRepository;
use App\Containers\Tracker\Models\TimeTracker;
use App\Port\Task\Abstracts\Task;

/**
 * Class CloseNonClosedTimeTrackerTasks.
 *
 * @author Mahmoud Zalt <[email protected]>
 */
class CloseNonClosedTimeTrackerTasks extends Task
{

    /**
     * @var  \App\Containers\Tracker\Data\Repositories\TimeTrackerRepository
     */
    private $timeTrackerRepository;

    /**
     * CloseNonClosedTimeTrackerTasks constructor.
     *
     * @param \App\Containers\Tracker\Data\Repositories\TimeTrackerRepository $timeTrackerRepository
     */
    public function __construct(TimeTrackerRepository $timeTrackerRepository)
    {
        $this->timeTrackerRepository = $timeTrackerRepository;
    }

    /**
     * @param $timeTracker
     *
     * @return  mixed
     */
    public function run($timeTracker)
    {
        // check if any previous session was not closed
        if ($timeTracker && $timeTracker->status == TimeTracker::PENDING) {
            $this->timeTrackerRepository->update(['status' => TimeTracker::FAILED], $timeTracker->id);
        }

        return $timeTracker;
    }
}


1			<?php
2
3			namespace App\Containers\Tracker\Tasks;
4
5			use App\Containers\Tracker\Data\Repositories\TimeTrackerRepository;
6			use App\Containers\Tracker\Models\TimeTracker;
7			use App\Port\Task\Abstracts\Task;
8
9			/**
10			* Class CloseNonClosedTimeTrackerTasks.
11			*
12			* @author Mahmoud Zalt <[email protected]>
13			*/
14			class CloseNonClosedTimeTrackerTasks extends Task
15			{
16
17			/**
18			* @var \App\Containers\Tracker\Data\Repositories\TimeTrackerRepository
19			*/
20			private $timeTrackerRepository;
21
22			/**
23			* CloseNonClosedTimeTrackerTasks constructor.
24			*
25			* @param \App\Containers\Tracker\Data\Repositories\TimeTrackerRepository $timeTrackerRepository
26			*/
27			public function __construct(TimeTrackerRepository $timeTrackerRepository)
28			{
29			$this->timeTrackerRepository = $timeTrackerRepository;
30			}
31
32			/**
33			* @param $timeTracker
34			*
35			* @return mixed
36			*/
37			public function run($timeTracker)
38			{
39			// check if any previous session was not closed
40			if ($timeTracker && $timeTracker->status == TimeTracker::PENDING) {
41			$this->timeTrackerRepository->update(['status' => TimeTracker::FAILED], $timeTracker->id);
42			}
43
44			return $timeTracker;
45			}
46			}
47

Porto-SAP / Hello-API

Push — master ( 0cd623...76b2d2 )

CloseNonClosedTimeTrackerTasks::run() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like