BatchQueryResult::getRows() - Code Metrics - Inspection of "fa28a225d7f01c08dc34afc445aeded000355ca1" - yiisoft/db-mssql - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Branch — master (fa28a2)

by Wilmer

created 2022-12-16 13:01 UTC

BatchQueryResult::getRows() A

↳ Parent: BatchQueryResult

Complexity

Conditions	6
Paths	7

Size

Total Lines	18
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	6

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	6
eloc	11
c	1
b	0
f	0
nc	7
nop	0
dl	0
loc	18
ccs	9
cts	9
cp	1
crap	6
rs	9.2222

<?php

declare(strict_types=1);

namespace Yiisoft\Db\Mssql;

use PDOException;
use Yiisoft\Db\Exception\InvalidCallException;
use Yiisoft\Db\Query\BatchQueryResult as BaseBatchQueryResult;

class BatchQueryResult extends BaseBatchQueryResult
{
    /**
     * @var int MSSQL error code for exception that is thrown when last batch is size less than specified batch size
     *
     * {@see https://github.com/yiisoft/yii2/issues/10023}
     */
    private int $mssqlNoMoreRowsErrorCode = -13;

    /**
     * Reads and collects rows for batch.
     *
     * @throws InvalidCallException
     *
     * @psalm-suppress MixedArrayAccess
     */
    protected function getRows(): array
    {
        $rows = [];
        $count = 0;

        try {
            do {
                $this->dataReader?->next();
                /** @psalm-var array|bool $row */
                $row = $this->dataReader?->current();
            } while ($row && ($rows[] = $row) && ++$count < $this->batchSize);
        } catch (PDOException $e) {
            if (($e->errorInfo[1] ?? null) !== $this->mssqlNoMoreRowsErrorCode) {
                throw $e;
            }
        }

        return $rows;
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Yiisoft\Db\Mssql;
6
7		use PDOException;
8		use Yiisoft\Db\Exception\InvalidCallException;
9		use Yiisoft\Db\Query\BatchQueryResult as BaseBatchQueryResult;
10
11		class BatchQueryResult extends BaseBatchQueryResult
12		{
13		/**
14		* @var int MSSQL error code for exception that is thrown when last batch is size less than specified batch size
15		*
16		* {@see https://github.com/yiisoft/yii2/issues/10023}
17		*/
18		private int $mssqlNoMoreRowsErrorCode = -13;
19
20		/**
21		* Reads and collects rows for batch.
22		*
23		* @throws InvalidCallException
24		*
25		* @psalm-suppress MixedArrayAccess
26		*/
27	3	protected function getRows(): array
28		{
29	3	$rows = [];
30	3	$count = 0;
31
32		try {
33		do {
34	3	$this->dataReader?->next();
35		/** @psalm-var array\|bool $row */
36	3	$row = $this->dataReader?->current();
37	3	} while ($row && ($rows[] = $row) && ++$count < $this->batchSize);
38	3	} catch (PDOException $e) {
39	3	if (($e->errorInfo[1] ?? null) !== $this->mssqlNoMoreRowsErrorCode) {
40		throw $e;
41		}
42		}
43
44	3	return $rows;
45		}
46		}
47

yiisoft / db-mssql

Branch — master (fa28a2)

BatchQueryResult::getRows() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like