Migratio\Resource\PushManager\PushingProcess::processHandler()

Complexity

Conditions	4
Paths	1

Size

Total Lines	28
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function processHandler()
    {
        $results = [];

The assignment to $results is dead and can be removed.

        return $this->errorHandler(function(){

The expression return $this->errorHandler(function(...) { /* ... */ }) also could return the type string which is incompatible with the documented return type void.

            
            foreach ($this->list as $table =>$datas){

                foreach ($datas as $data){

                    $queryBuilder = $this->schema->getGrammarPath().'\QueryBuilder';

                    $query = (new $queryBuilder($this->schema,$table,$data))->handle();

                    $status =($query['result']!==false) ? true : false;

                    $results[]= [
                        'success'=>$status,
                        'file'=>$data->getFile(),
                        'table'=>$table,
                        'type'=>$query['type'],
                        'syntax'=>$query['syntax'],
                        'message'=>$query['message']
                    ];
                }
            }

            return $results;

The variable $results does not seem to be defined for all execution paths leading up to this point.

        });
}