It seems like $row->getStyle() can be null; however, merge() does not accept null, maybe add an additional type check?
Unless you are absolutely sure that the expression can never be null because of
other conditions, we strongly recommend to add an additional type check to your
code:
/** @return stdClass|null */functionmayReturnNull(){}functiondoesNotAcceptNull(stdClass$x){}// With potential error.functionwithoutCheck(){$x=mayReturnNull();doesNotAcceptNull($x);// Potential error here.}// Safe - Alternative 1functionwithCheck1(){$x=mayReturnNull();if(!$xinstanceofstdClass){thrownew\LogicException('$x must be defined.');}doesNotAcceptNull($x);}// Safe - Alternative 2functionwithCheck2(){$x=mayReturnNull();if($xinstanceofstdClass){doesNotAcceptNull($x);}}
Loading history...
32
$row->setStyle($mergedStyle);
33
}
34
35
/**
36
* Returns whether a row has cells
37
*
38
* @param Row $row
39
* @return bool
40
*/
41
75
public function hasCells(Row $row)
42
{
43
75
return count($row->getCells()) !== 0;
44
}
45
46
/**
47
* Detect whether a row is considered empty.
48
* An empty row has either no cells at all - or only one empty cell
box /
spout
Unless you are absolutely sure that the expression can never be null because of other conditions, we strongly recommend to add an additional type check to your code: