PushingProcess::errorHandler() - Code Metrics - Inspection of "database migration" - php-resta/resta-core - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Push — master ( d6fde7...1e848b )

by Php Easy Api

created 2019-06-27 08:28 UTC

PushingProcess::errorHandler() A

↳ Parent: PushingProcess

Complexity

Conditions	4
Paths	4

Size

Total Lines	13
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	4
eloc	5
c	1
b	0
f	0
nc	4
nop	1
dl	0
loc	13
rs	10

<?php

namespace Migratio\Resource\PushManager;

use Migratio\GrammarStructure\Mysql\QueryBuilder;

trait PushingProcess
{
    /**
     * @return void
     */
    public function processHandler()
    {
        $results = [];


        return $this->errorHandler(function(){

            
            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;

        });
    }

    /**
     * @param callable $callback
     */
    public function errorHandler(callable $callback)
    {
        foreach ($this->list as $table => $objects)
        {
            foreach ($objects as $object)
            {
                if(count($object->getError())){
                    return 'error : '.$object->getFile().' -> '.$object->getError()[0].'';
                }
            }
        }

        return call_user_func($callback);
    }
}

1			<?php
2
3			namespace Migratio\Resource\PushManager;
4
5			use Migratio\GrammarStructure\Mysql\QueryBuilder;
6
7			trait PushingProcess
8			{
9			/**
10			* @return void
11			*/
12			public function processHandler()
13			{
14			$results = [];
			0 ignored issues – show Unused Code introduced 2019-06-27 08:32 UTC by Report Bug Copy Issue Report The assignment to `$results` is dead and can be removed. Loading history...
15
16			return $this->errorHandler(function(){
			0 ignored issues – show Bug Best Practice introduced 2019-06-27 08:32 UTC by Report Bug Copy Issue Report The expression `return $this->errorHandl...ion(...) { /* ... */ })` also could return the type `string` which is incompatible with the documented return type `void`. Loading history...
17
18			foreach ($this->list as $table =>$datas){
19
20			foreach ($datas as $data){
21
22			$queryBuilder = $this->schema->getGrammarPath().'\QueryBuilder';
23
24			$query = (new $queryBuilder($this->schema,$table,$data))->handle();
25
26			$status =($query['result']!==false) ? true : false;
27
28			$results[]= [
29			'success'=>$status,
30			'file'=>$data->getFile(),
31			'table'=>$table,
32			'type'=>$query['type'],
33			'syntax'=>$query['syntax'],
34			'message'=>$query['message']
35			];
36			}
37			}
38
39			return $results;
			0 ignored issues – show Comprehensibility Best Practice introduced 2019-06-27 08:32 UTC by Report Bug Copy Issue Report The variable `$results` does not seem to be defined for all execution paths leading up to this point. Loading history...
40			});
41			}
42
43			/**
44			* @param callable $callback
45			*/
46			public function errorHandler(callable $callback)
47			{
48			foreach ($this->list as $table => $objects)
49			{
50			foreach ($objects as $object)
51			{
52			if(count($object->getError())){
53			return 'error : '.$object->getFile().' -> '.$object->getError()[0].'';
54			}
55			}
56			}
57
58			return call_user_func($callback);
59			}
60			}

php-resta / resta-core

Push — master ( d6fde7...1e848b )

PushingProcess::errorHandler() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like