Kadet\Xmpp\Stanza\Iq

Complexity

Total Complexity

6

Size/Duplication

Total Lines	57
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	1

Test Coverage

Coverage

72.21%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getXmlCollocations()	0	7	1
A	__construct()	0	5	1
A	appendChild()	0	8	2
A	getQuery()	0	4	1
A	setQuery()	0	5	1

<?php
/**
 * Nucleus - XMPP Library for PHP
 *
 * Copyright (C) 2016, Some rights reserved.
 *
 * @author Kacper "Kadet" Donat <kacper@kadet.net>
 *
 * Contact with author:
 * Xmpp: me@kadet.net
 * E-mail: contact@kadet.net
 *
 * From Kadet with love.
 */

namespace Kadet\Xmpp\Stanza;


use Kadet\Xmpp\Jid;
use Kadet\Xmpp\Stanza\Iq\Query;
use function Kadet\Xmpp\Utils\filter\pass;
use Kadet\Xmpp\Xml\XmlElement;

/**
 * Represents IQ Stanza
 * @package Kadet\Xmpp\Stanza
 *
 * @property Query $query
 */
class Iq extends Stanza
{
    /**
     * Stanza constructor.
     * @param string $type Iq query type
     * @param array  $options {
     *     @var Jid     $from      Jid representing "from" stanza attribute
     *     @var Jid     $to        Jid representing "to" stanza attribute
     *     @var string  $id        Unique id, will be generated if omitted
     *     @var string  $type      Stanza type
     *     @var mixed   $content   Stanza content
     *     @var array   $arguments Stanza arguments
     *     @var Query   $query     Query associated with stanza
     * }
     */
    public function __construct(string $type, array $options = [])
    {
        $this->type = $type;
        parent::__construct('iq', $options);
    }

    protected function appendChild($element)
    {
        if(count($this->children) > 0) {
            throw new \RuntimeException('Iq stanzas cannot have more than one child.');
        }

        return parent::appendChild($element);
    }

    #region Query
    /**
     * @return Query
     */
    public function getQuery()
    {
        return $this->get(Query::class);

The expression $this->get(\Kadet\Xmpp\Stanza\Iq\Query::class); of type Kadet\Xmpp\Xml\XmlElement|false adds false to the return on line 66 which is incompatible with the return type documented by Kadet\Xmpp\Stanza\Iq::getQuery of type Kadet\Xmpp\Stanza\Iq\Query. It seems like you forgot to handle an error condition.

    }

    /**
     * @param Query $query
     */
    public function setQuery(Query $query)
    {
        $this->remove($this->query);
        $this->append($query);
    }
    #endregion

    public static function getXmlCollocations() : array
    {
        return array_merge(
            [[ Query::class, 'name' => pass(), 'uri' => pass() ]],
            parent::getXmlCollocations()
        );
    }
}