for testing and deploying your application
for finding and fixing issues
for empowering human code reviews
<?php
declare(strict_types=1);
namespace PhpOffice\PhpSpreadsheetTests\Worksheet;
use PhpOffice\PhpSpreadsheet\Spreadsheet;
use PhpOffice\PhpSpreadsheet\Worksheet\Worksheet;
use PhpOffice\PhpSpreadsheetTests\Functional\AbstractFunctional;
class Issue4112Test extends AbstractFunctional
{
/**
* Problem deleting all sheets then adding one.
*
* @dataProvider providerSheetNumber
*/
public function testIssue4112(?int $sheetNumber): void
$sheetNumber
If this is a false-positive, you can also ignore this issue in your code via the ignore-unused annotation
ignore-unused
public function testIssue4112(/** @scrutinizer ignore-unused */ ?int $sheetNumber): void
This check looks for parameters that have been defined for a function or method, but which are not used in the method body.
$mySpreadsheet = new Spreadsheet();
$mySpreadsheet->removeSheetByIndex(0);
$worksheet = new Worksheet($mySpreadsheet, 'addedsheet');
self::assertSame(-1, $mySpreadsheet->getActiveSheetIndex());
$mySpreadsheet->addSheet($worksheet, 0);
self::assertSame('addedsheet', $mySpreadsheet->getActiveSheet()->getTitle());
$row = 1;
$col = 1;
$worksheet->getCell([$col, $row])->setValue('id_uti');
self::assertSame('id_uti', $worksheet->getCell([$col, $row])->getValue());
$mySpreadsheet->disconnectWorksheets();
}
public static function providerSheetNumber(): array
return [
'problem case' => [0],
'normal case' => [null],
'negative 1 (as if there were no sheets)' => [-1],
'diffeent negative number' => [-4],
'positive number' => [4],
];
PHPOffice /
PhpSpreadsheet
This check looks for parameters that have been defined for a function or method, but which are not used in the method body.