Oliverde8\Component\PhpEtl\ChainOperation\Transformer\FilterDataOperation::processData()

Complexity

Conditions	5
Paths	2

Size

Total Lines	12
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function processData(DataItemInterface $item, ExecutionContext $context): ItemInterface
    {
        $data = $item->getData();

        $resultData = [];
        $result = $this->ruleApplier->apply($data, $resultData, $this->rule);

        if (($this->negate && $result == false) || (!$this->negate && $result == true)) {

It seems like you are loosely comparing $result of type null|string against false; this is ambiguous if the string can be empty. Consider using a strict comparison === instead.

            return $item;
        }

        return new ChainBreakItem();
    }
}