kint-php / kint

src/Parser/DOMIterator.php

<?php

class Kint_Parser_DOMIterator extends Kint_Parser_Plugin

Kint_Parser_DOMIterator does not seem to conform to the naming convention (^[A-Z][a-zA-Z0-9]*$).

{
    public function getTypes()
    {
        return array('object');
    }

    public function getTriggers()

The return type could not be reliably inferred; please add a @return annotation.

    {
        // Recursion should never happen, should always be stopped at the parent
        // DOMNode.  Depth limit on the other hand we're going to skip since
        // that would show an empty iterator and rather useless. Let the depth
        // limit hit the children (DOMNodeList only has DOMNode as children)
        return Kint_Parser::TRIGGER_COMPLETE & ~Kint_Parser::TRIGGER_RECURSION;
    }

    public function parse(&$var, Kint_Object &$o, $trigger)

Avoid variables with short names like $o. Configured minimum length is 3.

    {
        if (!($var instanceof DOMNamedNodeMap || $var instanceof DOMNodeList)) {
            return;
        }

        $o->size = $var->length;
        if ($o->size === 0) {
            $o->replaceRepresentation(new Kint_Object_Representation('Iterator'));
            $o->size = null;

            return;
        }

        // Depth limit
        // Make empty iterator representation since we need it in DOMNode to point out depth limits
        if ($this->parser->max_depth && $o->depth + 1 >= $this->parser->max_depth) {
            $b = new Kint_Object();

Avoid variables with short names like $b. Configured minimum length is 3.

            $b->name = $o->classname.' Iterator Contents';

The property classname does not seem to exist in Kint_Object.

            $b->access_path = 'iterator_to_array('.$o->access_path.')';
            $b->depth = $o->depth + 1;
            $b->hints[] = 'depth_limit';

            $r = new Kint_Object_Representation('Iterator');

Avoid variables with short names like $r. Configured minimum length is 3.

            $r->contents = array($b);
            $o->replaceRepresentation($r, 0);

            return;
        }

        // In 5.1 you can interate them but they're not traversable.
        // DomDoc. The gift that keeps on giving.
        if (!$var instanceof Traversable) {
            $data = array();
            foreach ($var as $item) {
                $data[] = $item;
            }
        } else {
            $data = iterator_to_array($var);
        }

        $r = new Kint_Object_Representation('Iterator');
        $o->replaceRepresentation($r, 0);

        foreach ($data as $key => $item) {
            $base_obj = new Kint_Object();

$base_obj does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            $base_obj->depth = $o->depth + 1;

$base_obj does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            $base_obj->name = $item->nodeName;

$base_obj does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            if ($o->access_path) {

The expression $o->access_path of type string|null is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

                if ($var instanceof DOMNamedNodeMap) {
                    $base_obj->access_path = $o->access_path.'->getNamedItem('.var_export($key, true).')';

$base_obj does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

                } elseif ($var instanceof DOMNodeList) {
                    $base_obj->access_path = $o->access_path.'->item('.var_export($key, true).')';

$base_obj does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

                } else {
                    $base_obj->access_path = 'iterator_to_array('.$o->access_path.')';

$base_obj does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

                }
            }

            $r->contents[] = $this->parser->parse($item, $base_obj);

$base_obj does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        }
    }
}