Conveyor\Actions\Traits\ProcedureActionTrait::__invoke()

Complexity

Conditions	1
Paths	1

Size

Total Lines	7
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function __invoke(array $data)
    {
        $this->validateData($data);

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

        $this->data = $data;

The property data does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        return $this->execute();

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

    }