[Question] Widget AttachmentsTableWithPreview

[Arekusei]

What is it the hell code?

        $hasFileBehavior = false;
        foreach ($this->model->getBehaviors() as $behavior) {
            if ($behavior->className() == FileBehavior::className()) {
                $hasFileBehavior = true;
            }
        }

Why not used instanceof?

[CTOlet]

Because it works! xD But instanceof is really the right way.

[Arekusei]

It works, but not extendable.

[Arekusei]

And for this code it will be good break foreach after $hasFileBehavior = true.

        $hasFileBehavior = false;
        foreach ($this->model->getBehaviors() as $behavior) {
            if ($behavior instanceof FileBehavior) {
                $hasFileBehavior = true;
                break;
            }
        }

[CTOlet]

Fixed 3ed19c5e2e5b424c7bd0c5dfbff17e49a52f57a7

[Arekusei]

Thank