BristolSU\AirTable\Jobs\FlushRows::handle()

Complexity

Conditions	2
Paths	2

Size

Total Lines	16
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	15
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	0

<?php
    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('Retrieving IDs');
        $ids = $airTable->getIdsFromTable();
        $this->log('Retrieved IDs');
        $this->log('Deleting Rows');
        foreach(array_chunk($ids, 50) as $idsToDelete) {

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

            dispatch((new DeleteRows(

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

                $idsToDelete,
                $this->apiKey,
                $this->baseId,
                $this->tableName)

This line of the multi-line function call does not seem to be indented correctly. Expected 12 spaces, but found 16.

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

            )->withDebug($this->debug));

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

        }
}