Dynamic\Salsify\Traits\Yieldable::yieldSingle()

Complexity

Conditions	6
Paths	12

Size

Total Lines	16
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

<?php
    public function yieldSingle($list, $callback = null)
    {
        if (!is_array($list) && !$list instanceof \Traversable) {
            $list = [$list];
        }

        foreach ($list as $item) {
            $injected = (yield $item);

            if ($injected === static::$STOP_GENERATOR) {
                break;
            }
        }

        if (is_callable($callback)) {
            $this->callback();

It seems like callback() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }