Triadev\Leopard\Console\Commands\Mapping\Rollback

Complexity

Total Complexity

8

Size/Duplication

Total Lines	69
Duplicated Lines	0 %

Test Coverage

Coverage

95.24%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	handle()	0	9	4
A	deleteLogsBySteps()	0	16	3

<?php

The PHP open tag does not have a corresponding PHP close tag

namespace Triadev\Leopard\Console\Commands\Mapping;

Missing file doc comment

use Illuminate\Console\ConfirmableTrait;
use Triadev\Leopard\Contract\Repository\MappingLogRepositoryContract;

class Rollback extends BaseCommand

Missing class doc comment

{

Opening brace should be on the same line as the declaration for class Rollback

    use ConfirmableTrait;
    
    /**
     * The console command name.
     *
     * @var string
     */
    protected $signature = 'triadev:mapping:rollback {--steps=}';

Protected member variable "signature" must contain a leading underscore

    
    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Rollback the last migration steps.';

Protected member variable "description" must contain a leading underscore

    
    /** @var MappingLogRepositoryContract */

The open comment tag must be the only content on the line

Missing short description in doc comment

The close comment tag must be the only content on the line

    private $mappingLogRepository;

Private member variable "mappingLogRepository" must contain a leading underscore

Private member variable "mappingLogRepository" must be prefixed with an underscore

    
    /**
     * Rollback constructor.
     * @param MappingLogRepositoryContract $mappingLogRepository

There must be exactly one blank line before the tags in a doc comment

Missing parameter comment

     */
    public function __construct(MappingLogRepositoryContract $mappingLogRepository)

Expected 2 blank lines before function; 1 found

    {
        parent::__construct();
        
        $this->mappingLogRepository = $mappingLogRepository;
    }

Expected 2 blank lines after function; 1 found

Expected 1 blank line before closing function brace; 0 found

Expected //end __construct()

    
    /**
     * Execute the console command.
     *
     * @throws \Throwable

Comment missing for @throws tag in function comment

     */

Missing @return tag in function comment

    public function handle()
    {
        if (!$this->confirmToProceed()) {

There must be a single space after a NOT operator; 0 found

            return;
        }
        
        $steps = $this->option('steps') ?: $this->mappingLogRepository->reset();

The value of a comparison must not be assigned to a variable

Inline IF statements are not allowed

        if (is_int($steps)) {
            $this->deleteLogsBySteps($steps);
        }
    }

Expected 2 blank lines after function; 1 found

Expected 1 blank line before closing function brace; 0 found

Expected //end handle()

    
    /**

Missing short description in doc comment

     * @param int $steps

Missing parameter comment

Expected "integer" but found "int" for parameter type

     * @throws \Throwable

Comment missing for @throws tag in function comment

     */

Missing @return tag in function comment

    private function deleteLogsBySteps(int $steps)

Private method name "Rollback::deleteLogsBySteps" must be prefixed with an underscore

    {
        $logs = $this->mappingLogRepository->all();
    
        if ($steps >= count($logs)) {
            $this->mappingLogRepository->reset();
        }
    
        $logs = array_slice(
            array_reverse($logs),
            0,
            $steps
        );
    
        foreach ($logs as $log) {
            $this->mappingLogRepository->delete($log['id']);
        }
    }

Expected 2 blank lines after function; 0 found

Expected 1 blank line before closing function brace; 0 found

Expected //end deleteLogsBySteps()

}

Expected //end class

As per coding style, files should not end with a newline character.