ExcludeWeekendConstraint::apply() - Code Metrics - Inspection of "[WIP] Feature constraints" - Samffy/recurrence - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#18)

by Samuel

created 2017-12-04 22:50 UTC

ExcludeWeekendConstraint::apply() A

↳ Parent: ExcludeWeekendConstraint

Complexity

Conditions	2
Paths	2

Size

Total Lines	10
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	10
ccs	4
cts	4
cp	1
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	5
nc	2
nop	2
crap	2

<?php

namespace Recurrence\Constraint;

use Recurrence\Model\Recurrence;

/**
 * Class ExcludeWeekendConstraint
 * @package Recurrence\Constraint
 */
class ExcludeWeekendConstraint implements RecurrenceConstraintInterface
{

    /**
     * @param Recurrence $recurrence
     * @param \Datetime $datetime
     * @return \Datetime
     */
    public function apply(Recurrence $recurrence, \Datetime $datetime)
    {
        if ($this->isWeekend($datetime)) {
            // Add 1 or 2 days to skip weekend (we didn't use `next monday` pattern of \Datetime::format cause it remove time)
            $days = (7 - (int) $datetime->format('N') + 1);
            $datetime->modify(sprintf('+%d days', $days));
        }

        return $datetime;
    }

    /**
     * @param \Datetime $datetime
     * @return bool
     */
    private function isWeekend($datetime) {
        return ((int) $datetime->format('N') >= 6);
    }
}


1		<?php
2
3		namespace Recurrence\Constraint;
4
5		use Recurrence\Model\Recurrence;
6
7		/**
8		* Class ExcludeWeekendConstraint
9		* @package Recurrence\Constraint
10		*/
11		class ExcludeWeekendConstraint implements RecurrenceConstraintInterface
12		{
13
14		/**
15		* @param Recurrence $recurrence
16		* @param \Datetime $datetime
17		* @return \Datetime
18		*/
19		public function apply(Recurrence $recurrence, \Datetime $datetime)
20		{
21	1	if ($this->isWeekend($datetime)) {
22		// Add 1 or 2 days to skip weekend (we didn't use `next monday` pattern of \Datetime::format cause it remove time)
23	1	$days = (7 - (int) $datetime->format('N') + 1);
24	1	$datetime->modify(sprintf('+%d days', $days));
25		}
26
27	1	return $datetime;
28		}
29
30		/**
31		* @param \Datetime $datetime
32		* @return bool
33		*/
34		private function isWeekend($datetime) {
35	1	return ((int) $datetime->format('N') >= 6);
36		}
37		}
38

Samffy / recurrence

Pull Request — master (#18)

ExcludeWeekendConstraint::apply() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like