scotteh / php-dom-wrapper

src/Traits/NodeTrait.php

<?php declare(strict_types=1);

namespace DOMWrap\Traits;

use DOMWrap\NodeList;

/**
 * Node Trait
 *
 * @package DOMWrap\Traits
 * @license http://opensource.org/licenses/BSD-3-Clause BSD 3 Clause
 * @property \DOMDocument $ownerDocument
 */
trait NodeTrait
{
    /**
     * @return NodeList
     */
    public function collection(): NodeList {
        return $this->newNodeList([$this]);

It seems like newNodeList() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * @return \DOMDocument
     */
    public function document(): ?\DOMDocument {
        if ($this->isRemoved()) {

It seems like isRemoved() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            return null;
        }

        return $this->ownerDocument;
    }

    /**
     * @param NodeList $nodeList
     *
     * @return NodeList|\DOMNode|null
     */
    public function result(NodeList $nodeList) {
        if ($nodeList->count()) {
            return $nodeList->first();
        }

        return null;
    }
}