i3Soft\CDA\Elements\Time

Complexity

Total Complexity

8

Size/Duplication

Total Lines	56
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getTime()	0	3	1
A	__construct()	0	3	1
A	setTime()	0	7	4
A	toDOMElement()	0	3	1
A	getElementTag()	0	3	1

<?php
/**
 * The MIT License
 *
 * Copyright 2016 julien.
 *
 * Permission is hereby granted, free of charge, to any person obtaining a copy
 * of this software and associated documentation files (the "Software"), to deal
 * in the Software without restriction, including without limitation the rights
 * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
 * copies of the Software, and to permit persons to whom the Software is
 * furnished to do so, subject to the following conditions:
 *
 * The above copyright notice and this permission notice shall be included in
 * all copies or substantial portions of the Software.
 *
 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
 * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
 * FITNESS FOR A PARTICULAR PURPOSE AND NON INFRINGEMENT. IN NO EVENT SHALL THE
 * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
 * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
 * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 * THE SOFTWARE.
 */

namespace i3Soft\CDA\Elements;

use i3Soft\CDA\DataType\Collection\Interval\IntervalOfTime;
use i3Soft\CDA\DataType\Collection\Interval\PeriodicIntervalOfTime;
use i3Soft\CDA\DataType\Quantity\DateAndTime\TimeStamp;

/**
 * @author julien
 */
class Time extends AbstractElement
{
  /**
   *
   * @var TimeStamp
   */
  protected $time;

  /**
   * Time constructor.
   *
   * @param null $time

Are you sure the doc-type for parameter $time is correct as it would always require null to be passed?

   */
  public function __construct ($time)
  {
    $this->setTime($time);
  }

  /**
   * @return TimeStamp|IntervalOfTime
   */
  public function getTime ()
  {
    return $this->time;
  }

  /**
   * @param TimeStamp|IntervalOfTime $in
   *
   * @return self
   */
  public function setTime ($in): self
  {
    if ($in instanceof TimeStamp || $in instanceof IntervalOfTime || $in instanceof PeriodicIntervalOfTime)

$in is always a sub-type of i3Soft\CDA\DataType\Collection\Interval\IntervalOfTime.

    {
      $this->time = $in;

It seems like $in can also be of type i3Soft\CDA\DataType\Collection\Interval\IntervalOfTime. However, the property $time is declared as type i3Soft\CDA\DataType\Quantity\DateAndTime\TimeStamp. Maybe add an additional type check?

    }
    return $this;
  }

  /**
   * @param \DOMDocument $doc
   *
   * @return \DOMElement
   */
  public function toDOMElement (\DOMDocument $doc): \DOMElement
  {
    return $this->createElement($doc, array('time'));
  }

  /**
   * @return string
   */
  protected function getElementTag (): string
  {
    return 'time';
  }
}