HoursWorkedCalculator::calculateHoursWorkedInWeek() - Code Metrics - rpalladino/scheduler-api - Measure and Improve Code Quality continuously with Scrutinizer

calculateHoursWorkedInWeek() A
last analyzed 2020-04-17 04:35 UTC

↳ Parent: HoursWorkedCalculator

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	11
rs	9.9
c	0
b	0
f	0
cc	1
nc	1
nop	2

<?php

namespace Scheduler\Domain\Model\Shift;

use DateTimeInterface;
use DateTimeImmutable;
use Scheduler\Domain\Model\User\User;

class HoursWorkedCalculator
{
    const DAY_OF_WEEK_START = 1;
    const DAY_OF_WEEK_END = 7;

    private $shiftMapper;

    public function __construct(ShiftMapper $shiftMapper)
    {
        $this->shiftMapper = $shiftMapper;
    }

    public function calculateHoursWorkedInWeek(User $employee, DateTimeInterface $weekOf)
    {
        list($start, $end) = $this->getWeekStartAndEndDates($weekOf);

        $shifts = $this->shiftMapper->findShiftsInTimePeriodByEmployeeId($start, $end, $employee->getId());
<?php

function getDate($date)
{
    if ($date !== null) {
        return new DateTime($date);
    }

    return false;
}
        $hours = array_reduce($shifts, function ($total, $shift) {
            return $total + $shift->getHours();
        });

        return new HoursWorkedSummary($start, $end, $hours);
<?php

function getDate($date)
{
    if ($date !== null) {
        return new DateTime($date);
    }

    return false;
}
    }

    private function getWeekStartAndEndDates(DateTimeInterface $weekOf)
    {
        $year = $weekOf->format('Y');
        $week = $weekOf->format('W');
        $date = new DateTimeImmutable("00:00:00");

        $start = $date->setISODate($year, $week, self::DAY_OF_WEEK_START);
        $end = $date->setISODate($year, $week, self::DAY_OF_WEEK_END);

        return [$start, $end];
    }
}


1			<?php
2
3			namespace Scheduler\Domain\Model\Shift;
4
5			use DateTimeInterface;
6			use DateTimeImmutable;
7			use Scheduler\Domain\Model\User\User;
8
9			class HoursWorkedCalculator
10			{
11			const DAY_OF_WEEK_START = 1;
12			const DAY_OF_WEEK_END = 7;
13
14			private $shiftMapper;
15
16			public function __construct(ShiftMapper $shiftMapper)
17			{
18			$this->shiftMapper = $shiftMapper;
19			}
20
21			public function calculateHoursWorkedInWeek(User $employee, DateTimeInterface $weekOf)
22			{
23			list($start, $end) = $this->getWeekStartAndEndDates($weekOf);
24
25			$shifts = $this->shiftMapper->findShiftsInTimePeriodByEmployeeId($start, $end, $employee->getId());
			0 ignored issues – show Security Bug introduced 2017-10-01 15:10 UTC by Report Bug Copy Issue Report It seems like `$start` defined by `$this->getWeekStartAndEndDates($weekOf)` on line `23` can also be of type `false`; however, `Scheduler\Domain\Model\S...imePeriodByEmployeeId()` does only seem to accept `object<DateTimeInterface>`, did you maybe forget to handle an error condition? This check looks for type mismatches where the missing type is `false`. This is usually indicative of an error condtion. Consider the follow example <?php function getDate($date) { if ($date !== null) { return new DateTime($date); } return false; } This function either returns a new `DateTime` object or false, if there was an error. This is a typical pattern in PHP programming to show that an error has occurred without raising an exception. The calling code should check for this returned `false` before passing on the value to another function or method that may not be able to handle a `false`. Loading history... Security Bug introduced 2017-10-01 15:10 UTC by Report Bug Copy Issue Report It seems like `$end` defined by `$this->getWeekStartAndEndDates($weekOf)` on line `23` can also be of type `false`; however, `Scheduler\Domain\Model\S...imePeriodByEmployeeId()` does only seem to accept `object<DateTimeInterface>`, did you maybe forget to handle an error condition? This check looks for type mismatches where the missing type is `false`. This is usually indicative of an error condtion. Consider the follow example <?php function getDate($date) { if ($date !== null) { return new DateTime($date); } return false; } This function either returns a new `DateTime` object or false, if there was an error. This is a typical pattern in PHP programming to show that an error has occurred without raising an exception. The calling code should check for this returned `false` before passing on the value to another function or method that may not be able to handle a `false`. Loading history...
26			$hours = array_reduce($shifts, function ($total, $shift) {
27			return $total + $shift->getHours();
28			});
29
30			return new HoursWorkedSummary($start, $end, $hours);
			0 ignored issues – show Security Bug introduced 2017-10-01 15:10 UTC by Report Bug Copy Issue Report It seems like `$start` defined by `$this->getWeekStartAndEndDates($weekOf)` on line `23` can also be of type `false`; however, `Scheduler\Domain\Model\S...dSummary::__construct()` does only seem to accept `object<DateTimeInterface>`, did you maybe forget to handle an error condition? This check looks for type mismatches where the missing type is `false`. This is usually indicative of an error condtion. Consider the follow example <?php function getDate($date) { if ($date !== null) { return new DateTime($date); } return false; } This function either returns a new `DateTime` object or false, if there was an error. This is a typical pattern in PHP programming to show that an error has occurred without raising an exception. The calling code should check for this returned `false` before passing on the value to another function or method that may not be able to handle a `false`. Loading history... Security Bug introduced 2017-10-01 15:10 UTC by Report Bug Copy Issue Report It seems like `$end` defined by `$this->getWeekStartAndEndDates($weekOf)` on line `23` can also be of type `false`; however, `Scheduler\Domain\Model\S...dSummary::__construct()` does only seem to accept `object<DateTimeInterface>`, did you maybe forget to handle an error condition? This check looks for type mismatches where the missing type is `false`. This is usually indicative of an error condtion. Consider the follow example <?php function getDate($date) { if ($date !== null) { return new DateTime($date); } return false; } This function either returns a new `DateTime` object or false, if there was an error. This is a typical pattern in PHP programming to show that an error has occurred without raising an exception. The calling code should check for this returned `false` before passing on the value to another function or method that may not be able to handle a `false`. Loading history...
31			}
32
33			private function getWeekStartAndEndDates(DateTimeInterface $weekOf)
34			{
35			$year = $weekOf->format('Y');
36			$week = $weekOf->format('W');
37			$date = new DateTimeImmutable("00:00:00");
38
39			$start = $date->setISODate($year, $week, self::DAY_OF_WEEK_START);
40			$end = $date->setISODate($year, $week, self::DAY_OF_WEEK_END);
41
42			return [$start, $end];
43			}
44			}
45

rpalladino / scheduler-api

calculateHoursWorkedInWeek() A last analyzed 2020-04-17 04:35 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

calculateHoursWorkedInWeek() A
last analyzed 2020-04-17 04:35 UTC