kint\parsers\objects\Kint_Objects_Closure

Complexity

Total Complexity

8

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

15%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
C	parse()	0	41	8

<?php

namespace kint\parsers\objects;

use kint\inc\KintObject;
use kint\Kint;

/**
 * Class Kint_Objects_Closure
 */
class Kint_Objects_Closure extends KintObject
{
  /**
   * @param $variable
   *
   * @return array|bool
   */
  public function parse(&$variable)
  {
    if (!$variable instanceof \Closure) {
      return false;
    }

    $this->name = 'Closure';
    $reflection = new \ReflectionFunction($variable);
    $ret = array(
        'Parameters' => array(),
    );

    $val = $reflection->getParameters();
    if ($val) {

The expression $val of type ReflectionParameter[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

      foreach ($val as $parameter) {
        // todo http://php.net/manual/en/class.reflectionparameter.php
        $ret['Parameters'][] = $parameter->name;
      }

    }

    $val = $reflection->getStaticVariables();
    if ($val) {

The expression $val of type string[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

      $ret['Uses'] = $val;
    }

    if (
        method_exists($reflection, 'getClousureThis')
        &&
        $val = $reflection->getClosureThis()
    ) {
      $ret['Uses']['$this'] = $val;
    }

    $val = $reflection->getFileName();
    if ($val) {
      $this->value = Kint::shortenPath($val) . ':' . $reflection->getStartLine();
    }

    return $ret;
  }
}