bristol-su / airtable

src/Control/AirtableHandler.php

<?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;
use Illuminate\Support\Facades\Log;

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();
        }

        $this->log('Flushing rows');
        
        dispatch_now(
            (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?

            ))->withDebug($this->config('debug', false))

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

        );

        $this->log('Flushed rows. Creating records');


        foreach(array_chunk($creating, 10) as $data) {

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

            dispatch((new CreateRecords(

The opening parenthesis of a multi-line function call should be the last content on the line.

                $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?

            ))->withDebug($this->config('debug', false)));

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

For multi-line function calls, the closing parenthesis should be on a new line.

        }

        $this->log('Created Records');

    }

    private function log(string $string)

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

Missing doc comment for function log()

    {
        if($this->config('debug', false)) {

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

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