kint\parsers\objects\Kint_Objects_Closure::parse()

Complexity

Conditions	8
Paths	17

Size

Total Lines	41
Code Lines	23

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	47.3039

Importance

Changes

0

<?php
  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;
  }
}