BristolSU\AirTable\Jobs\DeleteRows

Complexity

Total Complexity

7

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Test Coverage

Coverage

65.52%

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	handle()	0	8	1
A	withDebug()	0	4	1
A	log()	0	4	2
A	__construct()	0	7	1
A	retryUntil()	0	3	1
A	middleware()	0	9	1

<?php

Missing file doc comment

namespace BristolSU\AirTable\Jobs;


use BristolSU\AirTable\AirTable;
use Illuminate\Bus\Queueable;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Foundation\Bus\Dispatchable;
use Illuminate\Queue\InteractsWithQueue;
use Illuminate\Support\Facades\Log;
use Spatie\RateLimitedMiddleware\RateLimited;

class DeleteRows implements ShouldQueue

Missing doc comment for class DeleteRows

{
    use Dispatchable, Queueable, InteractsWithQueue;

    private string $apiKey;

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

    private string $baseId;

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

    private string $tableName;

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

    private array $ids;

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

    private bool $debug = false;

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

    public function __construct(array $ids

Missing doc comment for function __construct()

        , string $apiKey, string $baseId, string $tableName)

The closing parenthesis of a multi-line function declaration must be on a new line

    {

The closing parenthesis and the opening brace of a multi-line function declaration must be on the same line

        $this->apiKey = $apiKey;
        $this->baseId = $baseId;
        $this->tableName = $tableName;
        $this->ids = $ids;
    }

    public function middleware()

Missing doc comment for function middleware()

    {
        $rateLimitedMiddleware = (new RateLimited())
            ->key('airtable')
            ->allow(1)
            ->everySeconds(1)
            ->releaseAfterSeconds(3);

        return [$rateLimitedMiddleware];
    }

    public function handle(AirTable $airTable)

Missing doc comment for function handle()

    {
        $airTable->setApiKey($this->apiKey);
        $airTable->setBaseId($this->baseId);
        $airTable->setTableName($this->tableName);
        $this->log('Deleting Rows');
        $airTable->deleteRows($this->ids);
        $this->log('Deleted Rows');
    }

    public function withDebug(bool $debug)

Missing doc comment for function withDebug()

    {
        $this->debug = $debug;
        return $this;
    }
    
    public function retryUntil() :  \DateTime

Missing doc comment for function retryUntil()

    {
        return now()->addHours(5);
    }

    private function log(string $string)

Private method name "DeleteRows::log" must be prefixed with an underscore

Missing doc comment for function log()

    {
        if($this->debug) {

Expected "if (...) {\n"; found "if(...) {\n"

            Log::debug($string);
        }
    }
}