PHPHealth\CDA\RIM\Act\SubstanceAdministration

Complexity

Total Complexity

26

Size/Duplication

Total Lines	214
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	7

Importance

Changes

0

17 Methods

Rating	Name	Duplication	Size	Complexity
A	getClassCode()	0	4	1
A	getElementTag()	0	4	1
A	getRouteCode()	0	4	1
A	getApproachSiteCode()	0	4	1
A	getDoseQuantity()	0	4	1
A	getRateQuantity()	0	4	1
A	getTimeOfAdministration()	0	4	1
A	getFrequencyOfAdministration()	0	4	1
A	getConsumable()	0	4	1
A	setRouteCode()	0	5	1
A	setApproachSiteCode()	0	5	1
A	setDoseQuantity()	0	5	1
A	setRateQuantity()	0	5	1
A	setTimeOfAdministration()	0	4	1
A	setFrequencyOfAdministration()	0	6	1
A	setConsumable()	0	5	1
D	toDOMElement()	0	45	10

<?php
/*
 * The MIT License
 *
 * Copyright 2017 Julien Fastré <julien.fastre@champs-libres.coop>.
 *
 * 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 NONINFRINGEMENT. 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 PHPHealth\CDA\RIM\Act;

use PHPHealth\CDA\DataType\Code\CodedWithEquivalents;
use PHPHealth\CDA\DataType\Collection\Set;
use PHPHealth\CDA\DataType\Collection\Interval;
use PHPHealth\CDA\DataType\Quantity\PhysicalQuantity\PhysicalQuantity;
use PHPHealth\CDA\Elements\TemplateId;
use PHPHealth\CDA\Elements\Text;
use PHPHealth\CDA\Elements\EffectiveTime;
use PHPHealth\CDA\Elements\RouteCode;
use PHPHealth\CDA\RIM\Participation\Consumable;
use PHPHealth\CDA\Elements\DoseQuantity;

/**
 * 
 *
 * @author Julien Fastré <julien.fastre@champs-libres.coop>
 */
class SubstanceAdministration extends Act
{
    /**
     *
     * @var CodedWithEquivalents
     */
    private $routeCode;
    
    /**
     *
     * @var Set|CodedWithEquivalents
     */
    private $approachSiteCode;
    
    /**
     *
     * @var Interval|PhysicalQuantity
     */
    private $doseQuantity;
    
    /**
     *
     * @var Interval|PhysicalQuantity
     */
    private $rateQuantity;
    
    /**
     *
     * @var Consumable 
     */
    private $consumable;
    
    private $frequencyOfAdministration;
    
    
    public function getClassCode(): string
    {
        return 'SBADM';
    }

    protected function getElementTag(): string
    {
        return 'substanceAdministration';
    }
    
    /**
     * 
     * @return CodedWithEquivalents
     */
    public function getRouteCode(): CodedWithEquivalents
    {
        return $this->routeCode;
    }

    /**
     * 
     * @return Set|CodedWithEquivalents
     */
    public function getApproachSiteCode()
    {
        return $this->approachSiteCode;
    }

    /**
     * 
     * @return Interval|PhysicalQuantity
     */
    public function getDoseQuantity()
    {
        return $this->doseQuantity;
    }

    /**
     * 
     * @return Interval|PhysicalQuantity
     */
    public function getRateQuantity()
    {
        return $this->rateQuantity;
    }
    
    public function getTimeOfAdministration()
    {
        return parent::getEffectiveTime();

It seems like you call parent on a different method (getEffectiveTime() instead of getTimeOfAdministration()). Are you sure this is correct? If so, you might want to change this to $this->getEffectiveTime().

    }
    
    public function getFrequencyOfAdministration()
    {
        return $this->frequencyOfAdministration;
    }
    
    /**
     * 
     * @return Consumable
     */
    public function getConsumable()
    {
        return $this->consumable;
    }

    /**
     * 
     * @param CodedWithEquivalents $routeCode
     * @return $this
     */
    public function setRouteCode(CodedWithEquivalents $routeCode)
    {
        $this->routeCode = $routeCode;
        return $this;
    }

    /**
     * 
     * @param Set|CodedWithEquivalents $approachSiteCode
     * @return $this
     */
    public function setApproachSiteCode($approachSiteCode)
    {
        $this->approachSiteCode = $approachSiteCode;
        return $this;
    }

    /**
     * 
     * @param Interval|PhysicalQuantity $doseQuantity
     * @return $this
     */
    public function setDoseQuantity($doseQuantity)
    {
        $this->doseQuantity = $doseQuantity;
        return $this;
    }

    /**
     * 
     * @param Interval|PhysicalQuantity $rateQuantity
     * @return $this
     */
    public function setRateQuantity($rateQuantity)
    {
        $this->rateQuantity = $rateQuantity;
        return $this;
    }
    
    public function setTimeOfAdministration($time)

The parameter $time is not used and could be removed.

    {
        return parent::setEffectiveTime($effectiveTime);

The variable $effectiveTime does not exist. Did you forget to declare it?

It seems like you call parent on a different method (setEffectiveTime() instead of setTimeOfAdministration()). Are you sure this is correct? If so, you might want to change this to $this->setEffectiveTime().

    }
    
    public function setFrequencyOfAdministration($frequency)
    {
        $this->frequencyOfAdministration = $frequency;
        
        return $this;
    }

    /**
     * 
     * @param Consumable $consumable
     * @return $this
     */
    function setConsumable(Consumable $consumable)

It is generally recommended to explicitly declare the visibility for methods.

    {
        $this->consumable = $consumable;
        return $this;
    }
        
    public function toDOMElement(\DOMDocument $doc): \DOMElement
    {
        $el = $this->createElement($doc);
        
        if ($this->getTemplateIds() !== null) {
            foreach ($this->templateIds as $id) {
                $el->appendChild((new TemplateId($id))->toDOMElement($doc));
            }
        }
        
        if ($this->getText() !== null) {
            $el->appendChild((new Text($this->getText()))->toDOMElement($doc));

$this->getText() of type object<PHPHealth\CDA\DataType\TextAndMultimedia\EncapsuledData> is not a sub-type of object<PHPHealth\CDA\DataType\TextAndMultimedia\CharacterString>. It seems like you assume a child class of the class PHPHealth\CDA\DataType\TextAndMultimedia\EncapsuledData to be always present.

        }
        
        if ($this->getTimeOfAdministration() !== null) {
            $el->appendChild((new EffectiveTime($this->getTimeOfAdministration()))
                ->toDOMElement($doc));
        }
        
        if ($this->getFrequencyOfAdministration() !== null) {
            $effectiveTime = new EffectiveTime($this->getFrequencyOfAdministration());
            
            if ($this->getTimeOfAdministration() !== null) {
                $effectiveTime->setOperatorAppend();
            }
            
            $el->appendChild($effectiveTime->toDOMElement($doc));
        }
        
        if ($this->getRouteCode() !== null) {
            $el->appendChild((new RouteCode($this->getRouteCode()))
                ->toDOMElement($doc));
        }
        
        if ($this->getDoseQuantity() !== null) {
            $el->appendChild((new DoseQuantity($this->getDoseQuantity()))
                ->toDOMElement($doc));
        }
        
        if ($this->getConsumable() !== null) {
            $el->appendChild($this->getConsumable()->toDOMElement($doc));
        }
        
        return $el;
    }

}