BristolSU\AirTable\Control\AirtableHandler

Complexity

Total Complexity

3

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	save()	0	25	3

<?php

Missing file doc comment

namespace BristolSU\AirTable\Control;

use BristolSU\AirTable\Jobs\CreateRecords;
use BristolSU\AirTable\Jobs\FlushRows;
use BristolSU\ControlDB\Export\FormattedItem;
use BristolSU\ControlDB\Export\Handler\Handler;
use Carbon\Carbon;
use Illuminate\Support\Collection;

class AirtableHandler extends Handler

Missing doc comment for class AirtableHandler

{

    /**
     * Save each item to AirTable
     * 
     * @param FormattedItem[] $items

Missing parameter comment

Tag value for @param tag indented incorrectly; expected 2 spaces but found 1

     * @return mixed|void

Tag @return cannot be grouped with parameter tags in a doc comment

     */
    protected function save(array $items)
    {
        $creating = [];
        
        foreach($items as $item) {

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

            $creating[] = $item->toArray();
        }

        $creatingJobs = [];
        foreach(array_chunk($creating, 50) as $data) {

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

            $creatingJobs[] = new CreateRecords(
                $data,
                $this->config('apiKey'),

It seems like $this->config('apiKey') can also be of type null; however, parameter $apiKey of BristolSU\AirTable\Jobs\CreateRecords::__construct() does only seem to accept string, maybe add an additional type check?

                $this->config('baseId'),

It seems like $this->config('baseId') can also be of type null; however, parameter $baseId of BristolSU\AirTable\Jobs\CreateRecords::__construct() does only seem to accept string, maybe add an additional type check?

                $this->config('tableName')

It seems like $this->config('tableName') can also be of type null; however, parameter $tableName of BristolSU\AirTable\Jobs\CreateRecords::__construct() does only seem to accept string, maybe add an additional type check?

            );
        }
        
        dispatch(
            new FlushRows(
                $this->config('apiKey'),

It seems like $this->config('apiKey') can also be of type null; however, parameter $apiKey of BristolSU\AirTable\Jobs\FlushRows::__construct() does only seem to accept string, maybe add an additional type check?

                $this->config('baseId'),

It seems like $this->config('baseId') can also be of type null; however, parameter $baseId of BristolSU\AirTable\Jobs\FlushRows::__construct() does only seem to accept string, maybe add an additional type check?

                $this->config('tableName')

It seems like $this->config('tableName') can also be of type null; however, parameter $tableName of BristolSU\AirTable\Jobs\FlushRows::__construct() does only seem to accept string, maybe add an additional type check?

            )
        )->chain($creatingJobs);
    }
}