CalendarTimePeriod::setAttributes() - Code Metrics - sunnysideup/silverstripe-calendar - Measure and Improve Code Quality continuously with Scrutinizer

CalendarTimePeriod::setAttributes() A
last analyzed 2018-03-20 00:56 UTC

↳ Parent: CalendarTimePeriod

Complexity

Conditions	2
Paths	2

Size

Total Lines	9
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

class CalendarTimePeriod
namespace YourVendor;

class YourClass { }
{

    // Attributes

    private $startTime;
    private $endTime;

    // Constructor

    public function __construct(CalendarTime $startTime, CalendarTime $endTime)
    {
        $this->setAttributes($startTime, $endTime);
    }
    
    // Functions

    public function setAttributes(CalendarTime $startTime, CalendarTime $endTime)
    {
        if ($this->isValidPeriod($startTime, $endTime)) {
            $this->startTime = $startTime;
            $this->endTime = $endTime;
        } else {
            user_error('CalendarTimePeriod::setAttributes() : you cannot construct a \'CalendarTimePeriod\' with the $startTime attribute superior or equal to the $endTime attribute', E_USER_ERROR);
        }
    }
    
    public function isValidPeriod(CalendarTime $startTime, CalendarTime $endTime)
    {
        return $startTime < $endTime;
    }
    
    public function getStartTime()
    {
        return $this->startTime;
    }
    public function getEndTime()
    {
        return $this->endTime;
    }
}


1			<?php
2
3			class CalendarTimePeriod
			0 ignored issues – show Coding Style Compatibility introduced 2017-01-03 00:26 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
4			{
5
6			// Attributes
7
8			private $startTime;
9			private $endTime;
10
11			// Constructor
12
13			public function __construct(CalendarTime $startTime, CalendarTime $endTime)
14			{
15			$this->setAttributes($startTime, $endTime);
16			}
17
18			// Functions
19
20			public function setAttributes(CalendarTime $startTime, CalendarTime $endTime)
21			{
22			if ($this->isValidPeriod($startTime, $endTime)) {
23			$this->startTime = $startTime;
24			$this->endTime = $endTime;
25			} else {
26			user_error('CalendarTimePeriod::setAttributes() : you cannot construct a \'CalendarTimePeriod\' with the $startTime attribute superior or equal to the $endTime attribute', E_USER_ERROR);
27			}
28			}
29
30			public function isValidPeriod(CalendarTime $startTime, CalendarTime $endTime)
31			{
32			return $startTime < $endTime;
33			}
34
35			public function getStartTime()
36			{
37			return $this->startTime;
38			}
39			public function getEndTime()
40			{
41			return $this->endTime;
42			}
43			}
44

sunnysideup / silverstripe-calendar

CalendarTimePeriod::setAttributes() A last analyzed 2018-03-20 00:56 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

CalendarTimePeriod::setAttributes() A
last analyzed 2018-03-20 00:56 UTC