Pinepain\JsSandbox\Extractors\PlainExtractors\AssocExtractor::extract()

Complexity

Conditions	5
Paths	5

Size

Total Lines	36
Code Lines	20

Duplication

Lines	31
Ratio	86.11 %

Code Coverage

Tests	0
CRAP Score	30

Importance

Changes

0

<?php declare(strict_types=1);
    public function extract(Context $context, Value $value, PlainExtractorDefinitionInterface $definition, ExtractorInterface $extractor)
    {
        if ($value instanceof ObjectValue) {

This code seems to be duplicated across your project.

The class V8\ObjectValue does not exist. Did you forget a USE statement, or did you not list all dependencies?

            $own_properties = $value->getOwnPropertyNames($context);

            $length  = $own_properties->length();
            $isolate = $context->getIsolate();

            $out = [];

            $next = $definition->getNext();

            for ($i = 0; $i < $length; $i++) {
                /** @var \V8\PrimitiveValue $prop */
                $prop = $own_properties->get($context, new IntegerValue($isolate, $i));
                $item = $value->get($context, $prop);

                $prop_name = $prop->value();

                if ($next) {
                    try {
                        $out[$prop_name] = $extractor->extract($context, $item, $definition);
                    } catch (ExtractorException $e) {
                        throw new ExtractorException("Failed to convert assoc item #{$prop_name}: " . $e->getMessage());
                    }
                } else {
                    $out[$prop_name] = $item;
                }
            }

            return $out;

        }

        throw new ExtractorException('Value must be of object type, ' . $value->typeOf()->value() . ' given instead');
    }
}