BristolSU\AirTable\Control\AirtableHandler::save()

Complexity

Conditions	3
Paths	4

Size

Total Lines	25
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	17
CRAP Score	3

Importance

Changes	1
Bugs	0	Features	0

<?php
    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);
    }
}